Merge "Drop the UserRights hook, deprecated in 1.26"

Merge "PrefixSearch: Drop titleSearch(), deprecated in 1.23"

Merge "Linker::link: Drop support for string query param, deprecated in 1.20"

Follow-up I3e114219: Don't tell people to use the deprecated replacement of the deprecated removed code

Change-Id: I496ba02f9fc4837f7c7246567d19107ee2baa53d

Drop wfGetLBFactory, deprecated in 1.27

Change-Id: I53b5611edc9443f6a684192e5a03bb0100cb448d

Drop wfResetSessionID, deprecated in 1.27

Change-Id: I14a9c91a43504ee02aac809f5e1124e1cb958e19

Drop the UserLoadFromSession hook, deprecated in 1.27

Change-Id: Ic86d9ee1fe6763270e79e31296ecd027e5d41be2

SkinFactory: Drop getDefaultInstance(), deprecated in 1.27

Change-Id: I40313382c77a1521b80b70a55778db5da1a8a866

rdbms: make connection counting logic in LoadBalancer less stateful

Change-Id: I8428897fa5b6d09e5e3fb84b1adc6e55354eb44c

Localisation updates from https://translatewiki.net.

Change-Id: I6d78ce99ea53dbbf8f43dc422ef8c0dc6a9f5d2a

Merge "rdbms: Type IDatabase::buildLike param as variadic to Phan"

Merge "resourceloader: Add method ResourceLoaderModule::getVary"

Drop the UserRights hook, deprecated in 1.26

Change-Id: Ib4dc6f495c0d1d2aa1f41c1f61e09437f33c9077

PrefixSearch: Drop titleSearch(), deprecated in 1.23

Change-Id: I3e11421953bfe63856e29b928727c8264275cd25

Linker::link: Drop support for string query param, deprecated in 1.20

Change-Id: I910cead2c32a3525772ee0640c8ed3b90ad40a84

Merge "Replace very trivial mock builders with createMock()"

Merge "resourceloader: Throw exception in ResourceLoaderImage::getPath"

Merge "Use IResultWrapper in code comments instead of ResultWrapper"

Merge "rdbms: split out private LoadBalancer::getServerInfoStrict method"

resourceloader: Throw exception in ResourceLoaderImage::getPath

Throw an exception in ResourceLoaderImage::getPath when there is no
matching path instead of continue with null.

Change-Id: I677f4a53f4c90af27db0cc2fd8ef5f028fb49168

resourceloader: Add method ResourceLoaderModule::getVary

Change-Id: I79cd3ad7ad65533717bab52462ffaab5b2179789

Replace very trivial mock builders with createMock()

It does the exact same before and after. createMock() always disables
the default constructor.

Change-Id: I7b294cc3ee7f245f0aec55d12f1277fca5d39bce

rdbms: Type IDatabase::buildLike param as variadic to Phan

Follows-up 725a59f0c7.

* The real method in implementing classes is variadic.

* PHPUnit 6 on HHVM cannot create a mock for a class with a function
  that has a variadic parameter in its actual signature.
  See https://phabricator.wikimedia.org/T191668#5263929 for why.

  This is why the interface (which we use for mock creation)
  currently cannot use the variadic syntax in its signature.

* Documenting the interface as variadic while leaving the actual
  signature without it breaks stuff as well (PHPCS, Phan, ..).

* Documenting it as not variadic works for everyone, but makes
  Phan warn "PhanParamTooMany" at call sites when it is used
  as variadic (to be expected).

We can either try to disable this everywhere in repos that use
this method, but we have another option that might be cleaner.
We can document it as variadic in a way only Phan will consider
and consider only for its call-site analysis.

Hackadihack, Wickedidou, -- introducing `@phan-param`, per
<https://github.com/phan/phan/wiki/Annotating-Your-Source-Code>.

Bug: T226223
Change-Id: I33c729506c271de0f051b77d5f6aafd7a8705745

Merge "parser: inject the time for {{REVISIONTIMESTAMP}} on pre-save parse"

Localisation updates from https://translatewiki.net.

Change-Id: Iaaf41180bfc0a8931d4426d20fbe0c38641642e0

Merge "API: Abstract out parameter validation"

API: Abstract out parameter validation

With the introduction of a REST API into MediaWiki core, we're going to
want to share parameter validation logic rather than having similar code
in both the Action API and the REST API. This abstracts out parameter
validation logic as a library.

There will be at least two follow-up patches:
* One to add calls in the REST API, plus the interface for the REST API
  to do body validation. Should be reasonably straightforward.
