X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=maintenance%2FmigrateUserGroup.php;h=597a876df92e90519843f55ebd6bcedf575fe51d;hb=b4440a0f2ff822c1e82b809032413a0ddca2a35b;hp=dc20eee2f61e52111cbfbe374ccf059f5a7d5683;hpb=f56a905990f579a81f15ce636ab03758c15690df;p=lhc%2Fweb%2Fwiklou.git

diff --git a/maintenance/migrateUserGroup.php b/maintenance/migrateUserGroup.php
index dc20eee2f6..597a876df9 100644
--- a/maintenance/migrateUserGroup.php
+++ b/maintenance/migrateUserGroup.php
@@ -31,7 +31,7 @@ require_once __DIR__ . '/Maintenance.php';
 class MigrateUserGroup extends Maintenance {
 	public function __construct() {
 		parent::__construct();
-		$this->mDescription = "Re-assign users from an old group to a new one";
+		$this->addDescription( 'Re-assign users from an old group to a new one' );
 		$this->addArg( 'oldgroup', 'Old user group key', true );
 		$this->addArg( 'newgroup', 'New user group key', true );
 		$this->setBatchSize( 200 );
@@ -43,9 +43,9 @@ class MigrateUserGroup extends Maintenance {
 		$newGroup = $this->getArg( 1 );
 		$dbw = $this->getDB( DB_MASTER );
 		$start = $dbw->selectField( 'user_groups', 'MIN(ug_user)',
-			array( 'ug_group' => $oldGroup ), __FUNCTION__ );
+			[ 'ug_group' => $oldGroup ], __FUNCTION__ );
 		$end = $dbw->selectField( 'user_groups', 'MAX(ug_user)',
-			array( 'ug_group' => $oldGroup ), __FUNCTION__ );
+			[ 'ug_group' => $oldGroup ], __FUNCTION__ );
 		if ( $start === null ) {
 			$this->error( "Nothing to do - no users in the '$oldGroup' group", true );
 		}
@@ -60,11 +60,11 @@ class MigrateUserGroup extends Maintenance {
 
 			$this->beginTransaction( $dbw, __METHOD__ );
 			$dbw->update( 'user_groups',
-				array( 'ug_group' => $newGroup ),
-				array( 'ug_group' => $oldGroup,
-					"ug_user BETWEEN $blockStart AND $blockEnd" ),
+				[ 'ug_group' => $newGroup ],
+				[ 'ug_group' => $oldGroup,
+					"ug_user BETWEEN $blockStart AND $blockEnd" ],
 				__METHOD__,
-				array( 'IGNORE' )
+				[ 'IGNORE' ]
 			);
 			$affected += $dbw->affectedRows();
 			// Delete rows that the UPDATE operation above had to ignore.
@@ -72,20 +72,20 @@ class MigrateUserGroup extends Maintenance {
 			// Updating the row for the old group membership failed since
 			// user/group is UNIQUE.
 			$dbw->delete( 'user_groups',
-				array( 'ug_group' => $oldGroup,
-					"ug_user BETWEEN $blockStart AND $blockEnd" ),
+				[ 'ug_group' => $oldGroup,
+					"ug_user BETWEEN $blockStart AND $blockEnd" ],
 				__METHOD__
 			);
 			$affected += $dbw->affectedRows();
 			$this->commitTransaction( $dbw, __METHOD__ );
 
-			// Clear cache for the affected users (bug 40340)
+			// Clear cache for the affected users (T42340)
 			if ( $affected > 0 ) {
 				// XXX: This also invalidates cache of unaffected users that
 				// were in the new group and not in the group.
 				$res = $dbw->select( 'user_groups', 'ug_user',
-					array( 'ug_group' => $newGroup,
-						"ug_user BETWEEN $blockStart AND $blockEnd" ),
+					[ 'ug_group' => $newGroup,
+						"ug_user BETWEEN $blockStart AND $blockEnd" ],
 					__METHOD__
 				);
 				if ( $res !== false ) {