Revert r97711

WONTFIX:
 * (bug 30245) Use the correct way to construct a log page title.

 This was previously reverted for 1.19 and now reverting again for
 1.20, only this time proposing it not to be temporary.

 Right now this is already live on 2 wikis and as soon as it goes
 live on a non-English wiki (since English namespace name for Log
 matches the canonical name "Log", and as such causes no breakage yet
 for bots using the IRC feed), as soon as it goes live on a
 non-English wiki it will all log actions that bots are monitoring
 using the irc feed.

 (because no matter which log action is monitored, it all goes
 back to detecting the page title of Special:Log)

 If properly announced ahead of time it could be done, but we don't
 have that time anymore. Perhaps re-try in 1.21 or 1.22, but as far
 as I'm concerned, this bug can be kept WONTFIX as it only affects
 the IRC feed. And it's not worth the loss that will undoubtedly
 result from breaking this.
 Theoretically a certain amount of time for the announcement could be
 required, and if the change enables something or fixes a security
 problem, it could be justified, but for this it is just not worth
 it.

 The human-readable output from the API and on Special:Log are fully
 localized and even better so with the rewrite that Niklas did last
 year. The IRC feed however, has little to no gain from this and only
 causes breakages for the the large amount of bot usage from it, of
 which many are no longer maintained and as such will not be fixed,
 no matter how much time it is given. Granted those will probably
 break no matter what at some point, but hopefully we will have
 push notification *with* structured data[2] by that time to
 encourage rewrites of those bots.

Change-Id: I229e345c74b10f4d96b8d2d305b4a7623825b3f6
Links:
 * https://www.mediawiki.org/wiki/Special:Code/MediaWiki/97711
 * https://www.mediawiki.org/wiki/Requests_for_comment/Structured_data_push_notification_support_for_recent_changes

Merge "Add CSS class .mw-code (matching <pre>)"

Add CSS class .mw-code (matching <pre>)

- Add CSS class .mw-code.
  All skins that stye <pre> now also style .mw-code.
  This is because there are situations where a <pre> element is not
  allowed (e.g. when the child code contains <div>) by Tidy and W3C, but
  should still be styled as such.

- Required for fixing bug 19416 and bug 35875.
  SyntaxHighlighter Geshi has it's own custom markup that can't be relied on
  for both consistency (it can change over time) and due to its own
  over "!important"-ized css reset on it's own classes, so although it
  (sometimes!) does use a <pre>, that <pre> is completely reset so the skins
  css doesn't (and frankly shouldn't) apply to that.

  Up until last year this extension wrapped all of Geshi's internal output in
  a <div class="mw-geshi" />. Leaving all wmf wikis to have to create a hack
  in their local css (for every skins!) to reproduce the skins's <pre> style
  on .mw-geshi.

  Previously this was attempted to be fixed inside Geshi itself by wrapping
  all of Geshi's internal output in a <PRE class="mw-geshi"> instead of a
  <DIV class="mw-geshi">. That worked fine because that element didn't have
  any of geshi's internal classes on it so it was outside the scope of the
  reset and the skins' css applied fine.

  However when Tidy is enabled this got messed up badly and (part) of the
  Geshi output was ripped out of the containing <pre> by Tidy because the W3C
  doesn't allow a <div> to be inside a <pre>.

  Now fixed in a different way by instead providing a css class in all skins
  that when given to any element will give it the same styling that it does
  to <pre>. Separation of content and styling.

  Tested in all core skins, works great. The style of the geshi output is
  preserved and the container looks like any other <pre> in that skin.

Change-Id: I4ff5d0197ba8dc0dad3cafd03f2b120bc22e75cb

[FileBackend] Added support for concurrent file write operations.

* FS backends work via popen() and basic shell commands.
* Swift backends use the custom SwiftCloudFiles async features.
* Refactored storagePathsRead()/storagePathsChanged() to normalize the paths
  for correct dependency detection in FileOpBatch.
