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"

Merge "resourceloader: Ignore coverage for getBaseModulesInternal() wrapper"

rdbms: merge openConnection into getConnection in LoadBalancer

Add error suppression feature to getConnection() itself so that
callers can easily be migrated to it. Deprecate openConnection(),
which is poorly named and had a poor division of responsibility
with getConnection().

Split out some new private methods for readability:
* sanitizeConnectionFlags(), which warns about and strips unusable flags
* enforceConnectionFlags(), which handles some DBO_TRX sanity logic
* lazyLoadReplicationPositions(), which handles the chronology callback

Make sure "waitForPos" is loaded before pickReaderIndex() so that the
optimization of being extra picky about replication lag (in seconds)
actually applies.

Change-Id: I225ed6e6edf39cdf237f2322ea59f35d30f2e01a

Merge "phan: Add internal stubs for imagick, pcntl, redis & sockets"

Update documentation in DefaultSettings to reflect new Preprocessor default

This is a follow-up to Ica5d1ad5b1e677542962fc36d582a793f941155e and
updates a bit of documentation missed in the original patch.

Change-Id: Iaf2b07be4b6946782780a846f026c071edaaf567

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

Change-Id: I134e58949bab25e73874ea29d1b306e4fffa6ef9

Merge "language: Move some language-related classes to includes/language/"

Merge "Various cleanups to MediaWiki::preOutputCommit"

Merge "Use UUIDv4 for mediawiki.api-request log event ID."

Merge "Fix return type of UploadStash::listFiles"

Use UUIDv4 for mediawiki.api-request log event ID.

Bug: T222966
Change-Id: Ie753229ed15545a62349154d442ae9e297951455

Merge "Use IDatabase as return type for BotPassword::getDB"

Merge "resourceloader: Skip modules with circular deps in tree optimiser"

language: Move some language-related classes to includes/language/

Bug: T225756
Change-Id: Ica351483d83a3a912d58ebb194b30828541fbcf3

Merge "rdbms: Fix return type of DatabaseMysqlBase::mysqlFetchArray"

Merge "resourceloader: Fix return type in ResourceLoader-related classes"

resourceloader: Ignore coverage for getBaseModulesInternal() wrapper

The getBaseModules() method is covered already.

Change-Id: I92d9063b72cee9e3be7329927bf5b97028399c14

rdbms: Fix return type of DatabaseMysqlBase::mysqlFetchArray

Change-Id: I7797ae79e8b4c1a88a4f1da342ef4a4f5a78ed74

resourceloader: Fix return type in ResourceLoader-related classes

Change-Id: I543e321129cf8fadc1159017b622528f462ea116

Use IDatabase as return type for BotPassword::getDB

There is no guarantee that this is a maintenance able connection,
but drop, create or alter is also not needed in this case

Change-Id: I7dd294b3d28ef8c4e416e826f38ed39f824b2f8f

Merge "Return null from overwritten FormlessAction::onView"

Merge "Update type of PermissionManager::resultToError"

Merge "Fix return type of GenerateSitemap::getPageRes"

Merge "Fix return type of CeeFormatter::format"

phan: Add internal stubs for imagick, pcntl, redis & sockets

Phan will automatically use these stubs if the extensions are not installed.

Also added a brief README for how to generate the stubs in the first place.

Bug: T224893
Change-Id: I1d31d03801fbc7b18dfc758453c95439acd96d19

Fix return type of CeeFormatter::format

FormatterInterface defines the return type as mixed

Change-Id: I41735b9fe887f9436cf6294a46eb5419c8da251f

Return null from overwritten FormlessAction::onView

The abstract class is documented to use null for doing nothing, not void

Change-Id: I99f0747117f83f4f2ae554c759a8e1737e7dc8f7

Fix return type of UploadStash::listFiles

SpecialUploadStash is using the return and already checking for truthy

Change-Id: Ie26f327d5ec9d67c39cf757591d2a779aecda86f

Merge "ApiCSPReport: Support origin/path matching for false positives"

Fix return type of GenerateSitemap::getPageRes

Change-Id: I68729366f294b93aab5d2c76581c0847c11076d7

Merge "ApiCSPReport: Log user ID instead of name, and limit urls to origin"

Merge "WebInstaller: Add ARIA `role="main"` to overall div"

Merge "Introduce separate unit tests PHPUnit configuration"

WebInstaller: Add ARIA `role="main"` to overall div

Adding ARIA `main` role on appropriate element to keep in sync with
WMF deployed themes.

Change-Id: I9436fd4478e2b108a7ec588db0a628d7d5c26fcd

Introduce separate unit tests PHPUnit configuration

This changeset lays down the basic groundwork required to implement
T89432 and related tickets and is based on exploration done
at the Prague Hackathon. The goal is to identify tests in MediaWiki core
that can be run without having to install & configure MediaWiki and its dependencies,
and provide a way to execute these tests via the standard phpunit entry point,
allowing for faster development and integration with existing tooling like IDEs.

This changeset creates a  new subdirectory under phpunit/ and organizes it
into a separate test suite. The environment for this suite is set up
via a PHPUnit bootstrap file without a custom entry point. For B/C, this
directory is also registered in suite.xml, to ensure that existing CI jobs
still pick up tests in the new suite.

For initial testing, a single test class, PasswordFactoryTest, was moved
to this new suite.

You can run the new suite using the follwoing command:
$ vendor/bin/phpunit -d memory_limit=512M -c tests/phpunit/unit-tests.xml

Bug: T84948
Bug: T89432
Bug: T87781
Change-Id: I69b92db3e70093570e05cc0a64c7780a278b321a

rdbms: Document varargs for IDatabase::buildLike

This is needed in order for Phan not to consider calls to
IDatabase::buildLike as invalid. Interestingly, it does not
consider calls to Database::buildLike invalid.

Bug: T191668
Change-Id: I0e027f5ec66d20b1d11e3441086001f6a751e1f5

Various cleanups to MediaWiki::preOutputCommit

Do not send headers if they were already flushed. Split off some
chronology protection logic into a separate private method. Use
ILBFactory over LBFactory in a few places. Also, update various
code comments.

Bug: T225655
Change-Id: Iecb574e11d8ba09147ff7b84ad57d8845069ba99

Merge "Add link trail for Azerbaijani language"

Reduce selector specificity

Adapt to current Vector code.

Change-Id: Ie5a13f927810cea465fa984c68cdef3664042a41

Merge "shell: Remove documentation that doesn't add meaning to the method"

DevelopmentSettings: Document why wgRateLimits is disabled

Follows-up 9c52f982d8.

Bug: T225796
Change-Id: I6f7a75d58c61712233134a9d480ce68719d6cb6a

Merge "Remove unused/deprecated Replacer classes"

Merge "Installer: Use HTML/CSS instead of JavaScript to toggle the help message"

Merge "Add a report about slow PHPUnit tests"

Add a report about slow PHPUnit tests

johnkary/phpunit-speedtrap prints this information right before the
test results.

Bug: T225730
Change-Id: I288dcd6b3c9b2c0b31c6047eaf48a23e13af726e

Localisation updates from https://translatewiki.net.

Change-Id: I78a1cea264df6fa322b5df4a342882fb867a2c1a

Merge "Use [...] instead of array(...) in PHP comments and documentation"

Remove unused/deprecated Replacer classes

Change-Id: Ifec6c624810b2354f2fab2c1fd4a27267b625c4b