Merge "Report more information about composite blocks in block error messages"

Report more information about composite blocks in block error messages

For any messages that use CompositeBlock::getPermissionsError, include
details of the original blocks from which the composite block is made.

If there are any database blocks, give their IDs and also explain that
there may also be blocks due to IP blacklisting.

If there are no database blocks, then explain that the IP must be
blacklisted in multiple places.

Bug: T212326
Change-Id: Id6ad0019f8add4d5e000da5e872338e87cca485e

MessagesKo.php: periodic updates

Add Mute and sort BlockList alphabetically.

Change-Id: I8125979b61e489b2c99af258254c4bb83675b9c4
Signed-off-by: Yongmin Hong <revi@pobox.com>

Merge "Add mechanism for temporary user rights"

Add mechanism for temporary user rights

Add a mechanism for adding temporary user rights that only exist
for the current request. This is occasionally needed to let normal
users act with a bot flag; traditionally the fact that User::$mRights
was public has been abused to do it, but I88992403 broke that.

Bug: T227772
Change-Id: Ife8f9d8affa750701e4e5d646ed8cd153c1d867b

Hard-deprecate wgProfileOnly, deprecated in 1.23

Change-Id: I1b66931ca0d3ae3e044b14f5b97dd98861ec5e23

Hard-deprecate wgLocalInterwiki, deprecated in 1.23

Change-Id: Ib18c3122c4d9b8a50d314f67004cc73b8c14d40e

Merge "Hard-deprecate wgSkipSkin, deprecated in 1.23"

Merge "Update OOUI to v0.33.3"

Merge "Fix OO.ui.infuse error on misc-authed-ooui module"

Fix OO.ui.infuse error on misc-authed-ooui module

Follows-up 419849211e3a63.

Change-Id: I1e22c70fa8c467212a02b508c68a17abf79a77e8

Merge "Add mute preferences link to the tools menu"

Merge "Check namespace exists in HTMLNamespacesMultiselectField validation"

Update OOUI to v0.33.3

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

Bug: T212159
Bug: T225021
Bug: T226045
Bug: T226780
Bug: T226822
Bug: T227666
Bug: T227917
Depends-On: I7dde9d02f94c4e78ef480035d3887b68d0128233
Change-Id: I7800da87a6a10399f705b43e05c8592c766bae6f

Add mute preferences link to the tools menu

A link to Special:Mute is added to the tools menu on the sidebar
if the user is logged in and a registered relevant-user* is found
Mute preferences link will not be shown if $wgEnableSpecialMute is false

* From Skin::getRelevantUser: Special pages like Special:Contributions mark
  the user which  they are relevant to so that things like the toolbox can
  display the information they usually are only able to display on a
  user's userpage and talkpage.

Bug: T218270
Change-Id: I89eb8373963d28d52a026b60aa42d1c64b31b7c6

Check namespace exists in HTMLNamespacesMultiselectField validation

Bug: T219882
Change-Id: I710d0abed6bd40dcea5bdd1c59c8d936565961eb

Merge "Revert "RedirectSpecialPage: handle interwiki redirects.""

Revert "RedirectSpecialPage: handle interwiki redirects."

This reverts commit 41106688abbe6dfff61c5642924ced42af3f0d33.

The original case is changed by this commit from a MediaWiki fatal
exception with HTTP 500, to a blank 200 response due to silent
failure. Use of GoToInterwiki appears to be invalid at this point in
the code. Reverting to keep prod the same as last week, so as
to unblock the train.

Bug: T227700
Change-Id: Ieece956d2e2e4c21b5ed7a75890b9f11eaf07e66

Merge "resourceloader: Replace some Xml::encodeJs calls with RL's own encodeJson"

Linker: Force type Language for $lang in tocList and generateTOC

The call of wfGetLangObj( $lang ) is not necessary anymore.

$lang still defaults to user interface language on unset parameter.

This change is a follow-up to I15b65fec987641885374dfef9e1229ea405f7c30.

Change-Id: I6f12097a0e6cf7d6035d1164092c8b87c58e2bee

Merge "resourceloader: Remove support for 'object' in wgResourceModules"

resourceloader: Replace some Xml::encodeJs calls with RL's own encodeJson

Bug: T32956
Change-Id: I614fe0e80ff308b857639a27d7772f969899b468

Merge "Remove $purgeBlobs parameter from LocalisationCacheRecache hook"

Merge "Add ImgAuthModifyHeaders hook to img_auth.php to modify headers"

resourceloader: Remove support for 'object' in wgResourceModules

Obsolete since MediaWiki 1.17alpha, no known usage since.

Bug: T222637
Change-Id: Ie820b16022ced6767c32aee7f2497a99260b1641

Merge "resourceloader: Add test coverage for WikiModule::getType"

Merge "resourceloader: Clean up ResourceLoaderWikiModuleTest"

Hard-deprecate wgSkipSkin, deprecated in 1.23

Change-Id: Ia1013bea7216603f3b6a8148e0194f1e2a8e9b41

resourceloader: Add test coverage for WikiModule::getType

Change-Id: If004cc4ff3835091c01a4df84006be3f6fa173b2

resourceloader: Clean up ResourceLoaderWikiModuleTest

* Remove redundant any() calls.
* Use willReturn() instead of will(returnValue()).
* Use yield for providers and add missing test case descriptions.
* Use createMock() instead of wfGetDB() where the DB isn't needed.

* For provideIsKnownEmpty, re-order the cases and add a few extra
  cases, and document why they behave the way they do.

Change-Id: Iba42325a55bb3dfc50a8d2af46e1ddba8dda885a

Remove $purgeBlobs parameter from LocalisationCacheRecache hook

No longer allow hook functions to prevent message blobs from being
purged. Pass in an always-true variable for backwards compatibility,
which is then ignored.

Change-Id: I27ac9599711f2f0df2514a3934270af0ce03da7f

Merge "resourceloader: Add unit tests for ResourceLoaderFilePath class methods"

Add ImgAuthModifyHeaders hook to img_auth.php to modify headers

Change-Id: I3c6fd7b0c39d7fd52c484494233241093d152f88

Merge "rdbms: set QUERY_IGNORE_DBO_TRX flag in more places in DatabaseSqlite"

Merge "rdbms: fix IDatabase::setLBInfo() handling of null and allow clearing keys"

resourceloader: Add unit tests for ResourceLoaderFilePath class methods

~ testConstructor() - unit test for the constructor method.
~ testGetLocalPath() - unit test for the getLocalPath() method.
~ testGetRemotePath() - unit test for the getRemotePath() method.
~ testGetPath() - unit test for the getPath() method.

Change-Id: I0610938dd864931da7a7e1150ddb4d86ab9a2c5e

Fix JS error in MessagePoster when (optional) options param not given

Change-Id: I77c53862431402d18ff995b410bba9eca9c3beff
Follows-Up: I80c401c56e231b34822791faffebc84d4933b922

Merge "Drop NS_IMAGE and NS_IMAGE_TALK, deprecated in 1.14"

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

Drop NS_IMAGE and NS_IMAGE_TALK, deprecated in 1.14

Change-Id: Id980c3d9bbadae219181b07f22aa58cbc2efbdc6

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

rdbms: set QUERY_IGNORE_DBO_TRX flag in more places in DatabaseSqlite

Change-Id: Id965d35a3feb331ded31ecb39f342ce23a061712

rdbms: fix IDatabase::setLBInfo() handling of null and allow clearing keys

Change-Id: I20cb799b54cabb1172940f8ece93b7f45d7cf0ba

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"