Update OOUI to v0.31.0

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.31.0

Bug: T215645
Bug: T217471
Bug: T217584
Bug: T217952
Bug: T218271
Depends-On: Icc14a3662039080f1a65d098004640366351f0bb
Change-Id: Icc14a3662039080f1a65d098004640366351f0ba

Merge "Remove references to Html5Depurate"

Merge "Avoid a few overly complicated is_null() checks"

Remove references to Html5Depurate

Change-Id: I4f07c5ae606f5c86242bb7d299f4f30a044860b4

Localisation updates from https://translatewiki.net.

Change-Id: I0b3266b3300731491affb838bb287496fe942d3a

Merge "resourceloader: Remove redundant var-freeing in addScript()"

Merge "resourceloader: Reduce size of ES5 StringSet fallback"

Merge "resourceloader: Minor optimisation in variable declarations"

parser: closeParagraph already resets the lastSection

Change-Id: Ic24c9aa25852cc786a5ca438c2c1e9031f9e7c17

Merge "Replace sorting classes with better naming convention"

Localisation updates from https://translatewiki.net.

Change-Id: I2887536cb78f5b4780550b67a2fde8aaf7530a97

Merge "LogEntry: Restore logic to add change tags on udp-only publish"

Merge "ApiMain.php api/request logging event changes"

Merge "Add password policy setting `suggestChangeOnLogin`"

ApiMain.php api/request logging event changes

- Use 'Z' UTC timezone on dt
- Don't provide empty arrays if no data entries
- No need to set X-Request-Id in http.request_headers, it is meta.request_id

Bug: T214080
Bug: T212529
Change-Id: I37aa41a46ce74b372070e88a2f4da746fcbe2449

LogEntry: Restore logic to add change tags on udp-only publish

This was unintentionally removed in I8df2fd98.

Bug: T218110
Change-Id: Ie8678b4207032d6eb877d596eb8f115798c80a1e

Merge "Remove deprecated ParserOutput::legacyOptions"

Merge "Improve documentation of the PageContentLanguage hook"

Replace sorting classes with better naming convention

Improving CSS structure by removing code duplications.

Bug: T214218
Change-Id: I0501e1ab934aef3af289fee618b99578a1f0285c

Improve documentation of the PageContentLanguage hook

Clarify that the type of second parameter that is being passed
to the hooks can be anything because one hook can change it and
then it is passed to another hook without the hook caller having
possibility to check or modify the value.

Clarify that hooks should only return Language objects.

Rename $wgLang to $userLang in the hook parameter documentation to
avoid false posivite matches for the global.

Fix some typos, use Title::inNamespace and add a test assertion.

Also, the $content parameter is unused by all implementations of
this method, and on quick look never passed by any caller. I kept
it for now, however.

Bug: T214358
Change-Id: Iae49d2998c2b762565d232c0337d84d43a4a900c

Remove deprecated ParserOutput::legacyOptions

Change-Id: I5f32dd741f3ee795ec599aacb687d5cee2c52835

Merge "jobqueue: allow direct server configuration arrays to JobQueueDB"

Merge "Add sanity check to notifyUpdatesForRevision() for $oldRev"

Merge "resourceloader: Reword and expand doc comment about FileModule "packageFiles""

resourceloader: Reword and expand doc comment about FileModule "packageFiles"

Change-Id: Id9d68205504ecbddea6ca248e108f368fdcac2d2

Merge "logging: Start using LinkTarget & UserIdentity in ManualLogEntry"

Localisation updates from https://translatewiki.net.

Change-Id: I7ee423c084fbe33d4731e43ce5506f6a6f0c40be

Merge "Fix the order of the parameters for htmlform-title-badnamespace"

Merge "Bump minimum SQLite version to 3.8.0"

Merge "Fix small typos"

Merge "Linker: Use RequestContext::getMain()->getLanguage() instead of $wgLang"

OutputPage: Use $this->getUser()

Use $this->getUser() instead of $this->getContext()->getUser().
ContextSource::getUser is a shortcut for $this->getContext()->getUser().

Change-Id: I7d19116b634500cd1a435bebe235c7562889f7bb

