class FixDoubleRedirects extends Maintenance {
public function __construct() {
parent::__construct();
- $this->mDescription = "Script to fix double redirects";
+ $this->addDescription( 'Script to fix double redirects' );
$this->addOption( 'async', 'Don\'t fix anything directly, just queue the jobs' );
$this->addOption( 'title', 'Fix only redirects pointing to this page', false, true );
$this->addOption( 'dry-run', 'Perform a dry run, fix nothing' );
$title = null;
}
- $dbr = $this->getDB( DB_SLAVE );
+ $dbr = $this->getDB( DB_REPLICA );
// See also SpecialDoubleRedirects
- $tables = array(
+ $tables = [
'redirect',
'pa' => 'page',
'pb' => 'page',
- );
- $fields = array(
+ ];
+ $fields = [
'pa.page_namespace AS pa_namespace',
'pa.page_title AS pa_title',
'pb.page_namespace AS pb_namespace',
'pb.page_title AS pb_title',
- );
- $conds = array(
+ ];
+ $conds = [
'rd_from = pa.page_id',
'rd_namespace = pb.page_namespace',
'rd_title = pb.page_title',
'rd_interwiki IS NULL OR rd_interwiki = ' . $dbr->addQuotes( '' ), // bug 40352
'pb.page_is_redirect' => 1,
- );
+ ];
if ( $title != null ) {
$conds['pb.page_namespace'] = $title->getNamespace();
return;
}
- $jobs = array();
+ $jobs = [];
$processedTitles = "\n";
$n = 0;
foreach ( $res as $row ) {
$processedTitles .= "* [[$titleA]]\n";
- $job = new DoubleRedirectJob( $titleA, array(
+ $job = new DoubleRedirectJob( $titleA, [
'reason' => 'maintenance',
'redirTitle' => $titleB->getPrefixedDBkey()
- ) );
+ ] );
if ( !$async ) {
$success = ( $dryrun ? true : $job->run() );
// @todo FIXME: Hardcoded constant 10000 copied from DoubleRedirectJob class
if ( count( $jobs ) > 10000 ) {
$this->queueJobs( $jobs, $dryrun );
- $jobs = array();
+ $jobs = [];
}
}
protected function queueJobs( $jobs, $dryrun = false ) {
$this->output( "Queuing batch of " . count( $jobs ) . " double redirects.\n" );
- JobQueueGroup::singleton()->push( $dryrun ? array() : $jobs );
+ JobQueueGroup::singleton()->push( $dryrun ? [] : $jobs );
}
}