* Fix for r60163: in RC/RCL, hash together all the options, not just namespace, in...
[lhc/web/wiklou.git] / includes / Skin.php
index 6ff984b..d1a0016 100644 (file)
@@ -3,8 +3,9 @@
  * @defgroup Skins Skins
  */
 
-if ( ! defined( 'MEDIAWIKI' ) )
+if ( !defined( 'MEDIAWIKI' ) ) {
        die( 1 );
+}
 
 /**
  * The main skin class that provide methods and properties for all other skins.
@@ -24,16 +25,17 @@ class Skin extends Linker {
        /**#@-*/
        protected $mRevisionId; // The revision ID we're looking at, null if not applicable.
        protected $skinname = 'standard';
-       // @fixme Should be protected :-\
+       // @todo Fixme: should be protected :-\
        var $mTitle = null;
 
        /** Constructor, call parent constructor */
-       function Skin() { parent::__construct(); }
+       function __construct() {
+               parent::__construct();
+       }
 
        /**
         * Fetch the set of available skins.
         * @return array of strings
-        * @static
         */
        static function getSkinNames() {
                global $wgValidSkinNames;
@@ -53,7 +55,7 @@ class Skin extends Linker {
                                $matches = array();
                                if( preg_match( '/^([^.]*)\.php$/', $file, $matches ) ) {
                                        $aSkin = $matches[1];
-                                       $wgValidSkinNames[strtolower($aSkin)] = $aSkin;
+                                       $wgValidSkinNames[strtolower( $aSkin )] = $aSkin;
                                }
                        }
                        $skinDir->close();
@@ -62,7 +64,7 @@ class Skin extends Linker {
                }
                return $wgValidSkinNames;
        }
