Update composer/spdx-licenses to 1.4.0 and mediawiki/mediawiki-codesniffer to 21.0.0

https://github.com/composer/spdx-licenses/compare/1.3.0...1.4.0

Change-Id: I39f7a1310455159866bfed5224536e800befec0d

Merge "Upstream EasyDeflate library from VisualEditor"

Merge "Deprecate public access to some DifferenceEngine properties"

Deprecate public access to some DifferenceEngine properties

These had no business being public in the first place and there is
no usage in Gerrit. In case something does use them, it will probably
be broken by the refactoring that's soon to come, so the deprecation
warning makes it easier to figure out where the problem lies.

Change-Id: I72ba5cfbf91f4af16028ba4f8619df6a7168a786

Use setContentLang() instead of setMwGlobals()

This changes behavior in some tests by making them set $wgLanguageCode
as well as $wgContLang, but that seems like a good thing.

Bug: T200246
Change-Id: I936888f46ff9fefe2707efba837e2ce3a7ca5e3f

Merge "Use 'default' instead of 'value' in HTMLTextField"

Merge "Add helper trait for deprecating properties"

Make MessageCache use APC for big messages

Bug: T118893
Change-Id: I418d0ca490911d7ee4213fe1728cfbc637ef8b10

Localisation updates from https://translatewiki.net.

Change-Id: I69852319aa363902ca532563bee3938103e1be41

Use 'default' instead of 'value' in HTMLTextField

Bug: T193087
Change-Id: I631970797a778caa77d3605e0a5b8ca00e2cb56e

Merge "Make the 'other' option superessable in getSuggestedDurations"

Merge "Follow-up 9742d569b: Replace accidental \t in i18n with a space"

Merge "Deprecate $wgShowSQLErrors and $wgShowDBErrorBacktrace and make nonfunctional"

Deprecate $wgShowSQLErrors and $wgShowDBErrorBacktrace and make nonfunctional

Clarify and simplify exception output by deprecating
$wgShowSQLErrors and wgShowDBErrorBacktrace.
$wgShowExceptionDetails will now control most related output.
$wgShowHostnames will now solely control output of
MWExceptionRenderer::reportOutageHTML.

Bug: T165768
Change-Id: Idead2c11c499463dfa6293c3d4b33be3bde92e1a

Follow-up 9742d569b: Replace accidental \t in i18n with a space

Change-Id: Id8ad5916b7dd59a078fb600c7998e41f1b8abd3c

Merge "Make load* methods of DifferenceEngine idempotent"

Merge "cache: Minor docs for MessageCache interaction with WANObjectCache"

cache: Minor docs for MessageCache interaction with WANObjectCache

Follows-up 04bc03a29a.

Change-Id: I8fb810fc2e08277bb64fa18595483a2161cfb1cc

Merge "Fallback language for Santali should be English"

Merge "ApiBase: Use prefixed parameter name for 'missingparam' error"

Merge "Make OutputPage::mPagetitle private"

Merge "objectcache: improve logging and error handling in BagOStuff"

Fallback language for Santali should be English

Bug: T200290
Change-Id: I65005b8b31bd022c7978628f69c5d56d735063e3

Localisation updates from https://translatewiki.net.

Change-Id: Idbc346a6c0d5bf2bdc2278bc2ef3440579500dc7

objectcache: improve logging and error handling in BagOStuff

Bug: T198239
Bug: T199383
Change-Id: I0b4110396d03b98e83a7b614caf57d7e7c284361

Merge "PageArchive: Pass correct overrides to newRevisionFromArchiveRow()"

Merge "Avoid a redirect loop when the request URL is not normalized"

Make the 'other' option superessable in getSuggestedDurations

Bug: T193364
Change-Id: Ic2dbc961f7eebad11da53724b9cce2f804ffad39

Make OutputPage::mPagetitle private

Code search revealed only one outside access, which I've submitted a patch for.

Depends-On: I001deecad241b091070483d882088d2d6361ed5a
Change-Id: I0f7e565f4eacc1074c619772b2d11a47a05e3584

thiemowmde's legal name changed

I was a co-author only, and my contribution was not really
noteworthy, so I choose to remove the outdated name (in this
case only).

Change-Id: I9d9592d5b114adae633f6a4bdac2dd60f329bd7a

Merge "LogEventsList: Correct typo"

Merge "$wgHttpsPort should only be used in very special cases"

Merge "LogPager: Add backwards-compatibility for hide_[type]_log URL params"

