Merge "Replace use of deprecated methods in Linker and SkinFactory classes"

Merge "Simplify a few binary checks for bit 1"

Merge "Remove covers validator trait where it is not needed"

Merge "Remove useless use of current and end"

Merge "Turn logout link into a POST API call with refresh"

Turn logout link into a POST API call with refresh

It's better to get rid of any GET request with csrf token

Bug: T25227
Change-Id: I76464c750945d07a283b99b449f137593c685f02

Remove useless use of current and end

The ConfigRepository is not an Interable (any more), so these functions
don't have any useable effect.

Bug: T221045
Change-Id: I6e5bfd9d922d2838c65d52815590dcad1fe6d284

Typehint against IDatabase in special pages

Change-Id: I1ec906cc7b284a3de02bb67fb41b5cf9e67ff3d2

Merge "i18n: Drop img-auth-nologinnWL, unused AFAICT"

Merge "i18n: Drop number_of_watching_users_pageview, unused since wgPageShowWatchingUsers was dropped"

Update wikimedia/at-ease from 1.2.0 to 2.0.0

https://github.com/wikimedia/at-ease/releases/tag/v2.0.0
https://github.com/wikimedia/at-ease/compare/v1.2.0...v2.0.0

Change-Id: Ia49a156e76d0a4e257e91cc6a51050848bcb9a5e
Depends-On: If40364e2590e3c23035838f8ed26c4f69e730602

Drop wf(Suppress|Restore)Warnings, deprecated in 1.26

Change-Id: Idcc9debd90e6ee12adf5ab7c4ec640db12828997

HTMLForm: Add known MediaWiki version to wfDeprecated() call

In 3706dcb, this method was soft deprecated and hard deprecated at
the same time (1.28) but during the call to wfDeprecated for hard
deprecation, the version number was missed, though this will default
to false, it's good to use the version number which is known in this
case (1.28).

Change-Id: I535ad07e79d14bac8ec42beaeb4b3762a494b28b

Merge "jqueryMsg: Allow sneaking in non-qqx JavaScript messages in qqx mode"

Merge "IP: Remove unused deprecated ::isValidBlock() method"

Merge "Define date formats for Northern Sami"

Define date formats for Northern Sami

Following instructions from User:Yupik (at Wikimedia Hackathon 2019),
consulting [1], pages 10, 42, 43.

[1]: http://sh.pedit.no/web/NettskoleRessurs.axd?id=feb75bf8-6f42-402b-b622-29ce644c88d4

Bug: T223588
Change-Id: Ia63ea6ef09e95534ea0213bf28b26dbff3f3a7c4

Merge "SearchResultSet: MW version number should be a string in wfDeprecated call"

IP: Remove unused deprecated ::isValidBlock() method

This method was deprecated in 1.30 and no longer used in our code
bases.

Usage
=====

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

Change-Id: I26e11c0d88fe7c82c052c1cf8e0bbc33bd99de70

Merge "Revert "Always validate uploads over api""

Merge "Remove LegacyHookPreAuthenticationProvider class"

Simplify a few binary checks for bit 1

( $var & 1 ) is either 0 or 1, which can be used as a boolean value.
The main advantage of this is that there is no confusion with the
operator precedence. In `$var & 1 !== 1` the `!==` is executed first,
effectively turning it into `$var & 0`. This always succeeds.

Change-Id: I53c81a3891d42b2660eefc311f1f0f2523104894

Merge "build: Upgrade mediawiki/mediawiki-phan-config from 0.5.0 to 0.6.0 and make pass"

Remove covers validator trait where it is not needed

MediaWikiTestCase already contains this trait.

Change-Id: I08224ac12a58b41121697ab71d5c54ddbdd060ec

Revert "Always validate uploads over api"

The verification is broken with chunken uploads and ultimately
cause large files to no more be uploadable.

This reverts commit 38ec6d8a344d4eda0307dd3a72653dd2171305d6.

Bug: T223448
Change-Id: If414a8f751a3e1488a2ab099abd8b598c973c1f4

Remove LegacyHookPreAuthenticationProvider class

Deprecated from birth, hard-deprecated since 1.33.

Change-Id: I03e94c02370ff0057e18274e9b409e162a0d7af0

Merge "Fix typo: namesapce"

Fix typo: namesapce

Change-Id: Idb4e420c87b7043baf3289556b8ef8775126bf8e

Merge "Use HTTPS instead of protocol relative for WMF links"

Merge "objectcache: restore a simple version of the apc.serializer check in APCUBagOStuff"

Use HTTPS instead of protocol relative for WMF links

The WMF websites redirect from HTTP to HTTPS.

Change-Id: I916c97697b4ffa9774eec9c8e6ab650a08bb6edd

