Revert r96824: causes fatal errors when search query doesn't happen to be a valid title. Code doesn't appear to have been tested?

Typo fixed in message text of MediaWiki:logentry-suppress-delete
see http://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Logentry-suppress-delete/en

Use local context instead of global variables

Introduce MessageCache::getAllMessageKeys(), which returns all message keys stored in the MessageCache, and use this to power the amincludelocal parameter in meta=allmessages which adds messages that don't exist in PHP but exist as MediaWiki: pages only.

FU r89706: Cleaned up getParserOptions duplication problems

FU r84610: Renamed getImageTimeKeys() to getFileSearchOptions()

Fix typo in r97044 reported in twn

Fixed WikiPage::newFromId to use the factory() function

Switched from prepending http for protocol-relative $wgUploadStashScalerBaseUrl to wfExpandUrl()
Followup to r96934

FU r97022: Fallback to empty array for getExtraSortFields() result in __construct() if no value is set for the sort order/type.

Fixed incorrect userId reference

Added @todo items

Introduce Language::getMessageKeysFor() and use it in ApiQueryAllmessages

Whitespace/documentation

Revert r96715 per committer's (Chad's) CR comment

Followup r86304, guard against $title AND $wgTitle being null

Add profiling too

Revert part of r97046 which was not intented to be committed.
As side note, would be nice to have this configurable someway.

Revert r96798 - makes parser go infinite loop somewhere

Removed some more unused message made obsolete by r96847 (one was still referenced in r97044, but was already unused)

Added a bit defensive programming, as suggested by Aaron in r97040

RevisionDeleter:
* Fixed comment of getChanges and made it public
* Removed getLogMessage and expandMessageArray, now unused
LogFormatter:
* Cache parsed parameters locally, since this function is going to be called few times for every log entry
DefaultSettings:
* Could not find any use of suppress/file log
LogPage:
* Removed old formatting code, now unused
WikiPage:
* Converted (suppress|delete)/delete logs to new format

Added DeleteLogFormatter, handles part of suppression log as well.
Updated messages.
This also addresses bug 18578.

Fix variable name typo in r96847

Chrome seems to already support dir=auto, and I see no downsides using it for user comments.
Amir said that it is good.

There was a double space between log action text and user comment. Remove the space produced by linker.

Instead of throwing fatals, accept null titles in old log entries.
New log entries will still die with fatal few lines below, but that's good since that should never happen.

Drop JSON support. It was never used and causes problems like bug 30854 since PHP managed to screw up json_decode too.
It was not a good idea to autodetection of the format anyway, but it isn't such a problem with unserialize.

Partial revert r97035, followup r96930: make recentchanges tests pass again

Renaming SpecialRecentchangesTest file to match conventions
* There should be no two files with the same name
* Test suite files should be named after the main file containing the code being tested followed by "Test"

Merged wgWhatlinkshereLimit from Wikia. It defaults to the same value it was hardcoded to before.

Merged Wikia change.

Merged in mime-type from Wikia (for Sawfish Window Manager).

Possible fix for issue reported in r83762. I had a hard time confirming the problem/fix with profiling. Committing this now to get more attention.

Followup r96978, clean up code duplication by factoring out the code building load.php requests into ResourceLoader::makeLoaderURL() and makeLoaderQuery()

Removed parser hacks from r88740, r88741, r88780

Document the fact that some ResourceLoaderContext methods may return null

Localisation updates for core and extension messages from translatewiki.net

Consistency in wording of tool tips. Spotted by Gustrónico.

Fix the fixme on r88053: dependency handling was broken in debug mode in certain cases. More specifically, if A is a file module that depends on B, B is a wiki module that depends on C and C is a file module, the loading order is CBA (correct) in production mode but was BCA (wrong) in debug mode. Fixed this by URL-ifying scripts and styles for those modules in debug mode, as I said to on CR. What this means is that the initial debug=true request for a module will now always return arrays of URLs, never the JS or CSS itself. This was already the case for file modules (which returned arrays of URLs to the raw files), but not for other modules (which returned the JS and CSS itself). So for non-file modules, load.php?modules=foo&debug=true now returns some JS that instructs the loader to fetch the module's JS from load.php?modules=foo&debug=true&only=scripts and the CSS from ...&only=styles .

