Merge "Don't double wrap rollback links"

Merge "Add Hebrew aliases for some special pages"

Merge "Drop wgPasswordSalt, deprecated since 1.24"

Don't double wrap rollback links

Bug: T228766
Change-Id: Iaca54e1ceefc73815fe3d9898ca490b74d11dac6

Merge "objectcache: Use variadic signature for makeKey()"

objectcache: Use variadic signature for makeKey()

This should help fix the following issues that various repos
are getting from Phan as of late:

> Call with 5 arg(s) to \BagOStuff::makeKey() which only takes 2 arg(s)
> defined at ../../includes/libs/objectcache/BagOStuff.php:456
> <source="PhanParamTooMany"/>

Bug: T228563
Depends-On: I5cfba063821101325a5a7359e6b8ad71a0fb1b2f
Depends-On: Ifa5b96735376f2fbe3680799f960616ba8d357ff
Change-Id: Ic9df7f3ad7f356c7cbdfe1edfbe35821b931dda6

Merge "Make XmlDumpwriter resilient to blob store corruption."

Merge "Avoid the use of silence operator (@) and use AtEase methods"

Merge "filerepo: make ForeignApiRepo::httpGetCached() use "pcGroup""

Merge "filebackend: avoid use of LoadBalancer::newMainLB() in LockManagerGroup"

Merge "Replace wfWiki() usage with WikiMap in some additional places"

Add release note for custom OOUI themes using OOUIThemePaths (T100896)

Change-Id: I98f370310d93fc178e0baf55798808fa4ece2f92

Make XmlDumpwriter resilient to blob store corruption.

In the WMF databases, we have several revisions for which we cannot
load the content. They typically (but not necessarily) have
content_address = "tt:0" and content_sha1 = "" and rev_sha1 = ""
and content_size = 0 and rev_len = 0.

This patch makes sure we can still generate dumps in the presence of
such revisions.

Bug: T228720
Change-Id: Iaadad44eb5b5fe5a4f2e60da406ffc11f39c735b

Merge "Update LanguageTrTest::testDottedAndDotlessI for PHP 7.3"

Merge "Update OOUI to v0.33.4"

Avoid the use of silence operator (@) and use AtEase methods

Bug: T26159
Change-Id: I973cc607fd909d47faf2773a02835af83bbc301f

Update LanguageTrTest::testDottedAndDotlessI for PHP 7.3

PHP 7.3+ uses Unicode CaseFolding.txt for case mappings. For Turkic
languages(tr,az) the dotted i is given as a special case and we need
to implement it specifically for tr and az.

Updated the documentation and refactored the lcfirst and ucfirst methods
to use arrays containing the above mentioned special cases.

Bug: T207100
Change-Id: I317f2ca66b0adeaa79bc0f9e3dea5edfcd5e4693

Localisation updates from https://translatewiki.net.

Change-Id: Iaa4395494b2a07ab4107c5465fb09c7ba028cd0c

Update OOUI to v0.33.4

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

Bug: T171865
Bug: T224070
Depends-On: Iec2d70793459492c3cebaa735753e0afa542e7c4
Change-Id: Iec2d70793459492c3cebaa735753e0afa542e7ca

Merge "Add ISearchResultSet"

Merge "ChangesList: Add $groups parameter to FetchChangesList hook"

ChangesList: Add $groups parameter to FetchChangesList hook

This parameter is needed for consumers trying to return a $list object.

Bug: T228280
Change-Id: Ib68ff8fa782ac12479ef02247cdf78e405b64bad

Merge "Fix bad method call to \MediaWiki\Widget\Search\SearchResultWidget::render"

Merge "Deprecate SearchResultSet::free"

Localisation updates from https://translatewiki.net.

Change-Id: I08f1713b6e61980009fc99d50ccdf2f874dc1230

Add ISearchResultSet

Bug: T228626
Change-Id: I3306bf6107c97dd58adf578fd965bd11a422627d

Add Hebrew aliases for some special pages

