re: r93415
* Document what happens as the result of each value returned
* Require boolean true to continue, not just a true value or strings
  would succeed
* Trim the arguments since the header array already contains them.

Per Platonides, follow-up r93758: rename ParserOutput::addOutputPage() to ParserOutput::addOutputPageMetadata() to match its behaviour

Fix minor naming issue from r93314, add a bit to parameter documentation also

Follow up r93633. Allow only classic spaces.

Revert r93688 per Platonides

In ResourceLoaderWikiModule, use isCssOrJsPage() as well as isCssJsSubpage() to check whether a wiki page is a CSS or JS page. This is required to make ResourceLoaderWikiModule work for pages that are CSS/JS pages, but don't live in the MediaWiki namespace and aren't user subpages. This wasn't possible before r93759, which introduced a hook that allows extensions to override which pages are considered to be CSS/JS pages.

Factor out obtaining the database object in ResourceLoaderWikiModule into a getDB() method that can be overridden by subclasses. This is needed for the Gadgets extension in the RL2 branch, which subclasses ResourceLoaderWikiModule to create modules for foreign wiki pages.

Introduce a hook so extensions can customize what's considered to be a CSS/JS page. This is needed for the Gadgets extension in the RL2 branch

* (bug 15558) Allow includable special pages to be parameterized using wiki syntax
* Changed SpecialPageFactory::capturePath() to take the same parameters as SpecialPageFactory::executePath() (except the last one) and made it always return a bool instead of bool-or-string. The result HTML can be fetched from the OutputPage object of the context.
* Added module styles, scritps and messages members to ParserOutput in addition to modules. The first one is needed to display Special:RecentChanges correctly when transcluded since EnhancedChangesList::beginRecentChangesList() calls addModuleStyles( 'mediawiki.special.changeslist' )

Yes, this means that you can use {{Special:Recentchanges|enhanced=0}} to use the old changes list. For the ones that wonder, {{Special:Recentchanges|uselang=something}} will not work since the Language object is forced to the one used by the parser.

Fixes Bug #29311 - [OutputPage] Create a method to remove items from mModules

Patch from John Du Hart, reviewed by Roan, Applying at Roan's request.

* $wgSVGMaxSize is now applied to the smaller of width or height, making very wide pano/timeline/diagram SVGs renderable at saner sizes

Add --no-updates flag to importDump.php that allows the user to skip updating the links table. On my test dataset (an old (~2 years) MediaWiki namespace dump of enwiki) this speeds up the import from 9m35s to 10s.

Update whitespace, braces.

result of finished Chunk Upload has to be Success

In Special:Protectedpages and Special:Protectedtitles, show protection expiry times as local times instead of UTC.

Fixing comment (no Language::decodeExpiry).

Fixing Special:Protectedtitles to match Special:Protectedpages. This prevents an internal error (caused by timestamp translation), probably since r84258, and possibly some incompatibility to different databases.

use tabs. for consistant whitespaces

Extend upload api adding an option to upload files in chunks,
using UploadStash to track the unfinished upload.

Add direction mark in recent changes and in protection log summary.

Update punctuation for error messages.

* ApiTest: mark testApiGotCookie() broken, since it is
* ApiTest: partially cleanup testApiListPages() to make it not depend on broken testApiGotCookie(). Not sure what we're testing here, so still marking incomplete.
* ApiWatchTest: mark testWatchClear() as broken rather than skipped due to failing testWatchEdit(). This class needs some love <3

mediawiki.Title: Removing public setter functions. Title object should be immutable.
- To avoid repetition, instead of substituting, changed them to private helper functions
- Updated test suite to create new objects for different titles, rather than modifying the existing one.
- Adding missing unit tests, now 100% complete according to QUnit CompletnessTest
  + Test for Title.prototype.getNamespaceId
  + Test forTitle.prototype.getPrefixedText
  + Test forTitle.prototype.getExtension

Follows-up r90331 CR

breaks in this file cause odd things to happen with RTL, removing line breaks

add maxWidth param

Commit Preprocessor_DOM live hacks

Commit live hack

Commit to trunk also

Add Profiling to trunk from r93681

Merge live hack - Extra debugging on connection failures

Merge live hack - bumping version of minified stuff

Sqlite fails to open a connection when $user is given.
Reported by lokkju in #wikimedia-tech

Port 'editondblclick' js to ResourceLoader.
- Fixes (bug 27894) Move 'editondblclick' event listener down from body to
  div#bodyContent
- Patch by Nik <niknyby@gmail.com>
- Changes since patch:
  * Moved from OutputPage->headElement (where the bodyAttr-hack used to be) to addDefaultModules where other additions live
  * Added dependency on 'mediawiki.util', it must be initialized before mw.util.$content can be used
  * Whitespace conventions

