Replace deprecated wfGetLB() calls here and there

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

index 849362a..c95f1f5 100644 (file)
--- a/includes/skins/Skin.php
+++ b/includes/skins/Skin.php
@@ -34,7 +34,11 @@ use MediaWiki\MediaWikiServices;
   * @ingroup Skins
   */
  abstract class Skin extends ContextSource {
+       /**
+        * @var string|null
+        */
         protected $skinname = null;
+
         protected $mRelevantTitle = null;
         protected $mRelevantUser = null;
  
@@ -134,7 +138,17 @@ abstract class Skin extends ContextSource {
         }
  
         /**
-        * @return string Skin name
+        * @since 1.31
+        * @param string|null $skinname
+        */
+       public function __construct( $skinname = null ) {
+               if ( is_string( $skinname ) ) {
+                       $this->skinname = $skinname;
+               }
+       }
+
+       /**
+        * @return string|null Skin name
          */
         public function getSkinName() {
                 return $this->skinname;
@@ -449,7 +463,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
@@ -897,7 +911,7 @@ abstract class Skin extends ContextSource {
                         $s = '';
                 }
  
-               if ( wfGetLB()->getLaggedReplicaMode() ) {
+               if ( MediaWikiServices::getInstance()->getDBLoadBalancer()->getLaggedReplicaMode() ) {
                         $s .= ' <strong>' . $this->msg( 'laggedslavemode' )->parse() . '</strong>';
                 }
  
@@ -1057,10 +1071,10 @@ abstract class Skin extends ContextSource {
                         $targetUser = User::newFromId( $id );
                 }
  
-               # The sending user must have a confirmed email address and the target
-               # user must have a confirmed email address and allow emails from users.
-               return $this->getUser()->canSendEmail() &&
-                       $targetUser->canReceiveEmail();
+               # The sending user must have a confirmed email address and the receiving
+               # user must accept emails from the sender.
+               return $this->getUser()->canSendEmail()
+                       && SpecialEmailUser::validateTarget( $targetUser, $this->getUser() ) === '';
         }
  
         /**
@@ -1095,7 +1109,7 @@ abstract class Skin extends ContextSource {
                 $title = Title::newMainPage();
                 self::checkTitle( $title, '' );
  
-               return $title->getLocalURL( $urlaction );
+               return $title->getLinkURL( $urlaction );
         }
  
         /**
@@ -1215,7 +1229,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 ) {
@@ -1251,11 +1265,10 @@ abstract class Skin extends ContextSource {
         function buildSidebar() {
                 global $wgEnableSidebarCache, $wgSidebarCacheExpiry;
  
-               $that = $this;
-               $callback = function () use ( $that ) {
+               $callback = function () {
                         $bar = [];
-                       $that->addToSidebar( $bar, 'sidebar' );
-                       Hooks::run( 'SkinBuildSidebar', [ $that, &$bar ] );
+                       $this->addToSidebar( $bar, 'sidebar' );
+                       Hooks::run( 'SkinBuildSidebar', [ $this, &$bar ] );
  
                         return $bar;
                 };
@@ -1286,7 +1299,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 +1309,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
          */
@@ -1579,8 +1592,6 @@ abstract class Skin extends ContextSource {
  
                 $attribs = [];
                 if ( !is_null( $tooltip ) ) {
-                       # T27462: undo double-escaping.
-                       $tooltip = Sanitizer::decodeCharReferences( $tooltip );
                         $attribs['title'] = wfMessage( 'editsectionhint' )->rawParams( $tooltip )
                                 ->inLanguage( $lang )->text();
                 }
@@ -1612,7 +1623,7 @@ abstract class Skin extends ContextSource {
  
                 $result .= implode(
                         '<span class="mw-editsection-divider">'
-                               . wfMessage( 'pipe-separator' )->inLanguage( $lang )->text()
+                               . wfMessage( 'pipe-separator' )->inLanguage( $lang )->escaped()
                                 . '</span>',
                         $linksHtml
                 );