Merge "Pass $user from WikiPage::getContent to Revision::getContent"

[lhc/web/wiklou.git] / maintenance / cleanupRemovedModules.php

diff --git a/maintenance/cleanupRemovedModules.php b/maintenance/cleanupRemovedModules.php
index 9f22adf..2085da9 100644 (file)
--- a/maintenance/cleanupRemovedModules.php
+++ b/maintenance/cleanupRemovedModules.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * Maintenance script to create an account and grant it administrator rights
+ * Remove cache entries for removed ResourceLoader modules from the database.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -19,11 +19,17 @@
  *
  * @file
  * @ingroup Maintenance
- * @author Rob Church <robchur@gmail.com>
+ * @author Roan Kattouw
  */
 
-require_once( dirname( __FILE__ ) . '/Maintenance.php' );
+require_once( __DIR__ . '/Maintenance.php' );
 
+/**
+ * Maintenance script to remove cache entries for removed ResourceLoader modules
+ * from the database.
+ *
+ * @ingroup Maintenance
+ */
 class CleanupRemovedModules extends Maintenance {
 
        public function __construct() {
@@ -36,48 +42,53 @@ class CleanupRemovedModules extends Maintenance {
        public function execute() {
                $dbw = wfGetDB( DB_MASTER );
                $rl = new ResourceLoader();
-               $moduleNames = array_keys( $rl->getModules() );
+               $moduleNames = $rl->getModuleNames();
                $moduleList = implode( ', ', array_map( array( $dbw, 'addQuotes' ), $moduleNames ) );
-               $limit = min( 1, intval( $this->getOption( 'batchsize', 500 ) ) );
+               $limit = max( 1, intval( $this->getOption( 'batchsize', 500 ) ) );
                $maxlag = intval( $this->getOption( 'max-slave-lag', 5 ) );
-               
+
                $this->output( "Cleaning up module_deps table...\n" );
                $i = 1;
+               $modDeps = $dbw->tableName( 'module_deps' );
                do {
                        // $dbw->delete() doesn't support LIMIT :(
-                       $dbw->query( "DELETE FROM module_deps WHERE md_module NOT IN ($moduleList) LIMIT $limit", __METHOD__ );
+                       $where = $moduleList ? "md_module NOT IN ($moduleList)" : '1=1';
+                       $dbw->query( "DELETE FROM $modDeps WHERE $where LIMIT $limit", __METHOD__ );
                        $numRows = $dbw->affectedRows();
                        $this->output( "Batch $i: $numRows rows\n" );
                        $i++;
                        wfWaitForSlaves( $maxlag );
-               } while( $dbw->affectedRows() > 0 );
+               } while( $numRows > 0 );
                $this->output( "done\n" );
-               
+
                $this->output( "Cleaning up msg_resource table...\n" );
                $i = 1;
+
+               $mrRes = $dbw->tableName( 'msg_resource' );
                do {
-                       // $dbw->delete() doesn't support LIMIT :(
-                       $dbw->query( "DELETE FROM msg_resource WHERE mr_resource NOT IN ($moduleList) LIMIT $limit", __METHOD__ );
+                       $where = $moduleList ? "mr_resource NOT IN ($moduleList)" : '1=1';
+                       $dbw->query( "DELETE FROM $mrRes WHERE $where LIMIT $limit", __METHOD__ );
                        $numRows = $dbw->affectedRows();
                        $this->output( "Batch $i: $numRows rows\n" );
                        $i++;
                        wfWaitForSlaves( $maxlag );
-               } while( $dbw->affectedRows() > 0 );
+               } while( $numRows > 0 );
                $this->output( "done\n" );
-               
+
                $this->output( "Cleaning up msg_resource_links table...\n" );
                $i = 1;
+               $msgResLinks = $dbw->tableName( 'msg_resource_links' );
                do {
-                       // $dbw->delete() doesn't support LIMIT :(
-                       $dbw->query( "DELETE FROM msg_resource_links WHERE mrl_resource NOT IN ($moduleList) LIMIT $limit", __METHOD__ );
+                       $where = $moduleList ? "mrl_resource NOT IN ($moduleList)" : '1=1';
+                       $dbw->query( "DELETE FROM $msgResLinks WHERE $where LIMIT $limit", __METHOD__ );
                        $numRows = $dbw->affectedRows();
                        $this->output( "Batch $i: $numRows rows\n" );
                        $i++;
                        wfWaitForSlaves( $maxlag );
-               } while( $dbw->affectedRows() > 0 );
+               } while( $numRows > 0 );
                $this->output( "done\n" );
        }
 }
 
 $maintClass = "CleanupRemovedModules";
-require_once( DO_MAINTENANCE );
\ No newline at end of file
+require_once( RUN_MAINTENANCE_IF_MAIN );