r85047 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r85046‎ | r85047 | r85048 >
Date:20:56, 30 March 2011
Author:overlordq
Status:ok (Comments)
Tags:
Comment:
Followup to r85021, fix caller in updater maintenance script
Modified paths:
  • /trunk/phase3/maintenance/update.php (modified) (history)

Diff [purge]

Index: trunk/phase3/maintenance/update.php
@@ -102,10 +102,14 @@
103103 }
104104
105105 $shared = $this->hasOption( 'doshared' );
106 - $purge = !$this->hasOption( 'nopurge' );
107106
 107+ $updates = array('core','extensions');
 108+ if( !$this->hasOption('nopurge') ) {
 109+ $updates[] = 'purge';
 110+ }
 111+
108112 $updater = DatabaseUpdater::newForDb( $db, $shared, $this );
109 - $updater->doUpdates( $purge );
 113+ $updater->doUpdates( $updates );
110114
111115 foreach( $updater->getPostDatabaseUpdateMaintenance() as $maint ) {
112116 $child = $this->runChild( $maint );

Follow-up revisions

RevisionCommit summaryAuthorDate
r85081MFT last round of db and installer changes: r81755, r82596, r85021, r85047, r...demon19:33, 31 March 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r85021(bug 28237) Installer doesn't create extension tablesmaxsem17:32, 30 March 2011

Comments

#Comment by MaxSem (talk | contribs)   04:19, 31 March 2011

Thanks!

Status & tagging log