Merge "Revert "Add type hint against LinkTarget""
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Mon, 29 Feb 2016 23:58:59 +0000 (23:58 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 29 Feb 2016 23:58:59 +0000 (23:58 +0000)
1  2 
includes/cache/LinkBatch.php

@@@ -31,14 -31,14 +31,14 @@@ class LinkBatch 
        /**
         * 2-d array, first index namespace, second index dbkey, value arbitrary
         */
 -      public $data = array();
 +      public $data = [];
  
        /**
         * For debugging which method is using this class.
         */
        protected $caller;
  
 -      function __construct( $arr = array() ) {
 +      function __construct( $arr = [] ) {
                foreach ( $arr as $item ) {
                        $this->addObj( $item );
                }
        /**
         * @param LinkTarget $linkTarget
         */
-       public function addObj( LinkTarget $linkTarget ) {
-               $this->add( $linkTarget->getNamespace(), $linkTarget->getDBkey() );
+       public function addObj( $linkTarget ) {
+               if ( is_object( $linkTarget ) ) {
+                       $this->add( $linkTarget->getNamespace(), $linkTarget->getDBkey() );
+               } else {
+                       wfDebug( "Warning: LinkBatch::addObj got invalid LinkTarget object\n" );
+               }
        }
  
        /**
@@@ -71,7 -75,7 +75,7 @@@
                        return;
                }
                if ( !array_key_exists( $ns, $this->data ) ) {
 -                      $this->data[$ns] = array();
 +                      $this->data[$ns] = [];
                }
  
                $this->data[$ns][strtr( $dbkey, ' ', '_' )] = 1;
         */
        public function addResultToCache( $cache, $res ) {
                if ( !$res ) {
 -                      return array();
 +                      return [];
                }
  
                // For each returned entry, add it to the list of good links, and remove it from $remaining
  
 -              $ids = array();
 +              $ids = [];
                $remaining = $this->data;
                foreach ( $res as $row ) {
                        $title = Title::makeTitle( $row->page_namespace, $row->page_title );
         * @return bool|ResultWrapper
         */
        public function doQuery() {
 -              global $wgContentHandlerUseDB;
 +              global $wgContentHandlerUseDB, $wgPageLanguageUseDB;
  
                if ( $this->isEmpty() ) {
                        return false;
                // This is similar to LinkHolderArray::replaceInternal
                $dbr = wfGetDB( DB_SLAVE );
                $table = 'page';
 -              $fields = array( 'page_id', 'page_namespace', 'page_title', 'page_len',
 -                      'page_is_redirect', 'page_latest' );
 +              $fields = [ 'page_id', 'page_namespace', 'page_title', 'page_len',
 +                      'page_is_redirect', 'page_latest' ];
  
                if ( $wgContentHandlerUseDB ) {
                        $fields[] = 'page_content_model';
                }
 +              if ( $wgPageLanguageUseDB ) {
 +                      $fields[] = 'page_lang';
 +              }
  
                $conds = $this->constructSet( 'page', $dbr );