Merge "Avoid using stale data for revision visibility and actor data"

Merge "Streamline code involving .= string concatenations"

Streamline code involving .= string concatenations

This was inspired by Idbbdb31. Originally, I did a regex search for
code that did string concatenations like `$str = $str . …` and replaced
them all with the .= operator. A duplicate patch was uploaded by another
author. I rebeased this patch on top of the other, which leaves all
the manual optimizations I did.

Change-Id: Iaeb73d9c63302c9409bd1051b91e0d2bd77788a7

Avoid a few overly complicated is_null() checks

We can use the ?? syntax introduced with PHP 7.0 because it is already
supported by HHVM, even when HHVM is set to be compatible with PHP 5.6.

This was inspired by Ib117e05.

Change-Id: If980839fe6f86f2b4e86bbe99905a796e4708c7c

Bump minimum SQLite version to 3.8.0

3.3.7 is 12 years old, no chance in hell you will find it somewhere
that supports modern MediaWiki. Since Ubuntu 14.04 LTS has 3.8.2, this
seems like a reasonable conservative version.

Change-Id: I5fbb35bd2d4c0993889b79f463a3c35ce6765d3b

Merge "ProtectionForm: Add type hint to ::showLogExtract() header"

Localisation updates from https://translatewiki.net.

Change-Id: Ibf7f5a6cc435b330df709d40f8ea0687c65832c5

Avoid using stale data for revision visibility and actor data

Created class RevisionStoreCacheRecord to avoid returning stale
cached revision visibility and actor data when changes were
made after the cache was populated for that revision.

Bug: T216159
Change-Id: Iabed80e06a08ff0793dfe64db881cbcd535cb13f

Merge "mediawiki.cookie: Export config via packageFiles"

ProtectionForm: Add type hint to ::showLogExtract() header

https://codesearch.wmflabs.org/core/?q=showLogExtract%5C(&i=nope&files=&repos=
according to codesearch shows that this method takes as first argument an output
page object and also, objects are passed by reference by default so removed &.

Change-Id: I167ae6300b37d4e9292bd8ad14351f1ca35184d2

Remove a few method_exists() checks

These functions are present in every PHP version we support now.

Change-Id: I11add0fc9f6c3c3caff859db9c0475c4d2818577

Merge "installer: Code augmentation in wfInstallerMain function"

Merge "Add @var to class properties to clarify types"

Merge "Move class HistoryPager to own file"

Merge "Move class RangeDifference to own file"

Merge "Remove suppress of PhanRedefineFunctionInternal"

mediawiki.cookie: Export config via packageFiles

* Add a private setDefaults() method to allow mocking from
  unit tests.

* Use matching keys between the data export and the API.

* Reduce duplication in the code.

* Access the Sinon stub explicitly instead of via the public path.

* Remove use of the now-redundant QUnit.newMwEnvironment().

Change-Id: I600332cdb738f0b443549948b9f070f3ccfa12aa

Merge "maintenance: Replace deprecated global $wgParser with getParser() service"

maintenance: Replace deprecated global $wgParser with getParser() service

$wgParser is deprecated and using globals should also be avoid. So, replacing
the use of global $wgParser by a mediawiki service.

Change-Id: I34b61338e500567773249c71da0a7125cfd50d02

Merge "installer: Remove redundant $key variable"

installer: Remove redundant $key variable

Change-Id: Ie6d59a1fd94ccfb2a3b5248f1f98ae640a4d5c2c

Merge "NamespaceMultiselectWidget: Allow filtering by namespace ID or label"

Add sanity check to notifyUpdatesForRevision() for $oldRev

Change-Id: Ibd8975efb33d061ab16dc1a59899b6f78976ec5d

Merge "Block: Explicit convert Message to string"

Add password policy setting `suggestChangeOnLogin`

Password policy checks that fail and have `suggestChangeOnLogin` set to true will
prompt for a password change on login.

Below are some rules that apply to this setting in different scenarios:

- If only one policy fails and has `suggestChangeOnLogin = false`, a password change will
  not be requested
