Follow up r76207. It's called $skin here.

This paramter is completely unused, though. Any reason to keep it?

diff --git a/includes/OutputPage.php b/includes/OutputPage.php
index f126655..beba6fd 100644 (file)
--- a/includes/OutputPage.php
+++ b/includes/OutputPage.php
@@ -2302,7 +2302,7 @@ class OutputPage {
         * @param $useESI boolean
         * @return string html <script> and <style> tags
         */
-       protected function makeResourceLoaderLink( $skin, $modules, $only, $useESI = false ) {
+       protected function makeResourceLoaderLink( Skin $skin, $modules, $only, $useESI = false ) {
                global $wgUser, $wgLang, $wgLoadScript, $wgResourceLoaderUseESI,
                        $wgResourceLoaderInlinePrivateModules;
                // Lazy-load ResourceLoader
@@ -2332,7 +2332,7 @@ class OutputPage {
                                // Recursively call us for every item
                                $links = '';
                                foreach ( $modules as $name ) {
-                                       $links .= $this->makeResourceLoaderLink( $sk, $name, $only, $useESI );
+                                       $links .= $this->makeResourceLoaderLink( $skin, $name, $only, $useESI );
                                }
                                return $links;
                        }