More rigorous fix for ImageMagick parameter interpretation (bug 23148 etc.) based...
[lhc/web/wiklou.git] / includes / OutputPage.php
index f87a6d9..802740e 100644 (file)
@@ -186,12 +186,12 @@ class OutputPage {
         */
        public function addScriptFile( $file ) {
                global $wgStylePath, $wgStyleVersion;
-               if( substr( $file, 0, 1 ) == '/' ) {
+               if( substr( $file, 0, 1 ) == '/' || preg_match( '#^[a-z]*://#i', $file ) ) {
                        $path = $file;
                } else {
                        $path =  "{$wgStylePath}/common/{$file}";
                }
-               $this->addScript( Html::linkedScript( "$path?$wgStyleVersion" ) );
+               $this->addScript( Html::linkedScript( wfAppendQuery( $path, $wgStyleVersion ) ) );
        }
 
        /**
@@ -257,7 +257,7 @@ class OutputPage {
        /**
         * Set whether the output should only contain the body of the article,
         * without any skin, sidebar, etc.
-        * Used e.g. when calling with "action=raw".
+        * Used e.g. when calling with "action=render".
         *
         * @param $only Boolean: whether to output only the body of the article
         */
@@ -364,6 +364,16 @@ class OutputPage {
                return true;
        }
 
+       /**
+        * Override the last modified timestamp
+        *
+        * @param $timestamp String: new timestamp, in a format readable by
+        *        wfTimestamp()
+        */
+       public function setLastModified( $timestamp ) {
+               $this->mLastModified = wfTimestamp( TS_RFC2822, $timestamp );
+       }
+
 
        /**
         * Set the robot policy for the page: <http://www.robotstxt.org/meta.html>
@@ -435,7 +445,8 @@ class OutputPage {
        }
 
        /**
-        * "HTML title" means the contents of <title>. It is stored as plain, unescaped text and will be run through htmlspecialchars in the skin file.
+        * "HTML title" means the contents of <title>.
+        * It is stored as plain, unescaped text and will be run through htmlspecialchars in the skin file.
         */
        public function setHTMLTitle( $name ) {
                $this->mHTMLtitle = $name;
@@ -451,10 +462,10 @@ class OutputPage {
        }
 
        /**
-        * "Page title" means the contents of <h1>. It is stored as a valid HTML fragment.
-        * This function allows good tags like <sup> in the <h1> tag, but not bad tags like <script>.
-        * This function automatically sets <title> to the same content as <h1> but with all tags removed.
-        * Bad tags that were escaped in <h1> will still be escaped in <title>, and good tags like <i> will be dropped entirely.
+        * "Page title" means the contents of \<h1\>. It is stored as a valid HTML fragment.
+        * This function allows good tags like \<sup\> in the \<h1\> tag, but not bad tags like \<script\>.
+        * This function automatically sets \<title\> to the same content as \<h1\> but with all tags removed.
+        * Bad tags that were escaped in \<h1\> will still be escaped in \<title\>, and good tags like \<i\> will be dropped entirely.
         */
        public function setPageTitle( $name ) {
                # change "<script>foo&bar</script>" to "&lt;script&gt;foo&amp;bar&lt;/script&gt;"
@@ -472,7 +483,7 @@ class OutputPage {
        }
 
        /**
-        * Return the "page title", i.e. the content of the <h1> tag.
+        * Return the "page title", i.e. the content of the \<h1\> tag.
         *
         * @return String
         */
@@ -567,6 +578,25 @@ class OutputPage {
        }
 
 
+       /**
+        * Show an "add new section" link?
+        *
+        * @return Boolean
+        */
+       public function showNewSectionLink() {
+               return $this->mNewSectionLink;
+       }
+
+       /**
+        * Forcibly hide the new section link?
+        *
+        * @return Boolean
+        */
+       public function forceHideNewSectionLink() {
+               return $this->mHideNewSectionLink;
+       }
+
+
        /**
         * Add or remove feed links in the page header
         * This is mainly kept for backward compatibility, see OutputPage::addFeedLink()
@@ -624,6 +654,14 @@ class OutputPage {
                return count( $this->mFeedLinks ) > 0;
        }
 
+       /**
+        * Return URLs for each supported syndication format for this page.
+        * @return array associating format keys with URLs
+        */
+       public function getSyndicationLinks() {
+               return $this->mFeedLinks;
+       }
+
        /**
         * Will currently always return null
         *
@@ -798,24 +836,102 @@ class OutputPage {
                return $this->mCategories;
        }
 
-       public function suppressQuickbar() { $this->mSuppressQuickbar = true; }
-       public function isQuickbarSuppressed() { return $this->mSuppressQuickbar; }
 
-       public function disallowUserJs() { $this->mAllowUserJs = false; }
-       public function isUserJsAllowed() { return $this->mAllowUserJs; }
+       /**
+        * Suppress the quickbar from the output, only for skin supporting
+        * the quickbar
+        */
+       public function suppressQuickbar() {
+               $this->mSuppressQuickbar = true;
+       }
 
-       public function prependHTML( $text ) { $this->mBodytext = $text . $this->mBodytext; }
-       public function addHTML( $text ) { $this->mBodytext .= $text; }
-       public function clearHTML() { $this->mBodytext = ''; }
-       public function getHTML() { return $this->mBodytext; }
-       public function debug( $text ) { $this->mDebugtext .= $text; }
+       /**
+        * Return whether the quickbar should be suppressed from the output
+        *
+        * @return Boolean
+        */
+       public function isQuickbarSuppressed() {
+               return $this->mSuppressQuickbar;
+       }
+
+
+       /**
+        * Remove user JavaScript from scripts to load
+        */
+       public function disallowUserJs() {
+               $this->mAllowUserJs = false;
+       }
+
+       /**
+        * Return whether user JavaScript is allowed for this page
+        *
+        * @return Boolean
+        */
+       public function isUserJsAllowed() {
+               return $this->mAllowUserJs;
+       }
+
+
+       /**
+        * Prepend $text to the body HTML
+        *
+        * @param $text String: HTML
+        */
+       public function prependHTML( $text ) {
+               $this->mBodytext = $text . $this->mBodytext;
+       }
+
+       /**
+        * Append $text to the body HTML
+        *
+        * @param $text String: HTML
+        */
+       public function addHTML( $text ) {
+               $this->mBodytext .= $text;
+       }
+
+       /**
+        * Clear the body HTML
+        */
+       public function clearHTML() {
+               $this->mBodytext = '';
+       }
 
-       /* @deprecated */
+       /**
+        * Get the body HTML
+        *
+        * @return String: HTML
+        */
+       public function getHTML() {
+               return $this->mBodytext;
+       }
+
+
+       /**
+        * Add $text to the debug output
+        *
+        * @param $text String: debug text
+        */
+       public function debug( $text ) {
+               $this->mDebugtext .= $text;
+       }
+
+
+       /**
+        * @deprecated use parserOptions() instead
+        */
        public function setParserOptions( $options ) {
                wfDeprecated( __METHOD__ );
                return $this->parserOptions( $options );
        }
 
+       /**
+        * Get/set the ParserOptions object to use for wikitext parsing
+        *
+        * @param $options either the ParserOption to use or null to only get the
+        *                 current ParserOption object
+        * @return current ParserOption object
+        */
        public function parserOptions( $options = null ) {
                if ( !$this->mParserOptions ) {
                        $this->mParserOptions = new ParserOptions;
@@ -826,40 +942,78 @@ class OutputPage {
        /**
         * Set the revision ID which will be seen by the wiki text parser
         * for things such as embedded {{REVISIONID}} variable use.
-        * @param mixed $revid an integer, or NULL
-        * @return mixed previous value
+        *
+        * @param $revid Mixed: an positive integer, or null
+        * @return Mixed: previous value
         */
        public function setRevisionId( $revid ) {
                $val = is_null( $revid ) ? null : intval( $revid );
                return wfSetVar( $this->mRevisionId, $val );
        }
 
+       /**
+        * Get the current revision ID
+        *
+        * @return Integer
+        */
        public function getRevisionId() {
                return $this->mRevisionId;
        }
 
        /**
         * Convert wikitext to HTML and add it to the buffer
-        * Default assumes that the current page title will
-        * be used.
+        * Default assumes that the current page title will be used.
         *
-        * @param string $text
-        * @param bool   $linestart
+        * @param $text String
+        * @param $linestart Boolean: is this the start of a line?
         */
        public function addWikiText( $text, $linestart = true ) {
                $title = $this->getTitle(); // Work arround E_STRICT
                $this->addWikiTextTitle( $text, $title, $linestart );
        }
 
-       public function addWikiTextWithTitle($text, &$title, $linestart = true) {
-               $this->addWikiTextTitle($text, $title, $linestart);
+       /**
+        * Add wikitext with a custom Title object
+        *
+        * @param $text String: wikitext
+        * @param $title Title object
+        * @param $linestart Boolean: is this the start of a line?
+        */
+       public function addWikiTextWithTitle( $text, &$title, $linestart = true ) {
+               $this->addWikiTextTitle( $text, $title, $linestart );
        }
 
-       function addWikiTextTitleTidy($text, &$title, $linestart = true) {
+       /**
+        * Add wikitext with a custom Title object and 
+        *
+        * @param $text String: wikitext
+        * @param $title Title object
+        * @param $linestart Boolean: is this the start of a line?
+        */
+       function addWikiTextTitleTidy( $text, &$title, $linestart = true ) {
                $this->addWikiTextTitle( $text, $title, $linestart, true );
        }
 
-       public function addWikiTextTitle($text, &$title, $linestart, $tidy = false) {
+       /**
+        * Add wikitext with tidy enabled
+        *
+        * @param $text String: wikitext
+        * @param $linestart Boolean: is this the start of a line?
+        */
+       public function addWikiTextTidy( $text, $linestart = true ) {
+               $title = $this->getTitle();
+               $this->addWikiTextTitleTidy($text, $title, $linestart);
+       }
+
+       /**
+        * Add wikitext with a custom Title object
+        *
+        * @param $text String: wikitext
+        * @param $title Title object
+        * @param $linestart Boolean: is this the start of a line?
+        * @param $tidy Boolean: whether to use tidy
+        */
+       public function addWikiTextTitle( $text, &$title, $linestart, $tidy = false ) {
                global $wgParser;
 
                wfProfileIn( __METHOD__ );
@@ -880,8 +1034,45 @@ class OutputPage {
        }
 
        /**
-        * @todo document
-        * @param ParserOutput object &$parserOutput
+        * Add wikitext to the buffer, assuming that this is the primary text for a page view
+        * Saves the text into the parser cache if possible.
+        *
+        * @param $text String: wikitext
+        * @param $article Article object
+        * @param $cache Boolean
+        * @deprecated Use Article::outputWikitext
+        */
+       public function addPrimaryWikiText( $text, $article, $cache = true ) {
+               global $wgParser;
+
+               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 ) {
+                       $parserCache = ParserCache::singleton();
+                       $parserCache->save( $parserOutput, $article, $popts);
+               }
+
+               $this->addParserOutput( $parserOutput );
+       }
+
+       /**
+        * @deprecated use addWikiTextTidy()
+        */
+       public function addSecondaryWikiText( $text, $linestart = true ) {
+               wfDeprecated( __METHOD__ );
+               $this->addWikiTextTitleTidy($text, $this->getTitle(), $linestart);
+       }
+
+
+       /**
+        * Add a ParserOutput object, but without Html
+        *
+        * @param $parserOutput ParserOutput object
         */
        public function addParserOutputNoText( &$parserOutput ) {
                global $wgExemptFromUserRobotsControl, $wgContentNamespaces;
@@ -905,11 +1096,6 @@ class OutputPage {
                                $this->mTemplateIds[$ns] = $dbks;
                        }
                }
-               // Page title
-               $title = $parserOutput->getTitleText();
-               if ( $title != '' ) {
-                       $this->setPageTitle( $title );
-               }
 
                // Hooks registered in the object
                global $wgParserOutputHooks;
@@ -924,8 +1110,9 @@ class OutputPage {
        }
 
        /**
-        * @todo document
-        * @param ParserOutput &$parserOutput
+        * Add a ParserOutput object
+        *
+        * @param $parserOutput ParserOutput
         */
        function addParserOutput( &$parserOutput ) {
                $this->addParserOutputNoText( $parserOutput );
@@ -934,54 +1121,11 @@ class OutputPage {
                $this->addHTML( $text );
        }
 
-       /**
-        * Add wikitext to the buffer, assuming that this is the primary text for a page view
-        * Saves the text into the parser cache if possible.
-        *
-        * @param string  $text
-        * @param Article $article
-        * @param bool    $cache
-        * @deprecated Use Article::outputWikitext
-        */
-       public function addPrimaryWikiText( $text, $article, $cache = true ) {
-               global $wgParser;
-
-               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 ) {
-                       $parserCache = ParserCache::singleton();
-                       $parserCache->save( $parserOutput, $article, $popts);
-               }
-
-               $this->addParserOutput( $parserOutput );
-       }
-
-       /**
-        * @deprecated use addWikiTextTidy()
-        */
-       public function addSecondaryWikiText( $text, $linestart = true ) {
-               wfDeprecated( __METHOD__ );
-               $this->addWikiTextTitleTidy($text, $this->getTitle(), $linestart);
-       }
-
-       /**
-        * Add wikitext with tidy enabled
-        */
-       public function addWikiTextTidy(  $text, $linestart = true ) {
-               $title = $this->getTitle();
-               $this->addWikiTextTitleTidy($text, $title, $linestart);
-       }
-
 
        /**
         * Add the output of a QuickTemplate to the output buffer
         *
-        * @param QuickTemplate $template
+        * @param $template QuickTemplate
         */
        public function addTemplate( &$template ) {
                ob_start();
@@ -993,9 +1137,12 @@ class OutputPage {
        /**
         * Parse wikitext and return the HTML.
         *
-        * @param string $text
-        * @param bool   $linestart Is this the start of a line?
-        * @param bool   $interface ??
+        * @param $text String
+        * @param $linestart Boolean: is this the start of a line?
+        * @param $interface Boolean: use interface language ($wgLang instead of
+        *                   $wgContLang) while parsing language sensitive magic
+        *                   words like GRAMMAR and PLURAL
+        * @return String: HTML
         */
        public function parse( $text, $linestart = true, $interface = false ) {
                global $wgParser;
@@ -1010,7 +1157,16 @@ class OutputPage {
                return $parserOutput->getText();
        }
 
-       /** Parse wikitext, strip paragraphs, and return the HTML. */
+       /**
+        * Parse wikitext, strip paragraphs, and return the HTML.
+        *
+        * @param $text String
+        * @param $linestart Boolean: is this the start of a line?
+        * @param $interface Boolean: use interface language ($wgLang instead of
+        *                   $wgContLang) while parsing language sensitive magic
+        *                   words like GRAMMAR and PLURAL
+        * @return String: HTML
+        */
        public function parseInline( $text, $linestart = true, $interface = false ) {
                $parsed = $this->parse( $text, $linestart, $interface );
 
@@ -1023,12 +1179,10 @@ class OutputPage {
        }
 
        /**
-        * @param Article $article
-        * @param User    $user
-        *
         * @deprecated
         *
-        * @return bool True if successful, else false.
+        * @param $article Article
+        * @return Boolean: true if successful, else false.
         */
        public function tryParserCache( &$article ) {
                wfDeprecated( __METHOD__ );
@@ -1043,7 +1197,9 @@ class OutputPage {
        }
 
        /**
-        * @param int $maxage Maximum cache time on the Squid, in seconds.
+        * Set the value of the "s-maxage" part of the "Cache-control" HTTP header
+        *
+        * @param $maxage Integer: maximum cache time on the Squid, in seconds.
         */
        public function setSquidMaxage( $maxage ) {
                $this->mSquidMaxage = $maxage;
@@ -1051,12 +1207,18 @@ class OutputPage {
 
        /**
         * Use enableClientCache(false) to force it to send nocache headers
+        *
         * @param $state ??
         */
        public function enableClientCache( $state ) {
                return wfSetVar( $this->mEnableClientCache, $state );
        }
 
+       /**
+        * Get the list of cookies that will influence on the cache
+        *
+        * @return Array
+        */
        function getCacheVaryCookies() {
                global $wgCookiePrefix, $wgCacheVaryCookies;
                static $cookies;
@@ -1074,15 +1236,23 @@ class OutputPage {
                return $cookies;
        }
 
+       /**
+        * Return whether this page is not cacheable because "useskin" or "uselang"
+        * url parameters were passed
+        *
+        * @return Boolean
+        */
        function uncacheableBecauseRequestVars() {
                global $wgRequest;
-               return  $wgRequest->getText('useskin', false) === false
+               return $wgRequest->getText('useskin', false) === false
                        && $wgRequest->getText('uselang', false) === false;
        }
 
        /**
         * Check if the request has a cache-varying cookie header
         * If it does, it's very important that we don't allow public caching
+        *
+        * @return Boolean
         */
        function haveCacheVaryCookies() {
                global $wgRequest;
@@ -1102,6 +1272,12 @@ class OutputPage {
                return false;
        }
 
+       /**
+        * Add an HTTP header that will influence on the cache
+        *
+        * @param $header String: header name
+        * @param $option either an Array or null
+        */
        public function addVaryHeader( $header, $option = null ) {
                if ( !array_key_exists( $header, $this->mVaryHeader ) ) {
                        $this->mVaryHeader[$header] = $option;
@@ -1117,7 +1293,11 @@ class OutputPage {
                $this->mVaryHeader[$header] = array_unique( $this->mVaryHeader[$header] );
        }
 
-       /** Get a complete X-Vary-Options header */
+       /**
+        * Get a complete X-Vary-Options header
+        *
+        * @return String
+        */
        public function getXVO() {
                $cvCookies = $this->getCacheVaryCookies();
                
@@ -1139,14 +1319,14 @@ class OutputPage {
                return $xvo;
        }
 
-       /** bug 21672: Add Accept-Language to Vary and XVO headers
-               if there's no 'variant' parameter existed in GET.
-               
-               For example:
-                /w/index.php?title=Main_page should always be served; but
-                /w/index.php?title=Main_page&variant=zh-cn should never be served.
-               
-               patched by Liangent and Philip */
+       /**
+        * bug 21672: Add Accept-Language to Vary and XVO headers
+        * if there's no 'variant' parameter existed in GET.
+        *
+        * For example:
+        *   /w/index.php?title=Main_page should always be served; but
+        *   /w/index.php?title=Main_page&variant=zh-cn should never be served.
+        */
        function addAcceptLanguage() {
                global $wgRequest, $wgContLang;
                if( !$wgRequest->getCheck('variant') && $wgContLang->hasVariants() ) {
@@ -1162,6 +1342,9 @@ class OutputPage {
                }
        }
 
+       /**
+        * Send cache control HTTP headers
+        */
        public function sendCacheControl() {
                global $wgUseSquid, $wgUseESI, $wgUseETag, $wgSquidMaxage, $wgRequest, $wgUseXVO;
 
@@ -1222,7 +1405,6 @@ class OutputPage {
                        $response->header( 'Cache-Control: no-cache, no-store, max-age=0, must-revalidate' );
                        $response->header( 'Pragma: no-cache' );
                }
-               wfRunHooks('CacheHeadersAfterSet', array( $this ) );
        }
 
        /**
@@ -1394,7 +1576,8 @@ class OutputPage {
        /**
         * Actually output something with print(). Performs an iconv to the
         * output encoding, if needed.
-        * @param string $ins The string to output
+        *
+        * @param $ins String: the string to output
         */
        public function out( $ins ) {
                global $wgInputEncoding, $wgOutputEncoding, $wgContLang;
@@ -1424,9 +1607,9 @@ class OutputPage {
        }
 
        /**
-        * Deprecated, use wfReportTime() instead.
-        * @return string
-        * @deprecated
+        * @deprecated use wfReportTime() instead.
+        *
+        * @return String
         */
        public function reportTime() {
                wfDeprecated( __METHOD__ );
@@ -1437,7 +1620,7 @@ class OutputPage {
        /**
         * Produce a "user is blocked" page.
         *
-        * @param bool $return Whether to have a "return to $wgTitle" message or not.
+        * @param $return Boolean: whether to have a "return to $wgTitle" message or not.
         * @return nothing
         */
        function blockedPage( $return = true ) {
@@ -1499,9 +1682,9 @@ class OutputPage {
        /**
         * Output a standard error page
         *
-        * @param string $title Message key for page title
-        * @param string $msg Message key for page text
-        * @param array $params Message parameters
+        * @param $title String: message key for page title
+        * @param $msg String: message key for page text
+        * @param $params Array: message parameters
         */
        public function showErrorPage( $title, $msg, $params = array() ) {
                if ( $this->getTitle() ) {
@@ -1525,10 +1708,10 @@ class OutputPage {
        /**
         * Output a standard permission error page
         *
-        * @param array $errors Error message keys
+        * @param $errors Array: error message keys
+        * @param $action String: action that was denied or null if unknown
         */
-       public function showPermissionsErrorPage( $errors, $action = null )
-       {
+       public function showPermissionsErrorPage( $errors, $action = null ) {
                $this->mDebugtext .= 'Original title: ' .
                $this->getTitle()->getPrefixedText() . "\n";
                $this->setPageTitle( wfMsg( 'permissionserrors' ) );
@@ -1541,17 +1724,11 @@ class OutputPage {
                $this->addWikiText( $this->formatPermissionsErrorMessage( $errors, $action ) );
        }
 
-       /** @deprecated */
-       public function errorpage( $title, $msg ) {
-               wfDeprecated( __METHOD__ );
-               throw new ErrorPageError( $title, $msg );
-       }
-
        /**
         * Display an error page indicating that a given version of MediaWiki is
         * required to use it
         *
-        * @param mixed $version The version of MediaWiki needed to use the page
+        * @param $version Mixed: the version of MediaWiki needed to use the page
         */
        public function versionRequired( $version ) {
                $this->setPageTitle( wfMsg( 'versionrequired', $version ) );
@@ -1567,7 +1744,7 @@ class OutputPage {
        /**
         * Display an error page noting that a given permission bit is required.
         *
-        * @param string $permission key required
+        * @param $permission String: key required
         */
        public function permissionRequired( $permission ) {
                global $wgLang;
@@ -1591,16 +1768,14 @@ class OutputPage {
        }
 
        /**
-        * Use permissionRequired.
-        * @deprecated
+        * @deprecated use permissionRequired()
         */
        public function sysopRequired() {
                throw new MWException( "Call to deprecated OutputPage::sysopRequired() method\n" );
        }
 
        /**
-        * Use permissionRequired.
-        * @deprecated
+        * @deprecated use permissionRequired()
         */
        public function developerRequired() {
                throw new MWException( "Call to deprecated OutputPage::developerRequired() method\n" );
@@ -1642,14 +1817,12 @@ class OutputPage {
                        $this->returnToMain( null, $mainPage );
        }
 
-       /** @deprecated */
-       public function databaseError( $fname, $sql, $error, $errno ) {
-               throw new MWException( "OutputPage::databaseError is obsolete\n" );
-       }
-
        /**
-        * @param array $errors An array of arrays returned by Title::getUserPermissionsErrors
-        * @return string The wikitext error-messages, formatted into a list.
+        * Format a list of error messages
+        *
+        * @param $errors An array of arrays returned by Title::getUserPermissionsErrors
+        * @param $action String: action that was denied or null if unknown
+        * @return String: the wikitext error-messages, formatted into a list.
         */
        public function formatPermissionsErrorMessage( $errors, $action = null ) {
                if ($action == null) {
@@ -1692,9 +1865,10 @@ class OutputPage {
         *
         * @todo Needs to be split into multiple functions.
         *
-        * @param string $source    Source code to show (or null).
-        * @param bool   $protected Is this a permissions error?
-        * @param array  $reasons   List of reasons for this error, as returned by Title::getUserPermissionsErrors().
+        * @param $source    String: source code to show (or null).
+        * @param $protected Boolean: is this a permissions error?
+        * @param $reasons   Array: list of reasons for this error, as returned by Title::getUserPermissionsErrors().
+        * @param $action    String: action that was denied or null if unknown
         */
        public function readOnlyPage( $source = null, $protected = false, $reasons = array(), $action = null ) {
                global $wgUser;
@@ -1733,7 +1907,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</div>", array( 'readonlytext', $reason ) );
                }
 
                // Show source, if supplied
@@ -1766,6 +1940,17 @@ class OutputPage {
                }
        }
 
+       /** @deprecated */
+       public function errorpage( $title, $msg ) {
+               wfDeprecated( __METHOD__ );
+               throw new ErrorPageError( $title, $msg );
+       }
+
+       /** @deprecated */
+       public function databaseError( $fname, $sql, $error, $errno ) {
+               throw new MWException( "OutputPage::databaseError is obsolete\n" );
+       }
+
        /** @deprecated */
        public function fatalError( $message ) {
                wfDeprecated( __METHOD__ );
@@ -1836,12 +2021,15 @@ class OutputPage {
         *
         * @param $title Title to link
         * @param $query String: query string
+        * @param $text String text of the link (input is not escaped)
         */
-       public function addReturnTo( $title, $query = array() ) {
+       public function addReturnTo( $title, $query=array(), $text=null ) {
                global $wgUser;
                $this->addLink( array( 'rel' => 'next', 'href' => $title->getFullUrl() ) );
-               $link = wfMsgHtml( 'returnto', $wgUser->getSkin()->link(
-                       $title, null, array(), $query ) );
+               $link = wfMsgHtml( 
+                       'returnto', 
+                       $wgUser->getSkin()->link( $title, $text, array(), $query )
+               );
                $this->addHTML( "<p id=\"mw-returnto\">{$link}</p>\n" );
        }
 
@@ -1881,9 +2069,9 @@ class OutputPage {
        }
 
        /**
-        * @return string The doctype, opening <html>, and head element.
-        *
         * @param $sk Skin The given Skin
+        * @param $includeStyle Unused (?)
+        * @return String: The doctype, opening <html>, and head element.
         */
        public function headElement( Skin $sk, $includeStyle = true ) {
                global $wgDocType, $wgDTD, $wgContLanguageCode, $wgOutputEncoding, $wgMimeType;
@@ -1891,7 +2079,6 @@ class OutputPage {
                global $wgContLang, $wgUseTrackbacks, $wgStyleVersion, $wgHtml5, $wgWellFormedXml;
                global $wgUser, $wgRequest, $wgLang;
 
-               $this->addMeta( "http:Content-Type", "$wgMimeType; charset={$wgOutputEncoding}" );
                if ( $sk->commonPrintStylesheet() ) {
                        $this->addStyle( 'common/wikiprintable.css', 'print' );
                }
@@ -1904,11 +2091,12 @@ class OutputPage {
                }
 
                if ( $this->getHTMLTitle() == '' ) {
-                       $this->setHTMLTitle(  wfMsg( 'pagetitle', $this->getPageTitle() ));
+                       $this->setHTMLTitle( wfMsg( 'pagetitle', $this->getPageTitle() ) );
                }
 
                $dir = $wgContLang->getDir();
 
+               $htmlAttribs = array( 'lang' => $wgContLanguageCode, 'dir' => $dir );
                if ( $wgHtml5 ) {
                        if ( $wgWellFormedXml ) {
                                # Unknown elements and attributes are okay in XML, but unknown
@@ -1923,34 +2111,50 @@ class OutputPage {
                                # Much saner.
                                $ret .= "<!doctype html>\n";
                        }
-                       $ret .= "<html lang=\"$wgContLanguageCode\" dir=\"$dir\" ";
-                       if ( $wgHtml5Version ) $ret .= " version=\"$wgHtml5Version\" ";
-                       $ret .= ">\n";
+                       if ( $wgHtml5Version ) {
+                               $htmlAttribs['version'] = $wgHtml5Version;
+                       }
                } else {
                        $ret .= "<!DOCTYPE html PUBLIC \"$wgDocType\" \"$wgDTD\">\n";
-                       $ret .= "<html xmlns=\"{$wgXhtmlDefaultNamespace}\" ";
-                       foreach($wgXhtmlNamespaces as $tag => $ns) {
-                               $ret .= "xmlns:{$tag}=\"{$ns}\" ";
+                       $htmlAttribs['xmlns'] = $wgXhtmlDefaultNamespace;
+                       foreach ( $wgXhtmlNamespaces as $tag => $ns ) {
+                               $htmlAttribs["xmlns:$tag"] = $ns;
                        }
-                       $ret .= "xml:lang=\"$wgContLanguageCode\" lang=\"$wgContLanguageCode\" dir=\"$dir\">\n";
+                       $this->addMeta( 'http:Content-Type', "$wgMimeType; charset=$wgOutputEncoding" );
                }
+               $ret .= Html::openElement( 'html', $htmlAttribs ) . "\n";
 
-               $ret .= "<head>\n";
+               $openHead = Html::openElement( 'head' );
+               if ( $openHead ) {
+                       # Don't bother with the newline if $head == ''
+                       $ret .= "$openHead\n";
+               }
                $ret .= "<title>" . htmlspecialchars( $this->getHTMLTitle() ) . "</title>\n";
+
+               if ( $wgHtml5 ) {
+                       # More succinct than <meta http-equiv=Content-Type>, has the
+                       # same effect
+                       $ret .= Html::element( 'meta', array( 'charset' => $wgOutputEncoding ) ) . "\n";
+               }
+
                $ret .= implode( "\n", array(
                        $this->getHeadLinks(),
                        $this->buildCssLinks(),
                        $this->getHeadScripts( $sk ),
                        $this->getHeadItems(),
-               ));
-               if( $sk->usercss ){
+               ) );
+               if ( $sk->usercss ) {
                        $ret .= Html::inlineStyle( $sk->usercss );
                }
 
-               if ($wgUseTrackbacks && $this->isArticleRelated())
+               if ( $wgUseTrackbacks && $this->isArticleRelated() ) {
                        $ret .= $this->getTitle()->trackbackRDF();
+               }
 
-               $ret .= "</head>\n";
+               $closeHead = Html::closeElement( 'head' );
+               if ( $closeHead ) {
+                       $ret .= "$closeHead\n";
+               }
 
                $bodyAttrs = array();
 
@@ -1986,11 +2190,13 @@ class OutputPage {
                return $ret;
        }
 
-       /*
-        * gets the global variables and mScripts
+       /**
+        * Gets the global variables and mScripts; also adds userjs to the end if
+        * enabled
         *
-        * also adds userjs to the end if enabled:
-       */
+        * @param $sk Skin object to use
+        * @return String: HTML fragment
+        */
        function getHeadScripts( Skin $sk ) {
                global $wgUser, $wgRequest, $wgJsMimeType, $wgUseSiteJs;
                global $wgStylePath, $wgStyleVersion;
@@ -2016,13 +2222,16 @@ class OutputPage {
                                $this->addInlineScript( $wgRequest->getText( 'wpTextbox1' ) );
                        } else {
                                $userpage = $wgUser->getUserPage();
-                               $scriptpage = Title::makeTitleSafe(
-                                       NS_USER,
-                                       $userpage->getDBkey() . '/' . $sk->getSkinName() . '.js'
-                               );
-                               if ( $scriptpage && $scriptpage->exists() ) {
-                                       $userjs = Skin::makeUrl( $scriptpage->getPrefixedText(), 'action=raw&ctype=' . $wgJsMimeType );
-                                       $this->addScriptFile( $userjs );
+                               $names = array( 'common', $sk->getSkinName() );
+                               foreach( $names as $name ) {
+                                       $scriptpage = Title::makeTitleSafe(
+                                               NS_USER,
+                                               $userpage->getDBkey() . '/' . $name . '.js'
+                                       );
+                                       if ( $scriptpage && $scriptpage->exists() ) {
+                                               $userjs = $scriptpage->getLocalURL( 'action=raw&ctype=' . $wgJsMimeType );
+                                               $this->addScriptFile( $userjs );
+                                       }
                                }
                        }
                }
@@ -2031,6 +2240,9 @@ class OutputPage {
                return $scripts;
        }
 
+       /**
+        * Add default \<meta\> tags
+        */
        protected function addDefaultMeta() {
                global $wgVersion, $wgHtml5;
 
@@ -2135,15 +2347,12 @@ class OutputPage {
        }
 
        /**
-        * Return URLs for each supported syndication format for this page.
-        * @return array associating format keys with URLs
-        */
-       public function getSyndicationLinks() {
-               return $this->mFeedLinks;
-       }
-
-       /**
-        * Generate a <link rel/> for an RSS feed.
+        * Generate a <link rel/> for a feed.
+        *
+        * @param $type String: feed type
+        * @param $url String: URL to the feed
+        * @param $text String: value of the "title" attribute
+        * @return String: HTML fragment
         */
        private function feedLink( $type, $url, $text ) {
                return Html::element( 'link', array(
@@ -2157,9 +2366,10 @@ class OutputPage {
         * Add a local or specified stylesheet, with the given media options.
         * Meant primarily for internal use...
         *
-        * @param $media -- to specify a media type, 'screen', 'printable', 'handheld' or any.
-        * @param $conditional -- for IE conditional comments, specifying an IE version
-        * @param $dir -- set to 'rtl' or 'ltr' for direction-specific sheets
+        * @param $style String: URL to the file
+        * @param $media String: to specify a media type, 'screen', 'printable', 'handheld' or any.
+        * @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='' ) {
                $options = array();
@@ -2197,6 +2407,14 @@ class OutputPage {
                return implode( "\n", $links );
        }
 
+       /**
+        * Generate \<link\> tags for stylesheets
+        *
+        * @param $style String: URL to the file
+        * @param $options Array: option, can contain 'condition', 'dir', 'media'
+        *                 keys
+        * @return String: HTML fragment
+        */
        protected function styleLink( $style, $options ) {
                global $wgRequest;
 
@@ -2234,6 +2452,12 @@ class OutputPage {
                return $link;
        }
 
+       /**
+        * Transform "media" attribute based on request parameters
+        *
+        * @param $media String: current value of the "media" attribute
+        * @return String: modified value of the "media" attribute
+        */
        function transformCssMedia( $media ) {
                global $wgRequest, $wgHandheldForIPhone;
 
@@ -2272,7 +2496,6 @@ class OutputPage {
         * for when rate limiting has triggered.
         */
        public function rateLimited() {
-
                $this->setPageTitle(wfMsg('actionthrottled'));
                $this->setRobotPolicy( 'noindex,follow' );
                $this->setArticleRelated( false );
@@ -2285,24 +2508,6 @@ class OutputPage {
                $this->returnToMain( null, $this->getTitle() );
        }
 
-       /**
-        * Show an "add new section" link?
-        *
-        * @return bool
-        */
-       public function showNewSectionLink() {
-               return $this->mNewSectionLink;
-       }
-
-       /**
-       * Forcibly hide the new section link?
-       *
-       * @return bool
-       */
-       public function forceHideNewSectionLink() {
-               return $this->mHideNewSectionLink;
-       }
-
        /**
         * Show a warning about slave lag
         *
@@ -2310,7 +2515,7 @@ class OutputPage {
         * then the warning is a bit more obvious. If the lag is
         * lower than $wgSlaveLagWarning, then no warning is shown.
         *
-        * @param int $lag Slave lag
+        * @param $lag Integer: slave lag
         */
        public function showLagWarning( $lag ) {
                global $wgSlaveLagWarning, $wgSlaveLagCritical, $wgLang;
@@ -2368,7 +2573,7 @@ class OutputPage {
         *
         * Is equivalent to:
         *
-        *    $wgOut->addWikiText( "<div class='error'>\n" . wfMsgNoTrans( 'some-error' ) . '</div>' );
+        *    $wgOut->addWikiText( "<div class='error'>\n" . wfMsgNoTrans( 'some-error' ) . "</div>" );
         *
         * The newline after opening div is needed in some wikitext. See bug 19226.
         */
@@ -2399,23 +2604,20 @@ class OutputPage {
         * Include jQuery core. Use this to avoid loading it multiple times
         * before we get a usable script loader. 
         *
-        * @param array $modules List of jQuery modules which should be loaded
-        *
-        * Returns the list of modules which were not loaded.
+        * @param $modules Array: list of jQuery modules which should be loaded
+        * @return Array: the list of modules which were not loaded.
+        * @since 1.16
         */
        public function includeJQuery( $modules = array() ) {
-               global $wgScriptPath, $wgStyleVersion, $wgJsMimeType;
+               global $wgStylePath, $wgStyleVersion;
 
                $supportedModules = array( /** TODO: add things here */ );
                $unsupported = array_diff( $modules, $supportedModules );
 
-               $params = array(
-                       'type' => $wgJsMimeType,
-                       'src' => "$wgScriptPath/skins/common/jquery.min.js?$wgStyleVersion",
-               );
+               $url = "$wgStylePath/common/jquery.min.js?$wgStyleVersion";
                if ( !$this->mJQueryDone ) {
                        $this->mJQueryDone = true;
-                       $this->mScripts = Html::element( 'script', $params ) . "\n" . $this->mScripts;
+                       $this->mScripts = Html::linkedScript( $url ) . "\n" . $this->mScripts;
                }
                return $unsupported;
        }