public function __construct() {
parent::__construct();
- $this->mDescription = "Verify that database usernames are actually valid";
+ $this->addDescription( 'Verify that database usernames are actually valid' );
$this->setBatchSize( 1000 );
}
function execute() {
- $dbr = wfGetDB( DB_SLAVE );
+ $dbr = $this->getDB( DB_REPLICA );
$maxUserId = 0;
do {
$res = $dbr->select( 'user',
- array( 'user_id', 'user_name' ),
- array( 'user_id > ' . $maxUserId ),
+ [ 'user_id', 'user_name' ],
+ [ 'user_id > ' . $maxUserId ],
__METHOD__,
- array(
+ [
'ORDER BY' => 'user_id',
- 'LIMIT' => $this->mBatchSize,
- )
+ 'LIMIT' => $this->getBatchSize(),
+ ]
);
foreach ( $res as $row ) {
if ( !User::isValidUserName( $row->user_name ) ) {
- $this->error( sprintf( "%s: %6d: '%s'\n", wfWikiID(), $row->user_id, $row->user_name ) );
+ $this->output( sprintf( "Found: %6d: '%s'\n", $row->user_id, $row->user_name ) );
wfDebugLog( 'checkUsernames', $row->user_name );
}
}
}
}
-$maintClass = "CheckUsernames";
+$maintClass = CheckUsernames::class;
require_once RUN_MAINTENANCE_IF_MAIN;