have any pipes after being transformed by MessageCache, causes exception on
all pages.
This can happen with lines like:
**{{#if:yes|Something}}
Thank you to liangent for figuring out how to escape a | without {{!}} existing and | not working.
Changed written by Timo and reviewed by Hashar. This should be harmless.
To enable the feature:
$wgEnableJavaScriptTest = true;
Then head to:
[[Special:JavaScriptTest/qunit]]
* Added $context parameter to Action::factory() to allow callers passing a WikiPage object in addition to Article (otherwise this would throw a fatal error in getContext() since WikiPage::getContext() does not exist)
* Don't show 'whatlinkshere', 'recentchangeslinked' and 'trackbacklink' items if OutputPage::isArticleRelated() is false
* Simplify the code to check if we show show user related items since the actual code is just to check if we have an User object
* Moved the showEmailUser() check to the same location as other related user checks
* Made Skin::showEmailUser() accept an User object instead of doing object -> int -> object conversion
* Check if the items are present in BaseTemplate::getToolbox() instead of 'notspecialpage' and removed some empty() or isset() checks since all of these items are now set
* Left RawPage with back compat code for extensions that use it
* Removed calls to Skin for gen=(css|js). Calling action=raw&gen=(css|js) will now unconditionally return empty content
* Removed Skin::generateUserJs() and Skin::generateUserStylesheet() since they were only used in RawPage
* Apply code conventions (mostly whitespace and missing semi-colons)
* Minor other nitpicks
-- '! important' -> '!important'
-- Ucfirst -> lc; for color names and keywords
-- Remove uses of system color keywords (they are deprecated) [1] Most had fixed fallbacks already. Using those now.
-- Remove version query string (foo.png?2), last modified in r66151. Now obsolete with ResourceLoader
-- 0px to 0 where possible
-- Remove commented out css rules and declarations without any comment (debug stuff like /*div{ border:1px solid Red !important;}*/)
-- Remove empty blocks ( .foo { } )
[1] https://developer.mozilla.org/en/CSS/color_value#System_Colors
* use getCanonicalURL() - this part should be deployed to 1.17wmf1
* Do not show &oldid= in case of nonexisting pages, incl. file description pages from InstantCommons files
* Put OutputPage::$mIsArticleRelated declaration just below $mIsArticle's one since they are related
* Made DifferenceEngine consistent with action=view, i.e. setting that flag to true when displaying any version of the page, current or not
* Made Skin::getCopyright() always use history_copyright when displaying an old version, regardless to the diff parameter
* Changed some checks from namespace != NS_SPECIAL && action=view to isArticle() so that they don't get executed when e.g. throwing an Exception (or similar things)
* Removed Skin::getStylesheet(), not called from anywhere
* Removed Skin::setMembers() and moved the definition of the $userpage member to SkinTemplate::outputPage() since it's only used in that class
* Removed useless $out parameter of Skin::bottomScripts()
* Call Linker methods statically
* Use Linker::linkKnown() instead of Linker::link() if possible
* Escape 'mainpage' message when used for description
* Removed some variables from SkinTemplate::outputPage(): nscanonical, nsnumber, currevisionid that were used when the JavaScript global variables were defined in the template. Also changed pageclass and skinnameclass to be defined only when $useHeadElement is false
* Removed some unused members of SkinTemplate: $iscontent and $iseditable
* Refactored the part of SkinTemplate::outputPage() that sets copyright, viewcount, lastmod, credits and numberofwatchingusers to always define them to false first and set them when needed. Also modified the check to display the copyright; it will now only be displayed when $out->isArticle() is true and the page exists, since it is currently possible to display it on any page by adding the oldid parameter to the url.
