Merge "SpecialLinkSearch: Use LinkRenderer instead of MediaWikiPageLinkRenderer"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Thu, 23 Jun 2016 14:28:01 +0000 (14:28 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 23 Jun 2016 14:28:01 +0000 (14:28 +0000)
includes/specials/SpecialLinkSearch.php

index 1ecbee0..d4886f0 100644 (file)
@@ -30,11 +30,6 @@ class LinkSearchPage extends QueryPage {
        /** @var array|bool */
        private $mungedQuery = false;
 
-       /**
-        * @var PageLinkRenderer
-        */
-       protected $linkRenderer = null;
-
        function setParams( $params ) {
                $this->mQuery = $params['query'];
                $this->mNs = $params['namespace'];
@@ -49,39 +44,11 @@ class LinkSearchPage extends QueryPage {
                // using the setServices() method.
        }
 
-       /**
-        * Initialize or override the PageLinkRenderer LinkSearchPage collaborates with.
-        * Useful mainly for testing.
-        *
-        * @todo query logic and rendering logic should be split and also injected
-        *
-        * @param PageLinkRenderer $linkRenderer
-        */
-       public function setPageLinkRenderer(
-               PageLinkRenderer $linkRenderer
-       ) {
-               $this->linkRenderer = $linkRenderer;
-       }
-
-       /**
-        * Initialize any services we'll need (unless it has already been provided via a setter).
-        * This allows for dependency injection even though we don't control object creation.
-        */
-       private function initServices() {
-               global $wgContLang;
-               if ( !$this->linkRenderer ) {
-                       $titleFormatter = new MediaWikiTitleCodec( $wgContLang, GenderCache::singleton() );
-                       $this->linkRenderer = new MediaWikiPageLinkRenderer( $titleFormatter );
-               }
-       }
-
        function isCacheable() {
                return false;
        }
 
        public function execute( $par ) {
-               $this->initServices();
-
                $this->setHeaders();
                $this->outputHeader();
 
@@ -277,7 +244,7 @@ class LinkSearchPage extends QueryPage {
         */
        function formatResult( $skin, $result ) {
                $title = new TitleValue( (int)$result->namespace, $result->title );
-               $pageLink = $this->linkRenderer->renderHtmlLink( $title );
+               $pageLink = $this->getLinkRenderer()->makeLink( $title );
 
                $url = $result->url;
                $urlLink = Linker::makeExternalLink( $url, $url );