Merge "Add missing imports to MediaWikiServices"

diff --git a/includes/MediaWikiServices.php b/includes/MediaWikiServices.php
index 80190bf..4b6ddd4 100644 (file)
--- a/includes/MediaWikiServices.php
+++ b/includes/MediaWikiServices.php
@@ -15,6 +15,7 @@ use SearchEngineConfig;
 use SearchEngineFactory;
 use SiteLookup;
 use SiteStore;
+use SkinFactory;
 
 /**
  * Service locator for MediaWiki core services.
@@ -185,6 +186,13 @@ class MediaWikiServices extends ServiceContainer {
                return $this->getService( 'SearchEngineConfig' );
        }
 
+       /**
+        * @return SkinFactory
+        */
+       public function getSkinFactory() {
+               return $this->getService( 'SkinFactory' );
+       }
+
        ///////////////////////////////////////////////////////////////////////////
        // NOTE: When adding a service getter here, don't forget to add a test
        // case for it in MediaWikiServicesTest::provideGetters() and in

diff --git a/includes/ServiceWiring.php b/includes/ServiceWiring.php
index defe698..d53396f 100644 (file)
--- a/includes/ServiceWiring.php
+++ b/includes/ServiceWiring.php
@@ -91,7 +91,11 @@ return [
                // Create a search engine config from main config.
                $config = $services->getService( 'MainConfig' );
                return new SearchEngineConfig( $config );
-       }
+       },
+
+       'SkinFactory' => function( MediaWikiServices $services ) {
+               return new SkinFactory();
+       },
 
        ///////////////////////////////////////////////////////////////////////////
        // NOTE: When adding a service here, don't forget to add a getter function

diff --git a/includes/revisiondelete/RevDelList.php b/includes/revisiondelete/RevDelList.php
index 26e4b17..b555592 100644 (file)
--- a/includes/revisiondelete/RevDelList.php
+++ b/includes/revisiondelete/RevDelList.php
@@ -103,7 +103,7 @@ abstract class RevDelList extends RevisionListBase {
         * @return Status
         * @since 1.23 Added 'perItemStatus' param
         */
-       public function setVisibility( $params ) {
+       public function setVisibility( array $params ) {
                $bitPars = $params['value'];
                $comment = $params['comment'];
                $perItemStatus = isset( $params['perItemStatus'] ) ? $params['perItemStatus'] : false;

diff --git a/includes/skins/SkinFactory.php b/includes/skins/SkinFactory.php
index 366862d..cc993aa 100644 (file)
--- a/includes/skins/SkinFactory.php
+++ b/includes/skins/SkinFactory.php
@@ -21,6 +21,8 @@
  * @file
  */
 
+use MediaWiki\MediaWikiServices;
+
 /**
  * Factory class to create Skin objects
  *
@@ -42,16 +44,11 @@ class SkinFactory {
        private $displayNames = [];
 
        /**
-        * @var SkinFactory
+        * @deprecated in 1.27
+        * @return SkinFactory
         */
-       private static $self;
-
        public static function getDefaultInstance() {
-               if ( !self::$self ) {
-                       self::$self = new self;
-               }
-
-               return self::$self;
+               return MediaWikiServices::getInstance()->getSkinFactory();
        }
 
        /**

diff --git a/tests/phpunit/includes/MediaWikiServicesTest.php b/tests/phpunit/includes/MediaWikiServicesTest.php
index 0741445..f5c215b 100644 (file)
--- a/tests/phpunit/includes/MediaWikiServicesTest.php
+++ b/tests/phpunit/includes/MediaWikiServicesTest.php
@@ -29,6 +29,7 @@ class MediaWikiServicesTest extends PHPUnit_Framework_TestCase {
                        'SearchEngine' => [ 'newSearchEngine', SearchEngine::class ],
                        'SearchEngineFactory' => [ 'getSearchEngineFactory', SearchEngineFactory::class ],
                        'SearchEngineConfig' => [ 'getSearchEngineConfig', SearchEngineConfig::class ],
+                       'SkinFactory' => [ 'getSkinFactory', SkinFactory::class ],
                ];
        }
 
@@ -56,6 +57,7 @@ class MediaWikiServicesTest extends PHPUnit_Framework_TestCase {
                        'EventRelayerGroup' => [ 'EventRelayerGroup', EventRelayerGroup::class ],
                        'SearchEngineFactory' => [ 'SearchEngineFactory', SearchEngineFactory::class ],
                        'SearchEngineConfig' => [ 'SearchEngineConfig', SearchEngineConfig::class ],
+                       'SkinFactory' => [ 'SkinFactory', SkinFactory::class ],
                ];
        }