Merge "Unstub $wgLang for PageContentLanguage hook"

objectcache: restore a simple version of the apc.serializer check in APCUBagOStuff

Follow-up to 3192278eb21d

This seems to be about 30% faster on my local non-vm setup
when apc.serializer is set to "default".

Bug: T220470
Change-Id: If966187bd96a210915a72dc027544378c966a43c

Unstub $wgLang for PageContentLanguage hook

This is to avoid annoying fatal errors when someone annotated their
hook handler to only expect Language objects, but that expectation
is violated due to this code possibly passing StubUserLang to hooks,
some of which may also assign it to $pageLang.

Even with this in place, it is probably a good idea for hook handlers
to refrain from type hinting parameters that are passed by reference
because their types cannot be guaranteed.

Bug: T214358
Change-Id: I88405a8de4b13675eb5a9d11e9ddc87e20a85fb4

Merge "for two-stage dumps make the location of MWScript.php alterable"

Follow-up I2fef157c: Add missing getNamespaceInfo call to ContribsPager chain

Bug: T223440
Change-Id: I26e05a4312ba7f667b1b95cfbfdc24fae793c3b0

Add notice that FOR UPDATE is incompatible with aggregates

On some database types, combining locking reads with aggregate functions
is not allowed, and MediaWiki detects such cases and logs a deprecation
warning (on all database types), so we don’t really want people to use
this even if the target database type happens to support it. Add a
warning to the documentation.

Change-Id: Ib6fd2148aeb0c94c0a13b33aec08e3308c740b01

for two-stage dumps make the location of MWScript.php alterable

Previously it was fixed to be in a certain location relative to
the MW instllation; this would allow the path to the script to
be passed in as an argument. If not, the script reverts to its
previous behavior.

Bug: T204962
Change-Id: Ib9fcf073332670149544d56e375541d014600371

Localisation updates from https://translatewiki.net.

Change-Id: Ib8bf857616112887a5c758c00375fd1f14b2a164

Merge "ApiLogin: Remove the first example of outdated flow"

ApiLogin: Remove the first example of outdated flow

This first example demonstrates a deprecated flow that we should not
be encouraging and definetly shouldn't be before the example that we
want to promote

Change-Id: Ia1ec99a0fd88cee216fcb80aaebd0a00ce2942a4

Merge "Don't allow completing a partial stash upload"

Merge "Bind retry callback with correct this argument"

SearchResultSet: MW version number should be a string in wfDeprecated call

Seems this was a typo and I think 1.32 which is a double/float will be
implicitly converted to true (bool) because it will resolve 1.32 to 1 as
integer and then 1 which maps to true (bool).

To avoid this, use '1.32' instead of the integer form of the version.

Change-Id: Ifaf6ab0d36bc02bd1707f8caf375f65a30eb1af5

Merge "title: Convert binary regexp to use Unicode code points"

Allow the request ID to be passed in via the `X-Request-Id` header

For tracing and logging purposes, we want to be able to see/generate the
list of all of the requests that happen in the environment for a given
external incoming request. To that end, allow Mediawiki to accept the
request ID provided by the incoming request as its own.

Since this may be problematic for set-ups that don't have an entity in
front of MW that sanitises the headers on the way in, introduce a new
global variable, `$wgAllowExternalReqID`, that can disable this
behaviour. By default, the feature is disabled.

Bug: T201409
Change-Id: I605471fb8b5bbc290baeecc7d80d9d715cb240c9

Don't allow completing a partial stash upload

finishUploadToStash could be called on an incomplete upload.
E.g. upload 1 chunk (that gets you the filekey) and then call
finishUploadToStash's finishUpload.

This adds another check for the stash result. The result will
only be 'Success' if all chunks have been submitted (or when
the file was uploaded as a whole, not chunked)
If we're still missing some chunks, result will be 'Continue'.

Bug: T190988
Change-Id: Ia0a4df0306b3e0cc0f2453f986b73bedd88241d3

Localisation updates from https://translatewiki.net.

Change-Id: Ibd264a8b5ec69c0fa68d24cea424f78a49646171

Merge "localisation: Inject 'directory' option to LCStore classes"

jqueryMsg: Allow sneaking in non-qqx JavaScript messages in qqx mode

In commit bb084a35739e3f3c54c8680f756ebebb90e13f28 I assumed that
with uselang=qqx, any messages defined in JavaScript would be just
the message key wrapped in parentheses.

It turns out that extensions (e.g. VisualEditor) have been sneaking
in messages in content language using the system we assumed to only
contain messages in user language. This trick stopped working.

Detect this and return the real message, if it's defined.

Bug: T222944
Change-Id: I3f2a6ceb249402ef76e788c494fd5bdc519faf65

Bind retry callback with correct this argument