-       
+
        /**
         * Fetch the list of usable skins in regards to $wgSkipSkins.
         * Useful for Special:Preferences and other places where you
@@ -82,9 +84,8 @@ class Skin extends Linker {
         * Normalize a skin preference value to a form that can be loaded.
         * If a skin can't be found, it will fall back to the configured
         * default (or the old 'Classic' skin if that's broken).
-        * @param string $key
+        * @param $key String: 'monobook', 'standard', etc.
         * @return string
-        * @static
         */
        static function normalizeKey( $key ) {
                global $wgDefaultSkin;
@@ -105,9 +106,10 @@ class Skin extends Linker {
                $fallback = array(
                        0 => $wgDefaultSkin,
                        1 => 'nostalgia',
-                       2 => 'cologneblue' );
+                       2 => 'cologneblue'
+               );
 
-               if( isset( $fallback[$key] ) ){
+               if( isset( $fallback[$key] ) ) {
                        $key = $fallback[$key];
                }
 
@@ -120,9 +122,8 @@ class Skin extends Linker {
 
        /**
         * Factory method for loading a skin of a given type
-        * @param string $key 'monobook', 'standard', etc
+        * @param $key String: 'monobook', 'standard', etc.
         * @return Skin
-        * @static
         */
        static function &newFromKey( $key ) {
                global $wgStyleDirectory;
@@ -131,13 +132,15 @@ class Skin extends Linker {
 
                $skinNames = Skin::getSkinNames();
                $skinName = $skinNames[$key];
-               $className = 'Skin'.ucfirst($key);
+               $className = 'Skin' . ucfirst( $key );
 
                # Grab the skin class and initialise it.
                if ( !class_exists( $className ) ) {
                        // Preload base classes to work around APC/PHP5 bug
                        $deps = "{$wgStyleDirectory}/{$skinName}.deps.php";
-                       if( file_exists( $deps ) ) include_once( $deps );
+                       if( file_exists( $deps ) ) {
+                               include_once( $deps );
+                       }
                        require_once( "{$wgStyleDirectory}/{$skinName}.php" );
 
                        # Check if we got if not failback to default skin
@@ -168,7 +171,9 @@ class Skin extends Linker {
        function qbSetting() {
                global $wgOut, $wgUser;
 
-               if ( $wgOut->isQuickbarSuppressed() ) { return 0; }
+               if ( $wgOut->isQuickbarSuppressed() ) {
+                       return 0;
+               }
                $q = $wgUser->getOption( 'quickbar', 0 );
                return $q;
        }
@@ -180,7 +185,7 @@ class Skin extends Linker {
 
                # Generally the order of the favicon and apple-touch-icon links
                # should not matter, but Konqueror (3.5.9 at least) incorrectly
-               # uses whichever one appears later in the HTML source.  Make sure
+               # uses whichever one appears later in the HTML source. Make sure
                # apple-touch-icon is specified first to avoid this.
                if( false !== $wgAppleTouchIcon ) {
                        $out->addLink( array( 'rel' => 'apple-touch-icon', 'href' => $wgAppleTouchIcon ) );
@@ -229,6 +234,11 @@ class Skin extends Linker {
                $lb->execute();
        }
 
+       /**
+        * Adds metadata links (Creative Commons/Dublin Core/copyright) to the HTML
+        * output.
+        * @param $out Object: instance of OutputPage
+        */
        function addMetadataLinks( OutputPage $out ) {
                global $wgEnableDublinCoreRdf, $wgEnableCreativeCommonsRdf;
                global $wgRightsPage, $wgRightsUrl;
@@ -271,13 +281,13 @@ class Skin extends Linker {
        /**
         * Set some local variables
         */
-       protected function setMembers(){
+       protected function setMembers() {
                global $wgUser;
                $this->mUser = $wgUser;
                $this->userpage = $wgUser->getUserPage()->getPrefixedText();
                $this->usercss = false;
        }
-       
+
        /**
         * Set the title
         * @param Title $t The title to use
@@ -285,12 +295,16 @@ class Skin extends Linker {
        public function setTitle( $t ) {
                $this->mTitle = $t;
        }
-       
+
        /** Get the title */
        public function getTitle() {
                return $this->mTitle;
        }
 
+       /**
+        * Outputs the HTML generated by other functions.
+        * @param $out Object: instance of OutputPage
+        */
        function outputPage( OutputPage $out ) {
                global $wgDebugComments;
                wfProfileIn( __METHOD__ );
@@ -303,12 +317,6 @@ class Skin extends Linker {
 
                $out->out( $out->headElement( $this ) );
 
-               $out->out( "\n<body" );
-               $ops = $this->getBodyOptions();
-               foreach ( $ops as $name => $val ) {
-                       $out->out( " $name='$val'" );
-               }
-               $out->out( ">\n" );
                if ( $wgDebugComments ) {
                        $out->out( "<!-- Wiki debugging output:\n" .
                          $out->mDebugtext . "-->\n" );
@@ -319,7 +327,7 @@ class Skin extends Linker {
                $out->out( $out->mBodytext . "\n" );
 
                $out->out( $this->afterContent() );
-               
+
                $out->out( $afterContent );
 
                $out->out( $this->bottomScripts() );
@@ -331,44 +339,42 @@ class Skin extends Linker {
        }
 
        static function makeVariablesScript( $data ) {
-               global $wgJsMimeType;
-
-               $doneFirstVar = false;
-               $r = array( "<script type=\"$wgJsMimeType\">/*<![CDATA[*/\n" );
-               foreach ( $data as $name => $value ) {
-                       $encValue = Xml::encodeJsVar( $value );
-                       if ( $doneFirstVar )
-                               $r[] = ",\n$name=$encValue";
-                       else {
-                               $r[] = "var $name=$encValue";
-                               $doneFirstVar = true;
-                       }
+               if( $data ) {
+                       $r = array();
+                       foreach ( $data as $name => $value ) {
+                               $encValue = Xml::encodeJsVar( $value );
+                               $r[] = "$name=$encValue";
+                       }
+                       $js = 'var ' . implode( ",\n", $r ) . ';';
+                       return Html::inlineScript( "\n$js\n" );
+               } else {
+                       return '';
                }
-               # No need for ; since the script is terminating
-               $r[] = "\n/*]]>*/</script>\n";
-
-               return implode( $r );
        }
 
        /**
         * Make a <script> tag containing global variables
-        * @param array $data Associative array containing one element:
-        *     skinname => the skin name
+        * @param $skinName string Name of the skin
         * The odd calling convention is for backwards compatibility
         * @TODO @FIXME Make this not depend on $wgTitle!
         */
-       static function makeGlobalVariablesScript( $data ) {
-               global $wgScript, $wgTitle, $wgStylePath, $wgUser;
-               global $wgArticlePath, $wgScriptPath, $wgServer, $wgContLang, $wgLang, $wgVariant;
-               global $wgCanonicalNamespaceNames, $wgOut, $wgArticle;
+       static function makeGlobalVariablesScript( $skinName ) {
+               if ( is_array( $skinName ) ) {
+                       # Weird back-compat stuff.
+                       $skinName = $skinName['skinname'];
+               }
+               global $wgScript, $wgTitle, $wgStylePath, $wgUser, $wgScriptExtension;
+               global $wgArticlePath, $wgScriptPath, $wgServer, $wgContLang, $wgLang;
+               global $wgOut, $wgArticle;
                global $wgBreakFrames, $wgRequest, $wgVariantArticlePath, $wgActionPaths;
                global $wgUseAjax, $wgAjaxWatch;
                global $wgVersion, $wgEnableAPI, $wgEnableWriteAPI;
-               global $wgRestrictionTypes, $wgLivePreview;
+               global $wgRestrictionTypes;
                global $wgMWSuggestTemplate, $wgDBname, $wgEnableMWSuggest;
+               global $wgSitename;
 
                $ns = $wgTitle->getNamespace();
-               $nsname = isset( $wgCanonicalNamespaceNames[ $ns ] ) ? $wgCanonicalNamespaceNames[ $ns ] : $wgTitle->getNsText();
+               $nsname = MWNamespace::exists( $ns ) ? MWNamespace::getCanonicalName( $ns ) : $wgTitle->getNsText();
                $separatorTransTable = $wgContLang->separatorTransformTable();
                $separatorTransTable = $separatorTransTable ? $separatorTransTable : array();
                $compactSeparatorTransTable = array(
@@ -382,26 +388,29 @@ class Skin extends Linker {
                        implode( "\t", $digitTransTable ),
                );
 
+               $mainPage = Title::newFromText( wfMsgForContent( 'mainpage' ) );
                $vars = array(
-                       'skin' => $data['skinname'],
+                       'skin' => $skinName,
                        'stylepath' => $wgStylePath,
+                       'wgUrlProtocols' => wfUrlProtocols(),
                        'wgArticlePath' => $wgArticlePath,
                        'wgScriptPath' => $wgScriptPath,
+                       'wgScriptExtension' => $wgScriptExtension,
                        'wgScript' => $wgScript,
                        'wgVariantArticlePath' => $wgVariantArticlePath,
                        'wgActionPaths' => (object)$wgActionPaths,
                        'wgServer' => $wgServer,
                        'wgCanonicalNamespace' => $nsname,
-                       'wgCanonicalSpecialPageName' => SpecialPage::resolveAlias( $wgTitle->getDBkey() ),
+                       'wgCanonicalSpecialPageName' => $ns == NS_SPECIAL ?
+                               SpecialPage::resolveAlias( $wgTitle->getDBkey() ) : false, # bug 21115
                        'wgNamespaceNumber' => $wgTitle->getNamespace(),
                        'wgPageName' => $wgTitle->getPrefixedDBKey(),
                        'wgTitle' => $wgTitle->getText(),
                        'wgAction' => $wgRequest->getText( 'action', 'view' ),
                        'wgArticleId' => $wgTitle->getArticleId(),
                        'wgIsArticle' => $wgOut->isArticle(),
-                       'wgUserName' => $wgUser->isAnon() ? NULL : $wgUser->getName(),
-                       'wgUserGroups' => $wgUser->isAnon() ? NULL : $wgUser->getEffectiveGroups(),
-                       'wgUserVariant' => $wgVariant->getCode(),
+                       'wgUserName' => $wgUser->isAnon() ? null : $wgUser->getName(),
+                       'wgUserGroups' => $wgUser->isAnon() ? null : $wgUser->getEffectiveGroups(),
                        'wgUserLanguage' => $wgLang->getCode(),
                        'wgContentLanguage' => $wgContLang->getCode(),
                        'wgBreakFrames' => $wgBreakFrames,
@@ -411,31 +420,37 @@ class Skin extends Linker {
                        'wgEnableWriteAPI' => $wgEnableWriteAPI,
                        'wgSeparatorTransformTable' => $compactSeparatorTransTable,
                        'wgDigitTransformTable' => $compactDigitTransTable,
+                       'wgMainPageTitle' => $mainPage ? $mainPage->getPrefixedText() : null,
+                       'wgFormattedNamespaces' => $wgContLang->getFormattedNamespaces(),
+                       'wgNamespaceIds' => $wgContLang->getNamespaceIds(),
+                       'wgSiteName' => $wgSitename,
+                       'wgCategories' => $wgOut->getCategories(),
                );
-               if ( !( $wgContLang->hasVariants() ) ) {
-                       unset( $vars['wgUserVariant'] );
+               if ( $wgContLang->hasVariants() ) {
+                       $vars['wgUserVariant'] = $wgContLang->getPreferredVariant();
+               }
+
+               // if on upload page output the extension list & js_upload
+               if( SpecialPage::resolveAlias( $wgTitle->getDBkey() ) == 'Upload' ) {
+                       global $wgFileExtensions, $wgAjaxUploadInterface;
+                       $vars['wgFileExtensions'] = $wgFileExtensions;
                }
-               
-               if( $wgUseAjax && $wgEnableMWSuggest && !$wgUser->getOption( 'disablesuggest', false ) ){
+
+               if( $wgUseAjax && $wgEnableMWSuggest && !$wgUser->getOption( 'disablesuggest', false ) ) {
                        $vars['wgMWSuggestTemplate'] = SearchEngine::getMWSuggestTemplate();
                        $vars['wgDBname'] = $wgDBname;
                        $vars['wgSearchNamespaces'] = SearchEngine::userNamespaces( $wgUser );
                        $vars['wgMWSuggestMessages'] = array( wfMsg( 'search-mwsuggest-enabled' ), wfMsg( 'search-mwsuggest-disabled' ) );
                }
 
-               foreach( $wgRestrictionTypes as $type )
+               foreach( $wgRestrictionTypes as $type ) {
                        $vars['wgRestriction' . ucfirst( $type )] = $wgTitle->getRestrictions( $type );
-
-               if ( $wgLivePreview && $wgUser->getOption( 'uselivepreview' ) ) {
-                       $vars['wgLivepreviewMessageLoading'] = wfMsg( 'livepreview-loading' );
-                       $vars['wgLivepreviewMessageReady']   = wfMsg( 'livepreview-ready' );
-                       $vars['wgLivepreviewMessageFailed']  = wfMsg( 'livepreview-failed' );
-                       $vars['wgLivepreviewMessageError']   = wfMsg( 'livepreview-error' );
                }
 
                if ( $wgOut->isArticleRelated() && $wgUseAjax && $wgAjaxWatch && $wgUser->isLoggedIn() ) {
                        $msgs = (object)array();
-                       foreach ( array( 'watch', 'unwatch', 'watching', 'unwatching' ) as $msgName ) {
+                       foreach ( array( 'watch', 'unwatch', 'watching', 'unwatching',
+                               'tooltip-ca-watch', 'tooltip-ca-unwatch' ) as $msgName ) {
                                $msgs->{$msgName . 'Msg'} = wfMsg( $msgName );
                        }
                        $vars['wgAjaxWatch'] = $msgs;
@@ -447,31 +462,6 @@ class Skin extends Linker {
                return self::makeVariablesScript( $vars );
        }
 
-       function getHeadScripts( $allowUserJs ) {
-               global $wgStylePath, $wgUser, $wgJsMimeType, $wgStyleVersion;
-
-               $vars = self::makeGlobalVariablesScript( array( 'skinname' => $this->getSkinName() ) );
-
-               $r = array( "<script type=\"{$wgJsMimeType}\" src=\"{$wgStylePath}/common/wikibits.js?$wgStyleVersion\"></script>" );
-               global $wgUseSiteJs;
-               if( $wgUseSiteJs ) {
-                       $jsCache = $wgUser->isLoggedIn() ? '&smaxage=0' : '';
-                       $r[] = "<script type=\"$wgJsMimeType\" src=\"".
-                               htmlspecialchars( self::makeUrl( '-',
-                                       "action=raw$jsCache&gen=js&useskin=" .
-                                       urlencode( $this->getSkinName() ) ) ) .
-                               "\"><!-- site js --></script>";
-               }
-               if( $allowUserJs && $wgUser->isLoggedIn() ) {
-                       $userpage = $wgUser->getUserPage();
-                       $userjs = htmlspecialchars( self::makeUrl(
-                               $userpage->getPrefixedText().'/'.$this->getSkinName().'.js',
-                               'action=raw&ctype='.$wgJsMimeType ) );
-                       $r[] = '<script type="'.$wgJsMimeType.'" src="'.$userjs."\"></script>";
-               }
-               return $vars . "\t\t" . implode ( "\n\t\t", $r );
-       }
-
        /**
         * To make it harder for someone to slip a user a fake
         * user-JavaScript or user-CSS preview, a random token
@@ -479,25 +469,30 @@ class Skin extends Linker {
         * passed back with the preview request, we won't render
         * the code.
         *
-        * @param string $action
+        * @param $action String: 'edit', 'submit' etc.
         * @return bool
-        * @private
         */
-       function userCanPreview( $action ) {
+       public function userCanPreview( $action ) {
                global $wgRequest, $wgUser;
 
-               if( $action != 'submit' )
+               if( $action != 'submit' ) {
                        return false;
-               if( !$wgRequest->wasPosted() )
+               }
+               if( !$wgRequest->wasPosted() ) {
                        return false;
-               if( !$this->mTitle->userCanEditCssJsSubpage() )
+               }
+               if( !$this->mTitle->userCanEditCssSubpage() ) {
                        return false;
+               }
+               if( !$this->mTitle->userCanEditJsSubpage() ) {
+                       return false;
+               }
                return $wgUser->matchEditToken(
                        $wgRequest->getVal( 'wpEditToken' ) );
        }
 
        /**
-        * generated JavaScript action=raw&gen=js
+        * Generated JavaScript action=raw&gen=js
         * This returns MediaWiki:Common.js and MediaWiki:[Skinname].js concate-
         * nated together.  For some bizarre reason, it does *not* return any
         * custom user JS from subpages.  Huh?
@@ -507,27 +502,31 @@ class Skin extends Linker {
         * top.  For now Monobook.js will be maintained, but it should be consi-
         * dered deprecated.
         *
+        * @param $skinName String: If set, overrides the skin name
         * @return string
         */
-       public function generateUserJs() {
+       public function generateUserJs( $skinName = null ) {
                global $wgStylePath;
 
                wfProfileIn( __METHOD__ );
+               if( !$skinName ) {
+                       $skinName = $this->getSkinName();
+               }
 
                $s = "/* generated javascript */\n";
-               $s .= "var skin = '" . Xml::escapeJsString( $this->getSkinName() ) . "';\n";
+               $s .= "var skin = '" . Xml::escapeJsString( $skinName ) . "';\n";
                $s .= "var stylepath = '" . Xml::escapeJsString( $wgStylePath ) . "';";
                $s .= "\n\n/* MediaWiki:Common.js */\n";
-               $commonJs = wfMsgForContent( 'common.js' );
+               $commonJs = wfMsgExt( 'common.js', 'content' );
                if ( !wfEmptyMsg( 'common.js', $commonJs ) ) {
                        $s .= $commonJs;
                }
 
-               $s .= "\n\n/* MediaWiki:".ucfirst( $this->getSkinName() ).".js */\n";
+               $s .= "\n\n/* MediaWiki:" . ucfirst( $skinName ) . ".js */\n";
                // avoid inclusion of non defined user JavaScript (with custom skins only)
                // by checking for default message content
-               $msgKey = ucfirst( $this->getSkinName() ).'.js';
-               $userJS = wfMsgForContent( $msgKey );
+               $msgKey = ucfirst( $skinName ) . '.js';
+               $userJS = wfMsgExt( $msgKey, 'content' );
                if ( !wfEmptyMsg( $msgKey, $userJS ) ) {
                        $s .= $userJS;
                }
@@ -546,11 +545,11 @@ class Skin extends Linker {
                wfProfileOut( __METHOD__ );
                return $s;
        }
-       
+
        /**
         * Split for easier subclassing in SkinSimple, SkinStandard and SkinCologneBlue
         */
-       protected function reallyGenerateUserStylesheet(){
+       protected function reallyGenerateUserStylesheet() {
                global $wgUser;
                $s = '';
                if( ( $undopt = $wgUser->getOption( 'underline' ) ) < 2 ) {
@@ -560,7 +559,7 @@ class Skin extends Linker {
                if( $wgUser->getOption( 'highlightbroken' ) ) {
                        $s .= "a.new, #quickbar a.new { color: #CC2200; }\n";
                } else {
-                       $s .= <<<END
+                       $s .= <<<CSS
 a.new, #quickbar a.new,
 a.stub, #quickbar a.stub {
        color: inherit;
@@ -573,7 +572,7 @@ a.stub:after, #quickbar a.stub:after {
        content: "!";
        color: #772233;
 }
-END;
+CSS;
                }
                if( $wgUser->getOption( 'justify' ) ) {
                        $s .= "#article, #bodyContent, #mw_content { text-align: justify; }\n";
@@ -584,6 +583,10 @@ END;
                if( !$wgUser->getOption( 'editsection' ) ) {
                        $s .= ".editsection { display: none; }\n";
                }
+               $fontstyle = $wgUser->getOption( 'editfont' );
+               if ( $fontstyle !== 'default' ) {
+                       $s .= "textarea { font-family: $fontstyle; }\n";
+               }
                return $s;
        }
 
@@ -604,8 +607,8 @@ END;
                );
 
                // Add any extension CSS
-               foreach( $out->getExtStyle() as $tag ) {
-                       $out->addStyle( $tag['href'] );
+               foreach ( $out->getExtStyle() as $url ) {
+                       $out->addStyle( $url );
                }
 
                // If we use the site's dynamic CSS, throw that in, too
@@ -644,12 +647,13 @@ END;
                        $action = $wgRequest->getVal( 'action' );
                        # If we're previewing the CSS page, use it
                        if( $this->mTitle->isCssSubpage() && $this->userCanPreview( $action ) ) {
-                               $previewCss = $wgRequest->getText( 'wpTextbox1' );
                                // @FIXME: properly escape the cdata!
-                               $this->usercss = "/*<![CDATA[*/\n" . $previewCss . "/*]]>*/";
+                               $out->addInlineStyle( $wgRequest->getText( 'wpTextbox1' ) );
                        } else {
-                               $out->addStyle( self::makeUrl( $this->userpage . '/' . $this->getSkinName() .'.css',
-                                       'action=raw&ctype=text/css' ) );
+                               $out->addStyle( self::makeUrl(
+                                       $this->userpage . '/' . $this->getSkinName() . '.css',
+                                       'action=raw&ctype=text/css' )
+                               );
                        }
                }
 
@@ -667,32 +671,8 @@ END;
                $out->addStyle( 'common/common_rtl.css', '', '', 'rtl' );
        }
 
-       function getBodyOptions() {
-               global $wgUser, $wgOut, $wgRequest, $wgContLang;
-
-               extract( $wgRequest->getValues( 'oldid', 'redirect', 'diff' ) );
-
-               if ( 0 != $this->mTitle->getNamespace() ) {
-                       $a = array( 'bgcolor' => '#ffffec' );
-               }
-               else $a = array( 'bgcolor' => '#FFFFFF' );
-               if( $wgOut->isArticle() && $wgUser->getOption( 'editondblclick' ) &&
-                 $this->mTitle->quickUserCan( 'edit' ) ) {
-                       $s = $this->mTitle->getFullURL( $this->editUrlOptions() );
-                       $s = 'document.location = "' .Xml::escapeJsString( $s ) .'";';
-                       $a += array( 'ondblclick' => $s );
-               }
-               $a['onload'] = $wgOut->getOnloadHandler();
-               $a['class'] =
-                       'mediawiki' .
-                       ' '.( $wgContLang->isRTL() ? 'rtl' : 'ltr' ).
-                       ' '.$this->getPageClasses( $this->mTitle ) .
-                       ' skin-'. Sanitizer::escapeClass( $this->getSkinName() );
-               return $a;
-       }
-       
        function getPageClasses( $title ) {
-               $numeric = 'ns-'.$title->getNamespace();
+               $numeric = 'ns-' . $title->getNamespace();
                if( $title->getNamespace() == NS_SPECIAL ) {
                        $type = 'ns-special';
                } elseif( $title->isTalkPage() ) {
@@ -700,7 +680,7 @@ END;
                } else {
                        $type = 'ns-subject';
                }
-               $name = Sanitizer::escapeClass( 'page-'.$title->getPrefixedText() );
+               $name = Sanitizer::escapeClass( 'page-' . $title->getPrefixedText() );
                return "$numeric $type $name";
        }
 
@@ -727,7 +707,8 @@ END;
                $s = '';
                $qb = $this->qbSetting();
 
-               if( $langlinks = $this->otherLanguages() ) {
+               $langlinks = $this->otherLanguages();
+               if( $langlinks ) {
                        $rows = 2;
                        $borderhack = '';
                } else {
@@ -741,24 +722,26 @@ END;
 
                $shove = ( $qb != 0 );
                $left = ( $qb == 1 || $qb == 3 );
-               if( $wgContLang->isRTL() ) $left = !$left;
+               if( $wgContLang->isRTL() ) {
+                       $left = !$left;
+               }
 
                if( !$shove ) {
                        $s .= "<td class='top' align='left' valign='top' rowspan='{$rows}'>\n" .
-                         $this->logoText() . '</td>';
+                               $this->logoText() . '</td>';
                } elseif( $left ) {
                        $s .= $this->getQuickbarCompensator( $rows );
                }
-               $l = $wgContLang->isRTL() ? 'right' : 'left';
+               $l = $wgContLang->alignStart();
                $s .= "<td {$borderhack} align='$l' valign='top'>\n";
 
                $s .= $this->topLinks();
-               $s .= "<p class='subtitle'>" . $this->pageTitleLinks() . "</p>\n";
+               $s .= '<p class="subtitle">' . $this->pageTitleLinks() . "</p>\n";
 
-               $r = $wgContLang->isRTL() ? 'left' : 'right';
+               $r = $wgContLang->alignEnd();
                $s .= "</td>\n<td {$borderhack} valign='top' align='$r' nowrap='nowrap'>";
                $s .= $this->nameAndLogin();
-               $s .= "\n<br />" . $this->searchForm() . "</td>";
+               $s .= "\n<br />" . $this->searchForm() . '</td>';
 
                if ( $langlinks ) {
                        $s .= "</tr>\n<tr>\n<td class='top' colspan=\"2\">$langlinks</td>\n";
@@ -781,19 +764,20 @@ END;
                return $s;
        }
 
-
        function getCategoryLinks() {
                global $wgOut, $wgUseCategoryBrowser;
                global $wgContLang, $wgUser;
 
-               if( count( $wgOut->mCategoryLinks ) == 0 ) return '';
+               if( count( $wgOut->mCategoryLinks ) == 0 ) {
+                       return '';
+               }
 
                # Separator
                $sep = wfMsgExt( 'catseparator', array( 'parsemag', 'escapenoentities' ) );
 
                // Use Unicode bidi embedding override characters,
                // to make sure links don't smash each other up in ugly ways.
-               $dir = $wgContLang->isRTL() ? 'rtl' : 'ltr';
+               $dir = $wgContLang->getDir();
                $embed = "<span dir='$dir'>";
                $pop = '</span>';
 
@@ -821,12 +805,12 @@ END;
                        $s .= "<div id=\"mw-hidden-catlinks\" class=\"$class\">" .
                                wfMsgExt( 'hidden-categories', array( 'parsemag', 'escapenoentities' ), count( $allCats['hidden'] ) ) .
                                $colon . $embed . implode( "$pop $sep $embed", $allCats['hidden'] ) . $pop .
-                               "</div>";
+                               '</div>';
                }
 
                # optional 'dmoz-like' category browser. Will be shown under the list
                # of categories an article belong to
-               if( $wgUseCategoryBrowser ){
+               if( $wgUseCategoryBrowser ) {
                        $s .= '<br /><hr />';
 
                        # get a big array of the parents tree
@@ -850,7 +834,7 @@ END;
         * @param &skin Object: skin passed by reference
         * @return String separated by &gt;, terminate with "\n"
         */
-       function drawCategoryBrowser( $tree, &$skin ){
+       function drawCategoryBrowser( $tree, &$skin ) {
                $return = '';
                foreach( $tree as $element => $parent ) {
                        if( empty( $parent ) ) {
@@ -868,18 +852,21 @@ END;
        }
 
        function getCategories() {
-               $catlinks=$this->getCategoryLinks();
+               $catlinks = $this->getCategoryLinks();
 
                $classes = 'catlinks';
 
-               if( strpos( $catlinks, '<div id="mw-normal-catlinks">' ) === false &&
-                       strpos( $catlinks, '<div id="mw-hidden-catlinks" class="mw-hidden-cats-hidden">' ) !== false ) {
+               // Check what we're showing
+               global $wgOut, $wgUser;
+               $allCats = $wgOut->getCategoryLinks();
+               $showHidden = $wgUser->getBoolOption( 'showhiddencats' ) ||
+                                               $this->mTitle->getNamespace() == NS_CATEGORY;
+
+               if( empty( $allCats['normal'] ) && !( !empty( $allCats['hidden'] ) && $showHidden ) ) {
                        $classes .= ' catlinks-allhidden';
                }
 
-               if( !empty( $catlinks ) ){
-                       return "<div id='catlinks' class='$classes'>{$catlinks}</div>";
-               }
+               return "<div id='catlinks' class='$classes'>{$catlinks}</div>";
        }
 
        function getQuickbarCompensator( $rows = 1 ) {
@@ -903,10 +890,10 @@ END;
        protected function afterContentHook() {
                $data = '';
 
-               if( wfRunHooks( 'SkinAfterContent', array( &$data ) ) ){
+               if( wfRunHooks( 'SkinAfterContent', array( &$data ) ) ) {
                        // adding just some spaces shouldn't toggle the output
                        // of the whole <div/>, so we use trim() here
-                       if( trim( $data ) != '' ){
+                       if( trim( $data ) != '' ) {
                                // Doing this here instead of in the skins to
                                // ensure that the div has the same ID in all
                                // skins
@@ -929,13 +916,50 @@ END;
        protected function generateDebugHTML() {
                global $wgShowDebug, $wgOut;
                if ( $wgShowDebug ) {
-                       $listInternals = str_replace( "\n", "</li>\n<li>", htmlspecialchars( $wgOut->mDebugtext ) );
-                       return "\n<hr>\n<strong>Debug data:</strong><ul style=\"font-family:monospace;\"><li>" .
-                               $listInternals . "</li></ul>\n";
+                       $listInternals = $this->formatDebugHTML( $wgOut->mDebugtext );
+                       return "\n<hr />\n<strong>Debug data:</strong><ul style=\"font-family:monospace;\" id=\"mw-debug-html\">" .
+                               $listInternals . "</ul>\n";
                }
                return '';
        }
 
+       private function formatDebugHTML( $debugText ) {
+               $lines = explode( "\n", $debugText );
+               $curIdent = 0;
+               $ret = '<li>';
+               foreach( $lines as $line ) {
+                       $m = array();
+                       $display = ltrim( $line );
+                       $ident = strlen( $line ) - strlen( $display );
+                       $diff = $ident - $curIdent;
+
+                       if ( $display == '' ) {
+                               $display = "\xc2\xa0";
+                       }
+
+                       if ( !$ident && $diff < 0 && substr( $display, 0, 9 ) != 'Entering ' && substr( $display, 0, 8 ) != 'Exiting ' ) {
+                               $ident = $curIdent;
+                               $diff = 0;
+                               $display = '<span style="background:yellow;">' . htmlspecialchars( $display ) . '</span>';
+                       } else {
+                               $display = htmlspecialchars( $display );
+                       }
+
+                       if ( $diff < 0 ) {
+                               $ret .= str_repeat( "</li></ul>\n", -$diff ) . "</li><li>\n";
+                       } elseif ( $diff == 0 ) {
+                               $ret .= "</li><li>\n";
+                       } else {
+                               $ret .= str_repeat( "<ul><li>\n", $diff );
+                       }
+                       $ret .= $display . "\n";
+
+                       $curIdent = $ident;
+               }
+               $ret .= str_repeat( '</li></ul>', $curIdent ) . '</li>';
+               return $ret;
+       }
+
        /**
         * This gets called shortly before the </body> tag.
         * @return String HTML to be put before </body>
@@ -950,8 +974,7 @@ END;
         * @return String HTML-wrapped JS code to be put before </body>
         */
        function bottomScripts() {
-               global $wgJsMimeType;
-               $bottomScriptText = "\n\t\t<script type=\"$wgJsMimeType\">if (window.runOnloadHook) runOnloadHook();</script>\n";
+               $bottomScriptText = "\n" . Html::inlineScript( 'if (window.runOnloadHook) runOnloadHook();' ) . "\n";
                wfRunHooks( 'SkinAfterBottomScripts', array( $this, &$bottomScriptText ) );
                return $bottomScriptText;
        }
@@ -959,7 +982,7 @@ END;
        /** @return string Retrievied from HTML text */
        function printSource() {
                $url = htmlspecialchars( $this->mTitle->getFullURL() );
-               return wfMsg( 'retrievedfrom', '<a href="'.$url.'">'.$url.'</a>' );
+               return wfMsg( 'retrievedfrom', '<a href="' . $url . '">' . $url . '</a>' );
        }
 
        function printFooter() {
@@ -968,7 +991,9 @@ END;
        }
 
        /** overloaded by derived classes */
-       function doAfterContent() { return '</div></div>'; }
+       function doAfterContent() {
+               return '</div></div>';
+       }
 
        function pageTitleLinks() {
                global $wgOut, $wgUser, $wgRequest, $wgLang;
@@ -1073,9 +1098,13 @@ END;
        function printableLink() {
                global $wgOut, $wgFeedClasses, $wgRequest, $wgLang;
 
-               $printurl = $wgRequest->escapeAppendQuery( 'printable=yes' );
+               $s = array();
+
+               if ( !$wgOut->isPrintable() ) {
+                       $printurl = $wgRequest->escapeAppendQuery( 'printable=yes' );
+                       $s[] = "<a href=\"$printurl\" rel=\"alternate\">" . wfMsg( 'printableversion' ) . '</a>';
+               }
 
-               $s[] = "<a href=\"$printurl\" rel=\"alternate\">" . wfMsg( 'printableversion' ) . '</a>';
                if( $wgOut->isSyndicated() ) {
                        foreach( $wgFeedClasses as $format => $class ) {
                                $feedurl = $wgRequest->escapeAppendQuery( "feed=$format" );
@@ -1086,6 +1115,10 @@ END;
                return $wgLang->pipeList( $s );
        }
 
+       /**
+        * Gets the h1 element with the page title.
+        * @return string
+        */
        function pageTitle() {
                global $wgOut;
                $s = '<h1 class="pagetitle">' . $wgOut->getPageTitle() . '</h1>';
@@ -1096,7 +1129,7 @@ END;
                global $wgOut;
 
                $sub = $wgOut->getSubtitle();
-               if ( '' == $sub ) {
+               if ( $sub == '' ) {
                        global $wgExtraSubtitle;
                        $sub = wfMsgExt( 'tagline', 'parsemag' ) . $wgExtraSubtitle;
                }
@@ -1108,8 +1141,9 @@ END;
 
        function subPageSubtitle() {
                $subpages = '';
-               if( !wfRunHooks( 'SkinSubPageSubtitle', array( &$subpages ) ) )
+               if( !wfRunHooks( 'SkinSubPageSubtitle', array( &$subpages ) ) ) {
                        return $subpages;
+               }
 
                global $wgOut;
                if( $wgOut->isArticle() && MWNamespace::hasSubpages( $this->mTitle->getNamespace() ) ) {
@@ -1124,7 +1158,7 @@ END;
                                        $growinglink .= $link;
                                        $display .= $link;
                                        $linkObj = Title::newFromText( $growinglink );
-                                       if( is_object( $linkObj ) && $linkObj->exists() ){
+                                       if( is_object( $linkObj ) && $linkObj->exists() ) {
                                                $getlink = $this->link(
                                                        $linkObj,
                                                        htmlspecialchars( $display ),
@@ -1229,16 +1263,17 @@ END;
                global $wgRequest, $wgUseTwoButtonsSearchForm;
                $search = $wgRequest->getText( 'search' );
 
-               $s = '<form id="searchform'.$this->searchboxes.'" name="search" class="inline" method="post" action="'
+               $s = '<form id="searchform' . $this->searchboxes . '" name="search" class="inline" method="post" action="'
                  . $this->escapeSearchLink() . "\">\n"
-                 . '<input type="text" id="searchInput'.$this->searchboxes.'" name="search" size="19" value="'
+                 . '<input type="text" id="searchInput' . $this->searchboxes . '" name="search" size="19" value="'
                  . htmlspecialchars( substr( $search, 0, 256 ) ) . "\" />\n"
                  . '<input type="submit" name="go" value="' . wfMsg( 'searcharticle' ) . '" />';
 
-               if( $wgUseTwoButtonsSearchForm )
+               if( $wgUseTwoButtonsSearchForm ) {
                        $s .= '&nbsp;<input type="submit" name="fulltext" value="' . wfMsg( 'searchbutton' ) . "\" />\n";
-               else
+               } else {
                        $s .= ' <a href="' . $this->escapeSearchLink() . '" rel="search">' . wfMsg( 'powersearch-legend' ) . "</a>\n";
+               }
 
                $s .= '</form>';
 
@@ -1314,11 +1349,12 @@ END;
                if( !$wgDisableLangConversion && sizeof( $variants ) > 1 ) {
                        foreach( $variants as $code ) {
                                $varname = $wgContLang->getVariantname( $code );
-                               if( $varname == 'disable' )
+                               if( $varname == 'disable' ) {
                                        continue;
+                               }
                                $s = $wgLang->pipeList( array(
                                        $s,
-                                       '<a href="' . $this->mTitle->escapeLocalUrl( 'variant=' . $code ) . '">' . htmlspecialchars( $varname ) . '</a>'
+                                       '<a href="' . $this->mTitle->escapeLocalURL( 'variant=' . $code ) . '">' . htmlspecialchars( $varname ) . '</a>'
                                ) );
                        }
                }
@@ -1340,29 +1376,41 @@ END;
                        $element[] = $this->whatLinksHere();
                        $element[] = $this->watchPageLinksLink();
 
-                       if( $wgUseTrackbacks )
+                       if( $wgUseTrackbacks ) {
                                $element[] = $this->trackbackLink();
+                       }
 
-                       if ( $this->mTitle->getNamespace() == NS_USER
-                           || $this->mTitle->getNamespace() == NS_USER_TALK ){
+                       if (
+                               $this->mTitle->getNamespace() == NS_USER ||
+                               $this->mTitle->getNamespace() == NS_USER_TALK
+                       )
+                       {
                                $id = User::idFromName( $this->mTitle->getText() );
                                $ip = User::isIP( $this->mTitle->getText() );
 
-                               if( $id || $ip ) { # both anons and non-anons have contri list
+                               # Both anons and non-anons have contributions list
+                               if( $id || $ip ) {
                                        $element[] = $this->userContribsLink();
                                }
                                if( $this->showEmailUser( $id ) ) {
                                        $element[] = $this->emailUserLink();
                                }
                        }
-                       
+
                        $s = implode( $element, $sep );
 
                        if ( $this->mTitle->getArticleId() ) {
                                $s .= "\n<br />";
-                               if( $wgUser->isAllowed( 'delete' ) ) { $s .= $this->deleteThisPage(); }
-                               if( $wgUser->isAllowed( 'protect' ) ) { $s .= $sep . $this->protectThisPage(); }
-                               if( $wgUser->isAllowed( 'move' ) ) { $s .= $sep . $this->moveThisPage(); }
+                               // Delete/protect/move links for privileged users
+                               if( $wgUser->isAllowed( 'delete' ) ) {
+                                       $s .= $this->deleteThisPage();
+                               }
+                               if( $wgUser->isAllowed( 'protect' ) ) {
+                                       $s .= $sep . $this->protectThisPage();
+                               }
+                               if( $wgUser->isAllowed( 'move' ) ) {
+                                       $s .= $sep . $this->moveThisPage();
+                               }
                        }
                        $s .= "<br />\n" . $this->otherLanguages();
                }
@@ -1376,10 +1424,18 @@ END;
 
                $oldid = $wgRequest->getVal( 'oldid' );
                $diff = $wgRequest->getVal( 'diff' );
-               if ( ! $wgOut->isArticle() ) { return ''; }
-               if( !$wgArticle instanceOf Article ) { return ''; }
-               if ( isset( $oldid ) || isset( $diff ) ) { return ''; }
-               if ( 0 == $wgArticle->getID() ) { return ''; }
+               if ( !$wgOut->isArticle() ) {
+                       return '';
+               }
+               if( !$wgArticle instanceof Article ) {
+                       return '';
+               }
+               if ( isset( $oldid ) || isset( $diff ) ) {
+                       return '';
+               }
+               if ( 0 == $wgArticle->getID() ) {
+                       return '';
+               }
 
                $s = '';
                if ( !$wgDisableCounters ) {
@@ -1389,7 +1445,7 @@ END;
                        }
                }
 
-               if( $wgMaxCredits != 0 ){
+               if( $wgMaxCredits != 0 ) {
                        $s .= ' ' . Credits::getCredits( $wgArticle, $wgMaxCredits, $wgShowCreditsIfMax );
                } else {
                        $s .= $this->lastModified();
@@ -1397,9 +1453,13 @@ END;
 
                if( $wgPageShowWatchingUsers && $wgUser->getOption( 'shownumberswatching' ) ) {
                        $dbr = wfGetDB( DB_SLAVE );
-                       $res = $dbr->select( 'watchlist',
+                       $res = $dbr->select(
+                               'watchlist',
                                array( 'COUNT(*) AS n' ),
-                               array( 'wl_title' => $dbr->strencode( $this->mTitle->getDBkey() ), 'wl_namespace' => $this->mTitle->getNamespace() ),
+                               array(
+                                       'wl_title' => $dbr->strencode( $this->mTitle->getDBkey() ),
+                                       'wl_namespace' => $this->mTitle->getNamespace()
+                               ),
                                __METHOD__
                        );
                        $x = $dbr->fetchObject( $res );
@@ -1444,8 +1504,10 @@ END;
                        return $out;
                }
                // Allow for site and per-namespace customization of copyright notice.
-               wfRunHooks( 'SkinCopyrightFooter', array( $wgArticle->getTitle(), $type, &$msg, &$link ) );
-               
+               if( isset( $wgArticle ) ) {
+                       wfRunHooks( 'SkinCopyrightFooter', array( $wgArticle->getTitle(), $type, &$msg, &$link ) );
+               }
+
                $out .= wfMsgForContent( $msg, $link );
                return $out;
        }
@@ -1455,14 +1517,14 @@ END;
                $out = '';
                if ( isset( $wgCopyrightIcon ) && $wgCopyrightIcon ) {
                        $out = $wgCopyrightIcon;
-               } else if ( $wgRightsIcon ) {
+               } elseif ( $wgRightsIcon ) {
                        $icon = htmlspecialchars( $wgRightsIcon );
                        if ( $wgRightsUrl ) {
                                $url = htmlspecialchars( $wgRightsUrl );
                                $out .= '<a href="'.$url.'">';
                        }
                        $text = htmlspecialchars( $wgRightsText );
-                       $out .= "<img src=\"$icon\" alt='$text' />";
+                       $out .= "<img src=\"$icon\" alt=\"$text\" width=\"88\" height=\"31\" />";
                        if ( $wgRightsUrl ) {
                                $out .= '</a>';
                        }
@@ -1470,16 +1532,20 @@ END;
                return $out;
        }
 
+       /**
+        * Gets the powered by MediaWiki icon.
+        * @return string
+        */
        function getPoweredBy() {
                global $wgStylePath;
                $url = htmlspecialchars( "$wgStylePath/common/images/poweredby_mediawiki_88x31.png" );
-               $img = '<a href="http://www.mediawiki.org/"><img src="'.$url.'" alt="Powered by MediaWiki" /></a>';
+               $img = '<a href="http://www.mediawiki.org/"><img src="' . $url . '" height="31" width="88" alt="Powered by MediaWiki" /></a>';
                return $img;
        }
 
        function lastModified() {
                global $wgLang, $wgArticle;
-               if( $this->mRevisionId ) {
+               if( $this->mRevisionId && $this->mRevisionId != $wgArticle->getLatest() ) {
                        $timestamp = Revision::getTimestampFromId( $wgArticle->getTitle(), $this->mRevisionId );
                } else {
                        $timestamp = $wgArticle->getTimestamp();
@@ -1498,7 +1564,7 @@ END;
        }
 
        function logoText( $align = '' ) {
-               if ( '' != $align ) {
+               if ( $align != '' ) {
                        $a = " align='{$align}'";
                } else {
                        $a = '';
@@ -1514,7 +1580,7 @@ END;
        }
 
        /**
-        * show a drop-down box of special pages
+        * Show a drop-down box of special pages
         */
        function specialPagesList() {
                global $wgUser, $wgContLang, $wgServer, $wgRedirectScript;
@@ -1543,6 +1609,10 @@ END;
                return $s;
        }
 
+       /**
+        * Gets the link to the wiki's main page.
+        * @return string
+        */
        function mainPageLink() {
                $s = $this->link(
                        Title::newMainPage(),
@@ -1554,16 +1624,7 @@ END;
                return $s;
        }
 
-       function copyrightLink() {
-               $title = Title::newFromText( wfMsgForContent( 'copyrightpage' ) );
-               $s = $this->linkKnown(
-                       $title,
-                       wfMsg( 'copyrightpagename' )
-               );
-               return $s;
-       }
-
-       private function footerLink ( $desc, $page ) {
+       private function footerLink( $desc, $page ) {
                // if the link description has been set to "-" in the default language,
                if ( wfMsgForContent( $desc )  == '-') {
                        // then it is disabled, for all languages.
@@ -1580,14 +1641,23 @@ END;
                }
        }
 
+       /**
+        * Gets the link to the wiki's privacy policy page.
+        */
        function privacyLink() {
                return $this->footerLink( 'privacy', 'privacypage' );
        }
 
+       /**
+        * Gets the link to the wiki's about page.
+        */
        function aboutLink() {
                return $this->footerLink( 'aboutsite', 'aboutpage' );
        }
 
+       /**
+        * Gets the link to the wiki's general disclaimers page.
+        */
        function disclaimerLink() {
                return $this->footerLink( 'disclaimers', 'disclaimerpage' );
        }
@@ -1627,7 +1697,7 @@ END;
        function editUrlOptions() {
                global $wgArticle;
 
-               $options = array( 'action' => 'edit' ); 
+               $options = array( 'action' => 'edit' );
 
                if( $this->mRevisionId && ! $wgArticle->isCurrent() ) {
                        $options['oldid'] = intval( $this->mRevisionId );
@@ -1716,7 +1786,7 @@ END;
                                SpecialPage::getTitleFor( 'Movepage' ),
                                wfMsg( 'movethispage' ),
                                array(),
-                               array( 'target' => $this->mTitle->getPrefixedURL() ),
+                               array( 'target' => $this->mTitle->getPrefixedDBkey() ),
                                array( 'known', 'noclasses' )
                        );
                } else {
@@ -1757,7 +1827,7 @@ END;
        function showEmailUser( $id ) {
                global $wgUser;
                $targetUser = User::newFromId( $id );
-               return  $wgUser->canSendEmail() && # the sending user must have a confirmed email address
+               return $wgUser->canSendEmail() && # the sending user must have a confirmed email address
                        $targetUser->canReceiveEmail(); # the target user must have a confirmed email address and allow emails from users
        }
 
@@ -1773,7 +1843,7 @@ END;
 
        function watchPageLinksLink() {
                global $wgOut;
-               if ( ! $wgOut->isArticleRelated() ) {
+               if ( !$wgOut->isArticleRelated() ) {
                        return '(' . wfMsg( 'notanarticle' ) . ')';
                } else {
                        return $this->link(
@@ -1805,7 +1875,9 @@ END;
 
                $s = wfMsg( 'otherlanguages' ) . wfMsg( 'colon-separator' );
                $first = true;
-               if( $wgContLang->isRTL() ) $s .= '<span dir="LTR">';
+               if( $wgContLang->isRTL() ) {
+                       $s .= '<span dir="LTR">';
+               }
                foreach( $a as $l ) {
                        if ( !$first ) {
                                $s .= wfMsgExt( 'pipe-separator', 'escapenoentities' );
@@ -1816,11 +1888,15 @@ END;
                        $url = $nt->escapeFullURL();
                        $text = $wgContLang->getLanguageName( $nt->getInterwiki() );
 
-                       if ( '' == $text ) { $text = $l; }
-                       $style = $this->getExternalLinkAttributes( $l, $text );
+                       if ( $text == '' ) {
+                               $text = $l;
+                       }
+                       $style = $this->getExternalLinkAttributes();
                        $s .= "<a href=\"{$url}\"{$style}>{$text}</a>";
                }
-               if( $wgContLang->isRTL() ) $s .= '</span>';
+               if( $wgContLang->isRTL() ) {
+                       $s .= '</span>';
+               }
                return $s;
        }
 
@@ -1934,8 +2010,10 @@ END;
                return $title->getLocalURL( $urlaction );
        }
 
-       # If url string starts with http, consider as external URL, else
-       # internal
+       /**
+        * If url string starts with http, consider as external URL, else
+        * internal
+        */
        static function makeInternalOrExternalUrl( $name ) {
                if ( preg_match( '/^(?:' . wfUrlProtocols() . ')/', $name ) ) {
                        return $name;
@@ -2004,26 +2082,49 @@ END;
                }
 
                $bar = array();
-               $lines = explode( "\n", wfMsgForContent( 'sidebar' ) );
+               $this->addToSidebar( $bar, 'sidebar' );
+
+               wfRunHooks( 'SkinBuildSidebar', array( $this, &$bar ) );
+               if ( $wgEnableSidebarCache ) {
+                       $parserMemc->set( $key, $bar, $wgSidebarCacheExpiry );
+               }
+               wfProfileOut( __METHOD__ );
+               return $bar;
+       }
+       /**
+        * Add content from a sidebar system message
+        * Currently only used for MediaWiki:Sidebar (but may be used by Extensions)
+        *
+        * @param &$bar array
+        * @param $message String
+        */
+       function addToSidebar( &$bar, $message ) {
+               $lines = explode( "\n", wfMsgForContent( $message ) );
                $heading = '';
                foreach( $lines as $line ) {
-                       if( strpos( $line, '*' ) !== 0 )
+                       if( strpos( $line, '*' ) !== 0 ) {
                                continue;
+                       }
                        if( strpos( $line, '**') !== 0 ) {
                                $heading = trim( $line, '* ' );
-                               if( !array_key_exists( $heading, $bar ) ) $bar[$heading] = array();
+                               if( !array_key_exists( $heading, $bar ) ) {
+                                       $bar[$heading] = array();
+                               }
                        } else {
                                if( strpos( $line, '|' ) !== false ) { // sanity check
                                        $line = array_map( 'trim', explode( '|', trim( $line, '* ' ), 2 ) );
                                        $link = wfMsgForContent( $line[0] );
-                                       if( $link == '-' )
+                                       if( $link == '-' ) {
                                                continue;
+                                       }
 
                                        $text = wfMsgExt( $line[1], 'parsemag' );
-                                       if( wfEmptyMsg( $line[1], $text ) )
+                                       if( wfEmptyMsg( $line[1], $text ) ) {
                                                $text = $line[1];
-                                       if( wfEmptyMsg( $line[0], $link ) )
+                                       }
+                                       if( wfEmptyMsg( $line[0], $link ) ) {
                                                $link = $line[0];
+                                       }
 
                                        if ( preg_match( '/^(?:' . wfUrlProtocols() . ')/', $link ) ) {
                                                $href = $link;
@@ -2043,12 +2144,22 @@ END;
                                                'id' => 'n-' . strtr( $line[1], ' ', '-' ),
                                                'active' => false
                                        );
-                               } else { continue; }
+                               } else {
+                                       continue;
+                               }
                        }
                }
-               wfRunHooks( 'SkinBuildSidebar', array( $this, &$bar ) );
-               if ( $wgEnableSidebarCache ) $parserMemc->set( $key, $bar, $wgSidebarCacheExpiry );
-               wfProfileOut( __METHOD__ );
-               return $bar;
+       }
+
+       /**
+        * Should we include common/wikiprintable.css?  Skins that have their own
+        * print stylesheet should override this and return false.  (This is an
+        * ugly hack to get Monobook to play nicely with
+        * OutputPage::headElement().)
+        *
+        * @return bool
+        */
+       public function commonPrintStylesheet() {
+               return true;
        }
 }