* added redundand join condition for index usage as per comment on CR r88008
[lhc/web/wiklou.git] / includes / OutputPage.php
index c9b5c4b..a45a3ad 100644 (file)
@@ -18,7 +18,7 @@ if ( !defined( 'MEDIAWIKI' ) ) {
  *
  * @todo document
  */
-class OutputPage {
+class OutputPage extends ContextSource {
        /// Should be private. Used with addMeta() which adds <meta>
        var $mMetatags = array();
 
@@ -47,7 +47,13 @@ class OutputPage {
        var $mHTMLtitle = '';
 
        /// Should be private. Is the displayed content related to the source of the corresponding wiki article.
-       var $mIsarticle = true;
+       var $mIsarticle = false;
+
+       /**
+        * Should be private. Has get/set methods properly documented.
+        * Stores "article flag" toggle.
+        */
+       var $mIsArticleRelated = true;
 
        /**
         * Should be private. We have to set isPrintable(). Some pages should
@@ -125,9 +131,9 @@ class OutputPage {
        var $mTemplateIds = array();
        var $mImageTimeKeys = array();
 
-    var $mRedirectCode = '';
+       var $mRedirectCode = '';
 
-    var $mFeedLinksAppendQuery = null;
+       var $mFeedLinksAppendQuery = null;
 
        # What level of 'untrustworthiness' is allowed in CSS/JS modules loaded on this page?
        # @see ResourceLoaderModule::$origin
@@ -145,12 +151,6 @@ class OutputPage {
        // Parser related.
        var $mContainsOldMagic = 0, $mContainsNewMagic = 0;
 
-       /**
-        * Should be private. Has get/set methods properly documented.
-        * Stores "article flag" toggle.
-        */
-       var $mIsArticleRelated = true;
-
        /// lazy initialised, use parserOptions()
        protected $mParserOptions = null;
 
@@ -196,8 +196,6 @@ class OutputPage {
 
        var $mFileVersion = null;
 
-       private $mContext;
-
        /**
         * An array of stylesheet filenames (relative from skins path), with options
         * for CSS media, IE conditions, and RTL/LTR direction.
@@ -226,11 +224,12 @@ class OutputPage {
         * a OutputPage tied to that context.
         */
        function __construct( RequestContext $context = null ) {
-               if ( !isset($context) ) {
+               if ( $context === null ) {
                        # Extensions should use `new RequestContext` instead of `new OutputPage` now.
                        wfDeprecated( __METHOD__ );
+               } else {
+                       $this->setContext( $context );
                }
-               $this->mContext = $context;
        }
 
        /**
@@ -510,6 +509,15 @@ class OutputPage {
                $this->mModuleMessages = array_merge( $this->mModuleMessages, (array)$modules );
        }
 
+       /**
+        * Get an array of head items
+        *
+        * @return Array
+        */
+       function getHeadItemsArray() {
+               return $this->mHeadItems;
+       }
+
        /**
         * Get all header items in a string
         *
@@ -787,66 +795,15 @@ class OutputPage {
                return $this->mPagetitle;
        }
 
-       /**
-        * Get the RequestContext used in this instance
-        *
-        * @return RequestContext
-        */
-       private function getContext() {
-               if ( !isset($this->mContext) ) {
-                       wfDebug( __METHOD__ . " called and \$mContext is null. Using RequestContext::getMain(); for sanity\n" );
-                       $this->mContext = RequestContext::getMain();
-               }
-               return $this->mContext;
-       }
-
-       /**
-        * Get the WebRequest being used for this instance
-        *
-        * @return WebRequest
-        * @since 1.18
-        */
-       public function getRequest() {
-               return $this->getContext()->getRequest();
-       }
-
        /**
         * Set the Title object to use
         *
         * @param $t Title object
         */
-       public function setTitle( $t ) {
+       public function setTitle( Title $t ) {
                $this->getContext()->setTitle( $t );
        }
 
-       /**
-        * Get the Title object used in this instance
-        *
-        * @return Title
-        */
-       public function getTitle() {
-               return $this->getContext()->getTitle();
-       }
-
-       /**
-        * Get the User object used in this instance
-        *
-        * @return User
-        * @since 1.18
-        */
-       public function getUser() {
-               return $this->getContext()->getUser();
-       }
-
-       /**
-        * Get the Skin object used to render this instance
-        *
-        * @return Skin
-        * @since 1.18
-        */
-       public function getSkin() {
-               return $this->getContext()->getSkin();
-       }
 
        /**
         * Replace the subtile with $str
@@ -1279,6 +1236,7 @@ class OutputPage {
        public function parserOptions( $options = null ) {
                if ( !$this->mParserOptions ) {
                        $this->mParserOptions = new ParserOptions;
+                       $this->mParserOptions->setEditSection( false );
                }
                return wfSetVar( $this->mParserOptions, $options );
        }
@@ -1440,6 +1398,9 @@ class OutputPage {
                $this->mNoGallery = $parserOutput->getNoGallery();
                $this->mHeadItems = array_merge( $this->mHeadItems, $parserOutput->getHeadItems() );
                $this->addModules( $parserOutput->getModules() );
+               $this->addModuleScripts( $parserOutput->getModuleScripts() );
+               $this->addModuleStyles( $parserOutput->getModuleStyles() );
+               $this->addModuleMessages( $parserOutput->getModuleMessages() );
 
                // Template versioning...
                foreach ( (array)$parserOutput->getTemplateIds() as $ns => $dbks ) {
@@ -1504,13 +1465,6 @@ class OutputPage {
         * @return String: HTML
         */
        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() ) ) {
@@ -1858,7 +1812,7 @@ class OutputPage {
 
                if ( $this->mRedirect != '' ) {
                        # Standards require redirect URLs to be absolute
-                       $this->mRedirect = wfExpandUrl( $this->mRedirect );
+                       $this->mRedirect = wfExpandUrl( $this->mRedirect, PROTO_CURRENT );
                        if( $this->mRedirectCode == '301' || $this->mRedirectCode == '303' ) {
                                if( !$wgDebugRedirects ) {
                                        $message = HttpStatus::getMessage( $this->mRedirectCode );
@@ -1910,7 +1864,7 @@ class OutputPage {
                        wfRunHooks( 'BeforePageDisplay', array( &$this, &$sk ) );
 
                        wfProfileIn( 'Output-skin' );
-                       $sk->outputPage( $this );
+                       $sk->outputPage();
                        wfProfileOut( 'Output-skin' );
                }
 
@@ -1939,9 +1893,12 @@ class OutputPage {
        /**
         * Output a standard error page
         *
+        * showErrorPage( 'titlemsg', 'pagetextmsg', array( 'param1', 'param2' ) );
+        * showErrorPage( 'titlemsg', $messageObject );
+        *
         * @param $title String: message key for page title
-        * @param $msg String: message key for page text
-        * @param $params Array: message parameters
+        * @param $msg Mixed: message key (string) for page text, or a Message object
+        * @param $params Array: message parameters; ignored if $msg is a Message object
         */
        public function showErrorPage( $title, $msg, $params = array() ) {
                if ( $this->getTitle() ) {
@@ -1955,7 +1912,11 @@ class OutputPage {
                $this->mRedirect = '';
                $this->mBodytext = '';
 
-               $this->addWikiMsgArray( $msg, $params );
+               if ( $msg instanceof Message ){
+                       $this->addHTML( $msg->parse() );
+               } else {
+                       $this->addWikiMsgArray( $msg, $params );
+               }
 
                $this->returnToMain();
        }
@@ -2094,6 +2055,8 @@ class OutputPage {
         * @param $action    String: action that was denied or null if unknown
         */
        public function readOnlyPage( $source = null, $protected = false, $reasons = array(), $action = null ) {
+               global $wgEnableInterwikiTranscluding, $wgEnableInterwikiTemplatesTracking;
+
                $this->setRobotPolicy( 'noindex,nofollow' );
                $this->setArticleRelated( false );
 
@@ -2123,12 +2086,15 @@ class OutputPage {
                if( is_string( $source ) ) {
                        $this->addWikiMsg( 'viewsourcetext' );
 
+                       $pageLang = $this->getTitle()->getPageLanguage();
                        $params = array(
                                'id'   => 'wpTextbox1',
                                'name' => 'wpTextbox1',
                                'cols' => $this->getUser()->getOption( 'cols' ),
                                'rows' => $this->getUser()->getOption( 'rows' ),
-                               'readonly' => 'readonly'
+                               'readonly' => 'readonly',
+                               'lang' => $pageLang->getCode(),
+                               'dir' => $pageLang->getDir(),
                        );
                        $this->addHTML( Html::element( 'textarea', $params, $source ) );
 
@@ -2139,6 +2105,13 @@ class OutputPage {
 $templates
 </div>
 " );
+                       if ( $wgEnableInterwikiTranscluding && $wgEnableInterwikiTemplatesTracking ) {
+                               $distantTemplates = Linker::formatDistantTemplates( $article->getUsedDistantTemplates() );
+                               $this->addHTML( "<div class='distantTemplatesUsed'>
+$distantTemplates
+</div>
+" );
+                       }
                }
 
                # If the title doesn't exist, it's fairly pointless to print a return
@@ -2261,16 +2234,15 @@ $templates
         * @return String: The doctype, opening <html>, and head element.
         */
        public function headElement( Skin $sk, $includeStyle = true ) {
-               global $wgLang, $wgContLang, $wgUseTrackbacks;
-               $userdir = $wgLang->getDir();
+               global $wgContLang, $wgUseTrackbacks;
+               $userdir = $this->getLang()->getDir();
                $sitedir = $wgContLang->getDir();
 
                if ( $sk->commonPrintStylesheet() ) {
                        $this->addModuleStyles( 'mediawiki.legacy.wikiprintable' );
                }
-               $sk->setupUserCss( $this );
 
-               $ret = Html::htmlHeader( array( 'lang' => $wgLang->getCode(), 'dir' => $userdir ) );
+               $ret = Html::htmlHeader( array( 'lang' => $this->getLang()->getCode(), 'dir' => $userdir, 'class' => 'client-nojs' ) );
 
                if ( $this->getHTMLTitle() == '' ) {
                        $this->setHTMLTitle( wfMsg( 'pagetitle', $this->getPageTitle() ) );
@@ -2285,9 +2257,9 @@ $templates
                $ret .= Html::element( 'title', null, $this->getHTMLTitle() ) . "\n";
 
                $ret .= implode( "\n", array(
-                       $this->getHeadLinks( $sk, true ),
-                       $this->buildCssLinks( $sk ),
-                       $this->getHeadScripts( $sk ),
+                       $this->getHeadLinks( null, true ),
+                       $this->buildCssLinks(),
+                       $this->getHeadScripts(),
                        $this->getHeadItems()
                ) );
 
@@ -2302,20 +2274,6 @@ $templates
 
                $bodyAttrs = array();
 
-               # Crazy edit-on-double-click stuff
-               $action = $this->getRequest()->getVal( 'action', 'view' );
-
-               if (
-                       $this->getTitle()->getNamespace() != NS_SPECIAL &&
-                       in_array( $action, array( 'view', 'purge' ) ) &&
-                       $this->getUser()->getOption( 'editondblclick' )
-               )
-               {
-                       $editUrl = $this->getTitle()->getLocalUrl( $sk->editUrlOptions() );
-                       $bodyAttrs['ondblclick'] = "document.location = '" .
-                               Xml::escapeJsString( $editUrl ) . "'";
-               }
-
                # Classes for LTR/RTL directionality support
                $bodyAttrs['class'] = "mediawiki $userdir sitedir-$sitedir";
 
@@ -2363,7 +2321,7 @@ $templates
                        }
 
                        if ( $wgEnableMWSuggest && !$this->getUser()->getOption( 'disablesuggest', false ) ) {
-                               $this->addModules( 'mediawiki.page.mwsuggest' );
+                               $this->addModules( 'mediawiki.legacy.mwsuggest' );
                        }
                }
 
@@ -2371,6 +2329,11 @@ $templates
                        $this->addModules( 'mediawiki.action.view.rightClickEdit' );
                }
 
+               # Crazy edit-on-double-click stuff
+               if ( $this->isArticle() && $this->getUser()->getOption( 'editondblclick' ) ) {
+                       $this->addModules( 'mediawiki.action.view.dblClickEdit' );
+               }
+
                if ( $wgUseAJAXCategories ) {
                        global $wgAJAXCategoriesNamespaces;
 
@@ -2396,23 +2359,23 @@ $templates
 
        /**
         * TODO: Document
-        * @param $skin Skin
-        * @param $modules Array/string with the module name
+        * @param $modules Array/string with the module name(s)
         * @param $only String ResourceLoaderModule TYPE_ class constant
         * @param $useESI boolean
+        * @param $extraQuery Array with extra query parameters to add to each request. array( param => value )
         * @return string html <script> and <style> tags
         */
-       protected function makeResourceLoaderLink( Skin $skin, $modules, $only, $useESI = false ) {
+       protected function makeResourceLoaderLink( $modules, $only, $useESI = false, array $extraQuery = array() ) {
                global $wgLoadScript, $wgResourceLoaderUseESI,
                        $wgResourceLoaderInlinePrivateModules;
-               // Lazy-load ResourceLoader
-               // TODO: Should this be a static function of ResourceLoader instead?
                $baseQuery = array(
                        'lang' => $this->getContext()->getLang()->getCode(),
                        'debug' => ResourceLoader::inDebugMode() ? 'true' : 'false',
-                       'skin' => $skin->getSkinName(),
-                       'only' => $only,
-               );
+                       'skin' => $this->getSkin()->getSkinName(),
+               ) + $extraQuery;
+               if ( $only !== ResourceLoaderModule::TYPE_COMBINED ) {
+                       $baseQuery['only'] = $only;
+               }
                // Propagate printable and handheld parameters if present
                if ( $this->isPrintable() ) {
                        $baseQuery['printable'] = 1;
@@ -2435,7 +2398,7 @@ $templates
                                // Recursively call us for every item
                                $links = '';
                                foreach ( $modules as $name ) {
-                                       $links .= $this->makeResourceLoaderLink( $skin, $name, $only, $useESI );
+                                       $links .= $this->makeResourceLoaderLink( $name, $only, $useESI );
                                }
                                return $links;
                        }
@@ -2500,6 +2463,7 @@ $templates
                                                )
                                        );
                                }
+                               $links .= "\n";
                                continue;
                        }
                        // Special handling for the user group; because users might change their stuff
@@ -2552,12 +2516,11 @@ $templates
         * JS stuff to put in the <head>. This is the startup module, config
         * vars and modules marked with position 'top'
         *
-        * @param $sk Skin object to use
         * @return String: HTML fragment
         */
-       function getHeadScripts( Skin $sk ) {
+       function getHeadScripts() {
                // Startup - this will immediately load jquery and mediawiki modules
-               $scripts = $this->makeResourceLoaderLink( $sk, 'startup', ResourceLoaderModule::TYPE_SCRIPTS, true );
+               $scripts = $this->makeResourceLoaderLink( 'startup', ResourceLoaderModule::TYPE_SCRIPTS, true );
 
                // Load config before anything else
                $scripts .= Html::inlineScript(
@@ -2568,8 +2531,8 @@ $templates
 
                // Script and Messages "only" requests marked for top inclusion
                // Messages should go first
-               $scripts .= $this->makeResourceLoaderLink( $sk, $this->getModuleMessages( true, 'top' ), ResourceLoaderModule::TYPE_MESSAGES );
-               $scripts .= $this->makeResourceLoaderLink( $sk, $this->getModuleScripts( true, 'top' ), ResourceLoaderModule::TYPE_SCRIPTS );
+               $scripts .= $this->makeResourceLoaderLink( $this->getModuleMessages( true, 'top' ), ResourceLoaderModule::TYPE_MESSAGES );
+               $scripts .= $this->makeResourceLoaderLink( $this->getModuleScripts( true, 'top' ), ResourceLoaderModule::TYPE_SCRIPTS );
 
                // Modules requests - let the client calculate dependencies and batch requests as it likes
                // Only load modules that have marked themselves for loading at the top
@@ -2589,17 +2552,15 @@ $templates
         * JS stuff to put at the bottom of the <body>: modules marked with position 'bottom',
         * legacy scripts ($this->mScripts), user preferences, site JS and user JS
         *
-        * @param $sk Skin
-        *
         * @return string
         */
-       function getBottomScripts( Skin $sk ) {
+       function getBottomScripts() {
                global $wgUseSiteJs, $wgAllowUserJs;
 
                // Script and Messages "only" requests marked for bottom inclusion
                // Messages should go first
-               $scripts = $this->makeResourceLoaderLink( $sk, $this->getModuleMessages( true, 'bottom' ), ResourceLoaderModule::TYPE_MESSAGES );
-               $scripts .= $this->makeResourceLoaderLink( $sk, $this->getModuleScripts( true, 'bottom' ), ResourceLoaderModule::TYPE_SCRIPTS );
+               $scripts = $this->makeResourceLoaderLink( $this->getModuleMessages( true, 'bottom' ), ResourceLoaderModule::TYPE_MESSAGES );
+               $scripts .= $this->makeResourceLoaderLink( $this->getModuleScripts( true, 'bottom' ), ResourceLoaderModule::TYPE_SCRIPTS );
 
                // Modules requests - let the client calculate dependencies and batch requests as it likes
                // Only load modules that have marked themselves for loading at the bottom
@@ -2619,7 +2580,7 @@ $templates
 
                // Add site JS if enabled
                if ( $wgUseSiteJs ) {
-                       $scripts .= $this->makeResourceLoaderLink( $sk, 'site', ResourceLoaderModule::TYPE_SCRIPTS );
+                       $scripts .= $this->makeResourceLoaderLink( 'site', ResourceLoaderModule::TYPE_SCRIPTS );
                        if( $this->getUser()->isLoggedIn() ){
                                $userScripts[] = 'user.groups';
                        }
@@ -2627,17 +2588,23 @@ $templates
 
                // Add user JS if enabled
                if ( $wgAllowUserJs && $this->getUser()->isLoggedIn() ) {
-                       $action = $this->getRequest()->getVal( 'action', 'view' );
-                       if( $this->getTitle() && $this->getTitle()->isJsSubpage() && $sk->userCanPreview( $action ) ) {
+                       if( $this->getTitle() && $this->getTitle()->isJsSubpage() && $this->userCanPreview() ) {
                                # XXX: additional security check/prompt?
+                               // We're on a preview of a JS subpage
+                               // Exclude this page from the user module in case it's in there (bug 26283)
+                               $scripts .= $this->makeResourceLoaderLink( 'user', ResourceLoaderModule::TYPE_SCRIPTS, false,
+                                       array( 'excludepage' => $this->getTitle()->getPrefixedDBkey() )
+                               );
+                               // Load the previewed JS
                                $scripts .= Html::inlineScript( "\n" . $this->getRequest()->getText( 'wpTextbox1' ) . "\n" ) . "\n";
                        } else {
-                               # @todo FIXME: This means that User:Me/Common.js doesn't load when previewing
-                               # User:Me/Vector.js, and vice versa (bug26283)
-                               $userScripts[] = 'user';
+                               // Include the user module normally
+                               // We can't do $userScripts[] = 'user'; because the user module would end up
+                               // being wrapped in a closure, so load it raw like 'site'
+                               $scripts .= $this->makeResourceLoaderLink( 'user', ResourceLoaderModule::TYPE_SCRIPTS );
                        }
                }
-               $scripts .= $this->makeResourceLoaderLink( $sk, $userScripts, ResourceLoaderModule::TYPE_SCRIPTS );
+               $scripts .= $this->makeResourceLoaderLink( $userScripts, ResourceLoaderModule::TYPE_COMBINED );
 
                return $scripts;
        }
@@ -2691,18 +2658,42 @@ $templates
                }
 
                // Allow extensions to add their custom variables to the global JS variables
-               wfRunHooks( 'MakeGlobalVariablesScript', array( &$vars ) );
+               wfRunHooks( 'MakeGlobalVariablesScript', array( &$vars, &$this ) );
 
                return $vars;
        }
 
        /**
-        * @param $sk Skin
+        * To make it harder for someone to slip a user a fake
+        * user-JavaScript or user-CSS preview, a random token
+        * is associated with the login session. If it's not
+        * passed back with the preview request, we won't render
+        * the code.
+        *
+        * @return bool
+        */
+       public function userCanPreview() {
+               if ( $this->getRequest()->getVal( 'action' ) != 'submit'
+                       || !$this->getRequest()->wasPosted()
+                       || !$this->getUser()->matchEditToken(
+                               $this->getRequest()->getVal( 'wpEditToken' ) )
+               ) {
+                       return false;
+               }
+               if ( !$this->getTitle()->isJsSubpage() && !$this->getTitle()->isCssSubpage() ) {
+                       return false;
+               }
+
+               return !count( $this->getTitle()->getUserPermissionsErrors( 'edit', $this->getUser() ) );
+       }
+
+       /**
+        * @param $unused Unused
         * @param $addContentType bool
         *
         * @return string HTML tag links to be put in the header.
         */
-       public function getHeadLinks( Skin $sk, $addContentType = false ) {
+       public function getHeadLinks( $unused = null, $addContentType = false ) {
                global $wgUniversalEditButton, $wgFavicon, $wgAppleTouchIcon, $wgEnableAPI,
                        $wgSitename, $wgVersion, $wgHtml5, $wgMimeType,
                        $wgFeed, $wgOverrideSiteFeed, $wgAdvertisedFeedTypes,
@@ -2826,7 +2817,9 @@ $templates
                        $tags[] = Html::element( 'link', array(
                                'rel' => 'EditURI',
                                'type' => 'application/rsd+xml',
-                               'href' => wfExpandUrl( wfAppendQuery( wfScript( 'api' ), array( 'action' => 'rsd' ) ) ),
+                               // Output a protocol-relative URL here if $wgServer is protocol-relative
+                               // Whether RSD accepts relative or protocol-relative URLs is completely undocumented, though
+                               'href' => wfExpandUrl( wfAppendQuery( wfScript( 'api' ), array( 'action' => 'rsd' ) ), PROTO_RELATIVE ),
                        ) );
                }
 
@@ -2848,8 +2841,8 @@ $templates
                        } else {
                                $tags[] = Html::element( 'link', array(
                                        'rel' => 'canonical',
-                                       'href' => $this->getTitle()->getFullURL() )
-                               );
+                                       'href' => $this->getTitle()->getCanonicalUrl()
+                               ) );
                        }
                }
 
@@ -2974,17 +2967,53 @@ $templates
        /**
         * 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
         *
         * @return string
         */
-       public function buildCssLinks( $sk ) {
-               $ret = '';
+       public function buildCssLinks() {
+               global $wgUseSiteCss, $wgAllowUserCss, $wgAllowUserCssPrefs;
+
+               $this->getSkin()->setupSkinUserCss( $this );
+
                // Add ResourceLoader styles
                // Split the styles into four groups
                $styles = array( 'other' => array(), 'user' => array(), 'site' => array(), 'private' => array(), 'noscript' => array() );
+               $otherTags = ''; // Tags to append after the normal <link> tags
                $resourceLoader = $this->getResourceLoader();
-               foreach ( $this->getModuleStyles() as $name ) {
+
+               $moduleStyles = $this->getModuleStyles();
+
+               // Per-site custom styles
+               if ( $wgUseSiteCss ) {
+                       $moduleStyles[] = 'site';
+                       $moduleStyles[] = 'noscript';
+                       if( $this->getUser()->isLoggedIn() ){
+                               $moduleStyles[] = 'user.groups';
+                       }
+               }
+
+               // Per-user custom styles
+               if ( $wgAllowUserCss ) {
+                       if ( $this->getTitle()->isCssSubpage() && $this->userCanPreview() ) {
+                               // We're on a preview of a CSS subpage
+                               // Exclude this page from the user module in case it's in there (bug 26283)
+                               $otherTags .= $this->makeResourceLoaderLink( 'user', ResourceLoaderModule::TYPE_STYLES, false,
+                                       array( 'excludepage' => $this->getTitle()->getPrefixedDBkey() )
+                               );
+                               // Load the previewed CSS
+                               $otherTags .= Html::inlineStyle( $this->getRequest()->getText( 'wpTextbox1' ) );
+                       } else {
+                               // Load the user styles normally
+                               $moduleStyles[] = 'user';
+                       }
+               }
+
+               // Per-user preference styles
+               if ( $wgAllowUserCssPrefs ) {
+                       $moduleStyles[] = 'user.options';
+               }
+
+               foreach ( $moduleStyles as $name ) {
                        $group = $resourceLoader->getModule( $name )->getGroup();
                        // Modules in groups named "other" or anything different than "user", "site" or "private"
                        // will be placed in the "other" group
@@ -2995,7 +3024,7 @@ $templates
                // dynamically added styles to override statically added styles from other modules. So the order
                // has to be other, dynamic, site, private, user
                // Add statically added styles for other modules
-               $ret .= $this->makeResourceLoaderLink( $sk, $styles['other'], ResourceLoaderModule::TYPE_STYLES );
+               $ret = $this->makeResourceLoaderLink( $styles['other'], ResourceLoaderModule::TYPE_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
@@ -3006,15 +3035,25 @@ $templates
                // 'private' at present only contains user.options, so put that before 'user'
                // Any future private modules will likely have a similar user-specific character
                foreach ( array( 'site', 'noscript', 'private', 'user' ) as $group ) {
-                       $ret .= $this->makeResourceLoaderLink( $sk, $styles[$group],
+                       $ret .= $this->makeResourceLoaderLink( $styles[$group],
                                        ResourceLoaderModule::TYPE_STYLES
                        );
                }
+
+               // Add stuff in $otherTags (previewed user CSS if applicable)
+               $ret .= $otherTags;
                return $ret;
        }
 
        public function buildCssLinksArray() {
                $links = array();
+
+               // Add any extension CSS
+               foreach ( $this->mExtStyles as $url ) {
+                       $this->addStyle( $url );
+               }
+               $this->mExtStyles = array();
+
                foreach( $this->styles as $file => $options ) {
                        $link = $this->styleLink( $file, $options );
                        if( $link ) {
@@ -3034,8 +3073,7 @@ $templates
         */
        protected function styleLink( $style, $options ) {
                if( isset( $options['dir'] ) ) {
-                       global $wgLang;
-                       if( $wgLang->getDir() != $options['dir'] ) {
+                       if( $this->getLang()->getDir() != $options['dir'] ) {
                                return '';
                        }
                }