Merge "HTMLForm: Improve method documentation for setCollapsibleOptions()"

ImageListPager: Use explicit escaped() instead of implicit __toString()

For the messages listfiles-latestversion-yes and
listfiles-latestversion-no a wikitext parsing is not necessary.

Change-Id: I6f9268ebabaa64494d8680b04990944e7a463f32

HTMLForm: Improve method documentation for setCollapsibleOptions()

Follows-up 36d33daa03a63.

Previously the method was described as "Set whether the form can be collapsed"
taking a single boolean.

Yet, the boolean did not influence whether the form can be collapsed, as one
might expect.

Rather, this method always enable collapsible mode. The thing that is set
based on the passed value is the default state of that (unconditionally)
enabled collapsible mode.

Change-Id: I2e73d5481c44ed43769553b2bc25543a702c19b8

Merge "objectcache: make detectLocalServerCache() prefer apcu over apc"

objectcache: make detectLocalServerCache() prefer apcu over apc

Bug: T220470
Change-Id: I8b85859b3991d2360a74107299d44da2c60d0214

Merge "resourceloader: Add tests for StartUpModule dep tree optimisation"

Merge "registration: Add development requirements to extension.json"

Merge "resourceloader: Make MessageBlobStore global check key actually global"

resourceloader: Add tests for StartUpModule dep tree optimisation

Bug: T223402
Change-Id: I0450c052852068f3c78312f48a42645a8b77bc5c

Merge "Fix some docs"

Merge "Work around disappearing Special:Preferences buttons on RTL in Edge"

Merge "Reference T224165 when deprecating Parser::$mConf"

Work around disappearing Special:Preferences buttons on RTL in Edge

Bug: T220706
Change-Id: If36dbade88f7594dea658f899f891e800c98529d

Merge "npm: bump grunt-karma to 3.0.2"

Merge "Fix doc in maintenance/findHooks.php"

Localisation updates from https://translatewiki.net.

Change-Id: Ic10472cce88fd44322506beb2330cc49f5d22fb0

Fix some docs

Discovered by phan.

Change-Id: Iadc2b151db6922f3159f061f10408d45636967c5

ExternalStore: Pass external domain to getReadOnlyReason

Be consistent with other paths of code that supply the external domain
id to LoadBalancer instead of defaulting to the wiki's domain.

This was causing the getReadOnlyReason to fail because it tries to
connect to the local wiki database using external storage credentials.

Bug: T200471
Change-Id: Ie73ea931d5c93f967624ee67717f87c2d9a31559

Merge "objectcache: check apc.serializer in APCBagOStuff like APCUBagOStuff"

Reference T224165 when deprecating Parser::$mConf

Bug: T224165
Change-Id: I98225ad03e4060dbc956905da01fdfe671526694

resourceloader: Make MessageBlobStore global check key actually global

Because it was still per-wiki, which isn't useful and makes
it really slow to invalidate by requiring a 900-wiki iteration
during deployments all of which fully initialise MW just to
send a single Memcached command basically.

Bug: T222539
Change-Id: I02a3e7d83172ccd7d8d0fba3be1e3f1ebb77efb2

Merge "Configure LoadBalancer with $wgSharedTables after WAN cache autodetection"

Merge "jquery.tablesorter: Fix pre-JS selector to match wikitext-generated sortable tables"

objectcache: check apc.serializer in APCBagOStuff like APCUBagOStuff

This mirrors a1389b602411

Change-Id: Iad6b104337dfd38c74f363ce76c36aedcc3da425

Configure LoadBalancer with $wgSharedTables after WAN cache autodetection

Currently, $wgSharedTables - if configured - is injected into the LoadBalancer
before the WAN cache autodetection runs. As such, if $wgSharedTables is set
but $wgMainWANCache was not explicitly given, the LoadBalancer will force
a premature instantion of the WAN cache with an EmptyBagOStuff backend,
preventing the autodetection logic from setting a correct backend.
This change moves $wgSharedTables initialization after the WAN cache
autodetection to accunt for this scenario.

Change-Id: I59bb52f42905cda1c2aa47bc589a2f0b36b2a28f

Merge "Deprecate $wgWikiDiff2MovedParagraphDetectionCutoff"

Merge "Make some array type hints more specific"

Deprecate $wgWikiDiff2MovedParagraphDetectionCutoff

Bug: T194272
Change-Id: I174192dc2b91409e023c06b054bc1bba8bfc571f

Merge "Exclude FileImporter browser tests"

Make some array type hints more specific

Change-Id: Ia9ed6fa851316baa47e73d9a5988a863d15f298c

