From 9c0ba20be8781b66959331862130099807bb4ddb Mon Sep 17 00:00:00 2001 From: Alexandre Emsenhuber Date: Sat, 7 Jan 2012 15:43:05 +0000 Subject: [PATCH] * Use WikiPage instead of Article * Use getRedirectTarget() instead of followRedirect(); the latter may return a string, which is not wanted here * No need to call updateRedirectOn(); getRedirectTarget() already populates the redirect table if needed --- maintenance/refreshLinks.php | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/maintenance/refreshLinks.php b/maintenance/refreshLinks.php index 1d33a8ead8..c16b6963be 100644 --- a/maintenance/refreshLinks.php +++ b/maintenance/refreshLinks.php @@ -184,17 +184,15 @@ class RefreshLinks extends Maintenance { __METHOD__ ); return; } - $article = new Article( $title ); - $rt = $article->followRedirect(); + $page = WikiPage::factory( $title ); + $rt = $page->getRedirectTarget(); - if ( !$rt || !is_object( $rt ) ) { + if ( $rt === null ) { // $title is not a redirect // Delete any redirect table entry for it $dbw->delete( 'redirect', array( 'rd_from' => $id ), __METHOD__ ); - } else { - $article->updateRedirectOn( $dbw, $rt ); } } -- 2.20.1