I've used the same translation as the pages' titles (except for "GoToInterwiki" which doesn't have one).

Change-Id: Ibf7137636f351a31e86a4621a644b9c41cc3a73e

Fix bad method call to \MediaWiki\Widget\Search\SearchResultWidget::render

Change-Id: I6b3a65ba62a9686bdc65110cba9d61362e584051

Deprecate SearchResultSet::free

Was only called by SpecialSearch, according to IResultWrapper::free()
this method is rarely worth being called. Therefor it does not seem wise
to expose it in the upcoming interface defining a search result set.

Bug: T228626
Change-Id: I12d41a488025eb2d6dd543c9fbdc1c803c840316

Merge "changes: Usage of 'noudp' & 'pleasedontudp' no longer exist"

changes: Usage of 'noudp' & 'pleasedontudp' no longer exist

Since no usage exist and this was deprecated in 1.31, should it
be removed?

Now I think with the use of Phan and having $send type hinted to
a bool, if a string is used, phan will catch that? :)

Usage
=====

https://codesearch.wmflabs.org/search/?q=(%27noudp%27%7C%27pleasedontudp%27)&i=nope&files=&repos=

Change-Id: Id0482e69e2e2b1cdf0a9ece230e04323bf51324f

don't load revision text content unless requested to

Bug: T228614
Change-Id: Idef4d9684560110a16c6a7c074402c5a5a6e59db

Merge "Do not hardcode square brackets in "rollback link""

Drop wgPasswordSalt, deprecated since 1.24

Depends-On: Ic9952a1bd8563f218c3c581d4b206ebd6c6c9c44
Change-Id: I8b1d43aba62859645221caf5ff340903e4d61787

Follow-up ffd802a386: Actually disable legacy JS globals

Bug: T72470
Change-Id: I2703420afd260c1c9d0cdc17baceed12dc28f758

Merge "Exclude redirects from Special:Fewestrevisions"

Avoid usage of deprecated Revision::* constants

Change-Id: I2d06bbacef88573f286da25994a3ad434ed90e9c

Avoid usage of deprecated Revision::* constants

Change-Id: Ibb55d5a01f997f9c79bcf2f5e09d4f02d22152d2

filerepo: make ForeignApiRepo::httpGetCached() use "pcGroup"

Given the item size, it is best to keep the cache limit small

Change-Id: Icf57a5e54046d4909535f772f1556452ccb122df

api: Avoid usage of deprecated Revision::FOR_THIS_USER constant

Change-Id: I48b302ba3ea5762e2b576e972cf85d57cd73fb15

action: Avoid use of deprecated Revision::* constants, use RevisionRecord

Change-Id: I09e3182cd83977d143ca7b34b791ea2cd0df0ef8

Avoid usage of deprecated Revision::* constants, use RevisionRecord

Change-Id: I872fc89e5c02dd6a3ae9cd7e76640b95dc33f514

Merge "maintenance: Avoid usage of deprecated Revision::* constants"

maintenance: Avoid usage of deprecated Revision::* constants

Update deprecated Revision::* constants to use RevisionRecord::*
equivalents.

Change-Id: Icae1ebb088e4efadbc7c2806127a2073d337fa05

Merge "objectcache: reorganize WANObjectCache fields and avoid exposing internal constants"

Merge "MessageCache: Fix isMainCacheable() logic for non-content languages"

objectcache: reorganize WANObjectCache fields and avoid exposing internal constants

Change-Id: I95771fc8d032939e71adba3a416894004ea0847d

Merge "resourceloader: Move FilePath test to /tests/phpunit/unit"

resourceloader: Move FilePath test to /tests/phpunit/unit

Pure value object. No external dependencies.

Change-Id: If4120ca302c2fab2a41880c32e164d36566ed9de

Merge "installer: Add a defined check to overrideConfig method"

Merge "Increase test coverage for BlockManager class"

MessageCache: Fix isMainCacheable() logic for non-content languages

The way isMainCacheable() was used, it always returned false in
non-content languages, because it would try to find strings like
'hidetoc/fr' in the array of message keys (which contains strings like
'hidetoc').

The consequence of this was that MessageCache would check the database
for a MediaWiki:hidetoc/fr page even if it already knew that that page
didn't exist. This is a substantial performance hit when requesting lots
of messages, like when building version hashes for ResourceLoader's
startup module.