Merge "EditPage: Migrate Title::userCan() calls to PermissionManager"

Merge "externalstore: make ExternalStoreDB::getDomainId treat false the same as null"

Merge "Add additional configuation parameters to RESTBagOStuff"

Merge "Don't pass Config to Parser(Factory)"

Merge "registration: Fix validation of ext-foo values"

Merge "Fix logic in NamespaceInfo::getRestrictionLevels"

Merge "WikiPage: Make prepareContentForEdit method not accept an integer"

registration: Add development requirements to extension.json

Extensions can specify development dependencies in extension.json under
the "dev-requires" key. It's identical to the "requires" field.

Any requirement that is needed to pass tests, including but not limited
to, PHPUnit, QUnit, structure, phan, should be documented in this new
field. The main intention is that CI will ensure that all of these
dependencies are satisfied before running tests.

At standard runtime, the development requirements will be ignored by
MediaWiki, since it only checks for real requirements. Scripts can
manually check development requirements by calling
ExtensionRegistry::setCheckDevRequires( true ) before trying to load
things.

If both "requires" and "dev-requires" are present, MediaWiki will merge
the two together, so the environment will need to satisfy both before
proceeding.

Bug: T193824
Change-Id: I9b2936666ee3c96f5c976c7a17f11c437c2c7f48

registration: Fix validation of ext-foo values

Our json-schema library doesn't yet support "const" (at least not in a
released version), so use "enum" with one element, since it's functionally
equivalent per <https://json-schema.org/understanding-json-schema/reference/generic.html#constant-values>.

Change-Id: Ib9ebb5c6b9d07903fa47603fbe54a148e9c00352

Fix doc in maintenance/findHooks.php

Change-Id: Iad3f2fc0910049dbfd5039f3589dd3f3918f1392

WikiPage: Make prepareContentForEdit method not accept an integer

This has been deprecated since 1.25.

Bug: T220656
Change-Id: I94f5cb78adcfbec1fdafd89495c045e6f8fd9246

Merge "rdbms: improve query logging logic in Database"

Merge "watchlist: make getLatestNotificationTimestamp() method use the correct user"

npm: bump grunt-karma to 3.0.2

`npm install` currently fails due to an unsatisfied peer dependency.
See full output here:

https://phabricator.wikimedia.org/P8552

Bumping the required grunt-karma version to the latest (3.0.2) fixes
this.

Change-Id: I5b7f2deba2d766b809a964f53cb22f47f0617bd6

externalstore: make ExternalStoreDB::getDomainId treat false the same as null

Callers like SqlBlobStore sometimes pass in false for the current wiki

Bug: T200471
Change-Id: I3025c869df07de312471d00a0ab7107c1fa14a90

Removed 'public $user' as it is deprecated in 1.29

Change-Id: If9c59d432e4b8b91b8c81200bf3a8a910461eded

Add additional configuation parameters to RESTBagOStuff

Class RESTBagOStuff is needed for the Kask session storage
service. Add additional configuration parameters so that
they will work together.

Bug: T215533
Change-Id: Ifbb9f8847bd10d65cc5e24f4aa6cb20cb71de3ca

Merge "Correct autonym for rmy (Vlax Romani)"

Correct autonym for rmy (Vlax Romani)

Bug: T223524
Change-Id: Id128199119001bd318012d034ef727cdfa256f54

watchlist: make getLatestNotificationTimestamp() method use the correct user

This uses the proper stash entry and also fits within the 3 item process
cache in WatchedItemStore that contains last-visit maps for users.

Follows-up 588a46468252, which added use of the stash optimisation but
wrongly keyed against rc_actor/rc_user instead of $this->getUser,
which means that unlike the DB 'wl_notificationtimestamp' field (which
is result of joining against watchlist and user via $this->getUser), it
was now reading the cache key for a different user instead of the user
that is viewing their watchlist.

Bug: T218511
Bug: T223310
Change-Id: I5c4dd2c7384f7681cee21fb2caf10d4d4b64c724

Merge "Remove tryLastResult function from includes/specials/QueryPage.php"

Merge "rdbms: replace some return-based sanity checks in Database with exception-based checks"

rdbms: replace some return-based sanity checks in Database with exception-based checks

Change-Id: I51a541fb560438a34522ca442a8b8e572830deea

Remove tryLastResult function from includes/specials/QueryPage.php

It is deprecated in 1.27

Change-Id: Ia10696a3d8e700cdf889c15e9b16b9d8e3e7f94f

Merge "jobqueue: Remove duplicate property from DoubleRedirectJob"

