Update OOUI to v0.31.4

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

Bug: T163142
Bug: T208547
Bug: T220041
Bug: T220204
Bug: T220227
Bug: T220248
Bug: T220840
Change-Id: Ic94a2f6790334baad7bc9c3055e990f7d3917e17
Depends-On: I0a2e152e61700e723a9ad7cd061682efaa9faf4d

Merge "api: Remove 'recenteditcount' set for BC to be removed in 1.25"

Merge "Remove usages of 'text' flag in revision-related getQueryInfo() calls"

api: Remove 'recenteditcount' set for BC to be removed in 1.25

This was a TODO that needed cleanup in mediawiki version 1.25 but
code was still hanging around. Removing it as we're in .34

Change-Id: I0d996362612495b3c523aed5af3a0e30ee4ed79c

Remove usages of 'text' flag in revision-related getQueryInfo() calls

Field rev_text_id will no longer be populated once the legacy
schema is disabled, so joins against it will not work.
Remove all usages of the 'text' flag in calls to both
Revision::getQueryInfo() and RevisionStore::getQueryInfo()
so that these joins are no longer attempted.

Bug: T198342
Change-Id: I9be6a544c6f68555d4ea856f949f0040d05eac0f

Merge "changetags: $param type should be string and not array"

Localisation updates from https://translatewiki.net.

Change-Id: Ief398fcf2575c4ff49e6db20f7f4c3a64b7209d8

changetags: $param type should be string and not array

Per the call to updateTagsWithChecks(), $params should be a string
and not an array as soft type hinted. The previous calls updateTags()
which uses $param as a string or null.

Change-Id: I87fa39682d91b7cb3b0844d0aabbfaea0cb6f206

Merge "context: Replace deprecated use of SkinFactory::getDefaultInstance()"

Merge "diff: Wrap condition around brackets for more readability"

Merge "resourceloader: Remove 'wgMainPageTitle' mw.config key"

Merge "clientpool: Remove unused parameter from constructor header"

Merge "resources: Rename a few local 'filterFn' vars to 'filterFunction'"

resources: Rename a few local 'filterFn' vars to 'filterFunction'

Abbreviations are discouraged, so let’s use the full word.

Change-Id: I31fb4cca32d766559005d1291d6d22da3d74b5e6

Merge "Allow fragment-only TitleValues"

Merge "Normalize input to TitleParser::parseTitle()"

Merge "New Title::castFromLinkTarget/TitleValue"

Merge "Update MediaWikiTitleCodec to use NamespaceInfo"

Merge "Rename un-prefixed rcfilters selectors"

Merge "Tweak margins for collapsible OOUI HTMLForms some more"

Merge "JsonContent: Remove never-used caption styles"

Merge "resourceloader: Use 'fallback' as default for ResourceLoaderTestCase"

Merge "resourceloader: Declare ResourceLoaderImage fields explicitly"

Merge "Remove obsolete `color` property from horizontal rule"

Merge "Set global config for test to avoid failure"

clientpool: Remove unused parameter from constructor header

Per https://codesearch.wmflabs.org/search/?q=SquidPurgeClient&i=nope&files=&repos=,
constructing an SquidPargeClient requires only the $server argument and
doesn't make use of $options yet. Maybe this was a WIP but for now, it's
never used, so cleaning it up.

Change-Id: I2d86ac6b17ac8dce85c05887de8eee4bb3dd28a5

Localisation updates from https://translatewiki.net.

Change-Id: Ida17cce17aa1b580a88aa9c767362e739364d1e3

Tweak margins for collapsible OOUI HTMLForms some more

* Move the code from mediawiki.action.history.styles, as it's
  theoretically applicable to other similar forms
* Fix it so that it actually only applies when JS is available
* Make top and bottom margins/paddings even

Follow-up to 32bca54dc94b1befcf21483d9f08017a91ad73eb.

Change-Id: Ifde2303f78f971f1e82418c9bd5feddbd6f1f25d

Merge "specials: Add $subpage param to RedirectSpecialPage::getRedirectQuery"

Merge "db: Use $fname instead of __METHOD__ magic constant"