* Cleaned up SwiftFileBackend exception handling to make debugging easier.
* Added a quick and dirty performance testing script.
* Updated unit tests to include a run with parallelize=implicit.
* Improved file test failure output a bit.

Change-Id: I6a5ed743c30c598e0dd7301dbdb3631c460332fd

Use the style for htmlform.tip Vector wide.

Instead of only in preferences.

Change-Id: I54438d8661c6181eede73c992e60298c4a54535d

Merge "Follow up change Icc8dc4ae: add new message keys to maintenance scripts"

(bug 35240) Fix mw.loader state machine.

Main changes:

* handlePending() correctly handles "missing" and "error" states
and propagates error states up the dependency tree.

* handlePending() is called whenever a module enters one of the
states "ready", "error", or "missing" (in execute() and
mw.loader.state()).

* load() filters out not only undefined modules, but -- by the logic of
the comment there -- also modules in state "error" or "missing".

Minor changes:

* recurse() renamed to sortDependencies(), also uses a hash for
unresolved now instead of an array.

* execute() was never called with the second parameter "callback", hence
I've removed it.

* simplified the "are all dependencies 'ready'?" test and moved it to
its own function.

The change comes with additional QUnit tests for mw.loader. If I run
these tests against the current mediawiki.js, several of them fail. In
particular test #86 ("mw.loader real missing dependency") is
instructive: if the server returns "missing" for a module, dependent
modules never progress beyond "loaded", and if there are jobs (from
mw.loader.using()) depending on such a missing module (directly or
indirectly), neither ready() nor error() is ever called.

Running the tests against the changed mediawiki.js in this change, they
all succeed for me.

Patchset 2: whitespace changes, $_GET instead of $_REQUEST in
testloader.

Patchset 3: XML::encodeJsVar() in testloader, deepEqual() in Qunit
tests.

Patchset 4: rebase

Patchset 5: Amend commit message only (typo)

Change-Id: Ia67edfc07fc9237def04ed13bb2cee16e519d7af

Remove UI overrides for simple English.

This was introduced a while ago, but this code should simply fall back to
English completely.

Change-Id: If311c2b6847068bdca6dd44d294ca207639a9f5b

Follow up change Icc8dc4ae: add new message keys to maintenance scripts

Change-Id: Ic7040fb32044bf8b500c081881e4b29261f1296b

Merge "fix bug 29002 - in RTL wikis resizable are flipped"

Merge "HTTP: Fix ded6170bd2072971f72db794f3ab1625a8e13475, which broke all HTTP calls."

HTTP: Fix ded6170bd2072971f72db794f3ab1625a8e13475, which broke all HTTP calls.

* respHeaders contains arrays for each header (should've been
  respHeaders['content-length'][0] instead of respHeaders['content-length'])
  using getResponseHeader() instead.
* This also fixes InstantCommons, which was broken by this (bug 36653).
* Also cleaned up a bit of w/s in foreign repo code while in the area.

Change-Id: I429b0a36618cc2b873077516e9c1023dc5b7d693

Merge "(bug 36669) Installer CSS outdated"

Remove 'jquery.ui.button' as dependency for mediawiki.action.history

- Follows-up 91c38e63da79e1bf426907f052e156140716e653
  which last updated this module after which jquery.ui was no longer
  used.

Change-Id: If8c6b64ca76dd4c03e8c65145efba30e579e39a8

Merge "Upgrade to jQuery UI 1.8.20"

Added some more output to cleanupUploadStash.php

Change-Id: I8acf22df78a97c6e4003e7e5ed89800f1be41c6a

[FileRepo] Purging/transaction fixes.

* Purge old version thumbnails on move
* Pushed purge operations outside of the DB transaction for move/delete

Change-Id: I260940149599eea814226d3c5b7092f4d6544a72

(bug 36669) Installer CSS outdated

Change-Id: I7ac606f2e8538eb5fd10a3944cf9d8b42bbc5ed5

