Moved globals $utfCombiningClass, $utfCanonicalComp, $utfCanonicalDecomp, $utfCheckNF...
[lhc/web/wiklou.git] / includes / OutputPage.php
index 03de89c..be9c459 100644 (file)
@@ -1,6 +1,7 @@
 <?php
-if ( ! defined( 'MEDIAWIKI' ) )
+if ( !defined( 'MEDIAWIKI' ) ) {
        die( 1 );
+}
 
 /**
  * @todo document
@@ -8,7 +9,15 @@ if ( ! defined( 'MEDIAWIKI' ) )
 class OutputPage {
        var $mMetatags = array(), $mKeywords = array(), $mLinktags = array();
        var $mExtStyles = array();
-       var $mPagetitle = '', $mBodytext = '', $mDebugtext = '';
+       var $mPagetitle = '', $mBodytext = '';
+
+       /**
+        * Holds the debug lines that will be outputted as comments in page source if
+        * $wgDebugComments is enabled. See also $wgShowDebug.
+        * TODO: make a getter method for this
+        */
+       public $mDebugtext = '';
+
        var $mHTMLtitle = '', $mIsarticle = true, $mPrintable = false;
        var $mSubtitle = '', $mRedirect = '', $mStatusCode;
        var $mLastModified = '', $mETag = false;
