mediawiki.api.watch: Use formatversion=2 for API requests

With formatversion=2 the JSON response uses UTF-8 instead of escape sequences
with hex for encoding of non-ASCII characters (e.g. "\u00e4" for "ä").

formatversion=2 has a different syntax for boolean values:
  formatversion=1: "watched": ""
  formatversion=2: "watched": true

Consumers of this API that process the response with
  response.watched === ''
must change this to
  response.watched === true

Change-Id: I087ab538b9400d774f8c2c6a8c0f55a33fab6044

Merge "SpecialActiveUsers: Change checkboxes to dropdown"

SpecialActiveUsers: Change checkboxes to dropdown

Moved form to new function and
fixed text about cached version.

Bug: T116354
Change-Id: I29ae63472536f99f7f9546f30d86e7bd324d3094

Revert "EditPage: Show EditFilterMergedContent hook errors in an errorbox"

This reverts commit a02a7ff8eab9d589d77c03d4b4c58fc4a05e65c8.

Bug: T149473
Change-Id: I35a650c6ec478542351824e8ae8a8d8354ffa50f

Merge "Revert "Gallery: Use intrinsic width for gallery to center caption""

Revert "Gallery: Use intrinsic width for gallery to center caption"

Resulted in unforeseen changes to galleries using mode=packed
(and other modes other than traditional).

This reverts commit e6dd624baeca16efe462e3938969a72c9ef4ed1e.

Bug: T149412
Change-Id: I57dabbd3b5c0e52f3fad1b2144764bcc2fdecda0

Merge "Check MW version requires before manifest_version"

Merge "Add PHP DateInputWidget"

mediawiki.util: Fix replacement of $ signs in mw.util.getUrl

Bug: T149767
Change-Id: I91c1afa1f93966d1f01793d124fe8f2ae3dab0cf

WatchedItemStore: Update process cache when adding items

This way, when submitting POST to action=watch, the response will
not show "Unwatched" in the navigation 'content action' tabs.

This was previously the case on any wiki installation that has
more than 1 database as the skin would call User::isWatched(),
which would query stale data from a slave.

This doesn't yet account for the reverse use case (removing from
watchlist). That's to be addressed in a later commit.

Bug: T28292
Change-Id: Ie4b69c985815a77b70692db0c4dbf52e1a6a018d

Check MW version requires before manifest_version

Bug: T149751
Change-Id: I914be4da431dcd613bdf2d1fd5ba9f28cf5b936b

Add PHP DateInputWidget

Bug: T120733
Change-Id: I8f80e27304a916bba16ed255c8e78073f9da3813

Merge "Don't parse <nowiki><span class="error"></nowiki>"

Localisation updates from https://translatewiki.net.

Change-Id: I21d3b38d7ec6af6e2a8e6baa6092a6810eab1820

Merge "DBError expects IDatabase|null as its first argument"

Merge "Parser test file editor"

DBError expects IDatabase|null as its first argument

I came across LBFactory.php:357 while testing my
master/replica db setup. It was not configured correctly
and I got "Argument 1 passed to DBExpectedError::__construct()
must implement interface IDatabase, string given" instead
of the actual error.

Change-Id: I949e1803f84459f8bb83e957a88d1e927cad16f7

Merge "Document diff/DifferenceEngine.php function"

Don't parse <nowiki><span class="error"></nowiki>

Which means we can't check if a parser limit was exceeded while trying
to expand the content of a tag, but that's probably not a huge loss.
It'll just result in potentially strange output rather than an exception.

Bug: T149622
Change-Id: I7910dfa0f61b1cc9168c7ed1498b2bda27c47f0e

Localisation updates from https://translatewiki.net.

Change-Id: I3f98e468088f53f688b357ed1ebf7a36c0df0f0b

Merge "Rename JobQueueDB::getSlaveDB() to getReplicaDB()"

API: Error if sensitive AuthManager parameters are in the query string

There was only 1 hit in the feature usage log in the past 30 days for
this code path, so this should be good to go.

ApiLogin, on the other hand, received 27048 hits over the same time
period. So let's not do that one just yet.

Change-Id: I1ae3b928fda9ddc94c8182155637920713dd404d

Merge "Allow empty href in SVGs"