Change-Id: I5db9e4e967d5569d4f90590333a7af2ddab1641f

localisation: Inject 'directory' option to LCStore classes

Avoid globals for this, inject them from the factory function instead.

Bug: T218207
Change-Id: Ia961e8e08dcf1ca154d74ea6a3dadd2d59c1299c

Remove FileRepoStatus

Change-Id: I4712d7cf8177dd3a7b8157d980dacb88a902262c

Merge "Fix AbstractBlock param types in documentation"

Fix AbstractBlock param types in documentation

Change-Id: I503375485956d3c05da445542419fb62684ae34a

SearchResult: MW version number should be a string in wfDeprecated call

Seems this was a typo and I think 1.32 which is a double/float will be
implicitly converted to true (bool) because it will resolve 1.32 to 1 as
integer and then 1 which maps to true (bool).

To avoid this, use '1.32' instead of the integer form of the version.

Change-Id: I2420396e110284f582cd79820ffc6064e247b4b9

Merge "watchlist: fix nonsensical timestamp/boolean comparisons in EnhancedRecentChanges"

Localisation updates from https://translatewiki.net.

Change-Id: I2968beda4424a3eab0515fc1a8b597f2f8273e56

Merge "Update release notes following the refactor of Block"

Update release notes following the refactor of Block

This is a follow-up to I83bceb5.

Bug: T222737
Change-Id: Ic01a4bb196733a719860c21b7c9b8a41a2e1257a

resources: Drop jquery.async, deprecated since 1.33

Bug: T209699
Change-Id: Ia1a034de8d9996d9955744b7b2b453f4845d69d8

Merge "feedback: Fix misplaced mw.Title.getNameText() call"

Merge "Follow-up ee66a534: Drop reference to old 7zip.inc file name from phan config too"

Merge "Make ApiQueryUserInfo::getBlockInfo accept an AbstractBlock"

build: Upgrade mediawiki/mediawiki-phan-config from 0.5.0 to 0.6.0 and make pass

Fix five instances of PhanPluginDuplicateConditionalNullCoalescing;
escape the rest for now.

Bug: T219114
Change-Id: Ic4bb30c43c5315ce6b878b37b432c6e219414f8b

SearchEngine: Remove deprecated unused method `transformSearchTerm()`

I've checked around with code search tool and realized that SMW has
its own implementation of `transformSearchTerm()` which overwrites the
implementation of this method from core as it extends SearchEngine.

So removing this won't break SMW, see usage below;

Usage
=====

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

Bug: T220656
Change-Id: I3dbe04ecba07700167c894673e23c1eead95460f

Follow-up ee66a534: Drop reference to old 7zip.inc file name from phan config too

Change-Id: I4e018ab676f114a45edb9e7ca6025490134a2ef0

Merge "Add tests for HttpRequestFactoryTest."

Make ApiQueryUserInfo::getBlockInfo accept an AbstractBlock

This method was removed (I84ed216) before the Block class was
refactored (I83bceb5) and restored (If47a938) after the refactor.

After the refactor, this method should accept and AbstractBlock,
so it can handle a SystemBlock as well as a database Block.

Change-Id: Ice8a2e7c19e88406229bd836a45f377abdb72fb6

tests: Remove deprecated rarely used method stashMwGlobals()

This method was hard deprecated in 1.32 and seems it's no longer
used? Can we remove it now?

Usage
=====

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

Change-Id: I8d105a82093ea4cb3a38ed04e471e0213b29e8ec

Add tests for HttpRequestFactoryTest.

This also moves several tests out of HttpTest, so they are not lost when
the deprecated Http class is removed.

Bug: T222935
Change-Id: Ib4aac96ba1a978e8b6738ed446663afc3cc8c792

Merge "move 7zip.inc to a regular php file"

move 7zip.inc to a regular php file

Bug: T182814
Change-Id: I53746333fe3c1cdc357bf46d71a351231a845e2f

Merge "jobqueue: Remove deprecated unused method pushLazyJobs()"

jobqueue: Remove deprecated unused method pushLazyJobs()

Was deprecated in 1.33 and is no longer used anywhere. See usage
below;

Usage
=====

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

Bug: T220656
Change-Id: I4d3f6bc019b8ebc7e504d841fc75acdb08061cb4

Return result from HttpRequestFactory get and post methods

Bug: T222935
Change-Id: Idf1d00d04abbcf4e3391e3979bbab97e595916a5

title: Convert binary regexp to use Unicode code points

The hex sequences are the raw binary values for the Unicode code points.
Now that we have a more modern PHP at hand, we can use \x{FFFF} for
Unicode characters.

I believe the /S is not needed any more. It "precompiles" the regular
expression. But this is a pretty trivial regular expression.
Precompiling it is most probably even slower.