* Removed the magic behavior where ResourceLoaderModule::getScripts() and getStyles() could return an array of URLs where the documentation said they should return a JS/CSS string. Because I didn't restructure the calling code too much, the old magical behavior should still work.
* Instead, move this behavior to getScriptURLsForDebug() and getStyleURLsForDebug(). The default implementation constructs a single URL for a load.php request for the module with debug=true&only=scripts (or styles). The URL building code duplicates some things from OutputPage::makeResourceLoaderLink(), I'll clean that up later. ResourceLoaderFileModule overrides this method to return URLs to the raw files, using code that I removed from getScripts()/getStyles()
* Add ResourceLoaderModule::supportsURLLoading(), which returns true by default but may return false to indicate that a module does not support loading via a URL. This is needed to respect $this->debugRaw in ResourceLoaderFileModule (set to true for jquery and mediawiki), and obviously for the startup module as well, because we get bootstrapping problems otherwise (can't call mw.loader.implement() when the code for mw.loader isn't loaded yet)

Fix typo in error message

Address one of the fixmes on r88053: because addScript() is asynchronous, calling it repeatedly can lead to scripts loading out of order. The specific bug was that if you had a module with two script files, A and B (in that order; scripts are always loaded in the order in which they are specified for multi-file modules) and A is 40M while B is 1K, then B would be executed before A if the module is loaded after document ready (through e.g. mw.loader.using) and the files are not in the browser cache.

So instead of calling addScript() repeatedly in a loop, this commit calls addScript() recursively from its own callback, using a helper function called nestedAddScript(). This also means the if block that checks if the scripts array is empty can be removed, because nestedAddScript() handles empty arrays correctly.

Fix documentation, document return type

* Use local context instead of global variables
* Pass the context to the HTMLForm object

Move MessagesEnRTL.php to MessagesEn_rtl.php so that it can be accessed with the more sensible language code "en-rtl". This code is not in $wgLanguageNames and so cannot be selected in the UI. This change is to support a debugging extension which I will commit shortly.

Call Linker methods statically

Fix revert artefact of r96930

If the scaler URL is protocol-relative, things just break here.  Sets a sane default.

Revert r87992 and followups r87998, r89028 (Support abstraction for 'NOT IN' SQL structure). Per discussion on CR and elsewhere...we're not 100% sold on the new format yet.

Changing the database api like this should be carefully thought out before we get stuck with it for 6 more years and end up hating it.

Use a username different from the main BlockTest.php, otherwise concurrency errors occur (as seen on Jenkins builds)

Localisation updates for core and extension messages from translatewiki.net

* Some documentation
* Use Title->isMainPage() in Article->view()

Updated r86144 code due to reverts in r96889 (specifically of Parser.php change in r86072)

Remove unused variable and update code comment per r96888.

Reverted r86072, r86419 per CR. Lots of conflicts resolved here. Removes lineStart from r87346 code as well for preprocess parser tests.

* (bug 30865) Fix regression breaking category pages when there's no local page content

Output in CategoryViewer::getHtml() was trying to close an outside div, then re-open another one (with different dir/lang settings). Switched it to instead open a child div with the desired properties, so there's no danger of surprise inconsistency.

Turn ParserOptions::setSkin() into a no-op since mSkin is no more used

Add some documentation

Swap $this->file->name for $this->file->getName()

Start producing new style move log entries

Move RELEASE-NOTES from r96373 from RELEASE-NOTES-1.18 to RELEASE-NOTES-1.19 after it was merged in r96845

Already removed this support for very old move log entries from plain changes list, now also from enhanced.

* Move <small> styling from message to shared.css
* Wrap complete section into a div
* Wrap "Other resolutions:" into a span and remove it from printing. Useless on paper.
* Add plural to 'show-big-image-other'

Followup r94435: don't barf on jobs inserted before this revision that don't have a table param

Added visibility markers to some Linker methods

* (bug 30864) Use bengali numerals for <ol> for Assamese

* Use local context instead of global variables (where possible)
* Made getFormFields() non-static since the only call to it is in that file

(bug 28545) When using the uca-default collation, sortkey's starting with a
space (U+20) will now sort under an invisible header like in 1.16 rather than a U+6DE (ARABIC START OF RUB EL HIZB)

SpecialSearch: add two hooks, one to modify the power search box, one to change the message pointing to the page title searched for

WikimediaIncubator:
* use these hooks and SpecialSearchSetupEngine to improve search usability
* use a different message for the language code in preferences, per suggestion of Amir

* r96737: set OutputPage->addWikiText() as an interface message by default
* r96760: use 'noflip' for $flip in OutputPage->addInlineStyle() so it's always a string
* Add some parameter documentation

Pass the context to the HTMLForm object

(follow-up r96765). Appearently my release notes for that revision were horrible ;)

(follow-up r96767) Make {{DEFAULTSORT:..|noerror}} and noreplace i18n-izable.

Hopefully i did this right. I've never added a magic word before...

