(bug 32508) Fix updater output

site_stats updated is still a database action.
Purge cache is now the very last one.

Output:

	<snip>
	...site_stats is populated...done.
	Purging caches...done.

	Done.
This commit is contained in:
Antoine Musso 2012-01-12 10:59:13 +00:00
parent ca27814f63
commit eb43a7e8b4

View file

@ -250,6 +250,10 @@ abstract class DatabaseUpdater {
$this->setAppliedUpdates( $wgVersion, $this->updates );
if ( isset( $what['stats'] ) ) {
$this->checkStats();
}
if ( isset( $what['purge'] ) ) {
$this->purgeCache();
@ -257,9 +261,6 @@ abstract class DatabaseUpdater {
$this->rebuildLocalisationCache();
}
}
if ( isset( $what['stats'] ) ) {
$this->checkStats();
}
}
/**
@ -562,7 +563,7 @@ abstract class DatabaseUpdater {
* Check the site_stats table is not properly populated.
*/
protected function checkStats() {
$this->output( "Checking site_stats row..." );
$this->output( "...site_stats is populated..." );
$row = $this->db->selectRow( 'site_stats', '*', array( 'ss_row_id' => 1 ), __METHOD__ );
if ( $row === false ) {
$this->output( "data is missing! rebuilding...\n" );