Change-Id: I49435114b3bc31dcce8aa4e48091d509844a2a07

feedback: Fix misplaced mw.Title.getNameText() call

What is called "name" here is a file name without the extension. But
the feedback code is not about files. It's a regular page that might
contain a dot. If it does, the link will be broken, because everything
after the last dot will be stripped.

Change-Id: I9a7c7b73f5d899400ffdc4f4a92cb8b42e644b1c

move most of dumpTextPass to an include file so it can be subclassed

Bug: T164262
Change-Id: I9b3c54503dd9ad193af7765459613a7e73ffbb45

Merge "allow xml page content or metadata dumps to target specific namespaces"

Localisation updates from https://translatewiki.net.

Change-Id: I9dabebcc8d21def62949958e402e314797e71799

Updated special page name translations for Slovak language

Bug: T222793
Change-Id: I738aaa9767f3062be8ad9357072ec5b942e3e7da

Merge "changetags: Remove deprecated unused method `purgeTagUsageCache()`"

Merge "api: Remove deprecated `prepareUrlQuerySearchString()` from core"

Merge "Remove "Squiz.WhiteSpace.FunctionSpacing" from phpcs exclusions"

changetags: Remove deprecated unused method `purgeTagUsageCache()`

This method was hard deprecated in 1.33 in core and no longer used
anymore.

Usage
=====

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

Bug: T220656
Change-Id: I79b364b8a95d9bbf9265c1efa844ff59d11f48aa

api: Remove deprecated `prepareUrlQuerySearchString()` from core

This method was deprecated in 1.33 and is no longer used.

Usage
=====

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

Bug: T220656
Change-Id: I15a476438f3a579db41a43658e13b6ddc4fc62d4

Merge "Restore ApiQueryUserInfo::getBlockInfo() as a stub."

Restore ApiQueryUserInfo::getBlockInfo() as a stub.

Fixes unintended breaking change made by I84ed21641c44b2f65ebe.
ApiQueryUserInfo::getBlockInfo() is restoed as a hard deprecated stub.

This renames the method in the new ApiBlockInfoTrait to
getBlockDetails.

Depends-On: I9f40666a31bd4af50762c197c2ce5bf089a5e68c
Change-Id: If47a93878f87d69800e5f305404c22528dac5e94

Remove "Squiz.WhiteSpace.FunctionSpacing" from phpcs exclusions

Change-Id: I78b3315f26ab91b6b443f5b028a635552f82f5a3

Merge "watchlist: cleanup various method/variable names and comments in watchlist/store code"

Update cssjanus/cssjanus from 1.2.1 to 1.3.0

https://github.com/cssjanus/php-cssjanus/releases/tag/v1.3.0
https://github.com/cssjanus/php-cssjanus/compare/v1.2.1...v1.3.0

Bug: T215746
Change-Id: I352f79f6f34279e669057aee7c2f1570800c8a11
Depends-On: Id8aba2d9e99671a9c45e02b271dbf290a11228d7

Update some composer dependancies

Update composer/semver from 1.4.2 to 1.5.0

https://github.com/composer/semver/releases/tag/1.5.0
https://github.com/composer/semver/compare/1.4.2...1.5.0

Update composer/spdx-licenses from 1.4.0 to 1.5.1

https://github.com/composer/spdx-licenses/compare/1.4.0...1.5.1
https://github.com/composer/spdx-licenses/releases/tag/1.5.1

Update mediawiki/codesniffer from 25.0.0 to 26.0.0

https://github.com/wikimedia/mediawiki-tools-codesniffer/releases/tag/v26.0.0
https://github.com/wikimedia/mediawiki-tools-codesniffer/compare/25.0.0...v26.0.0

Disable failing Squiz.WhiteSpace.FunctionSpacing, can be fixed in a later
commit

Change-Id: I89062c8f28bef893f054650f2e81287291ba9ca0
Depends-On: I8f55e77283f83d853ce340f5c1b7704c65e45bc4

Merge "selenium: Add jpeg-js to devDependencies"

Merge "Add another Bengali synonym for HIDDENCAT in MessagesBn.php"

Merge "user: Remove deprecated and unused method `getPasswordValidity()`"

Merge "Remove deprecated unused method `getModuleScripts()`"

Merge "Add 'avoidhours' option to Language#formatTimePeriod"

Merge "Remove unused title parameter from EditPage::getEditToolbar()"

selenium: Add jpeg-js to devDependencies

It is needed for I92739d84947ac146e in WikibaseMediaInfo Selenium tests.
It can not be added there because of T199116.

Bug: T219815
Bug: T199116
Change-Id: Ib7c3b1b49c280bdc38c40c6aef900a605c1e3cbe