* One to adjust the Action API to use this. That'll be much less
  straightforward, as the Action API needs some MediaWiki-specific types
  (which the REST API might use too in the future) and needs to override
  the defaults on some of the library's checks (to maintain back-compat).

Bug: T142080
Bug: T223239
Change-Id: I5c0cc3a8d686ace97596df5832c450a6a50f902c
Depends-On: Iea05dc439688871c574c639e617765ae88a75ff7

Merge "Skin::outputPage: Drop support for specifying a context, deprecated in 1.20"

Skip ::testT69870() under Windows system

Anonymous pipe under Windows does not support asynchronous read and write[1],
and the default buffer is too small (~4K), the test will definitely block it.

Before T69870, anonymous pipe for Windows can no longer hold more than 4K of data.

[1] https://docs.microsoft.com/en-us/windows/desktop/ipc/anonymous-pipe-operations

Bug: T209159
Change-Id: Ie9de36b1e6b68db95c35a0044c5b0d86c0050d33

rdbms: split out private LoadBalancer::getServerInfoStrict method

Change-Id: I9f77a07c75696f4a8353872375a7306d7734625a

Use IResultWrapper in code comments instead of ResultWrapper

Change-Id: Idb813c20bef0d41d0f9f01440daab4fee6cdb38d

rdbms: rename safeWaitForMasterPos() to waitForMasterPos() in ILoadBalancer

Change-Id: I2ad0c6f369ba992895a5306a57f1af16a772844c

Merge "Revert "Fix testsuites for LogFormatter""

Revert "Fix testsuites for LogFormatter"

This reverts commit d0c2da82616a280f51feafa015dd67608c13e29b.

Reason for revert: Original patch set appeared to work then began failing multiple CI builds post-merge.  Simply reverting for now due for T226293.

Big: T226293
Change-Id: I3e33300cecfe87926fcb8851689d67c3b1258b0f

Merge "Remove support for calling OutputPage::wrapWikiMsg() with an options parameter"

Fix testsuites for LogFormatter

Change-Id: I064f563cb54a50c3292b1f9e24b0bb467f69af9b

Skin::outputPage: Drop support for specifying a context, deprecated in 1.20

Bug: T220656
Change-Id: Ie416c88874599dbb196fc0e5abac711345c155a0

Remove support for calling OutputPage::wrapWikiMsg() with an options parameter

Bug: T220656
Change-Id: If9103eb702f3879172d6e860d974603b7b32d6f5

Merge "rdbms: add replica server counting methods to ILoadBalancer"

Merge "Deprecate Sanitizer::setupAttributeWhitelist/attributeWhitelist"

Fix excessive whitespace in BitmapMetadataHandlerTest.php

Change-Id: Ic801069c1a290626f863723409f156b071dec7f5
Follows-Up: I6705a8d46f53ef807edab10f8f0cb0eba6b560dc

Merge "API: Update api-help-datatypes for Timestamp library changes"

Merge "Update ObjectFactory and ConvertibleTimestamp"

Merge "Some love to UserDupes"

Merge "GlobalFunctions: Remove deprecated wfCountDown() function from core"

Merge "Update documentation in DefaultSettings to reflect new Preprocessor default"

Merge "resourceloader: Remove my `@author` comments from unit tests"

Merge "resourceloader: Remove 'Series60' from Grade C regex"

GlobalFunctions: Remove deprecated wfCountDown() function from core

The global function `wfCountDown()` was deprecated in 1.31 and a replacement
made available (\Maintenance::countdown()) in 1.31 as well. Currently, the few
usage in BSF have been cleaned up, hence can be removed.

Usage
=====

https://codesearch.wmflabs.org/search/?q=wfCountDown&i=nope&files=&repos=

Depends-On: Iae61d36899036ecdc0341feb3f1bcaa6b38dbf44
Depends-On: Id202148c6a6ceaadcefcf7a4ab8360edd9fe3b0b
Bug: T220656
Change-Id: Ic7460f1d35cb6f2eb8bf60694cdd4c55b2e001c0

Merge "Filter out duplicate autoblocks when checking for blocks"

Filter out duplicate autoblocks when checking for blocks

Follow-up to I7654907.

Bug: T225919
Change-Id: I67e72d6c88e3cbfd9515a016b2782d1d9b123775

Remove global function wfBCP47, deprecated in 1.31

Change-Id: I8b9956bec18cc7bbc9aafeec124e6c1fef262dc6

Merge "languages: Use "your" instead of "my" in the updatedmarker value"