Merge "Checking out RELEASE-NOTES-1.19 from branch gerrit/REL1_19"

Checking out RELEASE-NOTES-1.19 from branch gerrit/REL1_19

- File was updated inside the branch, updating master.

Change-Id: Ie54e621a1ce5494d98bcd76d4c858c0ed423e581

Merge changes Ic19071c7,Icc8dc4ae

* changes:
  Message documentation for Icc8dc4ae (circular references in strip tags)
  (bug 35315) Detect circular references in strip tags

Merge "Typo in comment"

Merge "Re-enable css @import unit tests"

Typo in comment

Change-Id: Ic5e1451f86d7bcda374fd12b5a9b85fe199a7c1a

Localisation updates from translatewiki.net.

Change-Id: I787349e3479ad12431d82ead7c9ae58f3675fd66

Merge "Show descriptive error message on invalid title instead of showing an empty line"

Merge "Optimized SqlBagOStuff::getBatch() a bit."

Merge "(bug 35993) gettoken should go die in a fire (part 1)"

Merge "Make backupTestPass.inc require_once by itself its dependence on backup.inc Change path-less requires to full path ones."

Merge "[FileRepo] Locking and transaction fixes."

Merge "Add new-inline-tags to tidy.conf"

Show descriptive error message on invalid title instead of showing an empty line

Change-Id: Ided20c465fee37087239a0caef01c7427c5ceeef

Merge "(bug 36568) Fixed "Illegal string offset 'LIMIT'" warnings in updater"

Merge "Don't make two database requests to load the same object, again."

Merge "(bug 20189) Added 'Show/hide selected revisions' button and checkboxes to Special:Log."

Merge "[FileBackend] File locking fixes."

Merge "Make getContinueStr protected as it's used in subclass by TMH"

Merge "Include the namespace of the page in the "target" parameter of the form to confirm display of a deleted file."

Merge "ORDER BY/GROUP BY accept arrays"

Make getContinueStr protected as it's used in subclass by TMH

Change-Id: I3bd265805122d3ae12b986d1396ca634ca9f8097

Merge "Fix comment ResourceLoader::tryRespondFromFileCache()."

Merge "Added Linker::getInvalidTitleDescription() to display invalid title entries in various places."

Merge "Use CamelCase in both ConfirmEmail and InvalidateEmail page names."

Fix comment ResourceLoader::tryRespondFromFileCache().

Mention ResourceFileCache instead of ObjectFileCache for the $fileCache parameter since the signature is
function tryRespondFromFileCache( ResourceFileCache $fileCache, ResourceLoaderContext $context )

Change-Id: I8e738c847ea358117a2a9b54f41fdb10308e2656

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: I604ed8ddc572e89001e49df6740ab42307bc0330

Include the namespace of the page in the "target" parameter of the form to confirm display of a deleted file.

Currently the name of the file is sent without the "File:" prefix making MediaWiki think we want to display a page in the main namespace.

Change-Id: I5a42764c8e9289d13b95a1989be63fdf357bc6ef

Use CamelCase in both ConfirmEmail and InvalidateEmail page names.

Change-Id: I2685023eff58479f0eecd3d6ff416cbe22ee425a

[FileBackend] File locking fixes.

* Fixed unlocking logic in FSLockManager for case when an EX lock was made, then an SH one, and then the EX one was "unlocked"
* Avoid hiding useful unlink() warnings in FSLockManager
* Reduced locking use in test cleanup code
* Added a simple testLockCalls() test function
* Made a few cleanups & fixes to backend tests

Change-Id: I1110d9b537c450d9feca5a2fb35519c22435e81d

Message documentation for Icc8dc4ae (circular references in strip tags)

Change-Id: Ic19071c752df87bb8efe6852599563cd1d036b6e

(bug 35315) Detect circular references in strip tags