Localisation updates from https://translatewiki.net.

Change-Id: I8b632d4052c2634867c7d498305cf287d2fc58c9

Revert "Ensure LanguageCode::bcp47() returns a valid BCP 47 language code"

This reverts commit 8380f0173e79b66f0e2afd6c49cd88afb9f4f6f3.

Reason for revert: Caused T199941

Bug: T199941
Change-Id: I93af756a2d70d6bc91f828fe6ac19bf10ca8788f

Merge "Revert "Accept BCP 47 codes as aliases for nonstandard variants""

Revert "Accept BCP 47 codes as aliases for nonstandard variants"

This reverts commit 0818070c59eb4e8330496796b943951c01c7a573.

Reason for revert: Caused T199941

Bug: T199941
Change-Id: I24c178eb33890477de79cbb3122861c140578011

Merge "LogEventsList: Use DerivativeContext"

LogEventsList: Correct typo

The parameter to getTitleInputDesc() is actually unused, so this was
not breaking anything.

Change-Id: Ia301faf511e30b4fc4542ccb80b24cf807cf731c

ApiBase: Use prefixed parameter name for 'missingparam' error

Similar errors use the prefixed parameter name, this one should too.

Bug: T200155
Change-Id: Ia14d6a9c457af06e72428c1eae14bd3849b4595a

PageArchive: Pass correct overrides to newRevisionFromArchiveRow()

Revision::newFromArchiveRow took 'page' as an override for ar_page_id,
while RevisionStore::newRevisionFromArchiveRow() needs 'page_id'.

Thanks to sanity checks elsewhere in RevisionStore, this mistaken
override causes an exception to be thrown rather than undeleted
revisions potentially pointing to the wrong page.

Bug: T200072
Change-Id: I9d7543866c674f4d8aea9ec00fcc15cbf616ca66

LogEventsList: Use DerivativeContext

Bug: T200136
Change-Id: Ie2b7753684dc0257b0b53d9c9314feeb14d99182

Merge "HTMLMultiSelectField: Improve compat with GET forms with no wpFormIdentifier"

Added non-parallel fallback to MultiHttpClient when curl is unavailable

If the curl extension is not available, fall back to the existing
HttpRequestFactory and associated classes. Also added related phpunit tests.

Bug: T139169
Change-Id: I2f9d4acbb491bce28d7105e124c5cee7e16e86d7

Merge "Make MessageCache use getWithSetCallback() for big messages"

Localisation updates from https://translatewiki.net.

Change-Id: I93c905909b83a89898922e241610275186e5346b

Localisation updates from https://translatewiki.net.

Change-Id: I848c703ad10f39827730fcdacdd83b4ad47b346d

Merge "Explicitly mark Skin::makeVariablesScript() public"

Merge "Fix octal escape sequence overflow in FiltersTest.php"

Explicitly mark Skin::makeVariablesScript() public

Remove else. If contains an unconditional return

Change-Id: I57a04339395d947dffbc3595d321106d4fd8256c

Localisation updates from https://translatewiki.net.

Change-Id: I01a747cba68c60850761ac5dcef992bcebaaf6b5

Merge "Send API timing to statsd for all requests, not just writes"

Add maintenance script for resetting login/signup throttle

Bug: T194506
Change-Id: I6dda5f516ba47a452e438946379fcb696c8c87f9

Merge "filebackend: let any specific backend value of "readOnly" override the default"

Merge "JobQueueGroup: Allow readOnlyReason to be specified per JQ type"

Merge "Install cache/integration-tests as require-dev"

Install cache/integration-tests as require-dev

Bug: T199440
Change-Id: I1e50d7202a5d02088319ec9e90e0136519cba1ce

GlobalFunctions: use a closure instead of polluting global namespace

Change-Id: Ia587008de75a8bde90ad45d4932284f0b6401845

Send API timing to statsd for all requests, not just writes

The rate will be about 8k req/s, whereas currently graphite1001
receives about 350k metric updates per second, so we would expect
this to increase statsd load by about 2%.

This would have been very helpful in diagnosing API cluster overload
incidents such as the one on July 11.

Change-Id: I0a248e1bcec05915d98ec735f4cfb9800e64412d

Merge "language: Document sources of data and example usage"

Merge "resourceloader: CSSMin::getLocalFileReferences now strips anchors"

Merge "Language::listToText: Avoid PHP warnings from E_STRICT"

