Merge "Allow skins/extensions to define custom OOUI themes"

Merge "Drop remnants of wgHtml5/wgJsMimeType/wgXhtmlDefaultNamespace support, killed in 1.22"

Drop remnants of wgHtml5/wgJsMimeType/wgXhtmlDefaultNamespace support, killed in 1.22

Change-Id: I6bb849543fb3cfd50a4d99bb6a69e228913b9def

Merge "Add change-tags to MessagePoster"

Merge "Add success variables to message group and unify values"

Merge "objectcache: make getMultiWith(Union)SetCallback() usage easier"

Merge "objectcache: add regen_walltime metric to WANObjectCache::getWithSetCallback()"

Merge "Allow setting email even when blocked from sending emails"

Add success variables to message group and unify values

Adding `@*success` variables to “Message” variable group and
unifying values with Style Guide.
Also renaming `@colorErrorText` to @colorError` &
`@colorWarningText` to @colorWarning` to stay in naming
convention.

Bug: T226795
Change-Id: I49ca4b8f510888dff7a2487470174666143bcd76

Merge "resourceloader: Speed up dependency checks in structure/ResourcesTest"

Add change-tags to MessagePoster

Add the ability to add change tags to talk page messages submitted
via wikitext MessagePoster, via a new 'options' parameter to the post()
function.

Bug: T227161
Change-Id: I80c401c56e231b34822791faffebc84d4933b922

resourceloader: Speed up dependency checks in structure/ResourcesTest

Stats from wmf-quibble-core-vendor-mysql-php72-docker builds.
Before:
* testIllegalDependencies (+21ms)
* testMissingDependencies (+254ms)
After:
* testValidDependencies (+17ms)

Bug: T225730
Change-Id: Idf760a27c7ad16d4838ae82e7895b659934fbf93

Merge "Load GlobalFunctions.php to tests/phpunit/bootstrap.php"

Merge "resourceloader: Remove register() 'object' use in OutputPageTest"

Merge "Split HttpTest and SessionTest to unit and integration"

Merge "tests: Move unit/languages to unit/includes/language"

tests: Move unit/languages to unit/includes/language

These source classes they test were also moved from
languages/ to includes/languages/.

Bug: T225756
Change-Id: I6931d459bbfa243d2a28f391f92bce70f1e52256

Localisation updates from https://translatewiki.net.

Change-Id: I530ffee4e3aa0dd4b1e38ca12f70d02ad5c5447e

resourceloader: Remove register() 'object' use in OutputPageTest

Follows-up d6dd6e4d7220a7b, missed one :)

Bug: T222637
Change-Id: I105772d655ce1b64e3eb1d6222b57adfb69ec51d

Merge "Setup: Remove $wgDebugPrintHttpHeaders option"

objectcache: add regen_walltime metric to WANObjectCache::getWithSetCallback()

Change-Id: I7918a6ed7a5d3d845c640aeeb78c88f1345f5157

Merge "Import PSR NullLogger instead of using absolute class references"

objectcache: make getMultiWith(Union)SetCallback() usage easier

Add WANObjectCache::multiRemap() as an array_combine() wrapper for
easily working with IDs after getMultiWith(Union)SetCallback() calls.
Make the enforcement of uniqueness in makeMultiKeys() stricter and
discourage poor key design in comments. Add WANObjectCache::hash256()
method for getting good key component hashes.

Also avoid pointless use of ArrayIterator::getArrayCopy().

Change-Id: I61ffdbf4af4374864bac180df590b4dddc8da56b

Raise default reqTimeout in MultiHttpClient

Bug: T226979
Change-Id: If55329ff561ab3facdba5a89fd429577d986c43e

Merge "When title contains only slashes, Title::getRootText() shouldn't return false"

When title contains only slashes, Title::getRootText() shouldn't return false

Otherwise, Title::makeTitleSafe() will return null, which
breaks the assumption that Title::makeTitleSafe() always
returns something meaningful for strings
returned by Title::getRootText().

Bug: T227816
Change-Id: If79a12bb8d23f1eafc10017d56c62566f39347ad

Import PSR NullLogger instead of using absolute class references

Change-Id: Ie09170f51e203e0a4c9a780a53cc9f3829139c31

Remove $wgSiteStatsAsyncFactor feature and related $wgMainStash use

Also include ss_row_id = 1 in the UPDATE query to avoid gap locks

Bug: T227376
Change-Id: I7b730bab05e6d8b6799b623e0aff089c1103c3c2

