* (bug 26937) [Installer] Fix for Javascript-opened sections being incorrectly open...
authorBrion Vibber <brion@users.mediawiki.org>
Sun, 27 Mar 2011 19:22:14 +0000 (19:22 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Sun, 27 Mar 2011 19:22:14 +0000 (19:22 +0000)
commit4ac9bc6a0a9aae3ce852c1718f7bbaa195a71076
treea61a67e59008481ca629360f8f2828889050edc8
parenta051fe15d38914ef3e4fbce8dbc34543b14bd2df
* (bug 26937) [Installer] Fix for Javascript-opened sections being incorrectly open or closed when returning to page

The sections whose visibility was being controlled by the checkbox had an initial visibility hardcoded in the page output, which had been manually set to match the expected initial default state. I've switched these to instead check what the variable is set to and base the initial style visibility on that.
Could be better to encapsulate these settings together in some way, but this'll do for now.

Note that the way these are all done now means that the sections can never be used when JS is off; if a fully-functional installer is desired in a JS-free environment, it may be necessary to change these further to default to shown, and do the initial hiding in JavaScript (could have 'flashing' issues though: the bits showing up, then quickly hiding. laaame but oh well)
includes/installer/DatabaseInstaller.php
includes/installer/WebInstallerPage.php