Add some @since tags to ParserOutput::SUPPORTS_ constants

Change-Id: I2f6588fe563ed5c1dc5ef2a70e2ed59fdca99018

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

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"

Merge "Reduce selector specificity"

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"

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

Merge "Installer: Make short header and normal header of WebInstaller equivalent"

Installer: Make short header and normal header of WebInstaller equivalent

Change-Id: I8ae0cca582c2dab87a4b3b050030f6c96897939e

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

Change-Id: I0c83783051bf35fe785bc01644eeb2946902b6b2

Merge "Fix some issues with HTMLSelectAndOtherField default and validation"

Merge "resourceloader: Remove unused static $contexts var in ResourceLoaderImageTest"

Merge "FileRepo: Use Late Static Binding in File static constructors"

resourceloader: Remove unused static $contexts var in ResourceLoaderImageTest

Change-Id: I95abbfd6d2baafb9378072fdf1a1fc5ed046cec3

Merge "Filter out blocks with duplicate IDs when checking for blocks"

Merge "rdbms: clean up and simplify toString() handle for Database"

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

Change-Id: I924453f683f058586516bb12e54940449d29cfd9

Filter out blocks with duplicate IDs when checking for blocks

Bug: T225919
Change-Id: I76549072d53083e6057f4fd8fe963e8989daa25c

Merge "[bugfix] Fetch tag ID before calling undefineTag()"

Disable rate limiting in Development Settings

Bug: T225796
Change-Id: I2475a04066d4aaefeba372bd223ef68548a8cf18

Merge "Fix param type of search terms in search related classes"

Merge "rdbms: allow synchronous=OFF for sqlite"

Merge "search: clean up some type hints in search classes"

Merge "rdbms: do not close the connection in LoadBalancerSingle::__destruct"

rdbms: allow synchronous=OFF for sqlite

Change-Id: I58be09bb170399f3ad94fc3bcc4881932d357a9b

search: clean up some type hints in search classes

Change-Id: I357a78b42ea02a63c3b64e8982f04314cbbfdf9f

rdbms: do not close the connection in LoadBalancerSingle::__destruct

The connection instance is injected unlike with the parent class,
meaning that it does not own the connection. No-op the method.

Change-Id: I61914b986619f201ac9cc8c94d46873a3b9e8177

Only attempt to deduplicate if there is data in archive and revision

The idea is to avoid expensive calls to makeDummyRevisionRow, and speed up
installation of MediaWiki on CI.

Bug: T225901
Change-Id: I6f69281568218c89eb18353c06cabf7eb1926de8

Localisation updates from https://translatewiki.net.

Change-Id: I84ab99cdcd91ee6dab8ae284e8b1c6aea53cefac

[bugfix] Fetch tag ID before calling undefineTag()

undefineTag() also deletes the tag if it's not used anywhere,
which is breaking the rest of deleteTagEverywhere() in that case.

Bug: T225564
Change-Id: I7ca5db9efd0088b266e33c0a9ce78d73a4fa87c9

Use secure protocols on Wikimedia sites

Change-Id: I3af0038202027a737a306ae677f7cda2b9857404

Merge "Reduce HashRing test load to avoid several seconds of CPU"

FileRepo: Use Late Static Binding in File static constructors

The FileRepo extension API allows to specify factory methods that
FileRepo implementations will use to instantiate File instances.
Currently, the default static constructors in LocalFile and OldLocalFile
do not use Late Static Binding, so every subclass is forced to re-implement them,
even if they would not need any custom logic. Switching to Late Static Binding
(available since PHP 5.3) allows File implementors to reduce boilerplate
if they do not need to overwrite the existing logic.

Change-Id: Id8f6f5362b68269c2a3232796a1703be14116dd5

rdbms: clean up and simplify toString() handle for Database

Change-Id: Ia95e25505785aad71b70a74f4ec7a07a73e419e1

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

Changes in the behavior:
* The expanding/collapsing now works even with disabled JavaScript.
* There is no FOUC on loading anymore.
* The animation on show/hide is absent.

Change-Id: I1b88d8e2cdbb1c969670a7a8637dba10bf447330

Merge "resourceloader: Support 'versionCallback' for computed package files"

Fix some issues with HTMLSelectAndOtherField default and validation

Bring HTMLSelectAndOtherField in line with other HTMLFormFields by
ensuring that the default value is of the correct type and passes
validation checks.

Also make sure HTMLSelectAndOtherField::validate checks for the
final value, if the field has required set to true.

Bug: T222170
Bug: T225860
Change-Id: I949ee3df2b1f597982cf522b149c54b8e79d59bc

Merge "PHPVersionCheck: remove version_compare() existence check"

PHPVersionCheck: remove version_compare() existence check

It was added to PHP in 2001.

Change-Id: I3b14d726aa1cefd6b64e4bac39ef489b3c79ac1a