Merge "objectcache: make BagOStuff::getMulti() preserve order and omit keys with missing segments"

objectcache: make BagOStuff::getMulti() preserve order and omit keys with missing segments

A key missing segments should count as non-existing, meaning that it should
not appear in the return value at all

Change-Id: I0f08b3dba77e0d335ebb2d75061927ebd36570d2

Merge "Removed deprecated WikiMap::getWikiIdFromDomain() method"

Merge "Switch various LoadBalancer::getConnection() callers to getConnectionRef()"

Removed deprecated WikiMap::getWikiIdFromDomain() method

Also simplify WikiMap::isCurrentWikiDbDomain() slightly

Change-Id: I21358c5e5ae1d229228fc416894f50b4ee444182

Merge "objectcache: fix race conditions in RedisBagOStuff::incr()"

objectcache: fix race conditions in RedisBagOStuff::incr()

The exist() check was not atomic, so a non-expiring TTL could be
make by mistake on race conditions. Use the redis WATCH command
for CAS-style atomicity of the exists()/incrBy() cycle.

Also optimized RedisBagOStuff::incrWithInit().

Change-Id: Ia003e054a41d4b4bbe73508e39d6606d8cc47291

Merge "jobqueue: migrate root job deduplication to the WAN cache"

Load GlobalFunctions.php to tests/phpunit/bootstrap.php

That mostly enables testing global functions

Bug: T87781
Change-Id: Ib42c56a67926ebcdba53f4c6c54a5bff98cb77a3

Merge "rdbms: fix some phpstorm warnings database classes"

jobqueue: migrate root job deduplication to the WAN cache

If the root job timestamp keys are lost or otherwise unknown, they
will now be deductively recached with the best known values as jobs
are popped and executed. This means the running any of many child
jobs of a root job can restore the root timestamp if it was lost.
This does not need to use the main stash given this fact.

Bug: T227376
Change-Id: Iae0f3af15803af048ff49f3bf281b2bde18c87f2

objectcache: only process cache non-LB connections in SqlBagOStuff

LB should manage any such process caching otherwise as it owns the handles

Change-Id: Ic9cd84c7c3cba790eb127e70f6506270e5bec2a1

rdbms: fix some phpstorm warnings database classes

Change-Id: Ib3b8aaadda8101ed82158b1260f10f6d7be16783

Move the showMissingArticle() cache from $wgMainStash to "db-replicated"

Bug: T227376
Change-Id: Ice3b3b1bc8a09f66c4f02bb233a1992ed6d46ab7

Merge "upload: move UploadBase status store from $wgMainStash to "db-replicated""

Merge "objectcache: Hard deprecate `ObjectCache::getMainWANInstance()`"

Setup: Remove $wgDebugPrintHttpHeaders option

This is always true and has been for years. I don't know of any
reason why someone would want to disable it, as the information
is only ever logged when the debug log is enabled, at which point
why not?

Bug: T189966
Change-Id: I17b9f564a61a0f76f2fc8c0006dc13997857829a

upload: move UploadBase status store from $wgMainStash to "db-replicated"

This is only triggered for chunked uploads and does not need anything
more special the local objectcache table.

Also add comments and fix numerous IDEA warnings.

Bug: T227376
Change-Id: Ia61855293a265306c5a27a9dfc0139c4d0b04c4f

Split HttpTest and SessionTest to unit and integration

This way it moves 73 more tests to unit tests

Bug: T87781
Change-Id: I963488450bb355573d71ae31b57d71683ce51683

Merge "Chinese Conversion Table Update 2019-2"

Merge "Title: Title::getSubpage should not lose the interwiki prefix"

Title: Title::getSubpage should not lose the interwiki prefix

This issue was discovered while investigating T227700, and added some
confusion. This patch is necessary for Special:MyLanguage to behave
correctly in all cases, but it's not necessary for fixing the primary
critical problem.

Bug: T227700
Change-Id: Ib4cbeec47a877c473cbd501cc964cc66d169b99e

storage: remove wfWikiId() call from RevisionStore::checkDatabaseDomain()

This should only already be comparing DB domains

Change-Id: If2d40fe1ce4698d71d8a29f8a43b40a8fbb746e8

Merge "Setup: Make wfMemoryLimit() internal and simplify"

Merge "Setup: Move mb_internal_encoding() call earlier"

Merge "rdbms: switch to AtEast warning suppression in Database classes"

