(minor) Always use getSites() to get sites.
authordaniel <daniel.kinzler@wikimedia.de>
Wed, 12 Dec 2012 14:13:23 +0000 (15:13 +0100)
committerdaniel <daniel.kinzler@wikimedia.de>
Wed, 12 Dec 2012 14:13:23 +0000 (15:13 +0100)
Change-Id: If6acf458a762a3de6096f35f231d50ba59ed3256

includes/site/Sites.php

index a0ebfc1..edd3d64 100644 (file)
@@ -184,11 +184,9 @@ class Sites {
         * @return Site|false
         */
        public function getSite( $globalId, $source = 'cache' ) {
-               if ( $source === 'cache' && $this->sites !== false ) {
-                       return $this->sites->hasSite( $globalId ) ? $this->sites->getSite( $globalId ) : false;
-               }
+               $sites = $this->getSites( $source );
 
-               return SitesTable::singleton()->selectRow( null, array( 'global_key' => $globalId ) );
+               return $sites->hasSite( $globalId ) ? $sites->getSite( $globalId ) : false;
        }
 
 }