resoureloader: Consolidate styles-only queue at the top
[lhc/web/wiklou.git] / includes / resourceloader / ResourceLoaderModule.php
index 17e2696..3dd7a4b 100644 (file)
@@ -67,10 +67,6 @@ abstract class ResourceLoaderModule {
        // In-object cache for module content
        protected $contents = array();
 
-       // Whether the position returned by getPosition() is defined in the module configuration
-       // and not a default value
-       protected $isPositionDefined = false;
-
        /**
         * @var Config
         */
@@ -291,19 +287,6 @@ abstract class ResourceLoaderModule {
                return 'bottom';
        }
 
-       /**
-        * Whether the position returned by getPosition() is a default value or comes from the module
-        * definition. This method is meant to be short-lived, and is only useful until classes added
-        * via addModuleStyles with a default value define an explicit position. See getModuleStyles()
-        * in OutputPage for the related migration warning.
-        *
-        * @return bool
-        * @since  1.26
-        */
-       public function isPositionDefault() {
-               return !$this->isPositionDefined;
-       }
-
        /**
         * Whether this module's JS expects to work without the client-side ResourceLoader module.
         * Returning true from this function will prevent mw.loader.state() call from being
@@ -377,34 +360,33 @@ abstract class ResourceLoaderModule {
         *
         * These are only image files referenced by the module's stylesheet.
         *
-        * @param string $skin Skin name
+        * @param ResourceLoaderContext $context
         * @return array List of files
         */
-       protected function getFileDependencies( $skin ) {
-               // Try in-object cache first
-               if ( isset( $this->fileDeps[$skin] ) ) {
-                       return $this->fileDeps[$skin];
-               }
+       protected function getFileDependencies( ResourceLoaderContext $context ) {
+               $vary = $context->getSkin() . '|' . $context->getLanguage();
 
-               $dbr = wfGetDB( DB_SLAVE );
-               $deps = $dbr->selectField( 'module_deps',
-                       'md_deps',
-                       array(
-                               'md_module' => $this->getName(),
-                               'md_skin' => $skin,
-                       ),
-                       __METHOD__
-               );
-
-               if ( !is_null( $deps ) ) {
-                       $this->fileDeps[$skin] = self::expandRelativePaths(
-                               (array)FormatJson::decode( $deps, true )
+               // Try in-object cache first
+               if ( !isset( $this->fileDeps[$vary] ) ) {
+                       $dbr = wfGetDB( DB_SLAVE );
+                       $deps = $dbr->selectField( 'module_deps',
+                               'md_deps',
+                               array(
+                                       'md_module' => $this->getName(),
+                                       'md_skin' => $vary,
+                               ),
+                               __METHOD__
                        );
-               } else {
-                       $this->fileDeps[$skin] = array();
-               }
 
-               return $this->fileDeps[$skin];
+                       if ( !is_null( $deps ) ) {
+                               $this->fileDeps[$vary] = self::expandRelativePaths(
+                                       (array)FormatJson::decode( $deps, true )
+                               );
+                       } else {
+                               $this->fileDeps[$vary] = array();
+                       }
+               }
+               return $this->fileDeps[$vary];
        }
 
        /**
@@ -416,30 +398,32 @@ abstract class ResourceLoaderModule {
         * @param string $skin Skin name
         * @param array $deps Array of file names
         */
-       public function setFileDependencies( $skin, $deps ) {
-               $this->fileDeps[$skin] = $deps;
+       public function setFileDependencies( ResourceLoaderContext $context, $files ) {
+               $vary = $context->getSkin() . '|' . $context->getLanguage();
+               $this->fileDeps[$vary] = $files;
        }
 
        /**
         * Set the files this module depends on indirectly for a given skin.
         *
-        * @since 1.26
-        * @param string $skin Skin name
+        * @since 1.27
+        * @param ResourceLoaderContext $context
         * @param array $localFileRefs List of files
         */
-       protected function saveFileDependencies( $skin, $localFileRefs ) {
+       protected function saveFileDependencies( ResourceLoaderContext $context, $localFileRefs ) {
                // Normalise array
                $localFileRefs = array_values( array_unique( $localFileRefs ) );
                sort( $localFileRefs );
 
                try {
                        // If the list has been modified since last time we cached it, update the cache
-                       if ( $localFileRefs !== $this->getFileDependencies( $skin ) ) {
+                       if ( $localFileRefs !== $this->getFileDependencies( $context ) ) {
+                               $vary = $context->getSkin() . '|' . $context->getLanguage();
                                $dbw = wfGetDB( DB_MASTER );
                                $dbw->replace( 'module_deps',
                                        array( array( 'md_module', 'md_skin' ) ), array(
                                                'md_module' => $this->getName(),
-                                               'md_skin' => $skin,
+                                               'md_skin' => $vary,
                                                // Use relative paths to avoid ghost entries when $IP changes (T111481)
                                                'md_deps' => FormatJson::encode( self::getRelativePaths( $localFileRefs ) ),
                                        )
@@ -456,11 +440,11 @@ abstract class ResourceLoaderModule {
         * This is used to make file paths safe for storing in a database without the paths
         * becoming stale or incorrect when MediaWiki is moved or upgraded (T111481).
         *
-        * @since 1.26
+        * @since 1.27
         * @param array $filePaths
         * @return array
         */
-       protected static function getRelativePaths( Array $filePaths ) {
+       public static function getRelativePaths( Array $filePaths ) {
                global $IP;
                return array_map( function ( $path ) use ( $IP ) {
                        return RelPath\getRelativePath( $path, $IP );
@@ -470,11 +454,11 @@ abstract class ResourceLoaderModule {
        /**
         * Expand directories relative to $IP.
         *
-        * @since 1.26
+        * @since 1.27
         * @param array $filePaths
         * @return array
         */
-       protected static function expandRelativePaths( Array $filePaths ) {
+       public static function expandRelativePaths( Array $filePaths ) {
                global $IP;
                return array_map( function ( $path ) use ( $IP ) {
                        return RelPath\joinPath( $IP, $path );
@@ -526,7 +510,7 @@ abstract class ResourceLoaderModule {
        /**
         * Get module-specific LESS variables, if any.
         *
-        * @since 1.26
+        * @since 1.27
         * @param ResourceLoaderContext $context
         * @return array Module-specific LESS variables.
         */
@@ -618,11 +602,11 @@ abstract class ResourceLoaderModule {
                                                                foreach ( $style as $cssText ) {
                                                                        if ( is_string( $cssText ) ) {
                                                                                $stylePairs[$media][] =
-                                                                                       $rl->filter( 'minify-css', $cssText );
+                                                                                       ResourceLoader::filter( 'minify-css', $cssText );
                                                                        }
                                                                }
                                                        } elseif ( is_string( $style ) ) {
-                                                               $stylePairs[$media] = $rl->filter( 'minify-css', $style );
+                                                               $stylePairs[$media] = ResourceLoader::filter( 'minify-css', $style );
                                                        }
                                                }
                                        }
@@ -873,14 +857,13 @@ abstract class ResourceLoaderModule {
        protected function validateScriptFile( $fileName, $contents ) {
                if ( $this->getConfig()->get( 'ResourceLoaderValidateJS' ) ) {
                        // Try for cache hit
-                       // Use CACHE_ANYTHING since parsing JS is much slower than a DB query
-                       $key = wfMemcKey(
+                       $cache = ObjectCache::getLocalClusterInstance();
+                       $key = $cache->makeKey(
                                'resourceloader',
                                'jsparse',
                                self::$parseCacheVersion,
                                md5( $contents )
                        );
-                       $cache = wfGetCache( CACHE_ANYTHING );
                        $cacheEntry = $cache->get( $key );
                        if ( is_string( $cacheEntry ) ) {
                                return $cacheEntry;