diff options
Diffstat (limited to 'maintenance/initSiteStats.php')
| -rw-r--r-- | maintenance/initSiteStats.php | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/maintenance/initSiteStats.php b/maintenance/initSiteStats.php index 92268b3e..49e0e9d7 100644 --- a/maintenance/initSiteStats.php +++ b/maintenance/initSiteStats.php @@ -34,7 +34,10 @@ class InitSiteStats extends Maintenance { public function __construct() { parent::__construct(); $this->mDescription = "Re-initialise the site statistics tables"; - $this->addOption( 'update', 'Update the existing statistics (preserves the ss_total_views field)' ); + $this->addOption( + 'update', + 'Update the existing statistics (preserves the ss_total_views field)' + ); $this->addOption( 'noviews', "Don't update the page view counter" ); $this->addOption( 'active', 'Also update active users count' ); $this->addOption( 'use-master', 'Count using the master database' ); @@ -66,21 +69,19 @@ class InitSiteStats extends Maintenance { $this->output( "{$views}\n" ); } + if ( $this->hasOption( 'update' ) ) { + $this->output( "\nUpdating site statistics..." ); + $counter->refresh(); + $this->output( "done.\n" ); + } + if ( $this->hasOption( 'active' ) ) { - $this->output( "Counting active users..." ); + $this->output( "\nCounting and updating active users..." ); $active = SiteStatsUpdate::cacheUpdate( wfGetDB( DB_MASTER ) ); $this->output( "{$active}\n" ); } - $this->output( "\nUpdating site statistics..." ); - - if ( $this->hasOption( 'update' ) ) { - $counter->update(); - } else { - $counter->refresh(); - } - - $this->output( "done.\n" ); + $this->output( "\nDone.\n" ); } } |