Merge "Perform a permission check on the title when changing the page language"

[lhc/web/wiklou.git] / includes / skins / Skin.php
diff --git a/includes/skins/Skin.php b/includes/skins/Skin.php

index a8f9d0c..e1d0034 100644 (file)
--- a/includes/skins/Skin.php
+++ b/includes/skins/Skin.php
@@ -95,7 +95,7 @@ abstract class Skin extends ContextSource {
         static function normalizeKey( $key ) {
                 global $wgDefaultSkin, $wgFallbackSkin;
  
-               $skinNames = Skin::getSkinNames();
+               $skinNames = self::getSkinNames();
  
                 // Make keys lowercase for case-insensitive matching.
                 $skinNames = array_change_key_case( $skinNames, CASE_LOWER );
@@ -449,7 +449,7 @@ abstract class Skin extends ContextSource {
          * "<body>" tag, skins can override it if they have a need to add in any
          * body attributes or classes of their own.
          * @param OutputPage $out
-        * @param array $bodyAttrs
+        * @param array &$bodyAttrs
          */
         function addToBodyAttributes( $out, &$bodyAttrs ) {
                 // does nothing by default
@@ -1215,7 +1215,7 @@ abstract class Skin extends ContextSource {
         /**
          * make sure we have some title to operate on
          *
-        * @param Title $title
+        * @param Title &$title
          * @param string $name
          */
         static function checkTitle( &$title, $name ) {
@@ -1286,7 +1286,7 @@ abstract class Skin extends ContextSource {
          *
          * This is just a wrapper around addToSidebarPlain() for backwards compatibility
          *
-        * @param array $bar
+        * @param array &$bar
          * @param string $message
          */
         public function addToSidebar( &$bar, $message ) {
@@ -1296,7 +1296,7 @@ abstract class Skin extends ContextSource {
         /**
          * Add content from plain text
          * @since 1.17
-        * @param array $bar
+        * @param array &$bar
          * @param string $text
          * @return array
          */
@@ -1375,8 +1375,8 @@ abstract class Skin extends ContextSource {
                                         $bar[$heading][] = array_merge( [
                                                 'text' => $text,
                                                 'href' => $href,
-                                               'id' => 'n-' . Sanitizer::escapeId( strtr( $line[1], ' ', '-' ), 'noninitial' ),
-                                               'active' => false
+                                               'id' => Sanitizer::escapeIdForAttribute( 'n-' . strtr( $line[1], ' ', '-' ) ),
+                                               'active' => false,
                                         ], $extraAttribs );
                                 } else {
                                         continue;