Reorder some modules in Resources.php to match directory tree and alphabetical sorting.

Follows-up r93515 CR: Whitespace

[ResourceLoader2] Create getModuleNames() function
- Returns all keys from the (private) registry object as an array
- Needed for ResourceLoader 2 in order to do autosuggestions for dependancies and cheap non-http-request lookups to avoid duplicate keys when creating new modules/gadgets.

(bug 30160 ) Add public method to mw.loader to get module names from registry

Fixes Bug #30061 - Command line installer $wgScriptPath

Patch from Edward Z. Yang

Fixes Bug #30149 - Handling of parentheses for Korean, Chinese and Japanese

Adapts the pipe trick with parens for fullwidth parentheses.  Adds parsertests.

Patch from Dan Collins.

Fix 'exif-gpsdestdistance-k' to use american spelling

w/s changes: “svn diff -x-w” is clean

follow up r93626 - Forgot to credit Gero Scholz

fixes Bug #27543 - “Warning: in_array() [function.in-array]: Wrong datatype for second argument in ParserOptions.php”

AFAICT, result of ParserOutput::getUsedOptions() is only used by ParserOptions::optionsHash() but that method doesn't account for a non-array value.

Patch supplied on bug.

w/s changes: “svn diff -x-w” is clean

Update punctuation for a few fatal error messages.

Localization update for he.

Fixing Bug 27540 :  width of <gallery> always 100%, by setting display to inline-block.

FU r93017 : Reorder gallery css, to avoid unwanted indent.

* (bug 30131) XCache with variable caching disabled no longer used for variable caching (CACHE_ACCEL)

Patch from John Du Hart: https://bugzilla.wikimedia.org/attachment.cgi?id=8849&action=diff

If XCache is present, but the xcache.var_size setting is off, we'll now consider it as xcache being disabled for CACHE_ACCEL purposes. It won't trigger view of the accelerator option in the installer, and at runtime if using CACHE_ACCEL and no other modules are available, it'll throw an error so you know that it doesn't work!

Clarify dual-licensing (GPL&Apache) of CSSJanus. Has no significant contributions (i.e. changes other than very minor whitespace and Doxygen changes) by anyone other than me.

Follow-up r93527: $value can also be null when not set

* Made OutputPage extend ContextSource instead of duplicating its code; this also adds getLang() that was missing
* Use getLang() instead of $wgLang

* Call Linker methods statically
* Factorised calls to Skin::getTitle() in StandardTemplate::quickBar()

Localisation updates for core and extension messages from translatewiki.net (2011-07-30 18:43:00 UTC)

Back out r93533, r93531, r93530

Breaks unit tests as below, not going to be able to fix them before I disappear for the evening, so might aswell leave trunk clean

ArticleTablesTest testbug14404

Error:
ArticleTablesTest::testbug14404
Undefined offset: 0

/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/includes/ArticleTablesTest.php:31
/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/MediaWikiTestCase.php:60
/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/MediaWikiPHPUnitCommand.php:20
/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/phpunit.php:60

ParserTests testParserTest #552 - testParserTest with data set #551

Failure:
ParserTests::testParserTest with data set #551 ('RAW magic word', '{{RAW:QUERTY}}', '<p><a href="/index.php?title=Template:QUERTY&amp;action=edit&amp;redlink=1" class="new" title="Template:QUERTY (page does not exist)">Template:QUERTY</a>
</p>', '', '')
RAW magic word
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-<p><a href="/index.php?title=Template:QUERTY&amp;action=edit&amp;redlink=1" class="new" title="Template:QUERTY (page does not exist)">Template:QUERTY</a>
+<p><a href="/index.php?title=Template:RAW:QUERTY&amp;action=edit&amp;redlink=1" class="new" title="Template:RAW:QUERTY (page does not exist)">Template:RAW:QUERTY</a>
 </p>

/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/includes/parser/NewParserTest.php:545
/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/MediaWikiTestCase.php:60
/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/MediaWikiPHPUnitCommand.php:20
/home/ci/cruisecontrol-bin-2.8.3/projects/mw/source/tests/phpunit/phpunit.php:60

Add FIXME on undefined variable

Followup r93530

Fix SqliteUpdater

Fix Undefined index: globaltemplatelinks in LinksUpdate

Merge iwtransclusion branch into trunk

In Special:ComparePages:
* Validate title and revision when passed
* Don't display the diff if a field is not valid
* Pass the context to the HTMLForm and removed the setTitle() call

:facepalm: Installer checked for magic_quotes_runtime instead of register_globals

Increasing coverage of mw.Title with unit tests.