Explicitly detect circular references in strip tags and break the loop,
similar to how we deal with circular references in templates. This is
necessary to support Scribunto since we imagine we will provide an API
that allows strip markers to be forged.

The recursion depth limit is a consequence of changing the algorithm
from iterative to recursive, it's required to protect the stack against
deeply nested #tag invocations.

Change-Id: Icc8dc4aedbced55ad75b3b5a5429a376d06d9b31

(bug 20189) Added 'Show/hide selected revisions' button and checkboxes to Special:Log.

Change-Id: I2c4d4e1234d5ab9f7d514831351ea3b298811165

Fix for r113993: don't log UsageException instances

Change-Id: I205252534fcf7a0ee486938123c1e15cd036f4f1

Merge "Make $wgShowExceptionDetails=false more feasible for production"

Make $wgShowExceptionDetails=false more feasible for production

* Make the HTML error message prettier, with a nice red box and
  instructions to modify LocalSettings.php hidden in an HTML comment.
* Show the exception class name, since that's pretty safe.
* Show a random "log ID" to the user, and also send it to the exception
  log, to allow easier log correlation.
* Optionally send backtraces to the error log, enabled by default.

Change-Id: Ie92e46032b3d194c4217119567847a38a53be577

Fixing fatal error - you can't call getUser on a WebRequest object

Change-Id: I323e9c39cd89673b7e369df3b77573e51f4ea2e3

Merge "Treat truncated HTTP requests as errors."

Merge "add type check and bail out when title could not be created"

Merge "Let tablesUsed imply needsDB in MediaWikiTestCase."

Merge "fix 2 missing "local var" JavaScript statements"

Merge "Adding $this->tablesUsed to RevisionStorageTest."

Adding test case for basic WikiPage functionality, including database interaction.

New tests are added to avoid regressions when the ContentHandler facility
is introduced.

Changeset 3: use tablesUsed to allow database cleanup after test, as per
             Aaron's suggestion.

Changeset 4: use array_merge instead of +=.

Changeset 5: trying dumber rollback test to see of jenkins likes it better.

Change-Id: I0a9f8dfd30031baa4b12cc128a39059868149884

Let tablesUsed imply needsDB in MediaWikiTestCase.

If the test says it uses database tables, then it needs the database.

And while we are at it, make the check for the test group declaration
more lenient.

Patch set 2: tweaked according to demon's comments.

Change-Id: Ib123745d45b85cebeab5ec87ea7d8227ec3d1dea

Treat truncated HTTP requests as errors.

We detect such prematurely ended request when there's a Content-Length
greater than the body we got. A strict comparison would easily fail
when there was a content encoding layer, but hopefully we won't hit
a compressed request _larger_ than the original content.

See http://thread.gmane.org/gmane.org.wikimedia.mediawiki/39622
for breakage caused by truncated HTTP replies.

Change-Id: I71418424730d46a781bde5cbfda8038457ec79c5

Adding $this->tablesUsed to RevisionStorageTest.

This is a follow-up to I6934d03e (no 6414).

Declare tables in $this->tablesUsed to allow for database clenaup,
as Aaron suggested.

Change-Id: Iac05b0428b335879729f6d3b993eac0d3151429c

Merge "(bug 36603) Make DumpFilter::$sink public again"

(bug 36603) Make DumpFilter::$sink public again

This is required by dump unit testing for now. The change was introduced
by commit ab6e54e0 https://gerrit.wikimedia.org/r/#/c/6089/

We will have to update the Dump tests to no more reference sink directly
but use a proper accessing method that has yet to be written.

Change-Id: I1246739c4ea174b13af475db8cfcb48740ecc2df

Localisation updates from translatewiki.net.

Added language file for "sat", updated Names.php and RELEASE-NOTES.

Change-Id: I2dbe696fbe5b98352895fe1be2de96eb767b2b0b

Merge "Made WikiPage recall the source of the data used to load its state."

Merge "Adding tests for dumps"

Added Linker::getInvalidTitleDescription() to display invalid title entries in various places.