Remove spaces after cast operators

This fixes the outstanding mis-spaced cast operators to bring them
into line with the coding standards on mediawiki.org (and with the
more common usage within this codebase).

Bug: T149545
Change-Id: Ib7bcf95bbee83d20c05f6d621ce7b4e1fb58a347

Allow empty href in SVGs

bawolff is right, this shouldn't harm anything.

Bug: T149549
Change-Id: I377efdee7478940154ef5ec921bd0c8f7ec0c110

Merge "FormatMetadata: Do not format 'UserComment' as a number"

FormatMetadata: Do not format 'UserComment' as a number

It's a freeform text field.

Change-Id: Ic3150f48c2e38fa26728bc22017b992d8ddfd152

Merge "Don't show warning header for non-standard workflows on login-continue"

Merge "Add firstLetter data for ~50 additional languages"

Document diff/DifferenceEngine.php function

Change-Id: I999824d8ce569de93fe4c9a889edea582d07611c

Localisation updates from https://translatewiki.net.

Change-Id: I695ddf72027c55d160192f0ee4e88d6738a13fa6

Merge "Make determineCoreSchema a public function again in DatabasePostgres"

Make determineCoreSchema a public function again in DatabasePostgres

It was changed to a private function today, but it is called in
PostgresInstaller so it is causing this error

PHP Fatal error:  Call to private method
DatabasePostgres::determineCoreSchema() from context 'PostgresInstaller'
in /home/travis/build/paladox/mw/includes/installer/PostgresInstaller.php
on line 515

Caused by I2a067ca89a03f9ebf3f70a4f36ddae92e5b1e468

Change-Id: Ifac5a5c7dfeceeb8c7827d6d70e757633df77ca5

Merge "Move LinkBatch operations for QueryPage to a helper function"

Move LinkBatch operations for QueryPage to a helper function

This is easier to maintain as duplicated code everywhere.

This change also adds LinkBatch operations to specialpages, that does
not had it so far, but where it's useful.

Change-Id: I4936ecfb2be3a7709d99570c0561ab0cc9cd754f

Add missing Database::timestamp() calls

This was broken for Postgres installations

Change-Id: I0c1d442e3cf1d9a4d5359e7a8f8fd02a31d1ac65

Merge "Fix missing commit() flag in postgres savepoint class"

Merge "Add LinkBatch to Special:UnwatchedPages"

Localisation updates from https://translatewiki.net.

Change-Id: Ia735ef197567ad5151b508bc3b2851610f9b19a4

Add LinkBatch to Special:UnwatchedPages

Get all the information for blue links in one query.

Change-Id: I18cbaf69c5f75e00e48e27d4b5e7220f51c31808

Merge "Make LBFactoryTest.php pass for sqlite"

Fix missing commit() flag in postgres savepoint class

Previously, the transaction was left dangling, causing massive
unit failure.

Change-Id: I8fb6e69b44ff0ceb0cf7c4fb45f204e140472406

Don't show warning header for non-standard workflows on login-continue

