MediaWiki 1.30.2

[autoinstallsdev/mediawiki.git] / maintenance / populateLogUsertext.php

diff --git a/maintenance/populateLogUsertext.php b/maintenance/populateLogUsertext.php
index a491b2b0b931235b8009d745e8e43d45531db6a1..dd120fe0035ad4fbe93f2d6e599676d5a1275793 100644 (file)
--- a/maintenance/populateLogUsertext.php
+++ b/maintenance/populateLogUsertext.php
@@ -20,23 +20,39 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  * http://www.gnu.org/copyleft/gpl.html
  *
+ * @file
  * @ingroup Maintenance
  */
 
-require_once( dirname(__FILE__) . '/Maintenance.php' );
+require_once __DIR__ . '/Maintenance.php';
 
-class PopulateLogUsertext extends Maintenance {
+/**
+ * Maintenance script that makes the required database updates for
+ * Special:ProtectedPages to show all protected pages.
+ *
+ * @ingroup Maintenance
+ */
+class PopulateLogUsertext extends LoggedUpdateMaintenance {
        public function __construct() {
                parent::__construct();
-               $this->mDescription = "Populates the log_user_text";
+               $this->addDescription( 'Populates the log_user_text field' );
                $this->setBatchSize( 100 );
        }
 
-       public function execute() {
-               $db = wfGetDB( DB_MASTER );
+       protected function getUpdateKey() {
+               return 'populate log_usertext';
+       }
+
+       protected function updateSkippedMessage() {
+               return 'log_user_text column of logging table already populated.';
+       }
+
+       protected function doDBUpdates() {
+               $db = $this->getDB( DB_MASTER );
                $start = $db->selectField( 'logging', 'MIN(log_id)', false, __METHOD__ );
-               if( !$start ) {
+               if ( !$start ) {
                        $this->output( "Nothing to do.\n" );
+
                        return true;
                }
                $end = $db->selectField( 'logging', 'MAX(log_id)', false, __METHOD__ );
@@ -45,38 +61,27 @@ class PopulateLogUsertext extends Maintenance {
                $end += $this->mBatchSize - 1;
                $blockStart = $start;
                $blockEnd = $start + $this->mBatchSize - 1;
-               while( $blockEnd <= $end ) {
+               while ( $blockEnd <= $end ) {
                        $this->output( "...doing log_id from $blockStart to $blockEnd\n" );
                        $cond = "log_id BETWEEN $blockStart AND $blockEnd AND log_user = user_id";
-                       $res = $db->select( array('logging','user'), 
-                               array('log_id','user_name'), $cond, __METHOD__ );
-                       $batch = array();
-                       $db->begin();
-                       foreach( $res as $row ) {
-                               $db->update( 'logging', array('log_user_text' => $row->user_name),
-                                       array('log_id' => $row->log_id), __METHOD__ );
+                       $res = $db->select( [ 'logging', 'user' ],
+                               [ 'log_id', 'user_name' ], $cond, __METHOD__ );
+
+                       $this->beginTransaction( $db, __METHOD__ );
+                       foreach ( $res as $row ) {
+                               $db->update( 'logging', [ 'log_user_text' => $row->user_name ],
+                                       [ 'log_id' => $row->log_id ], __METHOD__ );
                        }
-                       $db->commit();
+                       $this->commitTransaction( $db, __METHOD__ );
                        $blockStart += $this->mBatchSize;
                        $blockEnd += $this->mBatchSize;
-                       wfWaitForSlaves( 5 );
-               }
-               if( $db->insert(
-                               'updatelog',
-                               array( 'ul_key' => 'populate log_usertext' ),
-                               __METHOD__,
-                               'IGNORE'
-                       )
-               ) {
-                       $this->output( "log_usertext population complete.\n" );
-                       return true;
-               } else {
-                       $this->output( "Could not insert log_usertext population row.\n" );
-                       return false;
+                       wfWaitForSlaves();
                }
+               $this->output( "Done populating log_user_text field.\n" );
+
+               return true;
        }
 }
 
 $maintClass = "PopulateLogUsertext";
-require_once( DO_MAINTENANCE );
-
+require_once RUN_MAINTENANCE_IF_MAIN;