Don't look for pipes in the root node.
[lhc/web/wiklou.git] / includes / OutputPage.php
index a7f7e2d..960b293 100644 (file)
@@ -4,62 +4,210 @@ if ( !defined( 'MEDIAWIKI' ) ) {
 }
 
 /**
+ * This class should be covered by a general architecture document which does
+ * not exist as of january 2011.  This is one of the Core class and should
+ * be read at least once by any new developers.
+ *
+ * This class is used to prepare the final rendering. A skin is then
+ * applied to the output parameters (links, javascript, html, categories ...).
+ * 
+ * Another class (fixme) handle sending the whole page to the client.
+ * 
+ * Some comments comes from a pairing session between Zak Greant and Ashar Voultoiz
+ * in november 2010.
+ *
  * @todo document
  */
 class OutputPage {
-       var $mMetatags = array(), $mKeywords = array(), $mLinktags = array();
+       /// Should be private. Used with addMeta() which adds <meta>
+       var $mMetatags = array();
+
+       /// <meta keyworkds="stuff"> most of the time the first 10 links to an article
+       var $mKeywords = array();
+
+       var $mLinktags = array();
+
+       /// Additional stylesheets. Looks like this is for extensions. Might be replaced by resource loader.
        var $mExtStyles = array();
-       var $mPagetitle = '', $mBodytext = '';
+
+       /// Should be private - has getter and setter. Contains the HTML title
+       var $mPagetitle = '';
+
+       /// Contains all of the <body> content. Should be private we got set/get accessors and the append() method.
+       var $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 = '';
+       public $mDebugtext = ''; // TODO: we might want to replace it by wfDebug() wfDebugLog()
+
+       /// Should be private. Stores contents of <title> tag
+       var $mHTMLtitle = '';
 
-       var $mHTMLtitle = '', $mIsarticle = true, $mPrintable = false;
-       var $mSubtitle = '', $mRedirect = '', $mStatusCode;
-       var $mLastModified = '', $mETag = false;
-       var $mCategoryLinks = array(), $mCategories = array(), $mLanguageLinks = array();
+       /// Should be private. Is the displayed content related to the source of the corresponding wiki article.
+       var $mIsarticle = true;
+
+       /**
+        * Should be private. We have to set isPrintable(). Some pages should
+        * never be printed (ex: redirections).
+        */
+       var $mPrintable = false;
+
+       /**
+        * Should be private. We have set/get/append methods.
+        *
+        * Contains the page subtitle. Special pages usually have some links here.
+        * Don't confuse with site subtitle added by skins.
+        */
+       var $mSubtitle = '';
+
+       var $mRedirect = '';
+       var $mStatusCode;
+
+       /**
+        * mLastModified and mEtag are used for sending cache control.
+        * The whole caching system should probably be moved in its own class.
+        */
+       var $mLastModified = '';
+
+       /**
+        * Should be private. No getter but used in sendCacheControl();
+        * Contains an HTTP Entity Tags (see RFC 2616 section 3.13) which is used
+        * as a unique identifier for the content. It is later used by the client
+        * to compare its cache version with the server version. Client sends
+        * headers If-Match and If-None-Match containing its local cache ETAG value.
+        *
+        * To get more information, you will have to look at HTTP1/1 protocols which
+        * is properly described in RFC 2616 : http://tools.ietf.org/html/rfc2616
+        */
+       var $mETag = false;
 
-       var $mScripts = '', $mLinkColours, $mPageLinkTitle = '', $mHeadItems = array();
+       var $mCategoryLinks = array();
+       var $mCategories = array();
+
+       /// Should be private. Associative array mapping language code to the page name
+       var $mLanguageLinks = array();
+
+       /**
+        * Should be private. Used for JavaScript (pre resource loader)
+        * We should split js / css.
+        * mScripts content is inserted as is in <head> by Skin. This might contains
+        * either a link to a stylesheet or inline css.
+        */
+       var $mScripts = '';
+
+       /**
+        * Inline CSS styles. Use addInlineStyle() sparsingly
+        */
+       var $mInlineStyles = '';
+
+       //
+       var $mLinkColours;
+
+       /**
+        * Used by skin template.
+        * Example: $tpl->set( 'displaytitle', $out->mPageLinkTitle );
+        */
+       var $mPageLinkTitle = '';
+
+       /// Array of elements in <head>. Parser might add its own headers!
+       var $mHeadItems = array();
+
+       // Next variables probably comes from the resource loader @todo FIXME
        var $mModules = array(), $mModuleScripts = array(), $mModuleStyles = array(), $mModuleMessages = array();
        var $mResourceLoader;
+
+       /** @fixme is this still used ?*/
        var $mInlineMsg = array();
 
        var $mTemplateIds = array();
 
+       /** Initialized with a global value. Let us override it.
+        *  Should probably get deleted / rewritten ... */
        var $mAllowUserJs;
+
+       /**
+        * This was for the old skins and for users with 640x480 screen.
+        * Please note old skins are still used and might prove useful for
+        * users having old computers or visually impaired.
+        */
        var $mSuppressQuickbar = false;
+
+       /**
+        * @EasterEgg I just love the name for this self documenting variable.
+        * @todo document
+        */
        var $mDoNothing = false;
+
+       // Parser related.
        var $mContainsOldMagic = 0, $mContainsNewMagic = 0;
+
+       /**
+        * Should be private. Has get/set methods properly documented.
+        * Stores "article flag" toggle.
+        */
        var $mIsArticleRelated = true;
-       protected $mParserOptions = null; // lazy initialised, use parserOptions()
 
+       /// lazy initialised, use parserOptions()
+       protected $mParserOptions = null;
+
+       /**
+        * Handles the atom / rss links.
+        * We probably only support atom in 2011.
+        * Looks like a private variable.
+        * @see $wgAdvertisedFeedTypes
+        */
        var $mFeedLinks = array();
 
+       // Gwicke work on squid caching? Roughly from 2003.
        var $mEnableClientCache = true;
+
+       /**
+        * Flag if output should only contain the body of the article.
+        * Should be private.
+        */
        var $mArticleBodyOnly = false;
 
        var $mNewSectionLink = false;
        var $mHideNewSectionLink = false;
+
+       /**
+        * Comes from the parser. This was probably made to load CSS/JS only
+        * if we had <gallery>. Used directly in CategoryPage.php
+        * Looks like resource loader can replace this.
+        */
        var $mNoGallery = false;
+
+       // should be private.
        var $mPageTitleActionText = '';
        var $mParseWarnings = array();
+
+       // Cache stuff. Looks like mEnableClientCache
        var $mSquidMaxage = 0;
+
+       // @todo document
+       var $mPreventClickjacking = true;
+
+       /// should be private. To include the variable {{REVISIONID}}
        var $mRevisionId = null;
+
+       /// Stores a Title object (of the current page).
        protected $mTitle = null;
 
        /**
         * An array of stylesheet filenames (relative from skins path), with options
         * for CSS media, IE conditions, and RTL/LTR direction.
         * For internal use; add settings in the skin via $this->addStyle()
+        *
+        * Style again! This seems like a code duplication since we already have
+        * mStyles. This is what makes OpenSource amazing.
         */
        var $styles = array();
 
        /**
-        * Whether to load jQuery core.
+        * Whether jQuery is already handled.
         */
        protected $mJQueryDone = false;
 
@@ -104,7 +252,6 @@ class OutputPage {
         * Set the HTTP status code to send with the output.
         *
         * @param $statusCode Integer
-        * @return nothing
         */
        public function setStatusCode( $statusCode ) {
                $this->mStatusCode = $statusCode;
@@ -114,8 +261,8 @@ class OutputPage {
         * Add a new <meta> tag
         * To add an http-equiv meta tag, precede the name with "http:"
         *
-        * @param $name tag name
-        * @param $val tag value
+        * @param $name String tag name
+        * @param $val String tag value
         */
        function addMeta( $name, $val ) {
                array_push( $this->mMetatags, array( $name, $val ) );
@@ -180,7 +327,7 @@ class OutputPage {
        }
 
        /**
-        * Get all links added by extensions
+        * Get all styles added by extensions
         *
         * @return Array
         */
@@ -232,7 +379,7 @@ class OutputPage {
         * @return Array of module names
         */
        public function getModules() {
-               return $this->mModules;
+               return array_values( array_unique( $this->mModules ) );
        }
 
        /**
@@ -251,7 +398,7 @@ class OutputPage {
         * @return array of module names
         */
        public function getModuleScripts() {
-               return $this->mModuleScripts;
+               return array_values( array_unique( $this->mModuleScripts ) );
        }
 
        /**
@@ -271,7 +418,7 @@ class OutputPage {
         * @return Array of module names
         */
        public function getModuleStyles() {
-               return $this->mModuleStyles;
+               return array_values( array_unique( $this->mModuleStyles ) );
        }
 
        /**
@@ -291,7 +438,7 @@ class OutputPage {
         * @return Array of module names
         */
        public function getModuleMessages() {
-               return $this->mModuleMessages;
+               return array_values( array_unique( $this->mModuleMessages ) );
        }
 
        /**
@@ -999,14 +1146,6 @@ class OutputPage {
                $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
         *
@@ -1117,43 +1256,6 @@ class OutputPage {
                wfProfileOut( __METHOD__ );
        }
 
-       /**
-        * 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->isCacheable() ) {
-                       $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
         *
@@ -1226,24 +1328,44 @@ class OutputPage {
         * @param $interface Boolean: use interface language ($wgLang instead of
         *                   $wgContLang) while parsing language sensitive magic
         *                   words like GRAMMAR and PLURAL
+        * @param $language  Language object: target language object, will override
+        *                   $interface
         * @return String: HTML
         */
-       public function parse( $text, $linestart = true, $interface = false ) {
+       public function parse( $text, $linestart = true, $interface = false, $language = null ) {
+               // Check one for one common cause for parser state resetting
+               $callers = wfGetAllCallers( 10 );
+               if ( strpos( $callers, 'Parser::extensionSubstitution' ) !== false ) {
+                       throw new MWException( "wfMsg* function with parsing cannot be used " .
+                               "inside a tag hook. Should use parser->recursiveTagParse() instead" );
+               }
+
                global $wgParser;
+
                if( is_null( $this->getTitle() ) ) {
                        throw new MWException( 'Empty $mTitle in ' . __METHOD__ );
                }
+
                $popts = $this->parserOptions();
                if ( $interface ) {
                        $popts->setInterfaceMessage( true );
                }
+               if ( $language !== null ) {
+                       $oldLang = $popts->setTargetLanguage( $language );
+               }
+
                $parserOutput = $wgParser->parse(
                        $text, $this->getTitle(), $popts,
                        $linestart, true, $this->mRevisionId
                );
+
                if ( $interface ) {
                        $popts->setInterfaceMessage( false );
                }
+               if ( $language !== null ) {
+                       $popts->setTargetLanguage( $oldLang );
+               }
+
                return $parserOutput->getText();
        }
 
@@ -1268,24 +1390,6 @@ class OutputPage {
                return $parsed;
        }
 
-       /**
-        * @deprecated
-        *
-        * @param $article Article
-        * @return Boolean: true if successful, else false.
-        */
-       public function tryParserCache( &$article ) {
-               wfDeprecated( __METHOD__ );
-               $parserOutput = ParserCache::singleton()->get( $article, $article->getParserOptions() );
-
-               if ( $parserOutput !== false ) {
-                       $this->addParserOutput( $parserOutput );
-                       return true;
-               } else {
-                       return false;
-               }
-       }
-
        /**
         * Set the value of the "s-maxage" part of the "Cache-control" HTTP header
         *
@@ -1426,7 +1530,7 @@ class OutputPage {
                                        continue;
                                } else {
                                        $aloption[] = 'string-contains=' . $variant;
-                                       
+
                                        // IE and some other browsers use another form of language code
                                        // in their Accept-Language header, like "zh-CN" or "zh-TW".
                                        // We should handle these too.
@@ -1442,6 +1546,41 @@ class OutputPage {
                }
        }
 
+       /**
+        * Set a flag which will cause an X-Frame-Options header appropriate for 
+        * edit pages to be sent. The header value is controlled by 
+        * $wgEditPageFrameOptions.
+        *
+        * This is the default for special pages. If you display a CSRF-protected 
+        * form on an ordinary view page, then you need to call this function.
+        */
+       public function preventClickjacking( $enable = true ) {
+               $this->mPreventClickjacking = $enable;
+       }
+
+       /**
+        * Turn off frame-breaking. Alias for $this->preventClickjacking(false).
+        * This can be called from pages which do not contain any CSRF-protected
+        * HTML form.
+        */
+       public function allowClickjacking() {
+               $this->mPreventClickjacking = false;
+       }
+
+       /**
+        * Get the X-Frame-Options header value (without the name part), or false 
+        * if there isn't one. This is used by Skin to determine whether to enable 
+        * JavaScript frame-breaking, for clients that don't support X-Frame-Options.
+        */
+       public function getFrameOptions() {
+               global $wgBreakFrames, $wgEditPageFrameOptions;
+               if ( $wgBreakFrames ) {
+                       return 'DENY';
+               } elseif ( $this->mPreventClickjacking && $wgEditPageFrameOptions ) {
+                       return $wgEditPageFrameOptions;
+               }
+       }
+
        /**
         * Send cache control HTTP headers
         */
@@ -1578,7 +1717,6 @@ class OutputPage {
                global $wgLanguageCode, $wgDebugRedirects, $wgMimeType;
                global $wgUseAjax, $wgAjaxWatch;
                global $wgEnableMWSuggest, $wgUniversalEditButton;
-               global $wgArticle;
 
                if( $this->mDoNothing ) {
                        return;
@@ -1617,8 +1755,7 @@ class OutputPage {
                $sk = $wgUser->getSkin();
 
                // Add base resources
-               $this->addModules( array( 'mediawiki.legacy.wikibits' ) );
-               $this->addModules( array( 'mediawiki.util' ) );
+               $this->addModules( array( 'mediawiki.legacy.wikibits', 'mediawiki.util' ) );
 
                // Add various resources if required
                if ( $wgUseAjax ) {
@@ -1627,7 +1764,7 @@ class OutputPage {
                        wfRunHooks( 'AjaxAddScript', array( &$this ) );
 
                        if( $wgAjaxWatch && $wgUser->isLoggedIn() ) {
-                               $this->addModules( 'mediawiki.legacy.ajaxwatch' );
+                               $this->addModules( 'mediawiki.action.watch.ajax' );
                        }
 
                        if ( $wgEnableMWSuggest && !$wgUser->getOption( 'disablesuggest', false ) ) {
@@ -1640,7 +1777,7 @@ class OutputPage {
                }
 
                if( $wgUniversalEditButton ) {
-                       if( isset( $wgArticle ) && $this->getTitle() && $this->getTitle()->quickUserCan( 'edit' )
+                       if( $this->isArticleRelated() && $this->getTitle() && $this->getTitle()->quickUserCan( 'edit' )
                                && ( $this->getTitle()->exists() || $this->getTitle()->quickUserCan( 'create' ) ) ) {
                                // Original UniversalEditButton
                                $msg = wfMsg( 'edit' );
@@ -1666,6 +1803,12 @@ class OutputPage {
                $wgRequest->response()->header( "Content-type: $wgMimeType; charset={$wgOutputEncoding}" );
                $wgRequest->response()->header( 'Content-language: ' . $wgLanguageCode );
 
+               // Prevent framing, if requested
+               $frameOptions = $this->getFrameOptions();
+               if ( $frameOptions ) {
+                       $wgRequest->response()->header( "X-Frame-Options: $frameOptions" );
+               }
+
                if ( $this->mArticleBodyOnly ) {
                        $this->out( $this->mBodytext );
                } else {
@@ -1702,32 +1845,6 @@ class OutputPage {
                print $outs;
        }
 
-       /**
-        * @todo document
-        */
-       public static function setEncodings() {
-               global $wgInputEncoding, $wgOutputEncoding;
-
-               $wgInputEncoding = strtolower( $wgInputEncoding );
-
-               if ( empty( $_SERVER['HTTP_ACCEPT_CHARSET'] ) ) {
-                       $wgOutputEncoding = strtolower( $wgOutputEncoding );
-                       return;
-               }
-               $wgOutputEncoding = $wgInputEncoding;
-       }
-
-       /**
-        * @deprecated use wfReportTime() instead.
-        *
-        * @return String
-        */
-       public function reportTime() {
-               wfDeprecated( __METHOD__ );
-               $time = wfReportTime();
-               return $time;
-       }
-
        /**
         * Produce a "user is blocked" page.
         *
@@ -1818,9 +1935,7 @@ class OutputPage {
                $this->mRedirect = '';
                $this->mBodytext = '';
 
-               array_unshift( $params, 'parse' );
-               array_unshift( $params, $msg );
-               $this->addHTML( call_user_func_array( 'wfMsgExt', $params ) );
+               $this->addWikiMsgArray( $msg, $params );
 
                $this->returnToMain();
        }
@@ -1905,7 +2020,7 @@ class OutputPage {
                $this->setPageTitle( wfMsg( 'loginreqtitle' ) );
                $this->setHtmlTitle( wfMsg( 'errorpagetitle' ) );
                $this->setRobotPolicy( 'noindex,nofollow' );
-               $this->setArticleFlag( false );
+               $this->setArticleRelated( false );
 
                $loginTitle = SpecialPage::getTitleFor( 'Userlogin' );
                $loginLink = $skin->link(
@@ -2065,53 +2180,6 @@ class OutputPage {
                $this->addModules( 'mediawiki.legacy.password' );
        }
 
-       /** @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__ );
-               throw new FatalError( $message );
-       }
-
-       /** @deprecated */
-       public function unexpectedValueError( $name, $val ) {
-               wfDeprecated( __METHOD__ );
-               throw new FatalError( wfMsg( 'unexpected', $name, $val ) );
-       }
-
-       /** @deprecated */
-       public function fileCopyError( $old, $new ) {
-               wfDeprecated( __METHOD__ );
-               throw new FatalError( wfMsg( 'filecopyerror', $old, $new ) );
-       }
-
-       /** @deprecated */
-       public function fileRenameError( $old, $new ) {
-               wfDeprecated( __METHOD__ );
-               throw new FatalError( wfMsg( 'filerenameerror', $old, $new ) );
-       }
-
-       /** @deprecated */
-       public function fileDeleteError( $name ) {
-               wfDeprecated( __METHOD__ );
-               throw new FatalError( wfMsg( 'filedeleteerror', $name ) );
-       }
-
-       /** @deprecated */
-       public function fileNotFoundError( $name ) {
-               wfDeprecated( __METHOD__ );
-               throw new FatalError( wfMsg( 'filenotfound', $name ) );
-       }
-
        public function showFatalError( $message ) {
                $this->setPageTitle( wfMsg( 'internalerror' ) );
                $this->setRobotPolicy( 'noindex,nofollow' );
@@ -2231,13 +2299,10 @@ class OutputPage {
                $ret .= Html::element( 'title', null, $this->getHTMLTitle() ) . "\n";
 
                $ret .= implode( "\n", array(
-                       $this->getHeadLinks(),
+                       $this->getHeadLinks( $sk ),
                        $this->buildCssLinks( $sk ),
-                       $this->getHeadItems(),
+                       $this->getHeadItems()
                ) );
-               if ( $sk->usercss ) {
-                       $ret .= Html::inlineStyle( $sk->usercss );
-               }
 
                if ( $wgUseTrackbacks && $this->isArticleRelated() ) {
                        $ret .= $this->getTitle()->trackbackRDF();
@@ -2309,7 +2374,7 @@ class OutputPage {
         */
        protected function makeResourceLoaderLink( Skin $skin, $modules, $only, $useESI = false ) {
                global $wgUser, $wgLang, $wgLoadScript, $wgResourceLoaderUseESI,
-                       $wgResourceLoaderInlinePrivateModules;
+                       $wgResourceLoaderInlinePrivateModules, $wgRequest;
                // Lazy-load ResourceLoader
                // TODO: Should this be a static function of ResourceLoader instead?
                // TODO: Divide off modules starting with "user", and add the user parameter to them
@@ -2319,17 +2384,24 @@ class OutputPage {
                        'skin' => $skin->getSkinName(),
                        'only' => $only,
                );
-               
+               // Propagate printable and handheld parameters if present
+               if ( $wgRequest->getBool( 'printable' ) ) {
+                       $query['printable'] = 1;
+               }
+               if ( $wgRequest->getBool( 'handheld' ) ) {
+                       $query['handheld'] = 1;
+               }
+
                if ( !count( $modules ) ) {
                        return '';
                }
-               
+
                if ( count( $modules ) > 1 ) {
                        // Remove duplicate module requests
                        $modules = array_unique( (array) $modules );
                        // Sort module names so requests are more uniform
                        sort( $modules );
-               
+
                        if ( ResourceLoader::inDebugMode() ) {
                                // Recursively call us for every item
                                $links = '';
@@ -2339,7 +2411,7 @@ class OutputPage {
                                return $links;
                        }
                }
-               
+
                // Create keyed-by-group list of module objects from modules list
                $groups = array();
                $resourceLoader = $this->getResourceLoader();
@@ -2458,7 +2530,7 @@ class OutputPage {
                        $action = $wgRequest->getVal( 'action', 'view' );
                        if( $this->mTitle && $this->mTitle->isJsSubpage() && $sk->userCanPreview( $action ) ) {
                                # XXX: additional security check/prompt?
-                               $this->addInlineScript( $wgRequest->getText( 'wpTextbox1' ) );
+                               $scripts .= Html::inlineScript( "\n" . $wgRequest->getText( 'wpTextbox1' ) . "\n" ) . "\n";
                        } else {
                                $scripts .= $this->makeResourceLoaderLink(
                                        $sk, array( 'user', 'user.options' ), 'scripts'
@@ -2469,7 +2541,7 @@ class OutputPage {
                if ( !$userOptionsAdded ) {
                        $scripts .= $this->makeResourceLoaderLink( $sk, 'user.options', 'scripts' );
                }
-               
+
                return $scripts;
        }
 
@@ -2517,7 +2589,7 @@ class OutputPage {
        /**
         * @return string HTML tag links to be put in the header.
         */
-       public function getHeadLinks() {
+       public function getHeadLinks( Skin $sk ) {
                global $wgFeed;
 
                // Ideally this should happen earlier, somewhere. :P
@@ -2587,7 +2659,6 @@ class OutputPage {
                                }
                        }
                }
-
                return implode( "\n", $tags );
        }
 
@@ -2638,14 +2709,17 @@ class OutputPage {
         * @param $style_css Mixed: inline CSS
         */
        public function addInlineStyle( $style_css ){
-               $this->mScripts .= Html::inlineStyle( $style_css );
+               $this->mInlineStyles .= Html::inlineStyle( $style_css );
        }
 
        /**
         * Build a set of <link>s for the stylesheets specified in the $this->styles array.
         * These will be applied to various media & IE conditionals.
+        * @param $sk Skin object
         */
        public function buildCssLinks( $sk ) {
+               $ret = '';
+               // Add ResourceLoader styles
                // Split the styles into three groups
                $styles = array( 'other' => array(), 'user' => array(), 'site' => array() );
                $resourceLoader = $this->getResourceLoader();
@@ -2653,15 +2727,24 @@ class OutputPage {
                        $group = $resourceLoader->getModule( $name )->getGroup();
                        // Modules in groups named "other" or anything different than "user" or "site" will
                        // be placed in the "other" group
-                       $styles[isset( $style[$group] ) ? $group : 'other'][] = $name;
-               }
-               // Add tags created using legacy methods
-               $tags = $this->buildCssLinksArray();
-               // Add ResourceLoader module style tags
-               $tags[] = $this->makeResourceLoaderLink(
-                       $sk, array_merge( $styles['other'], $styles['site'], $styles['user'] ), 'styles'
+                       $styles[isset( $styles[$group] ) ? $group : 'other'][] = $name;
+               }
+
+               // We want site and user styles to override dynamically added styles from modules, but we want
+               // dynamically added styles to override statically added styles from other modules. So the order
+               // has to be other, dynamic, site, user
+               // Add statically added styles for other modules
+               $ret .= $this->makeResourceLoaderLink( $sk, $styles['other'], 'styles' );
+               // Add normal styles added through addStyle()/addInlineStyle() here
+               $ret .= implode( "\n", $this->buildCssLinksArray() ) . $this->mInlineStyles;
+               // Add marker tag to mark the place where the client-side loader should inject dynamic styles
+               // We use a <meta> tag with a made-up name for this because that's valid HTML
+               $ret .= Html::element( 'meta', array( 'name' => 'ResourceLoaderDynamicStyles', 'content' => '' ) );
+               // Add site and user styles
+               $ret .= $this->makeResourceLoaderLink(
+                       $sk, array_merge( $styles['site'], $styles['user'] ), 'styles'
                );
-               return implode( "\n", $tags );
+               return $ret;
        }
 
        public function buildCssLinksArray() {
@@ -2692,7 +2775,7 @@ class OutputPage {
                }
 
                if( isset( $options['media'] ) ) {
-                       $media = $this->transformCssMedia( $options['media'] );
+                       $media = self::transformCssMedia( $options['media'] );
                        if( is_null( $media ) ) {
                                return '';
                        }
@@ -2724,7 +2807,7 @@ class OutputPage {
         * @param $media String: current value of the "media" attribute
         * @return String: modified value of the "media" attribute
         */
-       function transformCssMedia( $media ) {
+       public static function transformCssMedia( $media ) {
                global $wgRequest, $wgHandheldForIPhone;
 
                // Switch in on-screen display for media testing
@@ -2873,7 +2956,7 @@ class OutputPage {
         * @param $modules Array: list of jQuery modules which should be loaded
         * @return Array: the list of modules which were not loaded.
         * @since 1.16
-        * @deprecated No longer needed as of 1.17
+        * @deprecated @since 1.17
         */
        public function includeJQuery( $modules = array() ) {
                return array();