X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=maintenance%2FdeleteBatch.php;h=c3cbdebcdf9bb5954c5f6c84545714230cc3766c;hb=7ddb8c2d62433a32aad4458cf0efb8b51f0467e2;hp=6c89e671eabecabb364ced4255b9f7efb30d6dc2;hpb=f875f3d31867067116641af987dea4511324e96b;p=lhc%2Fweb%2Fwiklou.git

diff --git a/maintenance/deleteBatch.php b/maintenance/deleteBatch.php
index 6c89e671ea..c3cbdebcdf 100644
--- a/maintenance/deleteBatch.php
+++ b/maintenance/deleteBatch.php
@@ -39,7 +39,7 @@ 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" );
@@ -60,12 +60,12 @@ class DeleteBatch extends Maintenance {
 		$interval = $this->getOption( 'i', 0 );
 
 		if ( $username === false ) {
-			$user = User::newSystemUser( 'Delete page script', array( 'steal' => true ) );
+			$user = User::newSystemUser( 'Delete page script', [ 'steal' => true ] );
 		} else {
 			$user = User::newFromName( $username );
 		}
 		if ( !$user ) {
-			$this->error( "Invalid username", true );
+			$this->fatalError( "Invalid username" );
 		}
 		$wgUser = $user;
 
@@ -77,15 +77,14 @@ class DeleteBatch extends Maintenance {
 
 		# Setup
 		if ( !$file ) {
-			$this->error( "Unable to read file, exiting", true );
+			$this->fatalError( "Unable to read file, exiting" );
 		}
 
 		$dbw = $this->getDB( DB_MASTER );
 
 		# Handle each entry
-		// @codingStandardsIgnoreStart Ignore Generic.CodeAnalysis.ForLoopWithTestFunctionCall.NotAllowed
+		// phpcs:ignore Generic.CodeAnalysis.ForLoopWithTestFunctionCall
 		for ( $linenum = 1; !feof( $file ); $linenum++ ) {
-			// @codingStandardsIgnoreEnd
 			$line = trim( fgets( $file ) );
 			if ( $line == '' ) {
 				continue;
@@ -102,7 +101,7 @@ class DeleteBatch extends Maintenance {
 
 			$this->output( $title->getPrefixedText() );
 			if ( $title->getNamespace() == NS_FILE ) {
-				$img = wfFindFile( $title, array( 'ignoreRedirect' => true ) );
+				$img = wfFindFile( $title, [ 'ignoreRedirect' => true ] );
 				if ( $img && $img->isLocal() && !$img->delete( $reason ) ) {
 					$this->output( " FAILED to delete associated file... " );
 				}