resourceloader: CSSMin::getLocalFileReferences now strips anchors

When processing:

  url( 'foo.svg#anchors' )

getLocalFileReferences() would yield a file 'foo.svg#anchors'. It might
occur with upstream libraries, such as WikiFonts used by the Refreshed
skin.

When a path is found, strip the anchor entirely. Skip the case if that
results in an empty file.

remap() would properly skip the behavior "#default#', also skip url that
are solely an anchor such as '#other'.

Add a few test cases to cover CSSMin::getLocalFileReferences and cover
anchors usage in CSSMin::remap.

Bug: T115436
Change-Id: I1749ddc2b89021807f42d64131931ad7a99a7b43

language: Document sources of data and example usage

Change-Id: I0c72069d010285d9b88ff83bc1478410961065d3

parserTests: re-enable save parse and fix MCR errors

* Re-enable parse on save as requested in T159014, instead hide
  the feature behind an option.
* Add the MCR tables to the list of tables that need to be cloned. Fixes
  unconditional exception.
* Fix some undefined member variables, remove assignment to unused
  variable $this->databaseSetupDone.

Bug: T159014
Change-Id: I6be577ebcbadeb4e08079a82543d521fc2bb5948

LogPager: Add backwards-compatibility for hide_[type]_log URL params

Also, the $default value in LogEventsList#getFiltersDesc was not
being generated properly and would have always been an empty array.

Bug: T199856
Change-Id: Id286c76259406521f12cda67a4a715032e022637

Merge "Convert Title::getTitleCache() to using MapCacheLRU (again)"

Merge "ApiMain: Always create a new printer in getPrinterByName()"

Merge "Avoid passing "false" as keys to MapCacheLRU in RepoGroup"

Convert Title::getTitleCache() to using MapCacheLRU (again)

Re-applies commit 7a25cd388c8adcce6, reverted by 3648e5683c0ff11c0.

Change-Id: I087edb15fbe11ccac7be28540e20d990f5615c4e

Avoid passing "false" as keys to MapCacheLRU in RepoGroup

Bug: T200026
Change-Id: I40f6ad2a3d281d06c9b6eaf4f31d9796ea5e9e9e

Localisation updates from https://translatewiki.net.

Change-Id: I78f8728c250651e132b1104d2a26047ee938ac54

Merge "Convert Title::getTitleCache() to using MapCacheLRU (again)"

Merge "In mysql.php ignore SIGINT"

Language::listToText: Avoid PHP warnings from E_STRICT

Language::listToText() uses for(;;) which assumes the index in a list
starts at 0 and following keys are consecutive integers.

This assumption fails (and results in warnings with E_STRICT) when
SpecialListGroupRights::formatPermissions() uses the key-preserving
array_intersect() to create a subset of $allGroups.

Bug: T199559
Change-Id: I4eb03a95509d69653156a2764d58c0c5d0d1dfbc

Merge "RCFilters: Work around Chrome browser bug in filter menu"

Merge "Set default wgMultiContentRevisionSchemaMigrationStage to write-both"

JobQueueGroup: Allow readOnlyReason to be specified per JQ type

We use $wgReadOnly for various reasons, one of which is to disallow
writes in the currently-non-active DC. However, we should allow the
readOnlyReason configuration variable to be available per JobQueue type
and have it the code respect that.

In our current set-up, we use JobQueueEventBus which ever only uses the
enqueue execution path and does not depend upon which DC it is executed
in, so this will allow us to enqueue jobs in both DCs.

Note that this is an alternative approach to the one outlined in
Ibbad6063b6b154d7f7d172c79f7be324bf80eb7e

Bug: T199594
Change-Id: I8f1a57a81ea11c1c587c0057fa8bb3454b0e0b56

filebackend: let any specific backend value of "readOnly" override the default

Previously, only different reason strings could override, but there was
no way to override the read-only mode to being off.

Change-Id: I4266694034bfbdada90d4be96bcbc1a37f5b5409

Merge "docs: Remove table-layout documentation from core"

ApiMain: Always create a new printer in getPrinterByName()

ApiMain already caches the printer in ->mPrinter, so if
getPrinterByName() is being called more than once that's because we
really want a new printer instance, without any cached errors or other
behavior that results from reusing the same instance.

Bug: T199949
Change-Id: I779cbbaa8aab9b049a8eed732416edd828121ec4

Merge "config: Add new ConfigRepository"

Merge "registration: Expose config array to ExtensionRegistry"