Merge "config: ServiceOptions O(1) assert time in the common case"

config: ServiceOptions O(1) assert time in the common case

Check if the key array passed in the constructor is identical to the key
array passed to assertRequiredOptions(). This takes O(1) time if the key
arrays have the same underlying storage pointer, which is the common
case.

If the arrays have a different order but are otherwise identical, the
slow path is taken instead. The comparison will add O(N) overhead in
addition to the overhead of the array_diff() calls.

Change-Id: Icb9040ab66286b72a270e84f910cb578bed105b0

Merge "Remove OutputPage::enableSectionEditLinks(), ParserOptions::get/setEditSection()"

Follow up to 'Remove support for the deprecated Key header'

Clean up a few more code paths and documentation bits left behind by
Ia53d07cd8ce8ab1497294ea244c13c7499f632c7.

Change-Id: I2bb1749c45bb79b27c5a3b2e1b8ed3395e8c11e0

Deprecate Sanitizer::setupAttributeWhitelist/attributeWhitelist

These methods should be made private in the next release, but
hard-deprecate them for 1.34.

Tweak the return value of the attribute whitelist to be an
associative rather than a sequential array, which makes the
lookup of allowed attributes more efficient and avoids an
array_flip for every html element sanitized.

Bug: T221677
Change-Id: I17d734937accec6c2679dbe17328cf9554bd556a

Merge "Parser: Remove deprecated method Parser::fetchFile()"

Merge "installer: Remove deprecated method WebInstallerOutput::addWikiText()"

API: Update api-help-datatypes for Timestamp library changes

Instead of reproducing the documentation that's now at
https://www.mediawiki.org/wiki/Timestamp, let's just link to it.

Bug: T224730
Change-Id: I26c3bafc7cc9145b573c3ef84877633733be3cb1

resourceloader: Remove 'Series60' from Grade C regex

This existed to match UAs from the now-discontinued SymbianOS browser,
however, we already have 'SymbianOS' in the regex and all known
UA variants (which we have tests for) contain that phrase as well.

Change-Id: I643eb1e1df6a09873cda7d6fe2f325cd722b3ee5

Parser: Remove deprecated method Parser::fetchFile()

Deprecated in 1.32 and no longer used, see usage below;

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5CbfetchFile%5C(%20(.*%3F)%2C%20(.*%3F)%20%5C)&i=nope&files=&repos=

Change-Id: Id0f76ee5c0e2a230dabb563c9d9e5c0a584f6b2c

installer: Remove deprecated method WebInstallerOutput::addWikiText()

Was deprecated in 1.32 and no longer used, hence removing from MW core.

Change-Id: I93c5dc1d74fd856c07b09e4c3028e223d2939522

Some love to UserDupes

Change-Id: I9c0310c928a518e6f0ed13998603544fe441358e

Update ObjectFactory and ConvertibleTimestamp

* wikimedia/object-factory -> 2.0.0
* wikimedia/timestamp -> 3.0.0
* wikimedia/xmp-reader -> 0.6.3

Needed for I5c0cc3a8. mediawiki/vendor equivalent is Iea05dc43.

Bug: T224729
Bug: T144482
Depends-On: Iea05dc439688871c574c639e617765ae88a75ff7
Change-Id: I6705a8d46f53ef807edab10f8f0cb0eba6b560dc

Merge "Type hint against IResultWrapper in maintenance classes"

Merge "Type hint against IResultWrapper in api classes"

Merge "Remove $wgUseKeyHeader and OutputPage::getKeyHeader(), deprecated in 1.32"

Merge "Remove OutputPage::addWikitext()* functions, hard-deprecated in 1.32"

Merge "Update monolog/monolog from 1.22.1 -> 1.24.0"

Add some @since tags to ParserOutput::SUPPORTS_ constants

Change-Id: I2f6588fe563ed5c1dc5ef2a70e2ed59fdca99018

rdbms: add replica server counting methods to ILoadBalancer

This is slightly more robust and makes the intent much clearer
than random calling code checking getServerCount() all over the
place. In addition, this yields better separation of concern.

Also, cleanup the LoadBalancer constructer a bit and make the
validation a bit stricter.

Make some server index comparisons strict while at it.

Change-Id: Icc1a35bd65c6862ff81faa3ab9b2aa7cafe29443

Merge "resourceloader: Document which FileModule methods use a DB"

Merge "Fix wfLocalFile() replacement"

Merge "resourceloader: Switch Xml::encodeJsCall call in getDeprecationInformation"

Fix wfLocalFile() replacement