@@ -54,9 +63,10 @@ class OutputPage {
 
        private $mIndexPolicy = 'index';
        private $mFollowPolicy = 'follow';
-       private $mVaryHeader = array( 'Accept-Encoding' => array('list-contains=gzip'),
-                                                                 'Cookie' => null );
-
+       private $mVaryHeader = array(
+               'Accept-Encoding' => array( 'list-contains=gzip' ),
+               'Cookie' => null
+       );
 
        /**
         * Constructor
@@ -98,7 +108,6 @@ class OutputPage {
                $this->mStatusCode = $statusCode;
        }
 
-
        /**
         * Add a new <meta> tag
         * To add an http-equiv meta tag, precede the name with "http:"
@@ -147,7 +156,6 @@ class OutputPage {
                $haveMeta = true;
        }
 
-
        /**
         * Add raw HTML to the list of scripts (including \<script\> tag, etc.)
         *
@@ -183,15 +191,18 @@ class OutputPage {
         *
         * @param $file String: filename in skins/common or complete on-server path
         *              (/foo/bar.js)
+        * @param $version String: style version of the file. Defaults to $wgStyleVersion
         */
-       public function addScriptFile( $file ) {
+       public function addScriptFile( $file, $version = null ) {
                global $wgStylePath, $wgStyleVersion;
                if( substr( $file, 0, 1 ) == '/' || preg_match( '#^[a-z]*://#i', $file ) ) {
                        $path = $file;
                } else {
-                       $path =  "{$wgStylePath}/common/{$file}";
+                       $path = "{$wgStylePath}/common/{$file}";
                }
-               $this->addScript( Html::linkedScript( wfAppendQuery( $path, $wgStyleVersion ) ) );
+               if ( is_null( $version ) )
+                       $version = $wgStyleVersion;
+               $this->addScript( Html::linkedScript( wfAppendQuery( $path, $version ) ) );
        }
 
        /**
@@ -274,7 +285,6 @@ class OutputPage {
                return $this->mArticleBodyOnly;
        }
 
-
        /**
         * checkLastModified tells the client to use the client-cached page if
         * possible. If sucessful, the OutputPage is disabled so that
@@ -351,7 +361,7 @@ class OutputPage {
                # Not modified
                # Give a 304 response code and disable body output
                wfDebug( __METHOD__ . ": NOT MODIFIED, $info\n", false );
-               ini_set('zlib.output_compression', 0);
+               ini_set( 'zlib.output_compression', 0 );
                $wgRequest->response()->header( "HTTP/1.1 304 Not Modified" );
                $this->sendCacheControl();
                $this->disable();
@@ -374,7 +384,6 @@ class OutputPage {
                $this->mLastModified = wfTimestamp( TS_RFC2822, $timestamp );
        }
 
-
        /**
         * Set the robot policy for the page: <http://www.robotstxt.org/meta.html>
         *
@@ -386,10 +395,10 @@ class OutputPage {
        public function setRobotPolicy( $policy ) {
                $policy = Article::formatRobotPolicy( $policy );
 
-               if( isset( $policy['index'] ) ){
+               if( isset( $policy['index'] ) ) {
                        $this->setIndexPolicy( $policy['index'] );
                }
-               if( isset( $policy['follow'] ) ){
+               if( isset( $policy['follow'] ) ) {
                        $this->setFollowPolicy( $policy['follow'] );
                }
        }
@@ -422,7 +431,6 @@ class OutputPage {
                }
        }
 
-
        /**
         * Set the new value of the "action text", this will be added to the
         * "HTML title", separated from it with " - ".
@@ -473,7 +481,7 @@ class OutputPage {
                $nameWithTags = Sanitizer::normalizeCharReferences( Sanitizer::removeHTMLtags( $name ) );
                $this->mPagetitle = $nameWithTags;
 
-               $taction =  $this->getPageTitleActionText();
+               $taction = $this->getPageTitleActionText();
                if( !empty( $taction ) ) {
                        $name .= ' - '.$taction;
                }
@@ -542,7 +550,6 @@ class OutputPage {
                return $this->mSubtitle;
        }
 
-
        /**
         * Set the page as printable, i.e. it'll be displayed with with all
         * print styles included
@@ -560,7 +567,6 @@ class OutputPage {
                return $this->mPrintable;
        }
 
-
        /**
         * Disable output completely, i.e. calling output() will have no effect
         */
@@ -577,7 +583,6 @@ class OutputPage {
                return $this->mDoNothing;
        }
 
-
        /**
         * Show an "add new section" link?
         *
@@ -596,7 +601,6 @@ class OutputPage {
                return $this->mHideNewSectionLink;
        }
 
-
        /**
         * Add or remove feed links in the page header
         * This is mainly kept for backward compatibility, see OutputPage::addFeedLink()
@@ -721,7 +725,6 @@ class OutputPage {
                return $this->mIsArticleRelated;
        }
 
-
        /**
         * Add new language links
         *
@@ -751,7 +754,6 @@ class OutputPage {
                return $this->mLanguageLinks;
        }
 
-
        /**
         * Add an array of categories, with names in the keys
         *
@@ -774,7 +776,7 @@ class OutputPage {
                $pageTable = $dbr->tableName( 'page' );
                $where = $lb->constructSet( 'page', $dbr );
                $propsTable = $dbr->tableName( 'page_props' );
-               $sql = "SELECT page_id, page_namespace, page_title, page_len, page_is_redirect, pp_value
+               $sql = "SELECT page_id, page_namespace, page_title, page_len, page_is_redirect, page_latest, pp_value
                        FROM $pageTable LEFT JOIN $propsTable ON pp_propname='hiddencat' AND pp_page=page_id WHERE $where";
                $res = $dbr->query( $sql, __METHOD__ );
 
@@ -782,8 +784,10 @@ class OutputPage {
                $lb->addResultToCache( LinkCache::singleton(), $res );
 
                # Set all the values to 'normal'. This can be done with array_fill_keys in PHP 5.2.0+
-               $categories = array_combine( array_keys( $categories ),
-                       array_fill( 0, count( $categories ), 'normal' ) );
+               $categories = array_combine(
+                       array_keys( $categories ),
+                       array_fill( 0, count( $categories ), 'normal' )
+               );
 
                # Mark hidden categories
                foreach ( $res as $row ) {
@@ -799,9 +803,11 @@ class OutputPage {
                                $origcategory = $category;
                                $title = Title::makeTitleSafe( NS_CATEGORY, $category );
                                $wgContLang->findVariantLink( $category, $title, true );
-                               if ( $category != $origcategory )
-                                       if ( array_key_exists( $category, $categories ) )
+                               if ( $category != $origcategory ) {
+                                       if ( array_key_exists( $category, $categories ) ) {
                                                continue;
+                                       }
+                               }
                                $text = $wgContLang->convertHtml( $title->getText() );
                                $this->mCategories[] = $title->getText();
                                $this->mCategoryLinks[$type][] = $sk->link( $title, $text );
@@ -840,7 +846,6 @@ class OutputPage {
                return $this->mCategories;
        }
 
-
        /**
         * Suppress the quickbar from the output, only for skin supporting
         * the quickbar
@@ -858,7 +863,6 @@ class OutputPage {
                return $this->mSuppressQuickbar;
        }
 
-
        /**
         * Remove user JavaScript from scripts to load
         */
@@ -875,7 +879,6 @@ class OutputPage {
                return $this->mAllowUserJs;
        }
 
-
        /**
         * Prepend $text to the body HTML
         *
@@ -910,7 +913,6 @@ class OutputPage {
                return $this->mBodytext;
        }
 
-
        /**
         * Add $text to the debug output
         *
@@ -920,7 +922,6 @@ class OutputPage {
                $this->mDebugtext .= $text;
        }
 
-
        /**
         * @deprecated use parserOptions() instead
         */
@@ -1006,7 +1007,7 @@ class OutputPage {
         */
        public function addWikiTextTidy( $text, $linestart = true ) {
                $title = $this->getTitle();
-               $this->addWikiTextTitleTidy($text, $title, $linestart);
+               $this->addWikiTextTitleTidy( $text, $title, $linestart );
        }
 
        /**
@@ -1027,8 +1028,10 @@ class OutputPage {
                $popts = $this->parserOptions();
                $oldTidy = $popts->setTidy( $tidy );
 
-               $parserOutput = $wgParser->parse( $text, $title, $popts,
-                       $linestart, true, $this->mRevisionId );
+               $parserOutput = $wgParser->parse(
+                       $text, $title, $popts,
+                       $linestart, true, $this->mRevisionId
+               );
 
                $popts->setTidy( $oldTidy );
 
@@ -1052,13 +1055,15 @@ class OutputPage {
                wfDeprecated( __METHOD__ );
 
                $popts = $this->parserOptions();
-               $popts->setTidy(true);
-               $parserOutput = $wgParser->parse( $text, $article->mTitle,
-                       $popts, true, true, $this->mRevisionId );
-               $popts->setTidy(false);
-               if ( $cache && $article && $parserOutput->getCacheTime() != -1 ) {
+               $popts->setTidy( true );
+               $parserOutput = $wgParser->parse(
+                       $text, $article->mTitle,
+                       $popts, true, true, $this->mRevisionId
+               );
+               $popts->setTidy( false );
+               if ( $cache && $article && !$parserOutput->isCacheable() ) {
                        $parserCache = ParserCache::singleton();
-                       $parserCache->save( $parserOutput, $article, $popts);
+                       $parserCache->save( $parserOutput, $article, $popts );
                }
 
                $this->addParserOutput( $parserOutput );
@@ -1069,25 +1074,22 @@ class OutputPage {
         */
        public function addSecondaryWikiText( $text, $linestart = true ) {
                wfDeprecated( __METHOD__ );
-               $this->addWikiTextTitleTidy($text, $this->getTitle(), $linestart);
+               $this->addWikiTextTitleTidy( $text, $this->getTitle(), $linestart );
        }
 
-
        /**
         * Add a ParserOutput object, but without Html
         *
         * @param $parserOutput ParserOutput object
         */
        public function addParserOutputNoText( &$parserOutput ) {
-               global $wgExemptFromUserRobotsControl, $wgContentNamespaces;
-
                $this->mLanguageLinks += $parserOutput->getLanguageLinks();
                $this->addCategoryLinks( $parserOutput->getCategories() );
                $this->mNewSectionLink = $parserOutput->getNewSection();
                $this->mHideNewSectionLink = $parserOutput->getHideNewSection();
 
                $this->mParseWarnings = $parserOutput->getWarnings();
-               if ( $parserOutput->getCacheTime() == -1 ) {
+               if ( !$parserOutput->isCacheable() ) {
                        $this->enableClientCache( false );
                }
                $this->mNoGallery = $parserOutput->getNoGallery();
@@ -1121,7 +1123,7 @@ class OutputPage {
        function addParserOutput( &$parserOutput ) {
                $this->addParserOutputNoText( $parserOutput );
                $text = $parserOutput->getText();
-               wfRunHooks( 'OutputPageBeforeHTML',array( &$this, &$text ) );
+               wfRunHooks( 'OutputPageBeforeHTML', array( &$this, &$text ) );
                $this->addHTML( $text );
        }
 
@@ -1154,10 +1156,16 @@ class OutputPage {
                        throw new MWException( 'Empty $mTitle in ' . __METHOD__ );
                }
                $popts = $this->parserOptions();
-               if ( $interface) { $popts->setInterfaceMessage(true); }
-               $parserOutput = $wgParser->parse( $text, $this->getTitle(), $popts,
-                       $linestart, true, $this->mRevisionId );
-               if ( $interface) { $popts->setInterfaceMessage(false); }
+               if ( $interface ) {
+                       $popts->setInterfaceMessage( true );
+               }
+               $parserOutput = $wgParser->parse(
+                       $text, $this->getTitle(), $popts,
+                       $linestart, true, $this->mRevisionId
+               );
+               if ( $interface ) {
+                       $popts->setInterfaceMessage( false );
+               }
                return $parserOutput->getText();
        }
 
@@ -1192,7 +1200,7 @@ class OutputPage {
                wfDeprecated( __METHOD__ );
                $parserOutput = ParserCache::singleton()->get( $article, $article->getParserOptions() );
 
-               if ($parserOutput !== false) {
+               if ( $parserOutput !== false ) {
                        $this->addParserOutput( $parserOutput );
                        return true;
                } else {
@@ -1235,21 +1243,21 @@ class OutputPage {
                                ),
                                $wgCacheVaryCookies
                        );
-                       wfRunHooks('GetCacheVaryCookies', array( $this, &$cookies ) );
+                       wfRunHooks( 'GetCacheVaryCookies', array( $this, &$cookies ) );
                }
                return $cookies;
        }
 
        /**
         * Return whether this page is not cacheable because "useskin" or "uselang"
-        * url parameters were passed
+        * URL parameters were passed.
         *
         * @return Boolean
         */
        function uncacheableBecauseRequestVars() {
                global $wgRequest;
-               return $wgRequest->getText('useskin', false) === false
-                       && $wgRequest->getText('uselang', false) === false;
+               return $wgRequest->getText( 'useskin', false ) === false
+                       && $wgRequest->getText( 'uselang', false ) === false;
        }
 
        /**
@@ -1267,12 +1275,12 @@ class OutputPage {
                $cvCookies = $this->getCacheVaryCookies();
                foreach ( $cvCookies as $cookieName ) {
                        # Check for a simple string match, like the way squid does it
-                       if ( strpos( $cookieHeader, $cookieName ) ) {
-                               wfDebug( __METHOD__.": found $cookieName\n" );
+                       if ( strpos( $cookieHeader, $cookieName ) !== false ) {
+                               wfDebug( __METHOD__ . ": found $cookieName\n" );
                                return true;
                        }
                }
-               wfDebug( __METHOD__.": no cache-varying cookies found\n" );
+               wfDebug( __METHOD__ . ": no cache-varying cookies found\n" );
                return false;
        }
 
@@ -1285,12 +1293,10 @@ class OutputPage {
        public function addVaryHeader( $header, $option = null ) {
                if ( !array_key_exists( $header, $this->mVaryHeader ) ) {
                        $this->mVaryHeader[$header] = $option;
-               }
-               elseif( is_array( $option ) ) {
+               } elseif( is_array( $option ) ) {
                        if( is_array( $this->mVaryHeader[$header] ) ) {
                                $this->mVaryHeader[$header] = array_merge( $this->mVaryHeader[$header], $option );
-                       }
-                       else {
+                       } else {
                                $this->mVaryHeader[$header] = $option;
                        }
                }
@@ -1314,8 +1320,9 @@ class OutputPage {
                $headers = array();
                foreach( $this->mVaryHeader as $header => $option ) {
                        $newheader = $header;
-                       if( is_array( $option ) )
+                       if( is_array( $option ) ) {
                                $newheader .= ';' . implode( ';', $option );
+                       }
                        $headers[] = $newheader;
                }
                $xvo = 'X-Vary-Options: ' . implode( ',', $headers );
@@ -1333,14 +1340,15 @@ class OutputPage {
         */
        function addAcceptLanguage() {
                global $wgRequest, $wgContLang;
-               if( !$wgRequest->getCheck('variant') && $wgContLang->hasVariants() ) {
+               if( !$wgRequest->getCheck( 'variant' ) && $wgContLang->hasVariants() ) {
                        $variants = $wgContLang->getVariants();
                        $aloption = array();
                        foreach ( $variants as $variant ) {
-                               if( $variant === $wgContLang->getCode() )
+                               if( $variant === $wgContLang->getCode() ) {
                                        continue;
-                               else
+                               } else {
                                        $aloption[] = "string-contains=$variant";
+                               }
                        }
                        $this->addVaryHeader( 'Accept-Language', $aloption );
                }
@@ -1353,8 +1361,9 @@ class OutputPage {
                global $wgUseSquid, $wgUseESI, $wgUseETag, $wgSquidMaxage, $wgRequest, $wgUseXVO;
 
                $response = $wgRequest->response();
-               if ($wgUseETag && $this->mETag)
-                       $response->header("ETag: $this->mETag");
+               if ( $wgUseETag && $this->mETag ) {
+                       $response->header( "ETag: $this->mETag" );
+               }
 
                $this->addAcceptLanguage();
 
@@ -1368,8 +1377,10 @@ class OutputPage {
                }
 
                if( !$this->uncacheableBecauseRequestVars() && $this->mEnableClientCache ) {
-                       if( $wgUseSquid && session_id() == '' &&
-                         ! $this->isPrintable() && $this->mSquidMaxage != 0 && !$this->haveCacheVaryCookies() )
+                       if(
+                               $wgUseSquid && session_id() == '' && !$this->isPrintable() &&
+                               $this->mSquidMaxage != 0 && !$this->haveCacheVaryCookies()
+                       )
                        {
                                if ( $wgUseESI ) {
                                        # We'll purge the proxy cache explicitly, but require end user agents
@@ -1481,7 +1492,7 @@ class OutputPage {
                global $wgEnableMWSuggest, $wgUniversalEditButton;
                global $wgArticle, $wgJQueryOnEveryPage;
 
-               if( $this->mDoNothing ){
+               if( $this->mDoNothing ) {
                        return;
                }
                wfProfileIn( __METHOD__ );
@@ -1510,8 +1521,9 @@ class OutputPage {
                        return;
                } elseif ( $this->mStatusCode ) {
                        $message = self::getStatusMessage( $this->mStatusCode );
-                       if ( $message )
+                       if ( $message ) {
                                $wgRequest->response()->header( 'HTTP/1.1 ' . $this->mStatusCode . ' ' . $message );
+                       }
                }
 
                $sk = $wgUser->getSkin();
@@ -1526,7 +1538,7 @@ class OutputPage {
                                $this->addScriptFile( 'ajaxwatch.js' );
                        }
 
-                       if ( $wgEnableMWSuggest && !$wgUser->getOption( 'disablesuggest', false ) ){
+                       if ( $wgEnableMWSuggest && !$wgUser->getOption( 'disablesuggest', false ) ) {
                                $this->addScriptFile( 'mwsuggest.js' );
                        }
                }
@@ -1539,7 +1551,7 @@ class OutputPage {
                        if( isset( $wgArticle ) && $this->getTitle() && $this->getTitle()->quickUserCan( 'edit' )
                                && ( $this->getTitle()->exists() || $this->getTitle()->quickUserCan( 'create' ) ) ) {
                                // Original UniversalEditButton
-                               $msg = wfMsg('edit');
+                               $msg = wfMsg( 'edit' );
                                $this->addLink( array(
                                        'rel' => 'alternate',
                                        'type' => 'application/x-wiki',
@@ -1563,10 +1575,10 @@ class OutputPage {
                ob_start();
 
                $wgRequest->response()->header( "Content-type: $wgMimeType; charset={$wgOutputEncoding}" );
-               $wgRequest->response()->header( 'Content-language: '.$wgContLanguageCode );
+               $wgRequest->response()->header( 'Content-language: ' . $wgContLanguageCode );
 
-               if ($this->mArticleBodyOnly) {
-                       $this->out($this->mBodytext);
+               if ( $this->mArticleBodyOnly ) {
+                       $this->out( $this->mBodytext );
                } else {
                        // Hook that allows last minute changes to the output page, e.g.
                        // adding of CSS or Javascript by extensions.
@@ -1594,7 +1606,9 @@ class OutputPage {
                        $outs = $ins;
                } else {
                        $outs = $wgContLang->iconv( $wgInputEncoding, $wgOutputEncoding, $ins );
-                       if ( false === $outs ) { $outs = $ins; }
+                       if ( false === $outs ) {
+                               $outs = $ins;
+                       }
                }
                print $outs;
        }
@@ -1604,7 +1618,6 @@ class OutputPage {
         */
        public static function setEncodings() {
                global $wgInputEncoding, $wgOutputEncoding;
-               global $wgContLang;
 
                $wgInputEncoding = strtolower( $wgInputEncoding );
 
@@ -1644,7 +1657,9 @@ class OutputPage {
                if( $reason == '' ) {
                        $reason = wfMsg( 'blockednoreason' );
                }
-               $blockTimestamp = $wgLang->timeanddate( wfTimestamp( TS_MW, $wgUser->mBlock->mTimestamp ), true );
+               $blockTimestamp = $wgLang->timeanddate(
+                       wfTimestamp( TS_MW, $wgUser->mBlock->mTimestamp ), true
+               );
                $ip = wfGetIP();
 
                $link = '[[' . $wgContLang->getNsText( NS_USER ) . ":{$name}|{$name}]]";
@@ -1657,16 +1672,20 @@ class OutputPage {
                        // Search for localization in 'ipboptions'
                        $scBlockExpiryOptions = wfMsg( 'ipboptions' );
                        foreach ( explode( ',', $scBlockExpiryOptions ) as $option ) {
-                               if ( strpos( $option, ":" ) === false )
+                               if ( strpos( $option, ':' ) === false ) {
                                        continue;
-                               list( $show, $value ) = explode( ":", $option );
+                               }
+                               list( $show, $value ) = explode( ':', $option );
                                if ( $value == 'infinite' || $value == 'indefinite' ) {
                                        $blockExpiry = $show;
                                        break;
                                }
                        }
                } else {
-                       $blockExpiry = $wgLang->timeanddate( wfTimestamp( TS_MW, $blockExpiry ), true );
+                       $blockExpiry = $wgLang->timeanddate(
+                               wfTimestamp( TS_MW, $blockExpiry ),
+                               true
+                       );
                }
 
                if ( $wgUser->mBlock->mAuto ) {
@@ -1676,10 +1695,13 @@ class OutputPage {
                }
 
                /* $ip returns who *is* being blocked, $intended contains who was meant to be blocked.
-                * This could be a username, an ip range, or a single ip. */
+                * This could be a username, an IP range, or a single IP. */
                $intended = $wgUser->mBlock->mAddress;
 
-               $this->addWikiMsg( $msg, $link, $reason, $ip, $name, $blockid, $blockExpiry, $intended, $blockTimestamp );
+               $this->addWikiMsg(
+                       $msg, $link, $reason, $ip, $name, $blockid, $blockExpiry,
+                       $intended, $blockTimestamp
+               );
 
                # Don't auto-return to special pages
                if( $return ) {
@@ -1767,34 +1789,22 @@ class OutputPage {
                $groups = array_map( array( 'User', 'makeGroupLinkWiki' ),
                        User::getGroupsWithPermission( $permission ) );
                if( $groups ) {
-                       $this->addWikiMsg( 'badaccess-groups',
+                       $this->addWikiMsg(
+                               'badaccess-groups',
                                $wgLang->commaList( $groups ),
-                               count( $groups) );
+                               count( $groups )
+                       );
                } else {
                        $this->addWikiMsg( 'badaccess-group0' );
                }
                $this->returnToMain();
        }
 
-       /**
-        * @deprecated use permissionRequired()
-        */
-       public function sysopRequired() {
-               throw new MWException( "Call to deprecated OutputPage::sysopRequired() method\n" );
-       }
-
-       /**
-        * @deprecated use permissionRequired()
-        */
-       public function developerRequired() {
-               throw new MWException( "Call to deprecated OutputPage::developerRequired() method\n" );
-       }
-
        /**
         * Produce the stock "please login to use the wiki" page
         */
        public function loginToUse() {
-               global $wgUser, $wgContLang;
+               global $wgUser;
 
                if( $wgUser->isLoggedIn() ) {
                        $this->permissionRequired( 'read' );
@@ -1817,13 +1827,14 @@ class OutputPage {
                        array( 'known', 'noclasses' )
                );
                $this->addHTML( wfMsgWikiHtml( 'loginreqpagetext', $loginLink ) );
-               $this->addHTML( "\n<!--" . $this->getTitle()->getPrefixedUrl() . "-->" );
+               $this->addHTML( "\n<!--" . $this->getTitle()->getPrefixedUrl() . '-->' );
 
                # Don't return to the main page if the user can't read it
                # otherwise we'll end up in a pointless loop
                $mainPage = Title::newMainPage();
-               if( $mainPage->userCanRead() )
+               if( $mainPage->userCanRead() ) {
                        $this->returnToMain( null, $mainPage );
+               }
        }
 
        /**
@@ -1834,26 +1845,30 @@ class OutputPage {
         * @return String: the wikitext error-messages, formatted into a list.
         */
        public function formatPermissionsErrorMessage( $errors, $action = null ) {
-               if ($action == null) {
-                       $text = wfMsgNoTrans( 'permissionserrorstext', count($errors)). "\n\n";
+               if ( $action == null ) {
+                       $text = wfMsgNoTrans( 'permissionserrorstext', count( $errors ) ) . "\n\n";
                } else {
-                       global $wgLang;
                        $action_desc = wfMsgNoTrans( "action-$action" );
-                       $text = wfMsgNoTrans( 'permissionserrorstext-withaction', count($errors), $action_desc ) . "\n\n";
+                       $text = wfMsgNoTrans(
+                               'permissionserrorstext-withaction',
+                               count( $errors ),
+                               $action_desc
+                       ) . "\n\n";
                }
 
-               if (count( $errors ) > 1) {
+               if ( count( $errors ) > 1 ) {
                        $text .= '<ul class="permissions-errors">' . "\n";
 
-                       foreach( $errors as $error )
-                       {
+                       foreach( $errors as $error ) {
                                $text .= '<li>';
                                $text .= call_user_func_array( 'wfMsgNoTrans', $error );
                                $text .= "</li>\n";
                        }
                        $text .= '</ul>';
                } else {
-                       $text .= "<div class=\"permissions-errors\">\n" . call_user_func_array( 'wfMsgNoTrans', reset( $errors ) ) . "\n</div>";
+                       $text .= "<div class=\"permissions-errors\">\n" .
+                                       call_user_func_array( 'wfMsgNoTrans', reset( $errors ) ) .
+                                       "\n</div>";
                }
 
                return $text;
@@ -1888,25 +1903,16 @@ class OutputPage {
 
                // If no reason is given, just supply a default "I can't let you do
                // that, Dave" message.  Should only occur if called by legacy code.
-               if ( $protected && empty($reasons) ) {
+               if ( $protected && empty( $reasons ) ) {
                        $reasons[] = array( 'badaccess-group0' );
                }
 
-               if ( !empty($reasons) ) {
+               if ( !empty( $reasons ) ) {
                        // Permissions error
                        if( $source ) {
                                $this->setPageTitle( wfMsg( 'viewsource' ) );
                                $this->setSubtitle(
-                                       wfMsg(
-                                               'viewsourcefor',
-                                               $skin->link(
-                                                       $this->getTitle(),
-                                                       null,
-                                                       array(),
-                                                       array(),
-                                                       array( 'known', 'noclasses' )
-                                               )
-                                       )
+                                       wfMsg( 'viewsourcefor', $skin->linkKnown( $this->getTitle() ) )
                                );
                        } else {
                                $this->setPageTitle( wfMsg( 'badaccess' ) );
@@ -1916,7 +1922,7 @@ class OutputPage {
                        // Wiki is read only
                        $this->setPageTitle( wfMsg( 'readonly' ) );
                        $reason = wfReadOnlyReason();
-                       $this->wrapWikiMsg( "<div class='mw-readonly-error'>\n$1</div>", array( 'readonlytext', $reason ) );
+                       $this->wrapWikiMsg( "<div class='mw-readonly-error'>\n$1\n</div>", array( 'readonlytext', $reason ) );
                }
 
                // Show source, if supplied
@@ -1997,8 +2003,8 @@ class OutputPage {
        }
 
        public function showFatalError( $message ) {
-               $this->setPageTitle( wfMsg( "internalerror" ) );
-               $this->setRobotPolicy( "noindex,nofollow" );
+               $this->setPageTitle( wfMsg( 'internalerror' ) );
+               $this->setRobotPolicy( 'noindex,nofollow' );
                $this->setArticleRelated( false );
                $this->enableClientCache( false );
                $this->mRedirect = '';
@@ -2032,9 +2038,9 @@ class OutputPage {
         * @param $query String: query string
         * @param $text String text of the link (input is not escaped)
         */
-       public function addReturnTo( $title, $query=array(), $text=null ) {
+       public function addReturnTo( $title, $query = array(), $text = null ) {
                global $wgUser;
-               $this->addLink( array( 'rel' => 'next', 'href' => $title->getFullUrl() ) );
+               $this->addLink( array( 'rel' => 'next', 'href' => $title->getFullURL() ) );
                $link = wfMsgHtml(
                        'returnto',
                        $wgUser->getSkin()->link( $title, $text, array(), $query )
@@ -2079,12 +2085,12 @@ class OutputPage {
 
        /**
         * @param $sk Skin The given Skin
-        * @param $includeStyle Unused (?)
+        * @param $includeStyle Boolean: unused
         * @return String: The doctype, opening <html>, and head element.
         */
        public function headElement( Skin $sk, $includeStyle = true ) {
-               global $wgContLanguageCode, $wgOutputEncoding, $wgMimeType;
-               global $wgContLang, $wgUseTrackbacks, $wgStyleVersion, $wgHtml5;
+               global $wgOutputEncoding, $wgMimeType;
+               global $wgUseTrackbacks, $wgHtml5;
                global $wgUser, $wgRequest, $wgLang;
 
                if ( $sk->commonPrintStylesheet() ) {
@@ -2092,9 +2098,7 @@ class OutputPage {
                }
                $sk->setupUserCss( $this );
 
-               $dir = $wgContLang->getDir();
-               $htmlAttribs = array( 'lang' => $wgContLanguageCode, 'dir' => $dir );
-               $ret = Html::htmlHeader( $htmlAttribs );
+               $ret = Html::htmlHeader( array( 'lang' => wfUILang()->getCode() ) );
 
                if ( $this->getHTMLTitle() == '' ) {
                        $this->setHTMLTitle( wfMsg( 'pagetitle', $this->getPageTitle() ) );
@@ -2119,8 +2123,7 @@ class OutputPage {
                $ret .= implode( "\n", array(
                        $this->getHeadLinks(),
                        $this->buildCssLinks(),
-                       $this->getHeadScripts( $sk ),
-                       $this->getHeadItems(),
+                       $this->getHeadScripts( $sk ) . $this->getHeadItems(),
                ) );
                if ( $sk->usercss ) {
                        $ret .= Html::inlineStyle( $sk->usercss );
@@ -2140,13 +2143,17 @@ class OutputPage {
                # Crazy edit-on-double-click stuff
                $action = $wgRequest->getVal( 'action', 'view' );
 
-               if ( $this->getTitle()->getNamespace() != NS_SPECIAL
-               && !in_array( $action, array( 'edit', 'submit' ) )
-               && $wgUser->getOption( 'editondblclick' ) ) {
+               if (
+                       $this->getTitle()->getNamespace() != NS_SPECIAL &&
+                       !in_array( $action, array( 'edit', 'submit' ) ) &&
+                       $wgUser->getOption( 'editondblclick' )
+               )
+               {
                        $bodyAttrs['ondblclick'] = "document.location = '" . Xml::escapeJsString( $this->getTitle()->getEditURL() ) . "'";
                }
 
                # Class bloat
+               $dir = wfUILang()->getDir();
                $bodyAttrs['class'] = "mediawiki $dir";
 
                if ( $wgLang->capitalizeAllNouns() ) {
@@ -2180,23 +2187,25 @@ class OutputPage {
                global $wgUser, $wgRequest, $wgJsMimeType, $wgUseSiteJs;
                global $wgStylePath, $wgStyleVersion;
 
-               $scripts = Skin::makeGlobalVariablesScript( $sk->getSkinName() );
+               $scripts = Skin::makeGlobalVariablesScript( $sk->getSkinName() ) . "\n";
                $scripts .= Html::linkedScript( "{$wgStylePath}/common/wikibits.js?$wgStyleVersion" );
 
-               //add site JS if enabled:
+               // add site JS if enabled
                if( $wgUseSiteJs ) {
                        $jsCache = $wgUser->isLoggedIn() ? '&smaxage=0' : '';
-                       $this->addScriptFile(  Skin::makeUrl( '-',
+                       $this->addScriptFile(
+                               Skin::makeUrl(
+                                       '-',
                                        "action=raw$jsCache&gen=js&useskin=" .
                                        urlencode( $sk->getSkinName() )
-                                       )
-                               );
+                               )
+                       );
                }
 
-               //add user js if enabled:
+               // add user JS if enabled
                if( $this->isUserJsAllowed() && $wgUser->isLoggedIn() ) {
                        $action = $wgRequest->getVal( 'action', 'view' );
-                       if( $this->mTitle && $this->mTitle->isJsSubpage() and $sk->userCanPreview( $action ) ) {
+                       if( $this->mTitle && $this->mTitle->isJsSubpage() && $sk->userCanPreview( $action ) ) {
                                # XXX: additional security check/prompt?
                                $this->addInlineScript( $wgRequest->getText( 'wpTextbox1' ) );
                        } else {
@@ -2207,9 +2216,9 @@ class OutputPage {
                                                NS_USER,
                                                $userpage->getDBkey() . '/' . $name . '.js'
                                        );
-                                       if ( $scriptpage && $scriptpage->exists() ) {
+                                       if ( $scriptpage && $scriptpage->exists() && ( $scriptpage->getLength() > 0 ) ) {
                                                $userjs = $scriptpage->getLocalURL( 'action=raw&ctype=' . $wgJsMimeType );
-                                               $this->addScriptFile( $userjs );
+                                               $this->addScriptFile( $userjs, $scriptpage->getLatestRevID() );
                                        }
                                }
                        }
@@ -2233,7 +2242,7 @@ class OutputPage {
                $called = true;
 
                if ( !$wgHtml5 ) {
-                       $this->addMeta( 'http:Content-Style-Type', 'text/css' ); //bug 15835
+                       $this->addMeta( 'http:Content-Style-Type', 'text/css' ); // bug 15835
                }
                $this->addMeta( 'generator', "MediaWiki $wgVersion" );
 
@@ -2249,7 +2258,14 @@ class OutputPage {
                                "/<.*?" . ">/" => '',
                                "/_/" => ' '
                        );
-                       $this->addMeta( 'keywords', preg_replace(array_keys($strip), array_values($strip),implode( ",", $this->mKeywords ) ) );
+                       $this->addMeta(
+                               'keywords',
+                               preg_replace(
+                                       array_keys( $strip ),
+                                       array_values( $strip ),
+                                       implode( ',', $this->mKeywords )
+                               )
+                       );
                }
        }
 
@@ -2257,7 +2273,7 @@ class OutputPage {
         * @return string HTML tag links to be put in the header.
         */
        public function getHeadLinks() {
-               global $wgRequest, $wgFeed;
+               global $wgFeed;
 
                // Ideally this should happen earlier, somewhere. :P
                $this->addDefaultMeta();
@@ -2274,7 +2290,9 @@ class OutputPage {
                        $tags[] = Html::element( 'meta',
                                array(
                                        $a => $tag[0],
-                                       'content' => $tag[1] ) );
+                                       'content' => $tag[1]
+                               )
+                       );
                }
                foreach ( $this->mLinktags as $tag ) {
                        $tags[] = Html::element( 'link', $tag );
@@ -2291,7 +2309,8 @@ class OutputPage {
                                        $format,
                                        $link,
                                        # Used messages: 'page-rss-feed' and 'page-atom-feed' (for an easier grep)
-                                       wfMsg( "page-{$format}-feed", $this->getTitle()->getPrefixedText() ) );
+                                       wfMsg( "page-{$format}-feed", $this->getTitle()->getPrefixedText() )
+                               );
                        }
 
                        # Recent changes feed should appear on every page (except recentchanges,
@@ -2307,17 +2326,19 @@ class OutputPage {
 
                        if ( $wgOverrideSiteFeed ) {
                                foreach ( $wgOverrideSiteFeed as $type => $feedUrl ) {
-                                       $tags[] = $this->feedLink (
+                                       $tags[] = $this->feedLink(
                                                $type,
                                                htmlspecialchars( $feedUrl ),
-                                               wfMsg( "site-{$type}-feed", $wgSitename ) );
+                                               wfMsg( "site-{$type}-feed", $wgSitename )
+                                       );
                                }
                        } elseif ( $this->getTitle()->getPrefixedText() != $rctitle->getPrefixedText() ) {
                                foreach ( $wgAdvertisedFeedTypes as $format ) {
                                        $tags[] = $this->feedLink(
                                                $format,
                                                $rctitle->getLocalURL( "feed={$format}" ),
-                                               wfMsg( "site-{$format}-feed", $wgSitename ) ); # For grep: 'site-rss-feed', 'site-atom-feed'.
+                                               wfMsg( "site-{$format}-feed", $wgSitename ) # For grep: 'site-rss-feed', 'site-atom-feed'.
+                                       );
                                }
                        }
                }
@@ -2338,7 +2359,8 @@ class OutputPage {
                        'rel' => 'alternate',
                        'type' => "application/$type+xml",
                        'title' => $text,
-                       'href' => $url ) );
+                       'href' => $url )
+               );
        }
 
        /**
@@ -2350,16 +2372,19 @@ class OutputPage {
         * @param $condition String: for IE conditional comments, specifying an IE version
         * @param $dir String: set to 'rtl' or 'ltr' for direction-specific sheets
         */
-       public function addStyle( $style, $media='', $condition='', $dir='' ) {
+       public function addStyle( $style, $media = '', $condition = '', $dir = '' ) {
                $options = array();
                // Even though we expect the media type to be lowercase, but here we
                // force it to lowercase to be safe.
-               if( $media )
+               if( $media ) {
                        $options['media'] = $media;
-               if( $condition )
+               }
+               if( $condition ) {
                        $options['condition'] = $condition;
-               if( $dir )
+               }
+               if( $dir ) {
                        $options['dir'] = $dir;
+               }
                $this->styles[$style] = $options;
        }
 
@@ -2376,14 +2401,18 @@ class OutputPage {
         * These will be applied to various media & IE conditionals.
         */
        public function buildCssLinks() {
+               return implode( "\n", $this->buildCssLinksArray() );
+       }
+
+       public function buildCssLinksArray() {
                $links = array();
                foreach( $this->styles as $file => $options ) {
                        $link = $this->styleLink( $file, $options );
-                       if( $link )
-                               $links[] = $link;
+                       if( $link ) {
+                               $links[$file] = $link;
+                       }
                }
-
-               return implode( "\n", $links );
+               return $links;
        }
 
        /**
@@ -2395,13 +2424,11 @@ class OutputPage {
         * @return String: HTML fragment
         */
        protected function styleLink( $style, $options ) {
-               global $wgRequest;
-
                if( isset( $options['dir'] ) ) {
-                       global $wgContLang;
-                       $siteDir = $wgContLang->getDir();
-                       if( $siteDir != $options['dir'] )
+                       $siteDir = wfUILang()->getDir();
+                       if( $siteDir != $options['dir'] ) {
                                return '';
+                       }
                }
 
                if( isset( $options['media'] ) ) {
@@ -2475,13 +2502,13 @@ class OutputPage {
         * for when rate limiting has triggered.
         */
        public function rateLimited() {
-               $this->setPageTitle(wfMsg('actionthrottled'));
+               $this->setPageTitle( wfMsg( 'actionthrottled' ) );
                $this->setRobotPolicy( 'noindex,follow' );
                $this->setArticleRelated( false );
                $this->enableClientCache( false );
                $this->mRedirect = '';
                $this->clearHTML();
-               $this->setStatusCode(503);
+               $this->setStatusCode( 503 );
                $this->addWikiMsg( 'actionthrottledtext' );
 
                $this->returnToMain( null, $this->getTitle() );
@@ -2548,11 +2575,11 @@ class OutputPage {
         *
         * For example:
         *
-        *    $wgOut->wrapWikiMsg( "<div class='error'>\n$1</div>", 'some-error' );
+        *    $wgOut->wrapWikiMsg( "<div class='error'>\n$1\n</div>", 'some-error' );
         *
         * Is equivalent to:
         *
-        *    $wgOut->addWikiText( "<div class='error'>\n" . wfMsgNoTrans( 'some-error' ) . "</div>" );
+        *    $wgOut->addWikiText( "<div class='error'>\n" . wfMsgNoTrans( 'some-error' ) . "\n</div>" );
         *
         * The newline after opening div is needed in some wikitext. See bug 19226.
         */