* QUnit Completeness Test – Before:
Title.prototype.getNamespaceId
Title.prototype.getPrefixedText
Title.prototype.getExtension
Title.prototype.setNameAndExtension
Title.prototype.getUrl
Title.prototype.exists

* QUnit Completeness Test – After:
Title.prototype.getNamespaceId
Title.prototype.getPrefixedText
Title.prototype.getExtension

Release-notes for r93517

Adding fix for bug 27427. Fixes unit test. Patch provided by Michael M. through BugZilla.

* (bug 27427) mw.util.getParamValue shouldn't return value from hash even if param is only present in hash.

(Follows-up r93516)

Adding unit test for bug 27427. Currently broken. Fix (provided by Michael M. through BugZilla) will be committed afterwards.

* (bug 27427) mw.util.getParamValue shouldn't return value from hash even if param is only present in hash

mediawiki.js request() caching deep level object member access.
- Instead of requesting index 0 of dependancies-argument, and the value of that of the registry, and the dependancies object in that of which we read the length property is very deep. Even worse when doing that both in the head and in the body of a for()-loop.
- Instead caching reference access to dependancies object of the registry item.
- Plus a simple by-value of the length property

Localisation update for core and extension messages from translatewiki.net (2011-07-29 20:10:00)

Changed stash age parameter from a constant to a config option, switched to seconds for consistency.
followup to r92030

Fixed r93468, made setCachedLastEditTime() public

* Follow-up r93093: set the touch key for page moves too
* Tweaked related WikiPage.php comments

followup r93415 — there is no “$this” in a static method.

Provisional revert of r93319 -- change to TOC styles without coordinated cache updates.

Definitely shows different padding with the old cached markup and new styles in FF 5 though it's not a huge difference. Haven't tested in other browsers; IE needs testing in particular as it has funky special style bits.

Recommend:
* old markup should work uncahnged with the new styles -- this avoids any transition problems
* secondarily, consider updating parser cache version to force updates .... but you'd still want to do the above for pages that have been HTTP-cached!
* test all these renderings in all supported browsers and confirm that all is well with old & new markup

Revert r93386 -- PLEASE STOP COMMITTING THESE THINGS WITHOUT CONFIRMING THAT THEY DO WHAT NEEDS TO BE DONE, THAT THEY DON'T CAUSE PROBLEMS, AND THAT THEY WON'T JUST GET REVERTED FOR THE FOURTH OR FIFTH TIME. THANK YOU.

off-white background color matting should be applied only within the thumbnail box, not to the gallerybox; also there is no point in printing the off-white matting, so I removed it completely from the print version

Removed useless $out variable from Skin::getCopyright()

Localisation updates for core and extension messages from translatewiki.net (2011-07-28 19:51:00 UTC)

Followup r93412: Register new message key in maintenance file
Consistent spelling of e-mail: email -> e-mail

wtf, who made this typo?

follow up r93397 — missed msgid

Add hook that could be the solution for Bug #28026 (“Enable e-mail
notifications for watchlist (EnotifWatchlist) on all small wikis”) to
make logging sent emails easier.

further sanity checks for sending email … bomb out if we aren't given
any user with an email address.

tabs are tabs, not spaces

* followup r93397 — missing $dest
* Otherwise clean up $dest usage
* Clean up Debug messages

Added typehint per my own suggestion in bug 28827. Otherwise we get pretty useless errors if Title object is not passed here.

array of objects tostring conversion works correctly in php 5.2.3+

Reduce mail header differences by moving all the header creation code
to one place.

Follow-up r93386: fix editsection when user language direction != content lang dir

Redo r93284 ( Make editsection link more understandable by positioning it directly left ( RTL: right ) of the title ) in a more simple way and also include parserTests and release notes this time

Fu r92231: PHP Fatal error:  Call to a member function parse() on a non-object in /www/w/includes/specials/SpecialEditWatchlist.php on line 46

Maybe the method names need to be reconsidered to avoid errors like these?

r91728 : Fix a problem for api.php5 users and remove an unneeded function parameter from deliverResult

r91728 : Fix whitespace and two vars per CR

reword r93378 per suggestion of Nikerabbit

Release note for r92832

Fixed typo. w/s cleanup.

Fu r93351: remove extra space

Followup r93366: fix Opera bugs in textSelection. All tests pass in Opera now. Before, there were five failures
* replace \r\n with \n in the obtained output from getSelection. This fixes one failure
* In continuation of r93366, which updated post for added newlines in insertText, do the same for pre and also do this in the non-IE branch. This fixes one failure
* Look for \r as well as \n where line endings are concerned in the non-IE branch, just like in the IE branch. This fixes one failure
* Have the test case generator remap selection positions for Opera to account for the fact that a newline is double-counted. This fixes the remaining two failures