This bit of ImageListPager used to call wfLocalFile(), but was
accidentally replaced with the replacement for wfFindFile() instead.

Bug: T226102
Change-Id: Id50a5359fe2353ae88012d1f5a3331f570b73922

Disable flaky Selenium test

Change-Id: Ia3f2403e452f4476238b349c9d6a6f8135267456
Follows-Up: Iaf7e095b3bb34072eea6bcac76ba29358b14cc09
Bug: T226144

resourceloader: Remove my `@author` comments from unit tests

Previously removed from includes/ already.

Also remove odd `@package`, which we never use.
And remove `@since` which doesn't make sense for test-only files.

Change-Id: Ib7265d39329ecadd5279b11820f77f54189b55d2

Merge "AutoLoader: Skip tokenizing of irrelevant lines in ClassCollector"

resourceloader: Document which FileModule methods use a DB

Also, for the unit test, disable the two methods we use there
that can get called. The unintended side-effects of these two
methods was the only reason it used `@group Database`.
Removing that makes the test a bit faster as well.

Enforce this via MediaWikiServices for this suite to avoid an
untracked dependency slipping back in in the future.

Bug: T225730
Change-Id: I6c54466e9517d9899bc39f8f9bb946369c0a526d

Localisation updates from https://translatewiki.net.

Change-Id: I4b61ff429f73f4d8252206f6613551d20b5b230f

Merge "Remove deprecated ApiAction logging code from ApiMain.php"

Type hint against IResultWrapper in api classes

Change-Id: If3870e6b8d343dd12390f5155d97edbbe080ac77

Type hint against IResultWrapper in maintenance classes

Change-Id: I6a58e9cfe645056a854c80b2db1e71fb59bd226b

Merge "Reduce selector specificity"

Remove OutputPage::enableSectionEditLinks(), ParserOptions::get/setEditSection()

These methods were deprecated in 1.31, and most of the related code
was removed in b4e557f8f8a72d4539e0cf5f83add4eef8df418b but these three
methods appear to have been overlooked.

Change-Id: Iea6c8b1b628a7b6acf9b65497966af9fc4ab662e

Remove $wgUseKeyHeader and OutputPage::getKeyHeader(), deprecated in 1.32

These implemented a since-abandoned draft IETF spec, and the code was
broken due to (1) case-(in)sensitivity issues with the Accept-Language
header and (2) the BCP47 language code compatibility workaround we use.

Change-Id: Ia53d07cd8ce8ab1497294ea244c13c7499f632c7

Remove OutputPage::addWikitext()* functions, hard-deprecated in 1.32

This starts cleaning up the programmer-visible API for OutputPage
and removed some deprecated untidy parser modes.

Change-Id: Ib464b57248f114b68424ec1175d36ad86d1319ad

Style change tags correctly on Special:Newpages

Bug: T212613
Change-Id: I6905273bf9bb3d93480df201a2935d8b0e831805

rdbms: deprecate unused ILoadBalancer::safeGetLag method

Change-Id: Ib3bc2862548271613da30ad1be836d28a82e6cc9

AutoLoader: Skip tokenizing of irrelevant lines in ClassCollector

This makes AutoLoaderStructureTest in PHPUnit and the
generateLocalAutoload.php maintenance script much faster.

On my machine, it made it 35X faster (or time spent reduced by 97%).

Bug: T225730
Change-Id: Ife959bd17ce9c2ae952dfbd158ddb3d8475e8cb2

resourceloader: Switch Xml::encodeJsCall call in getDeprecationInformation

Also rename one of the test classes to have a more descriptive name.

Bug: T32956
Change-Id: I1a970c198300b7ef0d99c15609f1fb7fa8783b98

Merge "resourceloader: Define the default language and skin in constants"

Remove deprecated ApiAction logging code from ApiMain.php

ApiAction was disabled in WMF production in
https://gerrit.wikimedia.org/r/c/operations/mediawiki-config/+/516303

Bug: T222267
Change-Id: I527811ef84506cf35fe7b9b981fb405776fceb1a

resourceloader: Define the default language and skin in constants

This allows to use the default values at several places.

Change-Id: I53f29c2162c4cac9caf672e816ebfeb97c452389

Merge "rdbms: mention wait time in LoadBalancer::safeWaitForMasterPos() error logging"

Merge "rdbms: merge openConnection into getConnection in LoadBalancer"

Merge "resourceloader: Migrate use of 'raw' modules to 'raw' requests"

Merge "resourceloader: Remove redundant 'group' setting for StartupModule"