Code Coverage
 
Lines
Functions and Methods
Classes and Traits
Total
0.00% covered (danger)
0.00%
0 / 40
0.00% covered (danger)
0.00%
0 / 3
CRAP
0.00% covered (danger)
0.00%
0 / 1
FixInconsistentRedirects
0.00% covered (danger)
0.00%
0 / 37
0.00% covered (danger)
0.00%
0 / 3
110
0.00% covered (danger)
0.00%
0 / 1
 __construct
0.00% covered (danger)
0.00%
0 / 3
0.00% covered (danger)
0.00%
0 / 1
2
 getUpdateKey
0.00% covered (danger)
0.00%
0 / 1
0.00% covered (danger)
0.00%
0 / 1
2
 doDBUpdates
0.00% covered (danger)
0.00%
0 / 33
0.00% covered (danger)
0.00%
0 / 1
72
1<?php
2/**
3 * This program is free software; you can redistribute it and/or modify
4 * it under the terms of the GNU General Public License as published by
5 * the Free Software Foundation; either version 2 of the License, or
6 * (at your option) any later version.
7 *
8 * This program is distributed in the hope that it will be useful,
9 * but WITHOUT ANY WARRANTY; without even the implied warranty of
10 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 * GNU General Public License for more details.
12 *
13 * You should have received a copy of the GNU General Public License along
14 * with this program; if not, write to the Free Software Foundation, Inc.,
15 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
16 * http://www.gnu.org/copyleft/gpl.html
17 *
18 * @file
19 */
20
21require_once __DIR__ . '/Maintenance.php';
22
23/**
24 * Fix redirect pages with missing or incomplete row in the redirect table.
25 *
26 * @ingroup Maintenance
27 * @since 1.41
28 */
29class FixInconsistentRedirects extends LoggedUpdateMaintenance {
30
31    public function __construct() {
32        parent::__construct();
33        $this->addDescription( 'Fix redirect pages with missing or incomplete row in the redirect table' );
34        $this->setBatchSize( 100 );
35    }
36
37    protected function getUpdateKey() {
38        return __CLASS__;
39    }
40
41    protected function doDBUpdates() {
42        $dbr = $this->getReplicaDB();
43
44        $builder = $dbr->newSelectQueryBuilder()
45            ->caller( __METHOD__ )
46            ->from( 'page' )
47            ->where( [ 'page_is_redirect' => 1 ] );
48
49        $this->output( "Fixing inconsistent redirects ...\n" );
50
51        $estimateCount = $builder->estimateRowCount();
52        $this->output( "Estimated redirect page count: $estimateCount\n" );
53
54        $builder
55            ->limit( $this->getBatchSize() )
56            ->leftJoin( 'redirect', null, 'page_id=rd_from' )
57            ->select( [ 'rd_from', 'rd_interwiki', 'rd_fragment' ] );
58
59        // Using the page_redirect_namespace_len index to skip non-redirects
60        $index = [ 'page_is_redirect', 'page_namespace', 'page_len', 'page_id' ];
61        $builder->select( $index )->orderBy( $index );
62        $prevRow = [];
63
64        $total = 0;
65        $updated = 0;
66        do {
67            $res = ( clone $builder )
68                ->where( $prevRow ? [ $dbr->buildComparison( '>', $prevRow ) ] : [] )
69                ->caller( __METHOD__ )->fetchResultSet();
70
71            foreach ( $res as $row ) {
72                // Only attempt write queries if the row or rd_interwiki/rd_fragment fields are missing
73                // (we don't include this condition in the query to avoid slow queries and bad estimates)
74                if ( $row->rd_from === null || $row->rd_interwiki === null || $row->rd_fragment === null ) {
75                    RefreshLinks::fixRedirect( $this, $row->page_id );
76                    $updated++;
77                }
78            }
79            if ( isset( $row ) ) {
80                // Update the conditions to select the next batch
81                foreach ( $index as $field ) {
82                    $prevRow[ $field ] = $row->$field;
83                }
84            }
85
86            $this->waitForReplication();
87            $total += $res->numRows();
88            $this->output( "$updated/$total\n" );
89
90        } while ( $res->numRows() == $this->getBatchSize() );
91
92        $this->output( "Done, updated $updated of $total rows.\n" );
93        return true;
94    }
95}
96
97$maintClass = FixInconsistentRedirects::class;
98require_once RUN_MAINTENANCE_IF_MAIN;