Follow-up to r111091. Dont paste md5 in the code.

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

index 5d1f4bd..03ae3a4 100644 (file)
--- a/includes/SkinLegacy.php
+++ b/includes/SkinLegacy.php
@@ -84,6 +84,7 @@ class LegacyTemplate extends BaseTemplate {
         /**
          * This will be called immediately after the <body> tag.  Split into
          * two functions to make it easier to subclass.
+        * @return string
          */
         function beforeContent() {
                 return $this->doBeforeContent();
@@ -152,7 +153,9 @@ class LegacyTemplate extends BaseTemplate {
                 return $this->doAfterContent();
         }
  
-       /** overloaded by derived classes */
+       /** overloaded by derived classes
+        * @return string
+        */
         function doAfterContent() {
                 return '</div></div>';
         }
@@ -247,7 +250,7 @@ class LegacyTemplate extends BaseTemplate {
                                 }
                                 $s = $wgLang->pipeList( array(
                                         $s,
-                                       '<a href="' . $title->escapeLocalURL( 'variant=' . $code ) . '">' . htmlspecialchars( $varname ) . '</a>'
+                                       '<a href="' . htmlspecialchars( $title->getLocalURL( 'variant=' . $code ) ) . '">' . htmlspecialchars( $varname ) . '</a>'
                                 ) );
                         }
                 }
@@ -345,7 +348,7 @@ class LegacyTemplate extends BaseTemplate {
         }
  
         function otherLanguages() {
-               global $wgOut, $wgContLang, $wgHideInterlanguageLinks;
+               global $wgOut, $wgLang, $wgContLang, $wgHideInterlanguageLinks;
  
                 if ( $wgHideInterlanguageLinks ) {
                         return '';
@@ -360,8 +363,8 @@ class LegacyTemplate extends BaseTemplate {
                 $s = wfMsg( 'otherlanguages' ) . wfMsg( 'colon-separator' );
                 $first = true;
  
-               if ( $wgContLang->isRTL() ) {
-                       $s .= '<span dir="LTR">';
+               if ( $wgLang->isRTL() ) {
+                       $s .= '<span dir="ltr">';
                 }
  
                 foreach ( $a as $l ) {
@@ -379,7 +382,7 @@ class LegacyTemplate extends BaseTemplate {
                                 $text == '' ? $l : $text );
                 }
  
-               if ( $wgContLang->isRTL() ) {
+               if ( $wgLang->isRTL() ) {
                         $s .= '</span>';
                 }
  
@@ -388,6 +391,7 @@ class LegacyTemplate extends BaseTemplate {
  
         /**
          * Show a drop-down box of special pages
+        * @return string
          */
         function specialPagesList() {
                 global $wgScript;
@@ -486,7 +490,7 @@ class LegacyTemplate extends BaseTemplate {
          */
         function pageTitle() {
                 global $wgOut;
-               $s = '<h1 class="pagetitle">' . $wgOut->getPageTitle() . '</h1>';
+               $s = '<h1 class="pagetitle"><span dir="auto">' . $wgOut->getPageTitle() . '</span></h1>';
                 return $s;
         }
  
@@ -530,8 +534,10 @@ class LegacyTemplate extends BaseTemplate {
  
         /**
          * @deprecated in 1.19
+        * @return string
          */
         function getQuickbarCompensator( $rows = 1 ) {
+               wfDeprecated( __METHOD__, '1.19' );
                 return "<td width='152' rowspan='{$rows}'>&#160;</td>";
         }