This method will use two new messages 'invalidtitle-knownnamespace' and 'invalidtitle-unknownnamespace' depending on the fact that the given namespace number exists or not.
I did put that method in Linker because I plan to use it in various places, notably in other QueryPage and Pager subclasses.

Change-Id: I13e7cdc2c0a8e86dc5e4b144b6012f3864d2ec06

(bug 36568) Fixed "Illegal string offset 'LIMIT'" warnings in updater

No RELEASE-NOTES entry since this is going to be backported and RELEASE-NOTES-1.19 is out of sync.

Change-Id: I1ec6c824677606ac6f93cc43345942e4de8bd3f4

Merge "Localisation updates from translatewiki.net."

Merge "(bug 36537) Rename wfArrayToCGI to wfArrayToCgi"

Merge "Pass the result of the database queries in Title::getPreviousRevisionID() and Title::getNextRevisionID() into intval()."

Made WikiPage recall the source of the data used to load its state.

In WikiPage.php:
* Added WikiPage::$mDataLoadedFrom to store the source of the data used to load the state of the object and four new WikiPage::DATA_* constants for its possible values.
* Added WikiPage::convertSelectType() to convert 'fromdb', 'fromdbmaster' and 'forupdate' to the new WikiPage::DATA_* constants.
* Added $options to WikiPage::pageData(), WikiPage::pageDataFromTitle() and WikiPage::pageDataFromId() so that the "FOR UPDATE" option can be passed to DatabaseBase::select().
* Added new possibility "forupdate" to WikiPage::loadPageData() load the data from the master database using SELECT FOR UPDATE; this avoids to have to do this by LinkCache (via Title::getArticleID( Title::GAID_FOR_UPDATE ) )).
* Changed WikiPage::doDeleteArticleReal() to use this new feature so that all the data stored in WikiPage is up-to-date.

My point is also to deprecate the loading using SELECT FOR UPDATE in Title and remove LinkCache::forUpdate() at some point (there are still one usage in Title::moveTo(), two other in UploadFromUrlTest plus some in extensions).

