Add support for Number grouping(commafy) based on CLDR number grouping patterns like...
[lhc/web/wiklou.git] / includes / OutputPage.php
index 78900ff..8a97061 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.
@@ -225,12 +223,13 @@ class OutputPage {
         * Instead a new RequestContext should be created and it will implicitly create
         * a OutputPage tied to that context.
         */
-       function __construct( RequestContext $context = null ) {
-               if ( !isset($context) ) {
+       function __construct( IContextSource $context = null ) {
+               if ( $context === null ) {
                        # Extensions should use `new RequestContext` instead of `new OutputPage` now.
                        wfDeprecated( __METHOD__ );
+               } else {
+                       $this->setContext( $context );
                }
-               $this->mContext = $context;
        }
 
        /**
@@ -469,7 +468,7 @@ class OutputPage {
         *
         * @param $filter
         * @param $position
-        * 
+        *
         * @return Array of module names
         */
        public function getModuleStyles( $filter = false, $position = null ) {
@@ -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
@@ -1168,21 +1125,6 @@ class OutputPage {
         * @return Array of strings
         */
        public function getCategories() {
-               // mCategories will only be filled if we're coming from the parser
-               if ($this->getRequest()->getVal('action') != 'view') {
-                       $categories = array();
-                       $dbr = wfGetDB( DB_SLAVE );
-                       $res = $dbr->select(
-                               'categorylinks',
-                               array( 'cl_to' ),
-                               'cl_from='. $this->getTitle()->getArticleId(),
-                               __METHOD__
-                       );
-                       foreach( $res as $row ) {
-                               $categories[] = $row->cl_to;
-                       }
-                       $this->mCategories = $categories;
-               }
                return $this->mCategories;
        }
 
@@ -1294,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 );
        }
@@ -1358,7 +1301,7 @@ class OutputPage {
         * @return Array (dbKey => array('time' => MW timestamp or null, 'sha1' => sha1 or ''))
         * @since 1.18
         */
-       public function getImageTimeKeys() {
+       public function getFileSearchOptions() {
                return $this->mImageTimeKeys;
        }
 
@@ -1368,10 +1311,11 @@ class OutputPage {
         *
         * @param $text String
         * @param $linestart Boolean: is this the start of a line?
+        * @param $interface Boolean: is this text in the user interface language?
         */
-       public function addWikiText( $text, $linestart = true ) {
+       public function addWikiText( $text, $linestart = true, $interface = true ) {
                $title = $this->getTitle(); // Work arround E_STRICT
-               $this->addWikiTextTitle( $text, $title, $linestart );
+               $this->addWikiTextTitle( $text, $title, $linestart, /*tidy*/false, $interface );
        }
 
        /**
@@ -1414,8 +1358,10 @@ class OutputPage {
         * @param $title Title object
         * @param $linestart Boolean: is this the start of a line?
         * @param $tidy Boolean: whether to use tidy
+        * @param $interface Boolean: whether it is an interface message
+        *                                                              (for example disables conversion)
         */
-       public function addWikiTextTitle( $text, &$title, $linestart, $tidy = false ) {
+       public function addWikiTextTitle( $text, &$title, $linestart, $tidy = false, $interface = false ) {
                global $wgParser;
 
                wfProfileIn( __METHOD__ );
@@ -1424,6 +1370,7 @@ class OutputPage {
 
                $popts = $this->parserOptions();
                $oldTidy = $popts->setTidy( $tidy );
+               $popts->setInterfaceMessage( (bool) $interface );
 
                $parserOutput = $wgParser->parse(
                        $text, $title, $popts,
@@ -1455,6 +1402,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 ) {
@@ -1465,7 +1415,7 @@ class OutputPage {
                        }
                }
                // File versioning...
-               foreach ( (array)$parserOutput->getImageTimeKeys() as $dbk => $data ) {
+               foreach ( (array)$parserOutput->getFileSearchOptions() as $dbk => $data ) {
                        $this->mImageTimeKeys[$dbk] = $data;
                }
 
@@ -1513,19 +1463,13 @@ class OutputPage {
         * @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
+        *                   words like GRAMMAR and PLURAL. This also disables
+        *                                       LanguageConverter.
         * @param $language  Language object: target language object, will override
         *                   $interface
         * @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() ) ) {
@@ -1847,9 +1791,9 @@ class OutputPage {
         *
         * @param $code Integer: status code
         * @return String or null: message or null if $code is not in the list of
-        *         messages 
+        *         messages
         *
-        * @deprecated since 1.19 Use HttpStatus::getMessage() instead.
+        * @deprecated since 1.18 Use HttpStatus::getMessage() instead.
         */
        public static function getStatusMessage( $code ) {
                wfDeprecated( __METHOD__ );
@@ -1873,7 +1817,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 );
@@ -1925,7 +1869,7 @@ class OutputPage {
                        wfRunHooks( 'BeforePageDisplay', array( &$this, &$sk ) );
 
                        wfProfileIn( 'Output-skin' );
-                       $sk->outputPage( $this );
+                       $sk->outputPage();
                        wfProfileOut( 'Output-skin' );
                }
 
@@ -1954,9 +1898,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() ) {
@@ -1970,7 +1917,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();
        }
@@ -2034,15 +1985,14 @@ class OutputPage {
                $this->setArticleRelated( false );
 
                $loginTitle = SpecialPage::getTitleFor( 'Userlogin' );
-               $loginLink = Linker::link(
+               $loginLink = Linker::linkKnown(
                        $loginTitle,
                        wfMsgHtml( 'loginreqlink' ),
                        array(),
-                       array( 'returnto' => $this->getTitle()->getPrefixedText() ),
-                       array( 'known', 'noclasses' )
+                       array( 'returnto' => $this->getTitle()->getPrefixedText() )
                );
-               $this->addWikiMsgArray( 'loginreqpagetext', array( $loginLink ), array( 'replaceafter' ) );
-               $this->addHTML( "\n<!--" . $this->getTitle()->getPrefixedUrl() . '-->' );
+               $this->addHTML( wfMessage( 'loginreqpagetext' )->rawParams( $loginLink )->parse() .
+                       "\n<!--" . $this->getTitle()->getPrefixedUrl() . '-->' );
 
                # Don't return to the main page if the user can't read it
                # otherwise we'll end up in a pointless loop
@@ -2110,6 +2060,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 );
 
@@ -2139,12 +2091,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 ) );
 
@@ -2155,6 +2110,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
@@ -2277,14 +2239,15 @@ $templates
         * @return String: The doctype, opening <html>, and head element.
         */
        public function headElement( Skin $sk, $includeStyle = true ) {
-               global $wgUseTrackbacks, $wgLang;
+               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' => $wgLang->getDir() ) );
+               $ret = Html::htmlHeader( array( 'lang' => $this->getLang()->getCode(), 'dir' => $userdir, 'class' => 'client-nojs' ) );
 
                if ( $this->getHTMLTitle() == '' ) {
                        $this->setHTMLTitle( wfMsg( 'pagetitle', $this->getPageTitle() ) );
@@ -2299,9 +2262,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()
                ) );
 
@@ -2316,24 +2279,7 @@ $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
-               global $wgLang, $wgContLang;
-               $userdir = $wgLang->getDir();
-               $sitedir = $wgContLang->getDir();
                $bodyAttrs['class'] = "mediawiki $userdir sitedir-$sitedir";
 
                if ( $this->getContext()->getLang()->capitalizeAllNouns() ) {
@@ -2355,8 +2301,7 @@ $templates
         * Add the default ResourceLoader modules to this object
         */
        private function addDefaultModules() {
-               global $wgIncludeLegacyJavaScript,
-                       $wgUseAjax, $wgAjaxWatch, $wgEnableMWSuggest;
+               global $wgIncludeLegacyJavaScript, $wgUseAjax, $wgAjaxWatch, $wgEnableMWSuggest;
 
                // Add base resources
                $this->addModules( array(
@@ -2380,13 +2325,18 @@ $templates
                        }
 
                        if ( $wgEnableMWSuggest && !$this->getUser()->getOption( 'disablesuggest', false ) ) {
-                               $this->addModules( 'mediawiki.page.mwsuggest' );
+                               $this->addModules( 'mediawiki.legacy.mwsuggest' );
                        }
                }
 
-               if( $this->getUser()->getBoolOption( 'editsectiononrightclick' ) ) {
+               if ( $this->getUser()->getBoolOption( 'editsectiononrightclick' ) ) {
                        $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' );
+               }
        }
 
        /**
@@ -2403,30 +2353,15 @@ $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,
-               );
-               // Propagate printable and handheld parameters if present
-               if ( $this->isPrintable() ) {
-                       $baseQuery['printable'] = 1;
-               }
-               if ( $this->getRequest()->getBool( 'handheld' ) ) {
-                       $baseQuery['handheld'] = 1;
-               }
 
                if ( !count( $modules ) ) {
                        return '';
@@ -2442,7 +2377,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;
                        }
@@ -2472,14 +2407,26 @@ $templates
 
                $links = '';
                foreach ( $groups as $group => $modules ) {
-                       $query = $baseQuery;
                        // Special handling for user-specific groups
+                       $user = null;
                        if ( ( $group === 'user' || $group === 'private' ) && $this->getUser()->isLoggedIn() ) {
-                               $query['user'] = $this->getUser()->getName();
+                               $user = $this->getUser()->getName();
                        }
 
                        // Create a fake request based on the one we are about to make so modules return
                        // correct timestamp and emptiness data
+                       $query = ResourceLoader::makeLoaderQuery(
+                               array(), // modules; not determined yet
+                               $this->getContext()->getLang()->getCode(),
+                               $this->getSkin()->getSkinName(),
+                               $user,
+                               null, // version; not determined yet
+                               ResourceLoader::inDebugMode(),
+                               $only === ResourceLoaderModule::TYPE_COMBINED ? null : $only,
+                               $this->isPrintable(),
+                               $this->getRequest()->getBool( 'handheld' ),
+                               $extraQuery
+                       );
                        $context = new ResourceLoaderContext( $resourceLoader, new FauxRequest( $query ) );
                        // Drop modules that know they're empty
                        foreach ( $modules as $key => $module ) {
@@ -2492,8 +2439,6 @@ $templates
                                continue;
                        }
 
-                       $query['modules'] = ResourceLoader::makePackedModulesString( array_keys( $modules ) );
-
                        // Support inlining of private modules if configured as such
                        if ( $group === 'private' && $wgResourceLoaderInlinePrivateModules ) {
                                if ( $only == ResourceLoaderModule::TYPE_STYLES ) {
@@ -2507,6 +2452,7 @@ $templates
                                                )
                                        );
                                }
+                               $links .= "\n";
                                continue;
                        }
                        // Special handling for the user group; because users might change their stuff
@@ -2514,6 +2460,7 @@ $templates
                        // timestamp of these user-changable modules so we can ensure cache misses on change
                        // This should NOT be done for the site group (bug 27564) because anons get that too
                        // and we shouldn't be putting timestamps in Squid-cached HTML
+                       $version = null;
                        if ( $group === 'user' ) {
                                // Get the maximum timestamp
                                $timestamp = 1;
@@ -2521,15 +2468,21 @@ $templates
                                        $timestamp = max( $timestamp, $module->getModifiedTime( $context ) );
                                }
                                // Add a version parameter so cache will break when things change
-                               $query['version'] = wfTimestamp( TS_ISO_8601_BASIC, $timestamp );
+                               $version = wfTimestamp( TS_ISO_8601_BASIC, $timestamp );
                        }
-                       // Make queries uniform in order
-                       ksort( $query );
-
-                       $url = wfAppendQuery( $wgLoadScript, $query );
-                       // Prevent the IE6 extension check from being triggered (bug 28840)
-                       // by appending a character that's invalid in Windows extensions ('*')
-                       $url .= '&*';
+                       
+                       $url = ResourceLoader::makeLoaderURL(
+                               array_keys( $modules ),
+                               $this->getContext()->getLang()->getCode(),
+                               $this->getSkin()->getSkinName(),
+                               $user,
+                               $version,
+                               ResourceLoader::inDebugMode(),
+                               $only === ResourceLoaderModule::TYPE_COMBINED ? null : $only,
+                               $this->isPrintable(),
+                               $this->getRequest()->getBool( 'handheld' ),
+                               $extraQuery
+                       );
                        if ( $useESI && $wgResourceLoaderUseESI ) {
                                $esi = Xml::element( 'esi:include', array( 'src' => $url ) );
                                if ( $only == ResourceLoaderModule::TYPE_STYLES ) {
@@ -2559,12 +2512,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(
@@ -2575,8 +2527,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
@@ -2596,17 +2548,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
@@ -2626,7 +2576,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';
                        }
@@ -2634,17 +2584,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;
        }
@@ -2698,18 +2654,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,
@@ -2833,7 +2813,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 ),
                        ) );
                }
 
@@ -2855,8 +2837,8 @@ $templates
                        } else {
                                $tags[] = Html::element( 'link', array(
                                        'rel' => 'canonical',
-                                       'href' => $this->getTitle()->getFullURL() )
-                               );
+                                       'href' => $this->getTitle()->getCanonicalUrl()
+                               ) );
                        }
                }
 
@@ -2909,9 +2891,10 @@ $templates
 
                        if ( $wgOverrideSiteFeed ) {
                                foreach ( $wgOverrideSiteFeed as $type => $feedUrl ) {
+                                       // Note, this->feedLink escapes the url.
                                        $tags[] = $this->feedLink(
                                                $type,
-                                               htmlspecialchars( $feedUrl ),
+                                               $feedUrl,
                                                wfMsg( "site-{$type}-feed", $wgSitename )
                                        );
                                }
@@ -2973,25 +2956,66 @@ $templates
        /**
         * Adds inline CSS styles
         * @param $style_css Mixed: inline CSS
+        * @param $flip String: Set to 'flip' to flip the CSS if needed
         */
-       public function addInlineStyle( $style_css ){
+       public function addInlineStyle( $style_css, $flip = 'noflip' ) {
+               if( $flip === 'flip' && $this->getLang()->isRTL() ) {
+                       # If wanted, and the interface is right-to-left, flip the CSS
+                       $style_css = CSSJanus::transform( $style_css, true, false );
+               }
                $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
         *
         * @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
@@ -3002,7 +3026,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
@@ -3013,15 +3037,28 @@ $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;
        }
 
+       /**
+        * @return Array
+        */
        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 ) {
@@ -3041,8 +3078,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 '';
                        }
                }