Localisation updates from https://translatewiki.net.

Change-Id: I30b63a979dbf32b92ba5bea8799afdd51d00ce5f

Don't pass Config to Parser(Factory)

Change-Id: I5996b7cad8c8a61518d2997e955a4547c64a73a5

Exclude FileImporter browser tests

These test must be run from the extension's repo, because they use a custom
script to modify LocalSettings.php.

Bug: T190829
Change-Id: I920c7faa3ce91f1560f383575aa6a6adfcadd9c7

Merge "Allow hidden skins to show up in preferences"

Merge "Enable email settings for ApiBlockTest::testBlockWithEmailBlock"

Merge "Enable email in TemporaryPasswordPrimaryAuthenticationProviderTest"

Merge "Linker: Fix fatal error for "/*  */" in an edit summary"

SpecialAllMessages: Specify filter messages so we always escape them

As reported by phan-taint-check, HTMLForm options have to be escaped.

Bug: T223821
Change-Id: I663911a2c3d8a5c60d407e84ec528f71af2eafe9

Allow hidden skins to show up in preferences

It should be possible for power users to reveal hidden preferences
using the useskin query parameter. In future this will allow us
to use wgSkipSkins to deprecate poorly supported skins for new
users whilst not hard deprecating and removing skin support for
users who strongly want to continue to use them.

Change-Id: I1bbd4a09dff72f513c9413e0f826d8db38a5e04c

Merge "Follow-up 3535a5f327: Remove old CSS now caches have expired"

Linker: Fix fatal error for "/*  */" in an edit summary

Follows-up b6e1e99bec8, which switched the method from Title::makeTitleSafe
to TitleValue. The latter throws fatal on non-string $fragment.

Title::makeTitleSafe, on the other hand, uses makeName(), which casts
$fragment to a string, and ignores if it ends up as empty string
(boolean false becomes empty string and thus did "the right thing").

Bug: T222857
Change-Id: Iecc2140fabd31ef0f193740c7fab0fc698c38e51

Merge "i18n: Rename 11 messages for case-compliance"

Merge "Simplify a few list() that only care about the first element"

Merge "docs: Fix UploadVerifyUpload/UploadStashFile hook docs"

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

EditPage: Migrate Title::userCan() calls to PermissionManager

T208768 introduced the PermissionManager service that can now be used
for page specific permission checks. This change replaces calls to
Title::userCan() with the new service in the EditPage class.

Bug: T220191
Change-Id: I835d68d6c47785cf35386bca0431907fee87f0c1

Typehint against IDatabase in special pages

Change-Id: I1ec906cc7b284a3de02bb67fb41b5cf9e67ff3d2

i18n: Rename 11 messages for case-compliance

Can't yet enforce due to the API.

* allmessagesnotsupportedDB
* rcfilters-filtergroup-userExpLevel
* rcfilters-filtergroup-lastRevision
* number_of_watching_users_RCview
* apihelp-query+allfileusages-example-B
* apihelp-query+allimages-example-B
* apihelp-query+alllinks-example-B
* apihelp-query+allpages-example-B
* apihelp-query+allredirects-example-B
* apihelp-query+alltransclusions-example-B
* apihelp-query+allusers-example-Y
* apiwarn-wgDebugAPI

Change-Id: I95961dfeb43e9cfadc867a48866af6335022572a

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"

docs: Fix UploadVerifyUpload/UploadStashFile hook docs

1. FSFile::getPropsFromPath() is not used by any code any more.
https://codesearch.wmflabs.org/search/?q=FSFile%3A%3AgetPropsFromPath&i=1
The only remaining usage is in one test. We might as well remove the
function.

2. The $props array is passed to the hook for convenience, in case all
the file properties are already available. Fetching them from a file on
disc can be an expensive operation, and should be avoided if the
information is already available. But the caller does not guarantee this
is set. Other callees already know this can be a falsy value, notably
LocalFile::upload().

Change-Id: I43724d18467b6fb68a963b2206332cf553c81b2c

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"

jobqueue: Remove duplicate property from DoubleRedirectJob

All the stuff that is passed as $params to the constructor is stored
in $this->params anyway.

Change-Id: Idec1c243de7cdbb1e55dd754555536ce0ec2024d

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

Simplify a few list() that only care about the first element

The nice thing about explode() is that the resulting array is
guaranteed to contain at least one element. The array can not be
empty.

In some of these cases it might be possible to use strstr() instead,
but that returns an empty string when the needle character is not
found. explode() returns the original string in this case.

Change-Id: I6ad1f3273defeaf36e2305fd871eaaf9d3c1e134

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