- If more than one policy fails and one or more have `suggestChangeOnLogin` set to true`,
  a password change will be requested
- If `forceChange` is present in any of the failing policies, `suggestChangeOnLogin` value
  will be ignored and password change will be enforced
- if $wgInvalidPasswordReset is set to false `suggestChangeOnLogin` is ignored

IMPORTANT**
Before this patch, suggesting a password change was the default behavior (depending on
$wgInvalidPasswordReset), which means that the necessary changes to $wgPasswordPolicy
need to be in place before this patch is merged and gets to production.

Bug: T211621
Change-Id: I7a4a0a06273fa4e8bd0da3dac54cf5a1b78bb3fd

installer: Code augmentation in wfInstallerMain function

Cleanup null checks and string literals in wfInstallerMain function.

Change-Id: Ib117e05faf899b0c46820d337f1bd3ebd4272419

Fix the order of the parameters for htmlform-title-badnamespace

The name of the page is the first parameter, and the namespace is
the second parameter.

Change-Id: I738cdd920d0ce641837c5dc1da391602a15df794

installer: Get WebRequest in mw-config via RequestContext

Change-Id: I535a91d09cc74ab08b782973a946dcf3fc3e6b46

Remove suppress of PhanRedefineFunctionInternal

There is no longer an override

Change-Id: Ie7c99228135689d20ab6ffb287c978642851f3ee

resourceloader: Remove redundant var-freeing in addScript()

This was copied from jQuery 1.x, where it existed to reduce
memory usage in IE6-8. This was because it's JS engine was not
able to break the circular closure reference (the function
invocation has a reference to the script element, the script
element has an event listener with closure access to the script
element). This bug is no longer relevant.

Change-Id: I6fd66c5fbf3afb3c9e09bba6322dc3fac2fad4a6

resourceloader: Reduce size of ES5 StringSet fallback

Over 90% of browsers have ES6 Set implemented.
Make the fallback a bit smaller.

Change-Id: Icfd18cbeac6d95702ac409297a01c81829d5a41b

resourceloader: Minor optimisation in variable declarations

Trying out Safari after a while and, it's Dev Tools have a
way to see the compiler's type inference, showing that Safari
is not able to always detect some numbers as non-optional.

While I generally avoid optimising for an optimiser, I believe
being more explicit is always better or equal. Plus, it saves a
few bytes while at it.

Change-Id: I4084452a74c51953ff0a1609b3b52d696f14a7e1

Merge "mediawiki.cookie: Move file to its own directory"

Merge "SVGMetadataExtractorTest: remove method existence test"

Merge "Installer: Fix an obsolete comment"

Merge "Remove $wgMediaInTargetLanguage"

jobqueue: allow direct server configuration arrays to JobQueueDB

This is useful for using a separate sqlite DB.

Change-Id: I6a7bb1d4bddedca34d68b0d8460c96c6ef5bbcf6

Installer: Fix an obsolete comment

Change-Id: I0ae459b016f491ff3bc2c702e55be29fdcfec5eb

mediawiki.cookie: Move file to its own directory

In preparation for using RL packageFiles functionality, and
to make sure git can track the file, which it seems to be unable
to otherwise.

Change-Id: I966718db2028a45c3dce1c2c10218e5137e23c7d

Merge "Make PurgeJobUtils::invalidatePages avoid waiting on replication for no reason"

Make HTMLCacheUpdateJob avoid waiting on replication for no reason

Change-Id: Ica381dad0bdb0555add1c7925e0e48991e7e0964

Make PurgeJobUtils::invalidatePages avoid waiting on replication for no reason

Change-Id: I9ea4e4034ffc30a14b138279515447f990807e07

SVGMetadataExtractorTest: remove method existence test

It should be present in everything we support, as the earliest libxml
version would be 2.9.1 while >2.6.20 is required.

See https://3v4l.org/LdTIr

Change-Id: Ibc7152e9224ef8d5803d0e0ff8314cdf9eb8eac4

Merge "Make addIdentifierQuotes part of IDatabase"

Make addIdentifierQuotes part of IDatabase

This is so it can be used in places that typehint an IDatabase,
ultimately so that I can make phan-taint-check happy.

Bug: T216348
Change-Id: Iabf3e9a6add380322a12cdbb6a337bad690f1370

Merge "objectcache: Use constant for $ttl param default in WANObjectCache::set()"

objectcache: Use constant for $ttl param default in WANObjectCache::set()

Change-Id: I3f936088cd63af4d535d7f46fe5619e8bb866999

Merge "rdbms: optimize Database::selectDomain() to avoid extra queries"

Merge "Remove `list-style-type` declaration from elements.css"

Remove $wgMediaInTargetLanguage

It's a temporary feature flag not included in any release, just
removing it outright. The functonality will now be always enabled.

Bug: T205040
Change-Id: Ia9da82e6f6b2d270f1790a99fc8c35ad5e6aee5e

Merge "resourceloader: Omit default 'debug=false' from load.php urls"

Merge "resourceloader: Separate context from extraQuery in ClientHtmlTest"

Merge "ApiQueryLogEvents: Add IGNORE INDEX to avoid MariaDB optimizer bug"

Merge "resourceloader: Replace ResourceLoaderDebug config use with context"

Move class RangeDifference to own file

Change-Id: I5262c7484017a0d920cc24297515d5946f614ee6

Remove `list-style-type` declaration from elements.css

This doesn't make any sense here, as
a) we should apply `disc` in alignment to Design Style Guide
b) it's overwritten anyways in both, Vector and MonoBook, with
   different styles

Bug: T151100
Change-Id: Ied6810a78d8fbe5371838921cda468e6873d2e57

rdbms: optimize Database::selectDomain() to avoid extra queries

Also clean up and align the mysql subclass versions of the method.
Enforce that raw "USE" queries are not passed in, since they would
break the tracking (even before this change).

Change-Id: I11c9145c6c0525f27a4ec8d94c500a22a712b320

Merge "jquery.tablesorter: Optimise getElementSortKey() code"

Merge "Move CacheDependency classes to own files"

Merge "objectcache: add metrics for WAN cache deletes and check key touches/resets"

Merge "objectcache: move ERR_* constants to IExpiringStore"

resourceloader: Omit default 'debug=false' from load.php urls

Change-Id: I2a2f92b0f0438420105e6a4e4d97eb5f8c480917

resourceloader: Separate context from extraQuery in ClientHtmlTest

The test was dual-purposing the same array both to create
a ResourceLoaderContext object, and to pass further down as
$extraQuery parameter.

This meant that ResourceLoaderClientHtml::makeLoad, and indirectly
ResourceLoader::makeLoaderQuery, did not have any control over
the formatting of the query string, due to it forcing the key/value
pairs regardless (in addition to being redundant).

This is needed in order to omit debug=false by default, which is
the next commit.

Change-Id: I3c098a9a5055ec8f2b64319c6cadd13c030ba76a

Move CacheDependency classes to own files

Change-Id: Icacf36674f10fa19a09499d68d596d5e2cc235f0

objectcache: remove BagOStuff::modifySimpleRelayEvent()

This was only there as part of EventRelayer use by WAN cache, which
has since been removed. No other usage really makes sense.

Change-Id: I201425bb88739dd8febbfbc3d24a15e6d3d88d9c

objectcache: move ERR_* constants to IExpiringStore

Also removed unused ERR_* constant from WANObjectCache and fixed
comment in the constructor.

Change-Id: I4377fc3f533113a459b648fad084ba097ff51762

jquery.tablesorter: Optimise getElementSortKey() code

* Document why this uses $().data() instead of $.data(),
  as the rest of this file does.

* Move single-use $() to where it is needed.

* Use native Image.alt directly instead of going via jQuery
  and attributes. This also obviates the need for a default,
  as the DOM handles that already.

* Remove needless $.makeArray(), use $.map() directly which
  supports the same array-like objects as $.makeArray().

* Use native elem.textContent directly for text nodes,
  instead of $(elem).text().

Change-Id: I92da50e1c01f11585d2bac4f01dba1f671444a63

Localisation updates from https://translatewiki.net.

Change-Id: If38e8f3671ef5a53006836c5ac14bbadc6930098