Fu r93249: I see no reason to mark this private - protected should be the default value, only use private if you have some reason to. Avoids most of the breakage in out of tree extensions.

Fu r93249 failed grepping

Fix broken textSelection test case in IE. All textSelection tests are passing in IE now (at least on my local machine)

Applying code conventions:
* size() -> length
* strict comparison to undefined instead of typeof + string comparison
* merge var statements
* strict comparison to 0 and ''
* dot notation
* trailing whitespace

Make mw.user.options and mw.user.tokens work in debug mode in IE. Now that mw.user is a separate module, we have to make these depend on that and make sure dependencies are actually processed.
* Make ResourceLoaderUserOptionsModule and ResourceLoaderUserTokensModule depend on mw.user
* Load mw.user.{tokens,options,groups} load as TYPE_COMBINED instead of TYPE_SCRIPT. The latter wouldn't wrap the code in mw.loader.implement()
** ...but make sure 'user' (user scripts) is excluded, that one needs to not be wrapped in a closure
* Make TYPE_COMBINED actually work in makeResourceLoaderLink()
* Add a comment in makeModuleResponse() to explain what the weird is_array( $scripts ) stuff is all about
* Add FIXME about how mw.user.options should split off the CSS part into a separate module

AjaxCategories rewrite:

Solving syntax problems, performance improvements and applying code conventions:

* Replaced sprite image with separate images and letting ResourceLoader embed them with @embed (@embed means 0 http requests, less maintenance, none of the known limitations with sprites, and more readable code (named files rather than pixel offsets)

* Many functions were floating in the global namespace (like window.makeCaseInsensitive). A statement ends after a semi-colon(;). All functions declared after "catUrl" were assigned to the window object. I've instead turned the semi-colons back into comma's, merged some other var statements and moved them to the top of the closure. Changed local function declarations into function expressions for clarity.

* fetchSuggestions is called by $.fn.suggestions like ".call( $textbox, $textbox.val() )". So the context (this) isn't the raw element but the jQuery object, no need to re-construct with "$(this)" or "$(that)" which is slow and shouldn't even work. jQuery methods can be called on it directly. I've also replaced "$(this).val()" with the value-argument passed to fetchSuggestions which has this exact value already.

* Adding more function documentation. And changing @since to 1.19 as this was merged from js2-branch into 1.19-trunk and new features aren't backported to 1.18.

* Optimizing options/default construction to just "options = $.extend( {}, options )". Caching defaultOptions is cool, but doesn't really work if it's in a context/instance local variable. Moved it up to the module closure var statements, now it's static across all instances.

* In makeSuggestionBox(): Fixing invalid html fragments passed to jQuery that fail in IE. Shortcuts (like '<foo>' and '<foo/>') are only allowed for createElement triggers, not when creating longer fragments with content and/or attributes which are created through innerHTML, in the latter case the HTML must be completely valid and is not auto-corrected by IE.

* Using more jQuery chaining where possible.

* In buildRegex(): Using $.map with join( '|' ), (rather than $.each with += '|'; and substr).

* Storing the init instance of mw.ajaxCategories in mw.page for reference (rather than local/anonymous).

* Applied some best practices and "write testable code"
** Moved some of the functions created on the fly and assigned to 'this' into prototype (reference is cheaper)
** Making sure at least all 'do', 'set' and/or 'prototype' functions have a return value. Even if it's just a simple boolean true or context/this for chain-ability.
** Rewrote confirmEdit( .., .., .., ) as a prototyped method named "doConfirmEdit" which takes a single props-object with named valuas as argument, instead of list with 8 arguments.

* Removed trailing whitespace and other minor fixes to comply with the code conventions.
** Removed space between function name and caller: "foo ()" => foo())
** Changing "someArray.indexOf() + 1" into "someArr.indexOf() !== -1". We want a Boolean here, not a Number.
** Renamed all underscore-variables to non-underscore variants.

== Bug fixes ==

* When adding a category that is not already on the page as-is but of which the clean() version is already on the page, the script would fail. Fixed it by moving the checks up in handleCategoryAdd() and making sure that createCatLink() actually returned something.

* confirmEdit() wasn't working properly and had unused code (such as submitButton), removed hidden prepending to #catlinks, no need to, it can be dialog'ed directly from the jQuery object without being somewhere in the document.

* in doConfirmEdit() in submitFunction() and multiEdit: Clearing the input field after adding a category, so that when another category is being added it doesn't start with the previous value which is not allowed to be added again...

Followup r93343

Swap function_exists for MWInit::functionExists()