Localisation updates for core and extension messages from translatewiki.net

Per r96723: register new message key to maintenance script

(Bug 30364) LanguageConverter should depend on the page content language instead of the wiki content language. Remains largely the same, except for extensions using the PageContentLanguage hook like Translate and WikimediaIncubator.
* Made variant tabs hidden on special pages (has no or sometimes wrong/inconsistent effect there; plus it's mainly in the user language)
* Made redirects be in content language again (remove from Title->getPageLanguage())

fix r96712 copy paste error :(

Adress fixme on r92054: Style print

* Use local context instead of global variables
* Call Linker methods statically
* Added missing call to outputHeader()

* Follow-up r96420: don't show e-mail links to anon users
* Fix for r24612 (!) spotted while fixing the above: parenthesis arround the bitwise check are needed since ! operator has higher precedence than &

(bug 16755) Add syntax to DEFAULTSORT - {{DEFAULTSORT:Foo|noerror}} and {{DEFAULTSORT:Bar|noreplace}}
to in both cases prevent the "A sortkey already was specified earlier in the page, yadda yadda" message and in the second case, also do not replace the sortkey
if one was already specified.

(bug 30722) Add an identity collation that sorts things based on what the unicode code point is (aka pre-1.17 behaviour).

I'm tagging this 1.18 because the original bug was for iswiktionary wanting it, so it'd be nice to get it in 1.18.

Fix standard skin, which doesn't style ul/ol. The margin added in shared.css (see r91518 and previous) made standard have both margin and default browser css. This resets that.

Do r93269 in a better way (also fixes some rare cases). Use 'margin/padding' instead of 'margin-left/padding-left' so it doesn't have to rely on the direction.

* Per Aaron on r96261, use string 'flip' instead of boolean
* Apparently @noflip should only be set once, otherwise it behaves differently

mediawiki.Title.test: Add test for space collapsing
* When I first passed "Foobar___.js" to mw.Title and saw that it was stripping away (some) spaces, I thought it was a bug (since extension is part of the title and should be kept as is). Anyway, as it turns out it's not stripping them away because they are between 'title' and 'extension', it's a general rule that there should not two or more spaces next to each other in a title. Adding test to assert and confirm this behavior.

mediawiki.log.js: Add support for variadic arguments (just like the native console does)
* Use unshift() when calling window.console.log (that way logging an object will not result in "prefix> [object Object]" but in "prefix> { key: .., .. }" (which most consoles do, similar thing for logging elements for which most console API implementations offer special features)
* Use string concatenation when using our own fabricated log. Did not use unshift for both, because in the latter that would show a bad comma ("prefix:>, foo, bar"). The native API does not show a comma between two logged variables, they're just shown next to each other.

mediawiki.log.js: Refactor
* Add @author info etc. to the file header
* Merge var statements
* remove single quotes around object keys where not needed
* Changing bottom padding on <body> to be relatively added instead of absolutely (body might have a bottom padding already!)
* Add return statement after reaching out to window.console. No need for the rest of the function to be executed in that case. outdenting the rest after the new return statement back one tab (thus removing the else-case)

Simplify some contexts

Call Linker::titleAttrib() statically

Correctly parse interface messages as such, so it doesn't get converted according to the content language (like r96257/r94395/r94279)

Also whitespace fix in MessagesEn for r96723

Fix comment

Revert r96732

Invalidate user cache upon password change

Should help (maybe fix?) Bug 30850 - maintenance/changePassword.php does not purge (APC) cache

Add @since to the Deferred stuff, ping r96716

* Change the message 'yourvariant' to a more descriptive one (might be a bit too long however), and remove it as an optional message. Also added a help message to explain it.
* Make getVariantname fallback to the language name, and use that in Preferences. Shouldn't change the names in practice, but is a more logical implementation imho.
* (Fix a typo in Language.php)

Localization update for he.

* Use local context instead of global variables
* Call Linker methods statically

Moved global define into class constant, replaced a bunch of $fname with __METHOD__.

Add some other ServicesJson tests.

Refactor deferrable updates into classes & interfaces, also add helper method for the most common use case:

$wgDeferredUpdateList[] = new HTMLCacheUpdate( $title, "sometable" );

I kept b/c with $wgDeferredUpdateList for now, but seeing as only 3 exts in svn use it (FileSearch, FlaggedRevs and WikiScripts), I'd like to deprecate it pretty soon :)

Add a comment about something I noticed while spelunking the code

Add some basic tests to compare output of native json support and the Services_Json implementation, as I promised on wikitech-l. Could use plenty more test cases, but that should be trivial since I did it properly with data providers :)