Follows-up 4fc5ba8bf83102b02.

Bug: T228555
Change-Id: I20433175ca919acc1c995f4a9cd50ca53afcdd02

Merge "resourceloader: Complete test coverage for FilePath"

resourceloader: Complete test coverage for FilePath

The 'getLocalBasePath' and 'getRemoteBasePath' methods were not
yet covered.

Change-Id: If2eacca3a908048ec62b357e14e4e2322363e296

Merge "API: Fix 'user_id' field of ApiCSPReport"

Merge "API: Add test case for ApiCSPReportTest"

API: Fix 'user_id' field of ApiCSPReport

Was accidentally turned into an always-true boolean in 5f343617,
due to confusion with JavaScript's default '||' operator.

Change-Id: I24071e22f8bb7a296ae138303f63acdb8ea4950a

API: Add test case for ApiCSPReportTest

Covers the basic functioning of the class, as well as the specific
behaviours introduced or changed by 5f34361759 and 0ca1b8a0e621c9.

Also includes a (bad) expectation for 'user_id' (bool instead of string),
which is a bug caused by 5f34361759 that the next commit will fix.

Change-Id: I2c57c813b8a31e51a61778951227ccbd1217a547

Merge "DevelopmentSettings: Disable legacy javascript globals in CI and for devs"

DevelopmentSettings: Disable legacy javascript globals in CI and for devs

Bug: T72470
Change-Id: Iedd3a9c8e541bc4cc657753e6ae3389f0393ed64

Drop UploadBase:: and UploadFromChunks::stashFileGetKey() and stashSession(), deprecated in 1.28

Change-Id: I2411753a24092a5c37434b095e594831ecff04e1

Drop the UploadVerification hook, deprecated in 1.28

Change-Id: Ic1917fc95223cbf97845c2702ddf98accf8ba9c2

Merge "objectcache: add MediumSpecificBagOStuff base class for non-proxy subclasses"

Merge "objectcache: refactor WANObjectCache::fetchOrRegenerate() locking code stylistically"

objectcache: refactor WANObjectCache::fetchOrRegenerate() locking code stylistically

Change-Id: I5e289989ef91923b650a9c325febd7410d1b2caf

objectcache: add MediumSpecificBagOStuff base class for non-proxy subclasses

This make it much clearer what needs to be overridden

Change-Id: I3073f8a0605f557c6a3a93d0d8401cddd0fb8dbe

Merge "MediaWikiUnitTestCase: Use DIRECTORY_SEPARATOR instead of `/`"

MediaWikiUnitTestCase: Use DIRECTORY_SEPARATOR instead of `/`

`strpos( $reflection->getFilename(), '/unit/' )` will always
return FALSE under Windows

Change-Id: Ib13d32858e238b9a18094f829939f6bba08366ab

Merge "wdio-mediawiki: Release 0.4.0"

Merge "Improve links in several specialmute-*"

Merge "Allow empty input in namespaces multiselect on Special:Block"

Merge "Clarify WikiRevision::downloadSource() and related code"

Merge "resourceloader: Add Config parameter to packageFiles callbacks"

Merge "Clean up DeferredUpdates transaction handling"

Improve links in several specialmute-*

* Remove namespace prefix from user page links
* Add "user" as translatable text instead

Bug: T218265
Change-Id: If20b9023379f00e0b82992322e340362613dce09

resourceloader: Add Config parameter to packageFiles callbacks

Callbacks used to generate the contents of virtual files in packageFiles
modules only received a ResourceLoaderContext object. They could access
the Config object through ResourceLoaderContext::getConfig(), but that
method is deprecated. Pass the Config object as a second parameter to
these callbacks, so that they don't have to use a deprecated method.

Change-Id: Ia4666914e9b07f298ee5ae30ae8c70a3e83b0910

Do not hardcode square brackets in "rollback link"

Additional change:
* standardize on "optional" lowercase in qqq messages

Bug: T219948
Change-Id: I7d79c0ca534bff7165975943d5fa22dfe0a616db

Increase test coverage for BlockManager class

Change-Id: If2c50248448f3f633d9531039094f12c7d712c41