context: Replace deprecated use of SkinFactory::getDefaultInstance()

Replace with MediaWikiServices::getInstance()->getSkinFactory()

Change-Id: I2cf7afad249e52e13a56fbe92b1665dfebec793f

resourceloader: Use 'fallback' as default for ResourceLoaderTestCase

Follows-up fa05976f5f. This broke the Travis CI tests because
the Vector skin is not installed there (unlike in Wikimedia CI).

Make the test no longer reliant on that detail.

* Fix ResourceLoaderTestCase to use a default that actually
  works in plain core (previously 'vector' would be rejected
  as being an unknown skin, but then be re-normalised back to
  'vector' thanks to wgDefaultSkin. This was fixed in fa05976f5f,
  and the unknown skin now produces 'fallback' which is better.

* Update ResourceLoaderOOUIImageModuleTest to match this
  new default and also fix the test message while at it
  to mention 'skin', not 'image'.

Change-Id: I5fb69a6a38a42b1a5f325c0134e01ad880f65087

Merge "specials: Remove invalid return from RedirectSpecialPage::execute"

Merge "Fix calls to Title::getUserPermissionsErrors on SpecialImport"

Merge "resourceloader: Remove needless MW-specific fallback"

Merge "parser: use "-" for revision ID for non-preview edit filter parse during save"

Remove obsolete `color` property from horizontal rule

Removing `color` property definition, which dates back to IE 5/Mac
support times.
`background-color` is sufficient in any of our basic supported
browsers.

Change-Id: I013b9c5bc614299c94bb733fc724148e5c7b1343

resourceloader: Remove 'wgMainPageTitle' mw.config key

This was added in 2009 with cf75bdf4ba (r54384) and moved to the
startup module in 2010 with a99f9ec28b9 (r72772).

It is no longer used by any extension in Wikimedia Git, nor
elsewhere indexed by MediaWiki Codesearch. It also no longer has
any uses in mwgrep from public WMF wikis.

Bug: T220926
Depends-On: Ifdedccb513f5dd4636a37badc329ac5f1c4dd943
Change-Id: Id2729653cd450d31c1d903358ccf5e4fac564228

resourceloader: Remove needless MW-specific fallback

This is never used by the mw.loader client code. For any
manually crafted requests or fringe usage where the load.php
might be used to fetch some raw JS code that doesn't vary by
language or skin, proceed with qqx/fallback.

In the future load.php might deny these requests and then we
could make this a constructor parameter to ResourceLoaderContext.

Bug: T32956
Change-Id: I4e4ee758cd22278cea9592d4745b4f7fc00e0add

Merge "selenium: Replace regex matching with includes()"

Merge "ContibsPager: Fix slow query with actor migration read-new mode"

selenium: Replace regex matching with includes()

Bug: T217544
Change-Id: Iedcbb1f7e21fc768069346ac346c2e499a689556

ContibsPager: Fix slow query with actor migration read-new mode

The change in I1678f7ec broke the workaround in I992aa50f by adding
the rev_id column to the ORDER BY clause. We need to extend the aliasing
to rev_id => revactor_rev to re-fix it.

Bug: T220991
Change-Id: I62dc3803621da982e526d7402ab8e7b08f8336bc

resourceloader: Declare ResourceLoaderImage fields explicitly

Change-Id: I4153cbf6080dabd29f6fbe1fb546dc10957d780e

Merge "resourceloader: Re-use internal addLink() for mw.loader.load()"

Rename un-prefixed rcfilters selectors

Only -head and -container have on-wiki usage, so
keep deprecated aliases for those.

Bug: T220852
Change-Id: I67b120d67fbbdf108da98ddfc7eae24e668bfe11

Merge "JsonContent: Add missing prefix to selector"

db: Use $fname instead of __METHOD__ magic constant

The insert() method calls insertOneRow() method on line 391. It then
passes $fname which defaults to __METHOD__ in its function header and
not using $fname instead of __METHOD__ in insertOneRow() method makes
this parameter unused which of course should be used.

Change-Id: Ic5f5ee878e4f919a96900ba88755641ac85929df

specials: Remove invalid return from RedirectSpecialPage::execute

Follows-up f739a8f368a, and 22c9aa5ec07.

The execute() method here should generally be void, with the
exception of includable special pages, where the return value
is a string of HTML.

This was likely left-over early iteration of commit f739a8f368a,
which ended up returning from getRedirect() instead.

Change-Id: I68715910114e19a0421625e814fd8881068f6406

specials: Add $subpage param to RedirectSpecialPage::getRedirectQuery

This will make it easier to create redirects where $subpage is the title,
e.g. "Special:Example/Foo?x=y" to "index.php?title=Foo&x=y".

To do that conveniently, getRedirectQuery() needs access to $subpage.
The alternative is to do Title-parsing inside getRedirect(), which then
complicates this significantly as one has to deal with absence of a title
(null) and invalid titles (illegal chars etc.).

By using it plainly as query parameter (defaulting to null/omitted), this
is all deferred to index.php, which seems like a better separation of
concerns.

Motivated by SpecialMobileHistory in MobileFrontend (Ic0aea7ee340a).

Change-Id: I9fe78f479053fb55952ba78850d2fc281a039fe3

Fix calls to Title::getUserPermissionsErrors on SpecialImport

Right now, it uses a deprecated way to set rigor causing it to read from
master on GET, giving out errors

Change-Id: I89bbe20080998bde2156680c631700b52f4e4712

Allow fragment-only TitleValues

This matches behavior of Title, and is meaningful for creating links.
However, in other contexts such a TitleValue doesn't make sense, so
perhaps we want to write special cases in linking code instead.

Change-Id: I812ad9090463766f38ef677de318c006912b4cca

Normalize input to TitleParser::parseTitle()

This matches behavior of Title::newFromText(), which assists porting old
code to TitleParser.

Change-Id: I50e9af09df843ea575250e276c7cfce660c00efd

New Title::castFromLinkTarget/TitleValue

These behave the same as newFromLinkTarget/TitleValue, but accept null
as well (and then just return null). This makes things much easier when
converting code from using Title to LinkTarget, because you can wrap in
castFromLinkTarget without adding null checks.

Change-Id: Id61f91d40b81ad226532917c43e51f0b69af712c

filebackend: move more files to subdirectories

Change-Id: If965323809c5bfc9cb73ed0f357233d9325d3353

Merge "Cache: Make APCUBagOStuff::set return a useful value"

Merge "Split DairikiDiff to class per file"

JsonContent: Add missing prefix to selector

Change-Id: I977f6c565b93949a310d1207411f893cf83af996

Merge "selenium: Disable flaky wdio rollback tests that rely on pauses"

Merge "Restore func_get_args in HTMLFormField"

Split DairikiDiff to class per file

Change-Id: Ica426312ec69707b9a99dfe2a7deee945c38ba35

Merge "Split filebackend files to class per file"

selenium: Disable flaky wdio rollback tests that rely on pauses

Wait for an actual state to be reached, whether a JavaScript
expression to become true, or a certain state in the DOM.

Using pauses is an anti-pattern and inherently unstable.

Bug: T219440
Change-Id: Ibb2bd335f12442ab8ec2973031454cb58733211d

Rename PhpBugTests -> PhpXMLBugTester

Only contains one class (probably should be removed when we bump to PHP 7 fully)

Change-Id: Iab2bb837e2ed5d5556bd49f4236030b43aba29f1

Split filebackend files to class per file

Change-Id: Idf9f4177fb9a523ce41227bda2af923bf418396b

Merge "Split HTMLFormElement.php into class per file"

Merge "Move ClassCollector to its own file"

Merge "Move SectionProfileCallback to its own file"

Merge "DevelopmentSettings: Remove unused globals & declare $wgSQLMode"

Merge "Remove XMPReader back compat"

Localisation updates from https://translatewiki.net.

Change-Id: If213bbbe7552c4a6e747ca618ac0be22177d70b9

Move ClassCollector to its own file

Change-Id: I881f1ccc4aaf8babc40ad534a2a91f1ccf3baad4

Move SectionProfileCallback to its own file

Change-Id: Iccb2048eed6631caa4f20639f55866e76f9fabf0

Merge "Fix casing of Special Pages to match class name"

Merge "Remove HWLDFWordAccumulator, deprecated in 1.28"

Merge "Remove ObjectFactory"

Remove XMPReader back compat

Change-Id: I4e3ecbf6c13a7e7cfed88253eb6820e78e49e9c3

Cache: Make APCUBagOStuff::set return a useful value

It took me lots of time to debug a CI failure in
I9b3bc36b552901bc6ca7609ee51e80be2979a9c4. I was deceived by
$cache->set returning true, which according to the docs means that the
store was successful. But instead, this function just returns true, even
in case of failure. Make it return the result of apcu_store, which is
already in the format true = success, false = failure.

Change-Id: I2619845c12460e1acb5af696d2c11a5a4dee1bd3

Split StubObject.php to have one class in one file

Change-Id: If00625b700962a1c6d6412c6de30c28fa3df9f81

Remove HWLDFWordAccumulator, deprecated in 1.28

Change-Id: I1eb6ba3c4b8b4cbabf26033a27728e42ebcddb23

Remove ObjectFactory

Change-Id: Ibfdb55101c893a6724e1f999eba85f2f8eabb79b

Set global config for test to avoid failure

Change-Id: I4f883b0ecec5378e29625a8940a0c247967f3e71

Restore func_get_args in HTMLFormField

Mocking variadic arguments does not work in hhvm

Follow-Up: I066ec95a7beb7c0665146195a08e7cce1222c788
Change-Id: Ic3b689d003a4659abdc4c9344ffd83f24f448912

Fix casing of Special Pages to match class name

Change-Id: Ifc9e827202493e8f055a21875c54ff827a38d1f7

Split HTMLFormElement.php into class per file

Change-Id: Ia8a35056fa6df5b81cda3e5bfcaac91d0b898d51

Update MediaWikiTitleCodec to use NamespaceInfo

Depends-On: I759cde50e42020699138d32431c27428737f700f
Change-Id: I57d77754288449ec54b039802adae05d56fa5563

Split LocalFile.php to have one class in one file

Change-Id: Ic8e5220f2a1832dfc39f00720001235429ed2cab

Merge "resourceloader: Switch params for private addLink() function"

Merge "Move UploadStashFile to its own file"

Merge "Fix return type of UploadStashFile::remove()"

Merge "title: Allow passing MessageLocalizer to newMainPage()"

Move UploadStashFile to its own file

Change-Id: I557bcd8369014bf2ea43c5eb8336562cb6295890

Fix return type of UploadStashFile::remove()

Change-Id: Ib9da5827671a34798b58f7d736e4b663ac2c442d

Merge "Split logging classes to individual files"

title: Allow passing MessageLocalizer to newMainPage()

The method could not be used in session-less endpoints. This was
worked around once in the Startup module.

I plan to use this method in an extension module as well,
and would prefer not to duplicate core's logic for determining
the main page, outside this repository.

As general dependency-injection pattern, it seems desirable
to allow injecting a MessageLocalizer here.

Change-Id: I76cd02b2f489882e9404b93270f76aad9f0a4d9d

Merge "Remove PreferencesFormLegacy.php"

Merge "Avoid extra parse/save delay for users with non-canonical parser options"

Split logging classes to individual files

Change-Id: I3eaaf23612fe1aed65e49c06a9e5e565399af9ce

Remove PreferencesFormLegacy.php

Now unused, deprecated in 1.32

Change-Id: I4fc4bdf36b3832786c0266786cff57d512140d8e

Avoid extra parse/save delay for users with non-canonical parser options

If {{REVISIONID}} results in a re-parse, that re-parse will be post-send
unless the user has canonical parser options and will need the output for
page views anyway (e.g. the refresh after editing).

Also make getPreparedEdit() allow lazy-loading of the parser output via
a callback. A magic __get() method handles objects created the new way
but accessed by other code the old way.

Bug: T216306
Change-Id: I2012437c45dd605a6c0868dea47cf43dc67061d8