r97774 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r97773‎ | r97774 | r97775 >
Date:00:35, 22 September 2011
Author:brion
Status:ok
Tags:
Comment:
* (bug 30041) Fix installer cache selection field to keep the selected radio button, initialize the memcached server fold-out correctly when coming back to the page
Modified paths:
  • /trunk/phase3/includes/installer/WebInstallerPage.php (modified) (history)
  • /trunk/phase3/skins/common/config.css (modified) (history)

Diff [purge]

Index: trunk/phase3/skins/common/config.css
@@ -133,9 +133,6 @@
134134 padding: 0 !important;
135135 }
136136
137 -#config-memcachewrapper {
138 - display: none;
139 -}
140137 .config-download-link {
141138 font-size: 1.8em;
142139 margin-left: 2em;
Index: trunk/phase3/includes/installer/WebInstallerPage.php
@@ -920,6 +920,15 @@
921921 }
922922 $caches[] = 'memcached';
923923
 924+ // We'll hide/show this on demand when the value changes, see config.js.
 925+ $cacheval = $this->getVar( 'wgMainCacheType' );
 926+ if (!$cacheval) {
 927+ // We need to set a default here; but don't hardcode it
 928+ // or we lose it every time we reload the page for validation
 929+ // or going back!
 930+ $cacheval = 'none';
 931+ }
 932+ $hidden = ($cacheval == 'memcached') ? '' : 'display: none';
924933 $this->addHTML(
925934 # Advanced settings
926935 $this->getFieldSetStart( 'config-advanced-settings' ) .
@@ -929,10 +938,10 @@
930939 'label' => 'config-cache-options',
931940 'itemLabelPrefix' => 'config-cache-',
932941 'values' => $caches,
933 - 'value' => 'none',
 942+ 'value' => $cacheval,
934943 ) ) .
935944 $this->parent->getHelpBox( 'config-cache-help' ) .
936 - '<div id="config-memcachewrapper">' .
 945+ "<div id=\"config-memcachewrapper\" style=\"$hidden\">" .
937946 $this->parent->getTextArea( array(
938947 'var' => '_MemCachedServers',
939948 'label' => 'config-memcached-servers',

Follow-up revisions

RevisionCommit summaryAuthorDate
r98754MFT r93431, r94761, r97762, r97772, r97774, r97777, r97817reedy13:10, 3 October 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r97772* (bug 30041) Fix memcached validation in installer to accept resolvable host...brion00:11, 22 September 2011

Status & tagging log