rdbms: switch to AtEast warning suppression in Database classes

Change-Id: Ia32f1ba048a540438f78b11a1e94f80acfc7bf50

Setup: Make wfMemoryLimit() internal and simplify

Bug: T189966
Change-Id: I4fa3d66dbf2e2d05a0d1868329a16dfb1959498f

Merge "objectcache: Document WANObjectCache statsd metrics"

Setup: Move mb_internal_encoding() call earlier

This is an unconditional and unconfigurable changes to PHP behaviour.
The earlier the better so that there is less code that can run
without it.

Also improve some documentation of other setup logic bits being
documented by the referenced task.

Bug: T189966
Change-Id: Ia8e1478cf0841d80b1c61c266f2ece75be2303e1

Merge "Drop ChangePasswordForm hook, deprecated in 1.27"

Merge "Bring legacy/mediawiki.ui messages closer to Style Guide & ensure contrast"

objectcache: Document WANObjectCache statsd metrics

Bug: T197849
Change-Id: If0211f512500cce0decd70cebfe8c5731e85478b

Merge "objectcache: move version numbers to the main wrapper in WANObjectCache"

Switch various LoadBalancer::getConnection() callers to getConnectionRef()

This is the preferred method as it enforces read-only mode for DB_REPLICA
and handles LoadBalancer::reuseConnection() calls automatically.

Change-Id: Iab9439ba8e0810fa14c302661ed7a3534f6bfc0d

Bring legacy/mediawiki.ui messages closer to Style Guide & ensure contrast

Also updating mediawiki.ui variables and reflecting WikimediaUI Base
variable name structure for future merge.

Bug: T226795
Change-Id: Ifd01161f718f35fb3cce94efdad43a069ae8ae13

Drop ChangePasswordForm hook, deprecated in 1.27

Change-Id: Ia9281596e0d620b360ccd6cb7392d9516b9240b8

Merge "Migrate various callers away from wfWikiId() to WikiMap"

rdbms: add ILoadBalancer::getReplicaResumePos method

This does what ChronologyProtector wants more rigorously and is better
named. Not all replica servers will have the same position, so they
should be compared to get the highest one.

Simplify the getMasterPos() method to only return master positions
as the other current callers do not need anything else. It will now
connect if needed as well. This should make the method naming better.

Reducing the use of replica derived replication postitions (instead
of those from the master) makes certain GTID issues less likely,
such as the matter of obsolete domain IDs.

Increase general test coverage of LoadBalancer.

Bug: T224422
Change-Id: I5420721ee339a24d09c26c38709500c7bbe797c2

Merge "rdbms: better normalize and document constructor $params in Database"

rdbms: better normalize and document constructor $params in Database

Change-Id: I6531cd3a34d7d6bdf277db779301d88ca1e45a95

Fix stale client caching of history pages with $wgShowUpdateMarker enabled

Change-Id: I5b9d986e4f34166f2a1602129d2e01573bfe7ac5

Merge "Add helper for HTTPFileStreamer header syntax"

Merge "Update wikimedia/object-factory to 2.1.0"

Update wikimedia/object-factory to 2.1.0

Vendor patch: I5571d3ea4b50a866b2444af648b22b1c5a1b5ebe

Change-Id: If5884a31c3f468fc55502c27e978f2eb2668441d

Merge "RedirectSpecialPage: handle interwiki redirects."

RedirectSpecialPage: handle interwiki redirects.

Previously, WikiPage::performRequest() would assume that Titles returned
by RedirectSpecialPage::getRedirect() are local pages, and would set
$wgTitle to whatever was returned. That would lead to a confused state
where the skin would try to render for an interwiki Title.

Instead, WikiPage::performRequest() should wrap the interwiki redirect
in a call to Special:GoToInterwiki/xyz, just like
Title::getFullUrlForRedirect() does, but still avoid the HTTP redirect,
to avoid leaking private information via view counters (T109724).

There are two things to test:
1) call Special:MyLanguage with an interwiki prefix,
   e.g. Special:MyLanguage/wikipedia:XYZ.
2) create a page that contains an interwiki redirect,
   e.g. #REDIRECT [[wikipedia:XYZ]], then call Special:MyLanguage
   for that page.

For these tests, the user language should be the same as the content
language. That is the critical case. If the user language differs
from the content language, the problem would be obscured by another
bug which is addressed by Ib4cbeec47a877c473.

Bug: T227700
Change-Id: I2852c5a9774f0c76e49f1e3876fcfe85a305f9ce