In EditPage.php:
* Don't call WikiPage::clear() after fetching informations from master, this destroys all we work we did to get the correct data.
* Reload the whole data from master using SELECT FOR UPDATE directly in WikiPage and not only in Title. The problem was that before, the up-to-date information was only available in Title and not in WikiPage.
  Consider the following sequence from a MySQL prompt (where both revision 1 and 2 belong to page one, revision #2 being the current one):

mysql> UPDATE page SET page_latest=1 WHERE page_id=1;
mysql> COMMIT;
// Now grad the edit form for page #1 from the web and do some changes
mysql> BEGIN;
mysql> SELECT page_latest FROM page WHERE page_id=1 FOR UPDATE;
// Now submit the web form
mysql> UPDATE page SET page_latest=2 WHERE page_id=1;
mysql> COMMIT;

Before you ended-up with a "edit conflict" form with revision #1's text being displayed as current text (even if the texts are mergeable), due to the fact that
in the submit request the WikiPage object was loaded at the moment where page_latest was 1 (certainly due to MySQL's "consistent read" feature) making the
"UPDATE page SET ... WHERE page_id=1 AND page_latest=1" query of WikiPage::updateRevisionOn() return zero row, and thus WikiPage::doEdit returing a fatal Status object with message "edit-conflict".
Now the SELECT FOR UPDATE is done in the WikiPage, meaning that the object has the correct data and EditPage will correctly try to merge the revisions (and show the correct edit conflict if needed).

Change-Id: Ic4878ddb4dd96432b7ecaf43f9f359458d966999

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: Iad15427368d13a6cc11c8512b7265e8c7b8878a5

[FileRepo] Locking and transaction fixes.

* Make sure locks for file moves cover the whole operation.
* Pushed purging out of the move/delete/restore transactions.

Change-Id: I398c5627808fa79739f0dee632c4edf7416507c1

Merge "$wgGitRepositoryViewers to link to gitweb"

Merge "Added missing GPLv2 headers in some places."

Make backupTestPass.inc require_once by itself its dependence on backup.inc
Change path-less requires to full path ones.

Change-Id: Ida37a3ec9bb1d5c99f4bbbf6d967cb40d0442e38

Add new-inline-tags to tidy.conf

by default tidy does not support html5 tags.
TimedMediaHandler outputs video, audio and source
To make TimedMediaHandler work with $wgUseTidy = true,
those tags need to be allowed in tidy.conf:
new-inline-tags: video,audio,source,track

Fixes #30541

Change-Id: I528c9521e9277be308a6a75cc3fc57c83905b5df

$wgGitRepositoryViewers to link to gitweb

My git remote is configured just to be ssh://review/mediawiki/core.git
and I have "review" set up in $HOME/.ssh/config.

Unfortunately, I need to change git remote URLs to make sure
the repository is linked from Special:Version.

This shouldn't be necessary; either we should fallback to the
official MediaWiki git repository or we should add the configuration
option to adapt to local needs.

Change-Id: I2e0b6470c16ec36d0e94cceab844f4a4c4334067

Localisation updates from translatewiki.net.

Change-Id: I731ab96929dac09112de978b4ed387b044893d36

(bug 35993) gettoken should go die in a fire (part 1)

Mark gettoken param as deprecated, so users can fix their tools.
The second part will remove the support for gettoken from ApiMain,
but that is in the future.

Change-Id: I369ddc740c38d80aa8389544087ac6671cabc618

Move optionstoken from meta=userinfo to action=tokens.

Follow up to I0d6c654a7354ba77e65e338423952a6a78c1150f.
I have also added a URL to a help page on mw.org.

Change-Id: Ie223930cfc313aff150e2dcfd70b74bf4360a8a8

Added missing GPLv2 headers in some places.

Also made file/class documentation more consistent.

Change-Id: I1615bf268515ed7169e33173cea6ec30402451dc

(bug 33224) add variants of content language to meta=siteinfo

Change-Id: I0b92e1e626f0c5bc543c5d134eb44d7ca7722217

Fix typo and small error in release notes.

Change-Id: I6855a441c2924a0c7d659909b825c79e324a51a7

(bug 28814) add properties to output of action=parse

Change-Id: I27cf2daaf6f34596bc34aac230601d5688a0066f

(bug 34927) Output media_type for list=filearchive

Change-Id: I7c9b2f614ab319facfc9fb30893d82a9427e4253

(bug 36537) Rename wfArrayToCGI to wfArrayToCgi

- For consistency with wfCgiToArray()
  This way instead of the other way around to keep lowerCamelCase natural.

Change-Id: If4528f6572afeab42ef0602a427ac44da9830a3f

Use right case for Title::makeTitle

Change-Id: Ia91b3679f03eefb9970a790c784a92ea6fb0c274

fix 2 missing "local var" JavaScript statements

Change-Id: I0e8a5824be635e526c385f3168ffc6269928a05c

ORDER BY/GROUP BY accept arrays

renaming all variable which hold "' DESC'" to $sort
to keep it all the same

Change-Id: I75118f8cdd701f53949fe5cdd7155fb07f78ff65

Merge "Use $wgExpensiveParserFunctionLimit over ParserOptions"

Do not treat EXPLAIN as write query

Change-Id: Idb15955b94ee6977f3a4692df44d445d3450b1a5

Merge "Show Highest expansion depth in limit report"

Merge "Use local context to get messages and time formatting methods of Language class"

Merge "Add MWNamespace::isNonincludableNamespace"

Add MWNamespace::isNonincludableNamespace

Method is a wrapper around $wgNonincludableNamespaces,
replaced the one place in parser and
add it as info to api's meta=siteinfo

Change-Id: I501b811137c39f5c2d9ea35c78fef8ae22d21bfe