Merge "SiteConfigurationTest: Fix recurring typo ("an non-existing")"

Clean up DeferredUpdates transaction handling

Bail out in attemptUpdate() if the transaction state is dirty rather
that failing at some later point. Also, flush implicit transaction
rounds before calling DeferrableUpdate::doUpdate() for fresher data.

Bug: T225103
Change-Id: I4f5d2f9814a562069619f05e003663fcedbd3f64

Merge "Separate right for foreign user js redirects"

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

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

This commit reverts most of 7e647d2f0f5b, but keeps unrelated code
clean ups from it, as well as the conflicting changes from d8b952ae47.

From WMF production nutcracker:

> nc_redis.c parsed unsupported command 'WATCH'

The use of WATCH, in addition to failing the commands that use it,
also appears to also have caused a chain reaction making nutcracker
intermittently unavailable to other web requests.

Bug: T228303
Change-Id: Ic37efc2963b147e461837571ae0b65acf3f60cb4

Merge "GlobalFunctions: Hard deprecate `wfGlobalCacheKey()` function"

Merge "Fix param doc of TitleArrayFromResult::setCurrent"

Pass message params as array to RawMessage

To match documentation of RawMessage::__construct

Change-Id: I4d9fce6b63af89a0012c9eaac72c0b46f75b43ac

Fix param doc of TitleArrayFromResult::setCurrent

$row is a stdClass, not the ResultWrapper, which is a list of stdClass

Change-Id: Ic96354a6283fcf42ac1c75f524f1da45117100ce

Allow empty input in namespaces multiselect on Special:Block

Follow-up to 83ebbb519f

Bug: T219882
Change-Id: I5f3d907a8f8be02f9e2c23d2a66c42752daa38b5

Merge "Add support for extentions to change Special:Mute form"

Merge "objectcache: optimize MemcachedPeclBagOStuff::*Multi() write methods"

wdio-mediawiki: Release 0.4.0

a1e51bf94325130fa3d54d65c071d3ab57a4ab79 unpublished 3 RunJobs methods
They do not appear to be used according to wmf codesearch
and, given their limited usefulness, are unlikely to be used elsewhere.

Change-Id: Iaefe6f71320c2c3ab86468bf827b6e8d2e80095d

Add support for extentions to change Special:Mute form

The hook (SpecialMuteModifyFormFields) is used to append
the option to mute/unmute notifications from a specified user.

Special:Mute handles posting and saving the fields, the only
requirement is that the field name is the same as the property
that wants to be modified.

Currently there are only two notifications "blacklists":
* `email-blacklist` is directly handled in this page 'cause it is part of core.
* `echo-notifications-blacklist` is part of Echo, so this change is required
  to support it. See I77b3ccfdce9b501e

Bug: T220163
Change-Id: I2b3eee0802cb086091f35ecce13ae77a8e7d518d

GlobalFunctions: Hard deprecate `wfGlobalCacheKey()` function

The only extension that used this was ConfirmEdit and it has been
cleaned up with Ia7b276ee65fdf58c, so is it time we hard deprecate
it?

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5CbwfGlobalCacheKey%5Cb&i=nope&files=&repos=

Depends-On: Ia7b276ee65fdf58c4fc0859563930528d44a03ca
Change-Id: I3fda797c64ec62df4454625ad1a3bf89f6a88246

Delete very incomplete translation

Change-Id: Ic8df4cc9894a366148632dbb2a97803cd6feccef

Localisation updates from https://translatewiki.net.

Change-Id: I06dcc9ee33533abbfa09d8ae7bafd6f6547f0e22

Merge "objectcache: fix WinCacheBagOStuf::doGet() is_string() check"

objectcache: fix WinCacheBagOStuf::doGet() is_string() check

Allow integers to so that incr() and other cases work. This class
now passes all the unit tests again.

Change-Id: Ie7aae36180472a0e913fb8275bccee3fbc6fe7d9

Merge "Tests: Remove old TODO file and update README with integration notes"

Merge "Remove deprecated mw.Api.errors and mw.Api.warnings"

Merge "installer: Remove <doclink/> parser function and last use of it"