Merge "Some fixes to page updater docs"

Merge "resourceloader: Remove use of object registering in test suites"

Merge "mediawiki.dic: Remove Zero-related stuff"

resourceloader: Remove use of object registering in test suites

This was done as a "clever" shortcut to make sure tests a little
but shorter, but also made them less consistent with normal code.

Remove this in favour of 'class' or 'factory' options as needed.
Also remove a bunch of unneeded register() calls.

The tests cover everything affected by this change.

Side fix - isFileModule should reject modules with 'factory'
the same way it rejected raw objects and non-FileModule 'class'
cases already. This is now covered by tests as well.

Bug: T222637
Change-Id: I3996317dbcd780cc6e0f82c84e769c08a3fc42bb

objectcache: move version numbers to the main wrapper in WANObjectCache

Add FLD_VALUE_VERSION key to the value wrapper array to hold the version
number used in getWithSetCallback(). Remove the VFLD_* wrapper array from
FLD_VALUE for versioned values.

Keys stored with the old VFLD_VERSION and VFLD_DATA fields will be seen as
having the wrong version. The previous WAN cache code will see the new keys
that use FLD_VALUE_VERSION as having the wrong version too. In either case,
the usual variant key logic applies, so there should not be any issues.

This means that moving from a non-versioned to a versioned cache key is no
longer a breaking change when, for the same key, some code passes a version
number to getWithSetCallback() while other code does not.

Also:
* Make "pcTTL" respect the version number for sanity
* Make sure set() respects TTL_UNCACHEABLE for completeness
* Track slow regeneration callback runtime in FLD_GENERATION_TIME
* Remove is_callable() check overhead and rely on PHP Error instances
* Refactor unwrap() to return a more immediately useful value
* Simplify getNonProcessCachedKeys() signature by using $opts
* Split out PURGE_* constants for purge entries since those keys are
  never stored in any serialize value but are only in PHP arrays
* Rename doGetWithSetCallback() to be more succinct
* Rename and reorganize some variables for clarity

Change-Id: I4060b19583cdfd9fa36c91d7014441eeef4b3609

Migrate various callers away from wfWikiId() to WikiMap

Change-Id: Idc2980c05873c428fb5ffa6ab89f723d33217a8e

Add helper for HTTPFileStreamer header syntax

Adds a helper function for transforming an intuitive header array
to the peculiar syntax expected by HTTPFileStreamer and the related
FileRepo/FileBackend streaming methods.

Change-Id: Idac9281b0f1b3c93f4ec1d1c3f336db110e5d260

Merge "Log errors in DeferredUpdates::handleUpdateQueue()"

Merge "Reduce contention of getScopedLockAndFlush() callers by using the DB domain in the key"

Merge "Avoid using deprecated SiteStatsUpdate constructor"

Merge "Optimize JobQueue utilization in UserGroupMembership::insert()"

Merge "Remove unused methods and bogus method call in Title"

Merge "Fix various fatals and IDEA warnings in CopyFileBackend"

Reduce contention of getScopedLockAndFlush() callers by using the DB domain in the key

Change-Id: Ie9fb6a9ff384c72cca559f74d8e409d108207ae3

Merge "Add "help" links for Special:BrokenRedirects, Special:DoubleRedirects, and Special:ListRedirects"

Merge "Release notes for ResultWrapper indexing change"

Log errors in DeferredUpdates::handleUpdateQueue()

Other cleanups and fixes:
* Split up handleUpdate() method into run() and jobify()
* Handle Throwable errors
* Use 'cli' in stats keys in CLI mode instead of "get"
* Tweak some code comments

Change-Id: I7749465df2d7b58e66ee5ebdd3c3d25aea52eeb3

Merge "rdbms: cleanup some Database error message wording for consistency"

Merge "rdbms: cleanup DatabaseSqlite::lock() error handling"

Merge "WatchedItemStore: Fix fatal when revision is deleted"

rdbms: cleanup some Database error message wording for consistency

Change-Id: I7b338e6e856c62ecaab2ef97f76431c2220b430d

Merge "installer: make SqliteInstaller set "synchronous" for the job queue DB like the other DBs"

Merge "Add NS_MEDIAWIKI to LinkCache::isCacheable() to help MessageBlobStore cache regeneration"

Merge "Make MapCacheLRU::get() accept a default value and improve $maxAge default"

Merge "Correctly distinguish the latest revision in action=history"