r104145 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r104144‎ | r104145 | r104146 >
Date:09:41, 24 November 2011
Author:hashar
Status:ok
Tags:
Comment:
revert r102636

This move back the maintenance finalSetup code above core Setup.
If we really need to do any setup after core Setup has run, we should use
a new method in the Maintenance class. For example afterCoreSetup()
Modified paths:
  • /trunk/phase3/maintenance/doMaintenance.php (modified) (history)

Diff [purge]

Index: trunk/phase3/maintenance/doMaintenance.php
@@ -93,11 +93,10 @@
9494 {
9595 require( MWInit::interpretedPath( 'AdminSettings.php' ) );
9696 }
 97+$maintenance->finalSetup();
9798 // Some last includes
9899 require_once( MWInit::compiledPath( 'includes/Setup.php' ) );
99100
100 -$maintenance->finalSetup();
101 -
102101 // Much much faster startup than creating a title object
103102 $wgTitle = null;
104103

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r102636load up includes/Setup.php before Maintenance::finalSetup()...hashar11:18, 10 November 2011

Status & tagging log