For MediaWiki (recent comments | status changes | tags | authors | states | release notes | statistics)
Index: branches/new-installer/phase3/includes/installer/WebInstaller.php =================================================================== --- branches/new-installer/phase3/includes/installer/WebInstaller.php (revision 55128) +++ branches/new-installer/phase3/includes/installer/WebInstaller.php (revision 55129) @@ -338,7 +338,8 @@ $name = wfMsg( 'config-page-' . strtolower( $pageName ) ); if ( $enabled ) { $query = array( 'page' => $pageName ); - if ( !in_array( $pageName, $this->pageSequence ) ) { + if ( in_array( $pageName, $this->pageSequence ) || + in_array( $pageName, $this->otherPages ) ) { if ( in_array( $currentPageName, $this->pageSequence ) ) { $query['lastPage'] = $currentPageName; }
I don't think you have the faintest clue what you're doing. Those links were removed early on in the development process due to the problem of interactions between pages, where one page sets variables to be used by the next. The DBConnect and DBSettings pages in particular were a problem, if you allow forward links, then the user could skip back to set a different DB type and then skip the DBSettings page for that DB type, leading to uninitialised variables at the install stage. By making the user click the submit buttons on each page, thus accepting the defaults, you ensure that each page class has the opportunity to initialise variables.
Suggest revert.
Reverted in r55164
