Merge "Simplify redirect linking in InfoAction"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Mon, 20 Aug 2018 01:35:01 +0000 (01:35 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 20 Aug 2018 01:35:01 +0000 (01:35 +0000)
includes/actions/InfoAction.php

index dc661aa..11b8bad 100644 (file)
@@ -233,13 +233,14 @@ class InfoAction extends FormlessAction {
                ];
 
                // Is it a redirect? If so, where to?
-               if ( $title->isRedirect() ) {
+               $redirectTarget = $this->page->getRedirectTarget();
+               if ( $redirectTarget !== null ) {
                        $pageInfo['header-basic'][] = [
                                $this->msg( 'pageinfo-redirectsto' ),
-                               $linkRenderer->makeLink( $this->page->getRedirectTarget() ) .
+                               $linkRenderer->makeLink( $redirectTarget ) .
                                $this->msg( 'word-separator' )->escaped() .
                                $this->msg( 'parentheses' )->rawParams( $linkRenderer->makeLink(
-                                       $this->page->getRedirectTarget(),
+                                       $redirectTarget,
                                        $this->msg( 'pageinfo-redirectsto-info' )->text(),
                                        [],
                                        [ 'action' => 'info' ]