X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=maintenance%2FmigrateUserGroup.php;h=597a876df92e90519843f55ebd6bcedf575fe51d;hb=252fa957ad2442d25ce2d5b7f06ecfd71da2c1f9;hp=df6665a9690bb21663aec29760d3b6c3e41be669;hpb=5a4fdb5b4e82fd68e200a887cac9d4e106fed0af;p=lhc%2Fweb%2Fwiklou.git diff --git a/maintenance/migrateUserGroup.php b/maintenance/migrateUserGroup.php index df6665a969..597a876df9 100644 --- a/maintenance/migrateUserGroup.php +++ b/maintenance/migrateUserGroup.php @@ -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 ) {