When a user needs/wants to login again (reauthentication or user account switch),
the Login special page shows a header to explain what happened ("already logged
in" or "need to re-authenticate"). For now, the form also displays this header
for a continued login workflow (e.g. when the user needs to enter secondary auth-
entication data, like a one time password). This is misleading, as the user already
knows what and why he has to login (again). Other error/warning messages are maybe
not as visible as they should be.

This change omits the header message for the login-continue auth action.

Bug: T146317
Change-Id: I2115bbe7965e8d0326f71872a4d33767dd0f956b

Add firstLetter data for ~50 additional languages

Based on CLDR 29 data files.

This did the relatively easy languages in CLDR 29 (Which is most
of them). I skipped languages with complicated tailoring files.

Change-Id: I8367604f7d3a1cdef9cb4e15813893c8cbfff1ff

Rename JobQueueDB::getSlaveDB() to getReplicaDB()

Change-Id: I7a88c4876e76836c92cc0b45b4e72124a087121c

Make LBFactoryTest.php pass for sqlite

Change-Id: I4fe929e82218231f6c8afa64da8c0ccb42d2c362

Various small cleanups to DatabasePostgres

* Add missing method visibilites
* Removed redundant doc blocks
* Use empty string for mSchema for consistency with
  the base class

Change-Id: I2a067ca89a03f9ebf3f70a4f36ddae92e5b1e468

Merge "Postgres updater fixes to make update.php able to run"

Postgres updater fixes to make update.php able to run

* Remove redundant schema prefix from relname=x query. The
  schema filtering is already done via the JOIN. The relname
  portion is just the table name not <schema>.<table name>.
* Avoid explicit table schema qualification and rely on the
  search path, as MW 1.27 did. Previously it only used the
  global $wgDBschema var to pass to determineCoreSchema()
  instead of keeping it in mSchema.
* Clean up some code duplication in Database::tableName() and
  make the code comments clearer.
* Make DatabasePostgres::tableName() use parent::tableName()
  instead of a method that just wraps this method. The intent
  seems clearer this way.
* Remove unused return value in
  PostgresUpdater::rebuildTextSearch().

Bug: T148628
Change-Id: Id11d9576b7c2fdad22ff7f90727c12997217a632

Merge "Make updateCategoryCounts() have better lag checks"

Localisation updates from https://translatewiki.net.

Change-Id: Ic7e8cf96d2bd8ca7a583824eea1ea4a3ed6429ac

Merge "Add Parser to MediaWikiServices"

Merge "Introduce InterwikiLookupAdapter on top of SiteLookup"

Merge "Pass context to ChangeTags::tagDescription"

Add space between label and selector

Change-Id: Id5c8c857a2a1339b0ce626bac98f0869e27d5a75

Parser test file editor

Add editTests.php, which provides an interactive interface to the
parser tests, with semi-automated editing.

Change-Id: I1a20d007ba4627d562a16c03849bbad7aec0e516

Merge "PHPSessionHandler: Workaround PHP5 bug"

Merge "Change .tocnumber to use grey text for section numbers in Table of Contents boxes"

Pass context to ChangeTags::tagDescription

To avoid wfMessage calls.

Change-Id: Icaa984fa8c69ccb38fbfd3e30877a4c572501c28

Merge "Refactor rev_delete handling on deletion"

Localisation updates from https://translatewiki.net.

Change-Id: I83ef5fa88f65ec941b87888943273892857d5561

Remove tracking category stuff that accidentally slipped into 61adc1e14

Bug: T149310
Change-Id: I0a3725a72b1467c57280ae1880935dd5fa54ae9e

Refactor rev_delete handling on deletion

Refactoring handling of the rev_delete database field, which is not
needed to select, when the suppress option is set.

Also added a new const, to avoid a comment about that things should be
15 ...

Change-Id: Ia5bcb835172bf6c10e2e872555fc67e91806ebe1

Fix reference in docs of HashConfig::set

Change-Id: I82473e42a746ed4280ea22372d8d17d797070334

Merge "UploadBase: Permit SVG files with broken namespace definition (Inkscape bug)"

UploadBase: Permit SVG files with broken namespace definition (Inkscape bug)

Inkscape mangles namespace definitions created by Adobe Illustrator
(apparently it can't parse custom entities or something, maybe just
in 'xmlns' attributes). These files are still valid SVG, and not
a security issue (although Illustrator probably won't like them),
so it's okay to allow them.

Added tests with some example files.

* buggynamespace-original.svg
  File generated by Illustrator (edited by hand to reduce filesize).
  Based on <https://commons.wikimedia.org/w/?curid=16495597>.

* buggynamespace-okay.svg
  The original file, opened and saved in Inkscape (no other changes).

* buggynamespace-okay2.svg
  The original file, opened and saved in Inkscape twice.

* buggynamespace-bad.svg
  The original file, edited by hand to remove custom entities.
  This is not valid XML and should be rejected (although it's valid
  when parsed as HTML, and some image viewers might display it).

* buggynamespace-evilhtml.svg
  An SVG file using an entity declared namespace for a namespace
  we want to ban. Based on buggynamespace-original.svg.

Bug: T144827
Change-Id: I0eb9766cab86a58d729f10033c64f57d2076d917

Merge "New HTMLForm size filter field; add size filter to Special:Newpages"

Merge "Add short-circuit to DatabasePostgres::schemaExists()"

Merge "Factor out some methods in the header generation"

New HTMLForm size filter field; add size filter to Special:Newpages

The conversion of SpecialNewpages to HTMLForm seems to be half-finished.
It's not using HTMLForm to read in the request query, which means we have
to roll our own logic. Kind of defeats the purpose of using HTMLForm in
the first place.

When ProtectedPages is converted to HTMLForm (T117722), it can use this new
field type.

Bug: T12817
Change-Id: I069609fbb37b18c3df25156779ad7ac7cd5d6813

Add short-circuit to DatabasePostgres::schemaExists()

Change-Id: I5221f7d937be1e87689df5a21fd64e244dbd4c2a

Revert "Localisation updates from https://translatewiki.net."

Conflicts with https://gerrit.wikimedia.org/r/#/c/318151/

This reverts commit bc2fe61db26ef99f14b17be36476fc20a3bb4e0f.

Change-Id: Ib17efe29d4f78eb3508ffe57c3fa365b64aa1341

Localisation updates from https://translatewiki.net.

Change-Id: I62a31a3a58174c1c4481bcfb51ab94cc17cac7d9

Add a grant for viewing restricted log entries

In core this holds the 'suppressionlog' right. SpamBlacklist and
TitleBlacklist would add their permissions here as well.

Bug: T149235
Change-Id: I8e2304e7b6f2af0c3d21d7d7390e4979b8eaee39

Factor out some methods in the header generation

As preparation for some more minor refactoring there.

Change-Id: I31f585e5c4fcf724d9aaf670a21f4f0af42e98ce

Bump version number to 1.29.0-alpha for 1.29 development cycle

Branch point was dc0f9b3a3a75e80a0c5f09dd76b4df1fcc05080d

The following commits missed the branch point and should probably
be backported because master reports them as 1.28
 - 4290f686c07265d40718fc3358f196de41bbde57
 - 81698d4c1605f491709e636acd5ca0e6857e3821
 - 40da8bf039019a2845405854a2d636be4bd3b98d
 - 95db9833dd8d75ff3dc44d587fdc3ad80332a500
 - 7bd97758f7e1496c7d4a1fa4e7275c9d6c01c5c3

Change-Id: I51562ba357b5533500ef9dd1e29107dd05cc9e1e

Merge "Add documentation for wfClientAcceptsGzip()"

Add documentation for wfClientAcceptsGzip()

Change-Id: I18b9311d71278ca4c843908a11f77270f03cf534

Merge "Localisation updates from https://translatewiki.net."

Merge "Fix DatabaseSqlite IDEA warnings"

Localisation updates from https://translatewiki.net.

Change-Id: I832a9f7f9ac7c4afd2c358fb506ac3a11f713414

Add class to EditPage text area when editing an old revision

This adds a class 'mw-textarea-oldrev' to the editing area when an old
page revision is edited. It seems that the warning MediaWiki:Editingold
is not sufficient for some users and some css tweaks might be
appreciated, as is already done for protected pages.

Bug: T124226
Change-Id: Ifc53af0f827a5cebfd9cc220b0aa247210e8dcfc

Fix DatabaseSqlite IDEA warnings

Change-Id: Idd52a27c853653e9ba64d765f4f526f5e8a13bcd

Parser functions now format numbers according to page language

Also fixed incorrect return value documentation.

Bug: T62604
Change-Id: I1e16f20ca79436afe17eba711981b2ae43fed9e1

resourceloader: Deprecate mw.Map

This has never been used outside mediawiki.js for things like mw.config,
and mw.messages.

The public API of those objects will remain, but the constructor will
no longer be supported in the future in order to allow these to be
backed by native Map in the future (with a minimal polyfill rather than
a complete one).

* Remove 'mw.Map(valuesObject)'. (follows-up 63f8d7b9e5).
  This is incompatible with native Map and isn't actually used anywhere.

* Deprecate mw.Map#values. Undocumented and considered private.
  A few uses have appeared in the wild so keep a short deprecation period.

* Update the MediaWiki QUnit runner to construct new maps in setup/teardown
  instead of swapping the internal values object directly.
  Also update the runner to update mediawiki.jqueryMsg's parser default since
  it kept a reference to the first mw.messages object.

* Add 'logName' parameter to mw.log.deprecate(). Default logName
  of property key made sense for its original use case (deprecation warnings
  for global variables, which are a property of the window object). But
  for keys of any other object, we'll want to some context for the name.

* Remove redundant test expect numbers. (Follows-up 7c36375, c4c7007d)

* Ensure mw.Map.prototype.constructor points to mw.Map.
  This was broken by 5819c37b3, which accidentally made it point to Object.

Bug: T146432
Change-Id: I7ff0b230608f5ca582fc6752b59c4bb998c78ba2

mediawiki.storage: Catch exceptions on window.localStorage property access

Follows-up 430a0d3984, which created a private property that stored a
reference to the localStorage object outside the get and set methods.

Where previously it was used directly, and as such, covered by
the try/catch around getItem and setItem calls.

Bug: T148998
Change-Id: Ibe75408913d4ffbf1791e1a6470296b29f809185

mw.loader: Fix off-by-one error in splitModuleKey()

Follows-up 45bec76. This caused most localStorage cache entries from
mw.loader.store to be ignored because it'd store tje source code with
a version string that wrongly started with an "@".

As such, on the next page view, when comparing against the version
string from the startup manifest, it would mismatch and eventually
be pruned by mw.loader.store.

Change-Id: If14c80989c0bb28b7b209abe89f83d585c563987

Merge "Swapping defaultbranch for track"

Swapping defaultbranch for track

The former is a maintenance nightmare when branching.

Bug:T146293
Change-Id: Ic892c2f5778153f6e99107fe24dafc2c8797f86d

Merge "Clean up newWANCacheFromParams() getInstance() calls"

Merge "Small DB classes IDEA warning fixes and cleanups"

Merge "Cleanup unused LBFactory $domain arguments"

Cleanup unused LBFactory $domain arguments

Change-Id: I17b409a71e263ff5e0175f6a35fb9c5288f00c11

Merge "Remove redundant LoadBalancerSingle doc block"

Merge "API: Update ApiOptions docs for I45f69997"

Localisation updates from https://translatewiki.net.

Change-Id: I0f0c8c41d403de7df04aa49aebf3c21c9e5d3957

Comments: Added phabricator IDs to bz bug #s

Change-Id: Ia8d4e570e78c62e5c26bda0264fbf1efc74e58a2

Added array_replace_recursive merge strategy

For extension registry, add array_replace_recursive merge strategy,
as some extensions/configuration may prefer that to array_merge_recursive.

In some cases, configuration is merged from multiple extensions,
such as JsonConfig's $wgJsonConfigs configuration: ZeroBanner defines

"JsonZeroConfig": {
"namespace": 480,
"nsName": "Zero",
"isLocal": false,
"cacheKey": "1"
}

and mobile.php overrides it with
$wgJsonConfigs['JsonZeroConfig']['isLocal'] = false;
$wgJsonConfigs['JsonZeroConfig']['remote'] = [
'url' => 'https://zero.wikimedia.org/w/api.php',
'username' => $wmgZeroPortalApiUserName,
'password' => $wmgZeroPortalApiPassword,
];

Having identical value 'isLocal' would be converted into an array
if array_merge_recursive is used, but the replace strategy fixes it.

Change-Id: Ica6ddd0ae76f23e60de9b6235c6e2a3f2754a95d

API: Update ApiOptions docs for I45f69997

Now that we have an alternative multiple-value separator, the
documentation for action=options's change parameter should recommend
that instead of stating that values cannot contain pipes.

Change-Id: Ia0ff2c16be9cab3e1e303a5c7c618455167db31c

PHPSessionHandler: Workaround PHP5 bug

PHP5 has a bug in handling boolean return values for
SessionHandlerInterface methods, it expects 0 or -1 instead of true or
false. See <https://wiki.php.net/rfc/session.user.return-value>.

PHP7 and HHVM are not affected.

No tests are added here because the only case where it actually makes a
difference is a can-never-happen branch.

Also, since I'm touching it already, add a @codeCoverageIgnore for the
code no longer tested thanks to I6e153ec8.

Change-Id: Id87478964b3985ed8bf4dd00bbc09f65ddfcc130