config: Add new ConfigRepository

This, for now, has the focus for holding metadata of each
configuration option inside of MediaWiki (both extensions and core).
It's very close to the Config interface and friends, and uses these
implementations to retrieve the actual value of the
configuration option.

The goal with this change is to implement a basic architecture to
allow to display the values of the curent configuration of the MediaWiki
installation on-wiki (e.g. on a special page). It also provides a
central point, where the currently known options can be fetched
from.

The long-term goal, of course, would be to get a web interface to really
configure the MediaWiki installation. But, this is more like a dream,
then a plan (from the point of view of this commit).

Next steps would be:
 * Enable ExtensionRegistry to "register" the configuration options of
   extensions, so they're available in the repo (done).
 * Find a good way to get mediawiki/core configurations into this repo
 * Work out an overall architecture to display the different possible
   values. (I think about something like different formatters for types).

Change-Id: I9419508eaa85ffc55520db7f33b3e9530fc99f00

Fix octal escape sequence overflow in FiltersTest.php

The octal escape sequence must be less than or equal to 377.
Change \666 to \377.

Bug: T200004
Change-Id: I3804a0c0fab7ec74bc9dd1281d800fffd53f05fa

registration: Expose config array to ExtensionRegistry

Currently, the ExtensionProcessor processes the config array of the
extension and just sets the value as a global. In this way, however,
the metadata of the configs (description, public, ...) aren't available.

This change exposes the configurations in the extracted data as the
config key.

Bug: T155908
Change-Id: Ie41e6ecb8d08a5b20b56eb2281af1dfdd808b2c4

Make MessageCache use getWithSetCallback() for big messages

Change-Id: I8f93e1dffbcbbabf4d7cb361b001d8ca6e7ad954

docs: Remove table-layout documentation from core

This doesn't belong here and also doesn't make a lot of sense
with current browser support any more.
We shouldn't provide basic HTML documentation in 2018 as part of
core.

Change-Id: I1a3516e45902d16f0d9efbe97cb2dbd6efd566be

Convert Title::getTitleCache() to using MapCacheLRU (again)

Re-appplies commit 7a25cd388c8a, which was reverted because
HTMLTitleTextField was passing `null` as a title key, this was
fixed in I29ecd94cdf9e30.

Bug: T199763
Change-Id: I7fe8bbed3fb24240e47ba22de57da9dc8a9bea22

Remove JobQueueSecondTestQueue class.

After the switchover is completed we do not need this any more.
This class was marked deprecated as soon as it was created, so
removing it should be safe.

Change-Id: I3451a7a9d1e78bb2c7c988aeb1e3aac93bfc1b6f

Merge "registration: Add class docs for ExtensionJsonValidator"

Merge "Fix handling of already logged-in user in Special:Userlogin"

Deprecate wfArrayFilter() and wfArrayFilterByKey()

Now that all our supported PHP versions have array_filter()
with a third parameter, these functions aren't needed anymore.

Depends-On: I3b097a1a048baabcaca15dc214a3a1bb06e746cc
Depends-On: I0187e27ac47cbab099249572201d1a649226a734
Change-Id: I7cabd0252691a083cb749cf9d3a7a23f1d076c39

Merge "LogEventsList: Remove options not used by HTMLForm"

Merge "Make MultiWriteBagOStuff use the native merge() of each backend"

Merge "Convert MemcLockManager to using MapCacheLRU"

registration: Add class docs for ExtensionJsonValidator

Change-Id: Ic37544de1400600719764057a63cc546c26218d8

Fix handling of already logged-in user in Special:Userlogin

Due to a forgotten return clause, the full request handling
is executed after calling successfulAction() when the user
reloads the login page while logged in. With a typical setup
case this is masked by the fact that successfulAction() does
a redirect (although it probably still impacts performance),
for less common configurations this might result in the login
form rendered below a success message, or in calling extension
hooks which should not be called.

Bug: T185702
Change-Id: If19220c4cf17b08f7e03ca0d06381067f26dda1a

Make MultiWriteBagOStuff use the native merge() of each backend

This lets backends that support CAS take advantage of it.

Also fix race condition with merge() in asyncWrites mode where two
threads might add to a list/set but the first request finishes after
the second, which would erase an item during the set().

Bug: T198239
Change-Id: Ibd2539429746f12b4de74d0f031089cb80aa8b4b

Merge "LogEventsList: Add backwards-compatibility for log-show-hide messages"