Merge "Improve type hints in export related classes"

Merge "Add permission check for user is permitted to view the log type"

Merge "Type hint against LinkTarget in WatchedItemStore"

Localisation updates from https://translatewiki.net.

Change-Id: I32ed9eb31204d18421f9d986d049feca65fbb62e

Add permission check for user is permitted to view the log type

Note: formatter patch only

Note: cherry-picked I064f563cb here as well

Bug: T222038
Change-Id: I1c4e57a513e3a0e616b862a5b9d684f463ad9981

Merge "resourceloader: Also clear store in debug mode to simplify mw.loader.store.init()"

Merge "Update documentation for $wgAutopromote"

resourceloader: Also clear store in debug mode to simplify mw.loader.store.init()

This makes it consistent with the other two circumstances in which
mw.loader.store is disabled.

Also, it makes debug=true a more easily accessible means for users
to clear module storage, which can be useful at times (without requiring
console access).

Change-Id: I9c66815a2f6c60feef3e070cfe57c382a69cc752

Make wfGetDB() return a MaintainableDBConnRef instance (take 2)

This enforces the DB_* role checks of DBConnRef in more places.

This is a re-submission of 335066505a3ea, which was reverted due to T228928.

Change-Id: I556627dc6486e6f6539d1d2dd5aa6f009eff341e

Merge "Replace wfGetDB in ParserTestRunner"

Merge "MimeAnalyzer: fix ZIP parsing failure"

Merge "parser: inject a Logger into Parser instead of using wfDebug()"

MimeAnalyzer: fix ZIP parsing failure

unpack() actually returns an array with indexes starting from 1, not
zero, so unpack(...)[0] gives a notice and always returns null. It is
lucky that ZIPs normally have zero-length comments, so this would have
had little impact on file type detection aside from log spam.

Also, add a check to make sure the unpack() will not read beyond
the end of the file. Without this, unpack() could generate a warning.

The bug was introduced by me in f12db3804882272794b.

Add tests. The test files were generated by appending an EOCDR signature
and some extra bytes to 1bit-png.png.

Bug: T223728
Change-Id: I6fab63102d1d8eea92cdcce5ab6d1eb747a0a890

Merge "Do not block autopromote on partial blocks"

Update documentation for $wgAutopromote

Follow-up to 1ad840da68

Change-Id: I7c2dedf3824154c0c844f73a98e6824fe8ff141a

Merge "Remove deprecated handling of array keys for $wgProxyList"

Replace wfGetDB in ParserTestRunner

After I0d7dacee3, wfGetDB returns a connection reference instead of
a real connection, which means it can't be passed to CloneDatabase
anymore, and testcase / test runner classes should avoid it.

Bug: T228928
Change-Id: I42be4214e33a4d4f8d53fafc925c8d708b4c0106

Remove deprecated handling of array keys for $wgProxyList

Change-Id: Ic9cc2a5585180ab57fd361342cbac8210b094a5c

Localisation updates from https://translatewiki.net.

Change-Id: I70c061d14d3768c28234257004f35299c80a3201

Merge "Revert "Make wfGetDB() return a MaintainableDBConnRef instance""

Revert "Make wfGetDB() return a MaintainableDBConnRef instance"

This reverts commit 335066505a3ea6121f3c5b5eae43b0c37ca6e2eb.

The standalone parser test runner, as used by Parsoid Jenkins builds,
is (indirectly) using wfGetDB in a way that isn't supported (changing
the selected database). This needs to be fixed, but we can give it
a few days.

Change-Id: I07ff422dd56d5700e570100747aa49b7764ec80f

Merge "Add null to Parser::getTitle return doc"

Make wfGetDB() return a MaintainableDBConnRef instance

This enforces the DB_* role checks of DBConnRef in more places

Depends-on: I9328e709fe5d81099338a31deef24d34db22d784
Change-Id: I0d7dacee3ec4ef67dc0b0f6551ad046c74dc47dc

Merge "installer: Stop offering MyISAM option for new installs and fix bogus method calls"

.travis.yml: Don't allow failures for PHP 7.3 any more

Bug: T204884
Change-Id: I797855405cced7471ab40a7cf80980cf68c840ce

Merge "Users with partial access to user rights should not be blocked"

Merge "make XmlDumpwriter more resilient to blob store corruption"

Merge "Drop newlines in LocalisationCache debug logs"

Drop newlines in LocalisationCache debug logs

316d205115c migrated LocalisationCache from wfDebugLog() to the Logger
interface which no more requires the messages to be terminated with a
new line.

Change-Id: Ibe2f750e7ab16cf91801988cb726adb67f7cd9d2

Merge "Validate sort order in Special:Search"

make XmlDumpwriter more resilient to blob store corruption

Loading content can also throw InvalidArgumentException when
the cluster address is an unknown cluster.

Bug: T228720
Change-Id: I313f9a5a27b21a33e90639abae3f505640c30e23

Add null to Parser::getTitle return doc

Change-Id: I5086f636114cec83f0188d2309ea70aef27df13d

Fix and re-apply "RedirectSpecialPage: handle interwiki redirects"

This re-applies commit 41106688abbe6dfff61c5642924ced42af3f0d33
(thereby reverting commit 6c57748aeee6e4f2a197d64785102306fbd4a297)
and fixes it for local interwiki redirects by adding and using a
forcing parameter in Special:GoToInterwiki to treat local redirects
like external ones.

Bug: T227700
Change-Id: I4bc2ed998430fc2bac71baf850b8988fdb24c1ac

Merge "Drop mediawiki.dic, useless and poorly updated"

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

Validate sort order in Special:Search

Providing an invalid sort order to Special:Search could trigger an
exception from the search engine when trying to apply it. Validate the
sort order, much like API classes do, and let the user know that the
sort they requested could not be applied.

We also have a unreported error for invalid profile requested, so
added that warning to the display while here.

Bug: T228171
Change-Id: I79079eea8c03a90b5b65f1dad11c99e514de00e1

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

Drop mediawiki.dic, useless and poorly updated

Change-Id: Ie48eab4e989013e425d39dd92b9545079d47bb0c

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

installer: Stop offering MyISAM option for new installs and fix bogus method calls

The methods were renamed in 2010 (r68908, c4100fb83b5b12bd).

Change-Id: I076ec8d9c11f5bd33cb0ad19e5be8e239401b9c6

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"