X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FOutputPage.php;h=87712fdbff811b7ee31378a917d3e4c8e5cc090c;hb=4df8e9d5958e3d3415a15a9d92763aa2cd6ab940;hp=3e22f847989a3e60f88f4f0eb845661e9f81f197;hpb=c75465b7ad0615503b46c03e248dd8408d747b24;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/OutputPage.php b/includes/OutputPage.php index 3e22f84798..87712fdbff 100644 --- a/includes/OutputPage.php +++ b/includes/OutputPage.php @@ -13,12 +13,12 @@ if ( !defined( 'MEDIAWIKI' ) ) { * * @todo FIXME: Another class handles sending the whole page to the client. * - * Some comments comes from a pairing session between Zak Greant and Ashar Voultoiz + * Some comments comes from a pairing session between Zak Greant and Antoine Musso * in November 2010. * * @todo document */ -class OutputPage { +class OutputPage extends ContextSource { /// Should be private. Used with addMeta() which adds 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 @@ -61,7 +67,7 @@ class OutputPage { * Contains the page subtitle. Special pages usually have some links here. * Don't confuse with site subtitle added by skins. */ - var $mSubtitle = ''; + private $mSubtitle = array(); var $mRedirect = ''; var $mStatusCode; @@ -118,6 +124,7 @@ class OutputPage { // @todo FIXME: Next variables probably comes from the resource loader var $mModules = array(), $mModuleScripts = array(), $mModuleStyles = array(), $mModuleMessages = array(); var $mResourceLoader; + var $mJsConfigVars = array(); /** @todo FIXME: Is this still used ?*/ var $mInlineMsg = array(); @@ -125,6 +132,10 @@ class OutputPage { var $mTemplateIds = array(); var $mImageTimeKeys = array(); + var $mRedirectCode = ''; + + var $mFeedLinksAppendQuery = null; + # What level of 'untrustworthiness' is allowed in CSS/JS modules loaded on this page? # @see ResourceLoaderModule::$origin # ResourceLoaderModule::ORIGIN_ALL is assumed unless overridden; @@ -142,12 +153,9 @@ class OutputPage { var $mContainsOldMagic = 0, $mContainsNewMagic = 0; /** - * Should be private. Has get/set methods properly documented. - * Stores "article flag" toggle. + * lazy initialised, use parserOptions() + * @var ParserOptions */ - var $mIsArticleRelated = true; - - /// lazy initialised, use parserOptions() protected $mParserOptions = null; /** @@ -189,11 +197,10 @@ class OutputPage { /// should be private. To include the variable {{REVISIONID}} var $mRevisionId = null; + private $mRevisionTimestamp = null; 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. @@ -216,17 +223,26 @@ class OutputPage { 'Cookie' => null ); + /** + * If the current page was reached through a redirect, $mRedirectedFrom contains the Title + * of the redirect. + * + * @var Title + */ + private $mRedirectedFrom = null; + /** * Constructor for OutputPage. This should not be called directly. * 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; } /** @@ -465,7 +481,7 @@ class OutputPage { * * @param $filter * @param $position - * + * * @return Array of module names */ public function getModuleStyles( $filter = false, $position = null ) { @@ -506,6 +522,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 * @@ -744,7 +769,11 @@ class OutputPage { * @param $name string */ public function setHTMLTitle( $name ) { - $this->mHTMLtitle = $name; + if ( $name instanceof Message ) { + $this->mHTMLtitle = $name->setContext( $this->getContext() )->text(); + } else { + $this->mHTMLtitle = $name; + } } /** @@ -756,22 +785,35 @@ class OutputPage { return $this->mHTMLtitle; } + /** + * Set $mRedirectedFrom, the Title of the page which redirected us to the current page. + * + * param @t Title + */ + public function setRedirectedFrom( $t ) { + $this->mRedirectedFrom = $t; + } + /** * "Page title" means the contents of \. It is stored as a valid HTML fragment. * This function allows good tags like \ in the \ tag, but not bad tags like \. * This function automatically sets \ to the same content as \ but with all tags removed. * Bad tags that were escaped in \ will still be escaped in \, and good tags like \ will be dropped entirely. * - * @param $name string + * @param $name string|Message */ public function setPageTitle( $name ) { + if ( $name instanceof Message ) { + $name = $name->setContext( $this->getContext() )->text(); + } + # change "" to "<script>foo&bar</script>" # but leave "foobar" alone $nameWithTags = Sanitizer::normalizeCharReferences( Sanitizer::removeHTMLtags( $name ) ); $this->mPagetitle = $nameWithTags; # change "foo&bar" to "foo&bar" - $this->setHTMLTitle( wfMsg( 'pagetitle', Sanitizer::stripAllTags( $nameWithTags ) ) ); + $this->setHTMLTitle( $this->msg( 'pagetitle', Sanitizer::stripAllTags( $nameWithTags ) ) ); } /** @@ -783,83 +825,67 @@ 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 + * Replace the subtile with $str * - * @return Title + * @param $str String|Message: new value of the subtitle */ - public function getTitle() { - return $this->getContext()->getTitle(); + public function setSubtitle( $str ) { + $this->clearSubtitle(); + $this->addSubtitle( $str ); } /** - * Get the User object used in this instance + * Add $str to the subtitle * - * @return User - * @since 1.18 + * @deprecated in 1.19; use addSubtitle() instead + * @param $str String|Message to add to the subtitle */ - public function getUser() { - return $this->getContext()->getUser(); + public function appendSubtitle( $str ) { + $this->addSubtitle( $str ); } /** - * Get the Skin object used to render this instance + * Add $str to the subtitle * - * @return Skin - * @since 1.18 + * @param $str String|Message to add to the subtitle */ - public function getSkin() { - return $this->getContext()->getSkin(); + public function addSubtitle( $str ) { + if ( $str instanceof Message ) { + $this->mSubtitle[] = $str->setContext( $this->getContext() )->parse(); + } else { + $this->mSubtitle[] = $str; + } } /** - * Replace the subtile with $str + * Add a subtitle containing a backlink to a page * - * @param $str String: new value of the subtitle + * @param $title Title to link to */ - public function setSubtitle( $str ) { - $this->mSubtitle = /*$this->parse(*/ $str /*)*/; // @bug 2514 + public function addBacklinkSubtitle( Title $title ) { + $query = array(); + if ( $title->isRedirect() ) { + $query['redirect'] = 'no'; + } + $this->addSubtitle( $this->msg( 'backlinksubtitle' )->rawParams( Linker::link( $title, null, array(), $query ) ) ); } /** - * Add $str to the subtitle - * - * @param $str String to add to the subtitle + * Clear the subtitles */ - public function appendSubtitle( $str ) { - $this->mSubtitle .= /*$this->parse(*/ $str /*)*/; // @bug 2514 + public function clearSubtitle() { + $this->mSubtitle = array(); } /** @@ -868,7 +894,7 @@ class OutputPage { * @return String */ public function getSubtitle() { - return $this->mSubtitle; + return implode( "
\n\t\t\t\t", $this->mSubtitle ); } /** @@ -1240,6 +1266,19 @@ class OutputPage { $this->mBodytext .= $text; } + /** + * Shortcut for adding an Html::element via addHTML. + * + * @since 1.19 + * + * @param $element string + * @param $attribs array + * @param $contents string + */ + public function addElement( $element, $attribs = array(), $contents = '' ) { + $this->addHTML( Html::element( $element, $attribs, $contents ) ); + } + /** * Clear the body HTML */ @@ -1274,7 +1313,8 @@ class OutputPage { */ public function parserOptions( $options = null ) { if ( !$this->mParserOptions ) { - $this->mParserOptions = new ParserOptions; + $this->mParserOptions = ParserOptions::newFromContext( $this->getContext() ); + $this->mParserOptions->setEditSection( false ); } return wfSetVar( $this->mParserOptions, $options ); } @@ -1300,6 +1340,27 @@ class OutputPage { return $this->mRevisionId; } + /** + * Set the timestamp of the revision which will be displayed. This is used + * to avoid a extra DB call in Skin::lastModified(). + * + * @param $revid Mixed: string, or null + * @return Mixed: previous value + */ + public function setRevisionTimestamp( $timestmap ) { + return wfSetVar( $this->mRevisionTimestamp, $timestmap ); + } + + /** + * Get the timestamp of displayed revision. + * This will be null if not filled by setRevisionTimestamp(). + * + * @return String or null + */ + public function getRevisionTimestamp() { + return $this->mRevisionTimestamp; + } + /** * Set the displayed file version * @@ -1307,10 +1368,11 @@ class OutputPage { * @return Mixed: previous value */ public function setFileVersion( $file ) { + $val = null; if ( $file instanceof File && $file->exists() ) { $val = array( 'time' => $file->getTimestamp(), 'sha1' => $file->getSha1() ); } - return wfSetVar( $this->mFileVersion, $val ); + return wfSetVar( $this->mFileVersion, $val, true ); } /** @@ -1338,7 +1400,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; } @@ -1348,10 +1410,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 ); } /** @@ -1366,7 +1429,7 @@ class OutputPage { } /** - * Add wikitext with a custom Title object and + * Add wikitext with a custom Title object and tidy enabled. * * @param $text String: wikitext * @param $title Title object @@ -1394,8 +1457,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__ ); @@ -1404,6 +1469,7 @@ class OutputPage { $popts = $this->parserOptions(); $oldTidy = $popts->setTidy( $tidy ); + $popts->setInterfaceMessage( (bool) $interface ); $parserOutput = $wgParser->parse( $text, $title, $popts, @@ -1435,6 +1501,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 ) { @@ -1445,7 +1514,7 @@ class OutputPage { } } // File versioning... - foreach ( (array)$parserOutput->getImageTimeKeys() as $dbk => $data ) { + foreach ( (array)$parserOutput->getFileSearchOptions() as $dbk => $data ) { $this->mImageTimeKeys[$dbk] = $data; } @@ -1493,19 +1562,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() ) ) { @@ -1651,7 +1714,7 @@ class OutputPage { $this->mVaryHeader[$header] = $option; } } - $this->mVaryHeader[$header] = array_unique( $this->mVaryHeader[$header] ); + $this->mVaryHeader[$header] = array_unique( (array)$this->mVaryHeader[$header] ); } /** @@ -1690,12 +1753,12 @@ class OutputPage { * /w/index.php?title=Main_page&variant=zh-cn should never be served. */ function addAcceptLanguage() { - global $wgContLang; - if( !$this->getRequest()->getCheck( 'variant' ) && $wgContLang->hasVariants() ) { - $variants = $wgContLang->getVariants(); + $lang = $this->getTitle()->getPageLanguage(); + if( !$this->getRequest()->getCheck( 'variant' ) && $lang->hasVariants() ) { + $variants = $lang->getVariants(); $aloption = array(); foreach ( $variants as $variant ) { - if( $variant === $wgContLang->getCode() ) { + if( $variant === $lang->getCode() ) { continue; } else { $aloption[] = 'string-contains=' . $variant; @@ -1752,6 +1815,7 @@ class OutputPage { } elseif ( $this->mPreventClickjacking && $wgEditPageFrameOptions ) { return $wgEditPageFrameOptions; } + return false; } /** @@ -1827,9 +1891,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__ ); @@ -1841,7 +1905,7 @@ class OutputPage { * the object, let's actually output it: */ public function output() { - global $wgLanguageCode, $wgDebugRedirects, $wgMimeType; + global $wgLanguageCode, $wgDebugRedirects, $wgMimeType, $wgVaryOnXFP; if( $this->mDoNothing ) { return; @@ -1853,25 +1917,35 @@ class OutputPage { if ( $this->mRedirect != '' ) { # Standards require redirect URLs to be absolute - $this->mRedirect = wfExpandUrl( $this->mRedirect ); - if( $this->mRedirectCode == '301' || $this->mRedirectCode == '303' ) { - if( !$wgDebugRedirects ) { - $message = HttpStatus::getMessage( $this->mRedirectCode ); - $response->header( "HTTP/1.1 {$this->mRedirectCode} $message" ); + $this->mRedirect = wfExpandUrl( $this->mRedirect, PROTO_CURRENT ); + + $redirect = $this->mRedirect; + $code = $this->mRedirectCode; + + if( wfRunHooks( "BeforePageRedirect", array( $this, &$redirect, &$code ) ) ) { + if( $code == '301' || $code == '303' ) { + if( !$wgDebugRedirects ) { + $message = HttpStatus::getMessage( $code ); + $response->header( "HTTP/1.1 $code $message" ); + } + $this->mLastModified = wfTimestamp( TS_RFC2822 ); + } + if ( $wgVaryOnXFP ) { + $this->addVaryHeader( 'X-Forwarded-Proto' ); + } + $this->sendCacheControl(); + + $response->header( "Content-Type: text/html; charset=utf-8" ); + if( $wgDebugRedirects ) { + $url = htmlspecialchars( $redirect ); + print "\n\nRedirect\n\n\n"; + print "

Location: $url

\n"; + print "\n\n"; + } else { + $response->header( 'Location: ' . $redirect ); } - $this->mLastModified = wfTimestamp( TS_RFC2822 ); - } - $this->sendCacheControl(); - - $response->header( "Content-Type: text/html; charset=utf-8" ); - if( $wgDebugRedirects ) { - $url = htmlspecialchars( $this->mRedirect ); - print "\n\nRedirect\n\n\n"; - print "

Location: $url

\n"; - print "\n\n"; - } else { - $response->header( 'Location: ' . $this->mRedirect ); } + wfProfileOut( __METHOD__ ); return; } elseif ( $this->mStatusCode ) { @@ -1905,7 +1979,7 @@ class OutputPage { wfRunHooks( 'BeforePageDisplay', array( &$this, &$sk ) ); wfProfileIn( 'Output-skin' ); - $sk->outputPage( $this ); + $sk->outputPage(); wfProfileOut( 'Output-skin' ); } @@ -1932,25 +2006,50 @@ class OutputPage { } /** - * Output a standard error page + * Prepare this object to display an error page; disable caching and + * indexing, clear the current text and redirect, set the page's title + * and optionally an custom HTML title (content of the tag). * - * @param $title String: message key for page title - * @param $msg String: message key for page text - * @param $params Array: message parameters + * @param $pageTitle String|Message will be passed directly to setPageTitle() + * @param $htmlTitle String|Message will be passed directly to setHTMLTitle(); + * optional, if not passed the <title> attribute will be + * based on $pageTitle */ - public function showErrorPage( $title, $msg, $params = array() ) { + public function prepareErrorPage( $pageTitle, $htmlTitle = false ) { if ( $this->getTitle() ) { $this->mDebugtext .= 'Original title: ' . $this->getTitle()->getPrefixedText() . "\n"; } - $this->setPageTitle( wfMsg( $title ) ); - $this->setHTMLTitle( wfMsg( 'errorpagetitle' ) ); + + $this->setPageTitle( $pageTitle ); + if ( $htmlTitle !== false ) { + $this->setHTMLTitle( $htmlTitle ); + } $this->setRobotPolicy( 'noindex,nofollow' ); $this->setArticleRelated( false ); $this->enableClientCache( false ); $this->mRedirect = ''; - $this->mBodytext = ''; + $this->clearSubtitle(); + $this->clearHTML(); + } - $this->addWikiMsgArray( $msg, $params ); + /** + * Output a standard error page + * + * showErrorPage( 'titlemsg', 'pagetextmsg', array( 'param1', 'param2' ) ); + * showErrorPage( 'titlemsg', $messageObject ); + * + * @param $title String: message key for page title + * @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() ) { + $this->prepareErrorPage( $this->msg( $title ), $this->msg( 'errorpagetitle' ) ); + + if ( $msg instanceof Message ){ + $this->addHTML( $msg->parse() ); + } else { + $this->addWikiMsgArray( $msg, $params ); + } $this->returnToMain(); } @@ -1962,16 +2061,71 @@ class OutputPage { * @param $action String: action that was denied or null if unknown */ public function showPermissionsErrorPage( $errors, $action = null ) { - $this->mDebugtext .= 'Original title: ' . - $this->getTitle()->getPrefixedText() . "\n"; - $this->setPageTitle( wfMsg( 'permissionserrors' ) ); - $this->setHTMLTitle( wfMsg( 'permissionserrors' ) ); - $this->setRobotPolicy( 'noindex,nofollow' ); - $this->setArticleRelated( false ); - $this->enableClientCache( false ); - $this->mRedirect = ''; - $this->mBodytext = ''; - $this->addWikiText( $this->formatPermissionsErrorMessage( $errors, $action ) ); + global $wgGroupPermissions; + + // For some action (read, edit, create and upload), display a "login to do this action" + // error if all of the following conditions are met: + // 1. the user is not logged in + // 2. the only error is insufficient permissions (i.e. no block or something else) + // 3. the error can be avoided simply by logging in + if ( in_array( $action, array( 'read', 'edit', 'createpage', 'createtalk', 'upload' ) ) + && $this->getUser()->isAnon() && count( $errors ) == 1 && isset( $errors[0][0] ) + && ( $errors[0][0] == 'badaccess-groups' || $errors[0][0] == 'badaccess-group0' ) + && ( ( isset( $wgGroupPermissions['user'][$action] ) && $wgGroupPermissions['user'][$action] ) + || ( isset( $wgGroupPermissions['autoconfirmed'][$action] ) && $wgGroupPermissions['autoconfirmed'][$action] ) ) + ) { + $displayReturnto = null; + + # Due to bug 32276, if a user does not have read permissions, + # $this->getTitle() will just give Special:Badtitle, which is + # not especially useful as a returnto parameter. Use the title + # from the request instead, if there was one. + $request = $this->getRequest(); + $returnto = Title::newFromURL( $request->getVal( 'title', '' ) ); + if ( $action == 'edit' ) { + $msg = 'whitelistedittext'; + $displayReturnto = $returnto; + } elseif ( $action == 'createpage' || $action == 'createtalk' ) { + $msg = 'nocreatetext'; + } elseif ( $action == 'upload' ) { + $msg = 'uploadnologintext'; + } else { # Read + $msg = 'loginreqpagetext'; + $displayReturnto = Title::newMainPage(); + } + + $query = array(); + + if ( $returnto ) { + $query['returnto'] = $returnto->getPrefixedText(); + + if ( !$request->wasPosted() ) { + $returntoquery = $request->getValues(); + unset( $returntoquery['title'] ); + unset( $returntoquery['returnto'] ); + unset( $returntoquery['returntoquery'] ); + $query['returntoquery'] = wfArrayToCGI( $returntoquery ); + } + } + $loginLink = Linker::linkKnown( + SpecialPage::getTitleFor( 'Userlogin' ), + $this->msg( 'loginreqlink' )->escaped(), + array(), + $query + ); + + $this->prepareErrorPage( $this->msg( 'loginreqtitle' ) ); + $this->addHTML( $this->msg( $msg )->rawParams( $loginLink )->parse() ); + + # Don't return to a page the user can't read otherwise + # we'll end up in a pointless loop + if ( $displayReturnto && $displayReturnto->userCan( 'read' ) ) { + $this->returnToMain( null, $displayReturnto ); + } + } else { + $this->prepareErrorPage( $this->msg( 'permissionserrors' ) ); + $this->addWikiText( $this->formatPermissionsErrorMessage( $errors, $action ) ); + } } /** @@ -1981,11 +2135,7 @@ class OutputPage { * @param $version Mixed: the version of MediaWiki needed to use the page */ public function versionRequired( $version ) { - $this->setPageTitle( wfMsg( 'versionrequired', $version ) ); - $this->setHTMLTitle( wfMsg( 'versionrequired', $version ) ); - $this->setRobotPolicy( 'noindex,nofollow' ); - $this->setArticleRelated( false ); - $this->mBodytext = ''; + $this->prepareErrorPage( $this->msg( 'versionrequired', $version ) ); $this->addWikiMsg( 'versionrequiredtext', $version ); $this->returnToMain(); @@ -2002,34 +2152,11 @@ class OutputPage { /** * Produce the stock "please login to use the wiki" page + * + * @deprecated in 1.19; throw the exception directly */ public function loginToUse() { - if( $this->getUser()->isLoggedIn() ) { - throw new PermissionsError( 'read' ); - } - - $this->setPageTitle( wfMsg( 'loginreqtitle' ) ); - $this->setHtmlTitle( wfMsg( 'errorpagetitle' ) ); - $this->setRobotPolicy( 'noindex,nofollow' ); - $this->setArticleRelated( false ); - - $loginTitle = SpecialPage::getTitleFor( 'Userlogin' ); - $loginLink = Linker::link( - $loginTitle, - wfMsgHtml( 'loginreqlink' ), - array(), - array( 'returnto' => $this->getTitle()->getPrefixedText() ), - array( 'known', 'noclasses' ) - ); - $this->addWikiMsgArray( 'loginreqpagetext', array( $loginLink ), array( 'replaceafter' ) ); - $this->addHTML( "\n<!--" . $this->getTitle()->getPrefixedUrl() . '-->' ); - - # Don't return to the main page if the user can't read it - # otherwise we'll end up in a pointless loop - $mainPage = Title::newMainPage(); - if( $mainPage->userCanRead() ) { - $this->returnToMain( null, $mainPage ); - } + throw new PermissionsError( 'read' ); } /** @@ -2041,14 +2168,14 @@ class OutputPage { */ public function formatPermissionsErrorMessage( $errors, $action = null ) { if ( $action == null ) { - $text = wfMsgNoTrans( 'permissionserrorstext', count( $errors ) ) . "\n\n"; + $text = $this->msg( 'permissionserrorstext', count( $errors ) )->plain() . "\n\n"; } else { - $action_desc = wfMsgNoTrans( "action-$action" ); - $text = wfMsgNoTrans( + $action_desc = $this->msg( "action-$action" )->plain(); + $text = $this->msg( 'permissionserrorstext-withaction', count( $errors ), $action_desc - ) . "\n\n"; + )->plain() . "\n\n"; } if ( count( $errors ) > 1 ) { @@ -2056,13 +2183,13 @@ class OutputPage { foreach( $errors as $error ) { $text .= '<li>'; - $text .= call_user_func_array( 'wfMsgNoTrans', $error ); + $text .= call_user_func_array( array( $this, 'msg' ), $error )->plain(); $text .= "</li>\n"; } $text .= '</ul>'; } else { $text .= "<div class=\"permissions-errors\">\n" . - call_user_func_array( 'wfMsgNoTrans', reset( $errors ) ) . + call_user_func_array( array( $this, 'msg' ), reset( $errors ) )->plain() . "\n</div>"; } @@ -2102,12 +2229,10 @@ class OutputPage { if ( !empty( $reasons ) ) { // Permissions error if( $source ) { - $this->setPageTitle( wfMsg( 'viewsource' ) ); - $this->setSubtitle( - wfMsg( 'viewsourcefor', Linker::linkKnown( $this->getTitle() ) ) - ); + $this->setPageTitle( $this->msg( 'viewsource-title', $this->getTitle()->getPrefixedText() ) ); + $this->addBacklinkSubtitle( $this->getTitle() ); } else { - $this->setPageTitle( wfMsg( 'badaccess' ) ); + $this->setPageTitle( $this->msg( 'badaccess' ) ); } $this->addWikiText( $this->formatPermissionsErrorMessage( $reasons, $action ) ); } else { @@ -2119,18 +2244,21 @@ 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->getHtmlCode(), + 'dir' => $pageLang->getDir(), ); $this->addHTML( Html::element( 'textarea', $params, $source ) ); // Show templates used by this article - $article = new Article( $this->getTitle() ); - $templates = Linker::formatTemplates( $article->getUsedTemplates() ); + $page = WikiPage::factory( $this->getTitle() ); + $templates = Linker::formatTemplates( $page->getUsedTemplates() ); $this->addHTML( "<div class='templatesUsed'> $templates </div> @@ -2169,37 +2297,34 @@ $templates ? 'lag-warn-normal' : 'lag-warn-high'; $wrap = Html::rawElement( 'div', array( 'class' => "mw-{$message}" ), "\n$1\n" ); - $this->wrapWikiMsg( "$wrap\n", array( $message, $this->getContext()->getLang()->formatNum( $lag ) ) ); + $this->wrapWikiMsg( "$wrap\n", array( $message, $this->getLanguage()->formatNum( $lag ) ) ); } } public function showFatalError( $message ) { - $this->setPageTitle( wfMsg( 'internalerror' ) ); - $this->setRobotPolicy( 'noindex,nofollow' ); - $this->setArticleRelated( false ); - $this->enableClientCache( false ); - $this->mRedirect = ''; - $this->mBodytext = $message; + $this->prepareErrorPage( $this->msg( 'internalerror' ) ); + + $this->addHTML( $message ); } public function showUnexpectedValueError( $name, $val ) { - $this->showFatalError( wfMsg( 'unexpected', $name, $val ) ); + $this->showFatalError( $this->msg( 'unexpected', $name, $val )->text() ); } public function showFileCopyError( $old, $new ) { - $this->showFatalError( wfMsg( 'filecopyerror', $old, $new ) ); + $this->showFatalError( $this->msg( 'filecopyerror', $old, $new )->text() ); } public function showFileRenameError( $old, $new ) { - $this->showFatalError( wfMsg( 'filerenameerror', $old, $new ) ); + $this->showFatalError( $this->msg( 'filerenameerror', $old, $new )->text() ); } public function showFileDeleteError( $name ) { - $this->showFatalError( wfMsg( 'filedeleteerror', $name ) ); + $this->showFatalError( $this->msg( 'filedeleteerror', $name )->text() ); } public function showFileNotFoundError( $name ) { - $this->showFatalError( wfMsg( 'filenotfound', $name ) ); + $this->showFatalError( $this->msg( 'filenotfound', $name )->text() ); } /** @@ -2211,10 +2336,8 @@ $templates */ public function addReturnTo( $title, $query = array(), $text = null ) { $this->addLink( array( 'rel' => 'next', 'href' => $title->getFullURL() ) ); - $link = wfMsgHtml( - 'returnto', - Linker::link( $title, $text, array(), $query ) - ); + $link = $this->msg( 'returnto' )->rawParams( + Linker::link( $title, $text, array(), $query ) )->escaped(); $this->addHTML( "<p id=\"mw-returnto\">{$link}</p>\n" ); } @@ -2257,18 +2380,18 @@ $templates * @return String: The doctype, opening <html>, and head element. */ public function headElement( Skin $sk, $includeStyle = true ) { - global $wgUseTrackbacks; + global $wgContLang; + $userdir = $this->getLanguage()->getDir(); + $sitedir = $wgContLang->getDir(); if ( $sk->commonPrintStylesheet() ) { $this->addModuleStyles( 'mediawiki.legacy.wikiprintable' ); } - $sk->setupUserCss( $this ); - $lang = wfUILang(); - $ret = Html::htmlHeader( array( 'lang' => $lang->getCode(), 'dir' => $lang->getDir() ) ); + $ret = Html::htmlHeader( array( 'lang' => $this->getLanguage()->getHtmlCode(), 'dir' => $userdir, 'class' => 'client-nojs' ) ); if ( $this->getHTMLTitle() == '' ) { - $this->setHTMLTitle( wfMsg( 'pagetitle', $this->getPageTitle() ) ); + $this->setHTMLTitle( $this->msg( 'pagetitle', $this->getPageTitle() ) ); } $openHead = Html::openElement( 'head' ); @@ -2280,16 +2403,12 @@ $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() ) ); - if ( $wgUseTrackbacks && $this->isArticleRelated() ) { - $ret .= $this->getTitle()->trackbackRDF(); - } - $closeHead = Html::closeElement( 'head' ); if ( $closeHead ) { $ret .= "$closeHead\n"; @@ -2297,25 +2416,10 @@ $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"; - # Class bloat - $dir = wfUILang()->getDir(); - $bodyAttrs['class'] = "mediawiki $dir"; - - if ( $this->getContext()->getLang()->capitalizeAllNouns() ) { + if ( $this->getLanguage()->capitalizeAllNouns() ) { # A <body> class is probably not the best way to do this . . . $bodyAttrs['class'] .= ' capitalize-all-nouns'; } @@ -2334,8 +2438,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( @@ -2363,9 +2466,14 @@ $templates } } - 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' ); + } } /** @@ -2382,30 +2490,14 @@ $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 ) { - 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; - } + protected function makeResourceLoaderLink( $modules, $only, $useESI = false, array $extraQuery = array() ) { + global $wgResourceLoaderUseESI, $wgResourceLoaderInlinePrivateModules; if ( !count( $modules ) ) { return ''; @@ -2421,7 +2513,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; } @@ -2433,7 +2525,8 @@ $templates foreach ( (array) $modules as $name ) { $module = $resourceLoader->getModule( $name ); # Check that we're allowed to include this module on this page - if ( ( $module->getOrigin() > $this->getAllowedModules( ResourceLoaderModule::TYPE_SCRIPTS ) + if ( !$module + || ( $module->getOrigin() > $this->getAllowedModules( ResourceLoaderModule::TYPE_SCRIPTS ) && $only == ResourceLoaderModule::TYPE_SCRIPTS ) || ( $module->getOrigin() > $this->getAllowedModules( ResourceLoaderModule::TYPE_STYLES ) && $only == ResourceLoaderModule::TYPE_STYLES ) @@ -2451,14 +2544,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->getLanguage()->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 ) { @@ -2471,8 +2576,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 ) { @@ -2486,6 +2589,7 @@ $templates ) ); } + $links .= "\n"; continue; } // Special handling for the user group; because users might change their stuff @@ -2493,6 +2597,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; @@ -2500,15 +2605,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->getLanguage()->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 ) { @@ -2538,12 +2649,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( @@ -2554,8 +2664,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 @@ -2575,17 +2685,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 @@ -2605,7 +2713,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'; } @@ -2613,31 +2721,56 @@ $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; } /** - * Get an array containing global JS variables + * Add one or more variables to be set in mw.config in JavaScript. * - * Do not add things here which can be evaluated in - * ResourceLoaderStartupScript - in other words, without state. - * You will only be adding bloat to the page and causing page caches to + * @param $key {String|Array} Key or array of key/value pars. + * @param $value {Mixed} Value of the configuration variable. + */ + public function addJsConfigVars( $keys, $value ) { + if ( is_array( $keys ) ) { + foreach ( $keys as $key => $value ) { + $this->mJsConfigVars[$key] = $value; + } + return; + } + + $this->mJsConfigVars[$keys] = $value; + } + + + /** + * Get an array containing the variables to be set in mw.config in JavaScript. + * + * Do not add things here which can be evaluated in ResourceLoaderStartupScript + * - in other words, page-independent/site-wide variables (without state). + * You will only be adding bloat to the html page and causing page caches to * have to be purged on configuration changes. + * @return array */ protected function getJSVars() { - global $wgUseAjax, $wgEnableMWSuggest, $wgContLang; + global $wgUseAjax, $wgEnableMWSuggest; $title = $this->getTitle(); $ns = $title->getNamespace(); @@ -2648,6 +2781,22 @@ $templates $canonicalName = false; # bug 21115 } + $lang = $title->getPageLanguage(); + + // Pre-process information + $separatorTransTable = $lang->separatorTransformTable(); + $separatorTransTable = $separatorTransTable ? $separatorTransTable : array(); + $compactSeparatorTransTable = array( + implode( "\t", array_keys( $separatorTransTable ) ), + implode( "\t", $separatorTransTable ), + ); + $digitTransTable = $lang->digitTransformTable(); + $digitTransTable = $digitTransTable ? $digitTransTable : array(); + $compactDigitTransTable = array( + implode( "\t", array_keys( $digitTransTable ) ), + implode( "\t", $digitTransTable ), + ); + $vars = array( 'wgCanonicalNamespace' => $nsname, 'wgCanonicalSpecialPageName' => $canonicalName, @@ -2662,10 +2811,13 @@ $templates 'wgUserGroups' => $this->getUser()->getEffectiveGroups(), 'wgCategories' => $this->getCategories(), 'wgBreakFrames' => $this->getFrameOptions() == 'DENY', + 'wgPageContentLanguage' => $lang->getCode(), + 'wgSeparatorTransformTable' => $compactSeparatorTransTable, + 'wgDigitTransformTable' => $compactDigitTransTable, ); - if ( $wgContLang->hasVariants() ) { - $vars['wgUserVariant'] = $wgContLang->getPreferredVariant(); - } + if ( $lang->hasVariants() ) { + $vars['wgUserVariant'] = $lang->getPreferredVariant(); + } foreach ( $title->getRestrictionTypes() as $type ) { $vars['wgRestriction' . ucfirst( $type )] = $title->getRestrictions( $type ); } @@ -2675,24 +2827,55 @@ $templates if ( $title->isMainPage() ) { $vars['wgIsMainPage'] = true; } + if ( $this->mRedirectedFrom ) { + $vars['wgRedirectedFrom'] = $this->mRedirectedFrom->getPrefixedDBKey(); + } - // Allow extensions to add their custom variables to the global JS variables - wfRunHooks( 'MakeGlobalVariablesScript', array( &$vars ) ); + // Allow extensions to add their custom variables to the mw.config map. + // Use the 'ResourceLoaderGetConfigVars' hook if the variable is not + // page-dependant but site-wide (without state). + // Alternatively, you may want to use OutputPage->addJsConfigVars() instead. + wfRunHooks( 'MakeGlobalVariablesScript', array( &$vars, $this ) ); - return $vars; + // Merge in variables from addJsConfigVars last + return array_merge( $vars, $this->mJsConfigVars ); } /** - * @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, - $wgDisableLangConversion, $wgCanonicalLanguageLinks, $wgContLang, + $wgDisableLangConversion, $wgCanonicalLanguageLinks, $wgRightsPage, $wgRightsUrl; $tags = array(); @@ -2764,11 +2947,12 @@ $templates } # Universal edit button - if ( $wgUniversalEditButton ) { - if ( $this->isArticleRelated() && $this->getTitle() && $this->getTitle()->quickUserCan( 'edit' ) - && ( $this->getTitle()->exists() || $this->getTitle()->quickUserCan( 'create' ) ) ) { + if ( $wgUniversalEditButton && $this->isArticleRelated() ) { + $user = $this->getUser(); + if ( $this->getTitle()->quickUserCan( 'edit', $user ) + && ( $this->getTitle()->exists() || $this->getTitle()->quickUserCan( 'create', $user ) ) ) { // Original UniversalEditButton - $msg = wfMsg( 'edit' ); + $msg = $this->msg( 'edit' )->text(); $tags[] = Html::element( 'link', array( 'rel' => 'alternate', 'type' => 'application/x-wiki', @@ -2801,7 +2985,7 @@ $templates 'rel' => 'search', 'type' => 'application/opensearchdescription+xml', 'href' => wfScript( 'opensearch_desc' ), - 'title' => wfMsgForContent( 'opensearch-desc' ), + 'title' => $this->msg( 'opensearch-desc' )->inContentLanguage()->text(), ) ); if ( $wgEnableAPI ) { @@ -2812,30 +2996,35 @@ $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 ), ) ); } - # Language variants - if ( !$wgDisableLangConversion && $wgCanonicalLanguageLinks - && $wgContLang->hasVariants() ) { - $urlvar = $wgContLang->getURLVariant(); - - if ( !$urlvar ) { - $variants = $wgContLang->getVariants(); - foreach ( $variants as $_v ) { + # Language variants + if ( !$wgDisableLangConversion && $wgCanonicalLanguageLinks ) { + $lang = $this->getTitle()->getPageLanguage(); + if ( $lang->hasVariants() ) { + + $urlvar = $lang->getURLVariant(); + + if ( !$urlvar ) { + $variants = $lang->getVariants(); + foreach ( $variants as $_v ) { + $tags[] = Html::element( 'link', array( + 'rel' => 'alternate', + 'hreflang' => $_v, + 'href' => $this->getTitle()->getLocalURL( array( 'variant' => $_v ) ) ) + ); + } + } else { $tags[] = Html::element( 'link', array( - 'rel' => 'alternate', - 'hreflang' => $_v, - 'href' => $this->getTitle()->getLocalURL( '', $_v ) ) - ); + 'rel' => 'canonical', + 'href' => $this->getTitle()->getCanonicalUrl() + ) ); } - } else { - $tags[] = Html::element( 'link', array( - 'rel' => 'canonical', - 'href' => $this->getTitle()->getFullURL() ) - ); } } @@ -2872,7 +3061,7 @@ $templates $format, $link, # Used messages: 'page-rss-feed' and 'page-atom-feed' (for an easier grep) - wfMsg( "page-{$format}-feed", $this->getTitle()->getPrefixedText() ) + $this->msg( "page-{$format}-feed", $this->getTitle()->getPrefixedText() )->text() ); } @@ -2883,23 +3072,21 @@ $templates # like to promote instead of the RC feed (maybe like a "Recent New Articles" # or "Breaking news" one). For this, we see if $wgOverrideSiteFeed is defined. # If so, use it instead. - - $rctitle = SpecialPage::getTitleFor( 'Recentchanges' ); - if ( $wgOverrideSiteFeed ) { foreach ( $wgOverrideSiteFeed as $type => $feedUrl ) { + // Note, this->feedLink escapes the url. $tags[] = $this->feedLink( $type, - htmlspecialchars( $feedUrl ), - wfMsg( "site-{$type}-feed", $wgSitename ) + $feedUrl, + $this->msg( "site-{$type}-feed", $wgSitename )->text() ); } - } elseif ( $this->getTitle()->getPrefixedText() != $rctitle->getPrefixedText() ) { + } elseif ( !$this->getTitle()->isSpecial( 'Recentchanges' ) ) { foreach ( $wgAdvertisedFeedTypes as $format ) { $tags[] = $this->feedLink( $format, - $rctitle->getLocalURL( "feed={$format}" ), - wfMsg( "site-{$format}-feed", $wgSitename ) # For grep: 'site-rss-feed', 'site-atom-feed'. + $this->getTitle()->getLocalURL( "feed={$format}" ), + $this->msg( "site-{$format}-feed", $wgSitename )->text() # For grep: 'site-rss-feed', 'site-atom-feed'. ); } } @@ -2952,26 +3139,71 @@ $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->getLanguage()->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 ) { - $group = $resourceLoader->getModule( $name )->getGroup(); + + $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 ) { + $module = $resourceLoader->getModule( $name ); + if ( !$module ) { + continue; + } + $group = $module->getGroup(); // Modules in groups named "other" or anything different than "user", "site" or "private" // will be placed in the "other" group $styles[isset( $styles[$group] ) ? $group : 'other'][] = $name; @@ -2981,7 +3213,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 @@ -2992,15 +3224,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 ) { @@ -3020,8 +3265,7 @@ $templates */ protected function styleLink( $style, $options ) { if( isset( $options['dir'] ) ) { - $siteDir = wfUILang()->getDir(); - if( $siteDir != $options['dir'] ) { + if( $this->getLanguage()->getDir() != $options['dir'] ) { return ''; } } @@ -3109,16 +3353,11 @@ $templates * Like addWikiMsg() except the parameters are taken as an array * instead of a variable argument list. * - * $options is passed through to wfMsgExt(), see that function for details. - * * @param $name string * @param $args array - * @param $options array */ - public function addWikiMsgArray( $name, $args, $options = array() ) { - $options[] = 'parse'; - $text = wfMsgExt( $name, $options, $args ); - $this->addHTML( $text ); + public function addWikiMsgArray( $name, $args ) { + $this->addWikiText( $this->msg( $name, $args )->plain() ); } /**