MediaWiki 1.30.2

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

diff --git a/maintenance/deleteBatch.php b/maintenance/deleteBatch.php
index c8bb4803c301d26f1fcd6ac13b335040c1768207..0020446b41653d5d3555273928961e945a938c77 100644 (file)
--- a/maintenance/deleteBatch.php
+++ b/maintenance/deleteBatch.php
@@ -1,13 +1,13 @@
 <?php
 /**
- * Deletes a batch of pages
+ * Deletes a batch of pages.
  * Usage: php deleteBatch.php [-u <user>] [-r <reason>] [-i <interval>] [listfile]
  * where
- *     [listfile] is a file where each line contains the title of a page to be
- *             deleted, standard input is used if listfile is not given.
- *     <user> is the username
- *     <reason> is the delete reason
- *     <interval> is the number of seconds to sleep for after each delete
+ *   [listfile] is a file where each line contains the title of a page to be
+ *     deleted, standard input is used if listfile is not given.
+ *   <user> is the username
+ *   <reason> is the delete reason
+ *   <interval> is the number of seconds to sleep for after each delete
  *
  * 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
@@ -24,16 +24,22 @@
  * 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';
 
+/**
+ * Maintenance script to delete a batch of pages.
+ *
+ * @ingroup Maintenance
+ */
 class DeleteBatch extends Maintenance {
 
        public function __construct() {
                parent::__construct();
-               $this->mDescription = "Deletes a batch of pages";
+               $this->addDescription( 'Deletes a batch of pages' );
                $this->addOption( 'u', "User to perform deletion", false, true );
                $this->addOption( 'r', "Reason to delete page", false, true );
                $this->addOption( 'i', "Interval to sleep between deletions" );
@@ -49,9 +55,20 @@ class DeleteBatch extends Maintenance {
                chdir( $oldCwd );
 
                # Options processing
-               $user = $this->getOption( 'u', 'Delete page script' );
+               $username = $this->getOption( 'u', false );
                $reason = $this->getOption( 'r', '' );
                $interval = $this->getOption( 'i', 0 );
+
+               if ( $username === false ) {
+                       $user = User::newSystemUser( 'Delete page script', [ 'steal' => true ] );
+               } else {
+                       $user = User::newFromName( $username );
+               }
+               if ( !$user ) {
+                       $this->error( "Invalid username", true );
+               }
+               $wgUser = $user;
+
                if ( $this->hasArg() ) {
                        $file = fopen( $this->getArg(), 'r' );
                } else {
@@ -62,41 +79,39 @@ class DeleteBatch extends Maintenance {
                if ( !$file ) {
                        $this->error( "Unable to read file, exiting", true );
                }
-               $wgUser = User::newFromName( $user );
-               $dbw = wfGetDB( DB_MASTER );
+
+               $dbw = $this->getDB( DB_MASTER );
 
                # Handle each entry
+               // @codingStandardsIgnoreStart Ignore Generic.CodeAnalysis.ForLoopWithTestFunctionCall.NotAllowed
                for ( $linenum = 1; !feof( $file ); $linenum++ ) {
+                       // @codingStandardsIgnoreEnd
                        $line = trim( fgets( $file ) );
                        if ( $line == '' ) {
                                continue;
                        }
-                       $page = Title::newFromText( $line );
-                       if ( is_null( $page ) ) {
+                       $title = Title::newFromText( $line );
+                       if ( is_null( $title ) ) {
                                $this->output( "Invalid title '$line' on line $linenum\n" );
                                continue;
                        }
-                       if ( !$page->exists() ) {
+                       if ( !$title->exists() ) {
                                $this->output( "Skipping nonexistent page '$line'\n" );
                                continue;
                        }
 
-
-                       $this->output( $page->getPrefixedText() );
-                       $dbw->begin();
-                       if ( $page->getNamespace() == NS_FILE ) {
-                               $art = new ImagePage( $page );
-                               $img = wfFindFile( $art->mTitle );
-                               if ( !$img || !$img->delete( $reason ) ) {
-                                       $this->output( "FAILED to delete image file... " );
+                       $this->output( $title->getPrefixedText() );
+                       if ( $title->getNamespace() == NS_FILE ) {
+                               $img = wfFindFile( $title, [ 'ignoreRedirect' => true ] );
+                               if ( $img && $img->isLocal() && !$img->delete( $reason ) ) {
+                                       $this->output( " FAILED to delete associated file... " );
                                }
-                       } else {
-                               $art = new Article( $page );
                        }
-                       $success = $art->doDeleteArticle( $reason );
-                       $dbw->commit();
+                       $page = WikiPage::factory( $title );
+                       $error = '';
+                       $success = $page->doDeleteArticle( $reason, false, 0, true, $error, $user );
                        if ( $success ) {
-                               $this->output( "\n" );
+                               $this->output( " Deleted!\n" );
                        } else {
                                $this->output( " FAILED to delete article\n" );
                        }
@@ -104,10 +119,10 @@ class DeleteBatch extends Maintenance {
                        if ( $interval ) {
                                sleep( $interval );
                        }
-                       wfWaitForSlaves( 5 );
-}
+                       wfWaitForSlaves();
+               }
        }
 }
 
 $maintClass = "DeleteBatch";
-require_once( RUN_MAINTENANCE_IF_MAIN );
+require_once RUN_MAINTENANCE_IF_MAIN;