Split down patch-comment-table.sql

Bug: T227662
Change-Id: I7617616df57f7468d06e9b52426b6851bfef0e7d

Merge "Clear block cookie if the value is invalid"

Merge "Add instrumentation to Special:Mute"

Merge "Put block cookie expiry into UTC for comparison with current time"

Merge "rdbms: simplify LoadBalancer::getLaggedReplicaMode()"

Merge "Tests: Set dbname for DatabaseSqliteTest"

Merge "ApiSandbox: Load mediawiki.hlist RL module"

Merge "objectcache: fix WRITE_ALLOW_SEGMENTS in BagOStuff cas() and add() methods"

Put block cookie expiry into UTC for comparison with current time

Change-Id: Ia01a364a01ebacec8783ecdcfe4a129cfb746eb8

Merge "TitleOptionWidget: Documentation type fix"

Merge "RCFilters: add namespace filters for all contents and all discussions"

Merge "hooks.txt: Convert docs to modern extension registration style"

objectcache: fix WRITE_ALLOW_SEGMENTS in BagOStuff cas() and add() methods

Add MediumSpecificBagOStuff::getValueOrSegmentList() helper method.

Also:
* Use $keysMissing variable correctly in CachedBagOStuff::getMulti()
  to avoid extra overhead.
* Optimize mergeViaCas() when the current value matches the new one.

Change-Id: I5c4bd74379bc459216ac0278150ce3aecff3b851

TitleOptionWidget: Documentation type fix

Change-Id: I6ea4a1749a373c64e12d5c0ee1e5d55953523205

ApiSandbox: Load mediawiki.hlist RL module

The default l10n for the main module's description uses the hlist class,
so load it.

It might make more sense to have it as a module dependency of an
appropriate RL module like mediawiki.apipretty instead of requiring
every caller to load it manually, but it seems that's not really done
because RL is weird about handling CSS-only modules as dependencies.

Bug: T230782
Change-Id: I6493d7d789f3cebee737645528cddfb1d3acb1d7

hooks.txt: Convert docs to modern extension registration style

This changes the examples in hooks.txt from using the old format of
manually entering additions into `wgHooks` to instead use the new
`"Hooks“: {}` object format.

Bug: T230397
Change-Id: I48a9986e4243eb933088d36b4bb095b345ab62fd

Merge "objectcache: make newFromId()/newWANcacheFromId()/getDefaultKeyspace() private"

objectcache: make newFromId()/newWANcacheFromId()/getDefaultKeyspace() private

Change-Id: I30b99f23dbd9637ca8178d3a3650b4c38ec43e7d

Merge "Prime connections and update description of McTest script"

Merge "Tests for TempFSFile"

Merge "docs: Fix typos for 'parameter' and 'perform'"

Merge "logging: Fix typo for `getPerformer()` interface method declaration"

docs: Fix typos for 'parameter' and 'perform'

Bug: T201491
Change-Id: I37ed48907bf7c1a1d4ebab7b10b41a77623eba8a

Tests: Set dbname for DatabaseSqliteTest

Bug: T230780
Change-Id: I93d3771233c74443f76b0a402a4820e1e1e30ee7

Tests for TempFSFile

These are in preparation for making a TempFSFileFactory service, thus
the odd break-up into two files. I split it into a separate commit so
that we could verify that the same tests pass before and after the
conversion to service.

Tests cover everything except getUsableTempDirectory() (which I don't
see how to test), and register_shutdown_function()-related stuff (which
seems actually impossible to test without starting a new PHP process).

Change-Id: If61b7ea3e332adc2bceefc8e6879a9e9443c99dd

Localisation updates from https://translatewiki.net.

Change-Id: I5d21bfc6534244188a6d639590681edeba3e9e7d

Merge "Test that classes use all their ServiceOptions"

Clear block cookie if the value is invalid

When a block cookie is present and the block is invalid or doesn't exists
or the cookie value is invalid or malformed, the cookie is removed.

Bug: T227678
Change-Id: Icaff594686c16a0eb8551b2a4392a14a969b43b0

Test that classes use all their ServiceOptions

Classes that use ServiceOptions need to declare a list of keys that they
use, typically in self::$constructorOptions. If keys are missing from
that list that are supposed to be present, an exception will be thrown
when someone tries to access them. If keys are present on the list that
are never used, no error is flagged. This means if a dependency on a
given configuration option is removed and nobody updates the list, the
service will keep thinking it depends on that option when it doesn't.
This is messy at best, like an unused variable.

A new and easy-to-use TestAllServiceOptionsUsed trait fixes that
problem. It will log all the ServiceOptions accesses while the test
class runs and raise an error if there are any keys that were never
accessed.

In retrospect, it was probably not worth the time to write this, but
it's a sunk cost now.

Change-Id: Idcaf9a0e2f687069869e6f8057908ffee7dd5f11

Return better errors from MovePage::isValidMove()

Previously the errors returned were incorrect or redundant in a number
of cases.

Change-Id: Ief96e69b0ae09afb9642f9ed93b2419a36351292

Add a bunch of MovePage tests

The expected values in many cases are silly, because our code is
currently silly and could use some refactoring. These are marked with
@todo. In one case the return value is even wrong (moving to an invalid
non-empty name is considered valid).

Change-Id: I9649a4de12bbcd6263c85de37d7b9365d9c0aeb4

Use RepoGroup service in MovePage

Change-Id: I2b59de374d49652b61f074a9da5483c0d991e7a6

Introduce MovePageFactory

This will help make MovePage more testable.

In the course of abstracting the logic out of ParserFactoryTest to
FactoryArgTestTrait so it could be used in MovePageFactoryTest, I made
them all unit tests instead of integration. This required some
modification to the Parser constructor so that it didn't access
MediaWikiServices unnecessarily.

Change-Id: Idaa1633f32dfedfa37516bb9180cfcfbe7ca31aa

logging: Fix typo for `getPerformer()` interface method declaration

Not sure but the previous wording didn't make much sense or is it
correct that way?

Change-Id: Ia5bea91fb426bb2fe742e656e5a826b8c07d88ee

Merge "Use injected services instead of static method"

Merge "Convert MessageCache to service"

Merge "Add recursion check to createService()"

Add instrumentation to Special:Mute

We are already tracking pageviews and with following change we should be able
to answer the following questions:
* Of the users who land on this page, what percentage of users actually
  mute or unmute someone
* Of the users who mute a user, which option(s) did they check/uncheck in order
  to mute/unmute the user

EventLogging Schema: https://meta.wikimedia.org/wiki/Schema:SpecialMuteSubmit

Bug: T224958
Change-Id: I655dbd999fd5d3d8f792c4f53b7cc502fe05afd5

Merge "Revert "Clean up ApiEditPage content handler undo logic""

Revert "Clean up ApiEditPage content handler undo logic"

This reverts commit 01564dcbf76cd279c88320f96f0844cfadf4f7c9.

Reason for revert: I find the empty case far clearer as to intention than this change, and I doubt this is such a hot path that optimization should take precedence over clarity. If we really want to get rid of the empty case, I'd have structured it somewhat clearly as

 if (
     // always allow undo via api, T230702
     !( $params['undo'] > 0 ) &&
     // [something else here to explain this clause]
     $contentHandler->supportsDirectApiEditing() === false
 ) {

But even that is IMO less clear. At the very least this seems to deserve more discussion.

Bug: T230702
Change-Id: Icc16250faad31abf7308c4af9be2e7658540346c

Merge "Remove 'redundant' attribute from BaseTemplate::makeListItem() links in case this is not done in SkinTemplate::buildContentActionUrls()"

Merge "Add suppressredirect right to the createeditmovepage grant"

Merge "MySQL connect: Parse the Unix domain socket path correctly"

Merge "Clean up ApiEditPage content handler undo logic"

Clean up ApiEditPage content handler undo logic

Bug: T230702
Change-Id: I79d528866c54f40c5adcffde8410b9e2d5336ea7

Merge "Select "move subpages" by default when moving a page"

Localisation updates from https://translatewiki.net.

Change-Id: I298e4b5253f73c1e2d3a385a4ea5eb1369e432aa

Merge "Fix doc for MediaWikiIntegrationTestCase::getTestSysop"

Fix doc for MediaWikiIntegrationTestCase::getTestSysop

Bug: T230670
Change-Id: Icd57ef2f6268d362a0912931911c46f93165d4ff

Convert MessageCache to service

Depends-On: Ia70e6c75f6e8a533f20cd44ebb05e013678e9951
Depends-On: I546eda0377f3a50843144b1450d3fbe8e4e02a8a
Change-Id: I305539a8598535a73e5cd280b2becdafa740ef97

Add recursion check to createService()

This will throw when trying to create a service while already in the
process of creating that same service, i.e., if there's a circular
service dependency. This would have saved me a whole bunch of debugging
time. :)

Change-Id: Id148d4f221f35f4069f3e0ab0069d13ca271df3d

Localisation updates from https://translatewiki.net.

Change-Id: Iee50e7133afa7491e1a29e5d4600bf99cc259dbd

Merge "skins: Hard-deprecate QuickTemplate::msgWiki()"

Prime connections and update description of McTest script

Change-Id: I4a53b5e4c02a9e2d1163ab9458dd309f651345bc

Merge "Fix typos in code"

skins: Hard-deprecate QuickTemplate::msgWiki()

Add wfDeprecated() call as MediaWiki 1.33 has been released.

Change-Id: Ied80b7b23bad21af113d4d6f6e19c22064e3125c

Merge "Remove title protection correctly for undeletions and imports"

Localisation updates from https://translatewiki.net.

Change-Id: I487111cd57cb185b6a9c3222e4527fca72214de3

Merge "Handle changed defaults in Argon2PasswordTest::testPartialConfig()"

Merge "Fix IDatabase::affectedRows() documentation"

Handle changed defaults in Argon2PasswordTest::testPartialConfig()

PHP 7.2.21 and 7.3.8 changed the default settings for PASSWORD_ARGON2I.
Load the default settings at runtime so the test can work.

Bug: T230487
Change-Id: I55a0f1af160c822113c9f86f8f8cce558da61736

RecentChangesUpdateJob: Fix GROUP BY

When a column specified in GROUP BY is both a table column and a
SELECT-field, MySQL and PostgreSQL both interpret it as the table column
rather than the SELECT-field.

In PostgreSQL this raises an error, which is good since it lets us know
it needs fixing. But MySQL goes ahead and groups by the table field
which gives us the wrong result.

Bug: T230618
Change-Id: Id500556b2795b86849329eece3b651b08e29a7f7

ActiveUsersPager: Fix outer GROUP BY

MySQL doesn't care, but PG actually enforces rules about GROUP BY
columns.

Bug: T230618
Change-Id: If769db525bf53c906ad3e0e1d1a6e92194f622a8

Merge "Remove unused variables from setupTemplate"

Check all slots in DerivedPadeDataUpdater::isCountable.

For example when the page is moved, the slots are not actually
modified, so countability of the page after the update should
not be based only on modified slots, but instead on all the page
slots.

Bug: T212706
Change-Id: Ie352ec9307b89b04af09ed85b542ca2907856bbf

Remove 'redundant' attribute from BaseTemplate::makeListItem() links
in case this is not done in SkinTemplate::buildContentActionUrls()

Not all skins are going to use buildContentActionUrls() in the first
place, as they may not be laying out the usual action tabs as, well,
the usual action tabs. Given that this is being added in core, it
should likewise be cleaned up in core.

Bug: T214202
Change-Id: I630410aea2852f7a72fea67bbf256b3561ea0f3f

Remove unused variables from setupTemplate

Both $repository and $cache_dir are never used, no other skin/
extension makes use of that, variables are defined just because of
PHP Strict standards.

IMHO there is no need to define those, all Skins that extend
Skin class define optional params so PHP will not complain.

Change-Id: I059415a795e5836b9399890ecfd4fb5ee1888ac3

Remove title protection correctly for undeletions and imports

Undeletions and imports can both produce "new" pages with more than
1 revision. The fact that $options['created'] for the associated
DerivedPageDataUpdater instance is unexpectedly changed to false is due
to the fact that the latest revision often has a nonzero rev_parent_id.
This causes WikiPage::onArticleCreate not to be applied correctly, and
so title protection is not correctly removed. The part about "created"
being forced is therefore being fixed.

Bug: T200088
Change-Id: I8c899893486976285eb6fac87d641e93a616676d

Merge "Introduce TitleParser::makeTitleValueSafe()"

Fix IDatabase::affectedRows() documentation

Bug: T229456
Change-Id: I992e81e5dbcd9db74644b2990044cf685567db5a

Merge "Comma list handled via CSS in change list pages"

Merge "rdbms: ignore DBO_NOBUFFER flag in IDatabase"

Comma list handled via CSS in change list pages

This avoids unnecessary commas in the mobile interface.

Bug: T230564
Change-Id: Id486e9a5b8fd337c41c8749d0462a40d6e68dffb

Merge "Add Parser::resetOutput() and getStripState()"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I4f911ecc1aec466dabc18fd76853bc2b1defb906

Merge "Correct English grammar in linkstoimage"

Merge "parser: use 'vary-revision-sha1' in Parser::statelessFetchTemplate"

MySQL connect: Parse the Unix domain socket path correctly

Some Unix domain socket paths contain colons, requiring MediaWiki to use a modified parsing delimiter. For example,
Google AppEngine provides a /cloudsql/ path to a Unix domain socket used for accessing a Cloud SQL database.
See https://cloud.google.com/sql/docs/mysql/connect-app-engine
The connection name takes the form of project:region:database which introduces two additional colons to the socket path.
Previously, the code presumed the existence of only one colon in a valid Unix domain socket path.

Bug: T230306
Change-Id: I2411ac1b583fd5c143563cb3155ef4e8a29a9b08

SECURITY: Add permission check for suppressed account

Bug: T230402
Change-Id: I6a13859be81e5c746bdf0993eb5416fecdac2306

RCFilters: add namespace filters for all contents and all discussions

These filters provide a shortcut to see or highlight all entries associated
with a subject or a talk namespace.

They are defined as real filters on the client-side and are superset
of all other namespace filters.

They are expanded to the list of namespaces they represent on the server-side.

Bug: T201520
Change-Id: Id9e04a38c2d7c8165c27af98959aa5715225faa8

Merge "API: Always select rc_user from database (regardless of rcprop=user)"

rdbms: ignore DBO_NOBUFFER flag in IDatabase

Callers should use query batching instead. Without buffering, either
the RAM burden is shifted onto the RDBMS server (with MVCC vacuuming
also being stalled) or the RDBMS server has to keep adding locks as
the cursor advances. Also, if a caller does not read all the results
(possibly due to an exception), then the SQL commands sent/read get
out of sync, which is too fragile.

There are no DBO_NOBUFFER callers in WMF gerrit repos.

Change-Id: I3712149633d0f01bb6990e324e53dd58abba9cfd

rdbms: deprecate seldom used bufferResults() method

Change-Id: If67c91fa020f09a89fe2dd50fe4ad30b75d676ef

Add rate limiter to Special:ConfirmEmail

General hardening measure for Special:ConfirmEmail, similar to what's
already in place for Special:ChangeEmail.

Bug: T226733
Change-Id: I465e4748840e214531e930608386455084563bc6

Merge "Add some tests for DerivedPageDataUpdater::isCountable."

Use injected services instead of static method

Depends-On: I4b915244ceee4c1857178dd68dcdf57f1ee32200
Change-Id: I053894de3665c6f8f505ee6191f2ade843db62c0

Merge "Localisation updates from https://translatewiki.net."

Merge "Use ExtensionRegistry instead of class name"

Merge "Drop last bits of oracle"

Localisation updates from https://translatewiki.net.

Change-Id: Ia9d0fee0c299f4f610acbc7cf151f3010f6b8e17

Add some tests for DerivedPageDataUpdater::isCountable.

Change-Id: I82ed6fd604406d02faf4e966ec4845c8bc779a8b

Merge "Add help links to special pages"

Merge "Rename variables for Title objects in PermissionManager"

Drop last bits of oracle

Undefined constant

Change-Id: I66f17a7647bdcf254695dd0d3a0d49d41a35fff3

Merge "Add drop of mssql/oracle to RELEASE-NOTES"

Add drop of mssql/oracle to RELEASE-NOTES

Bug: T230418
Change-Id: Ie7771853048a27a8d8ee75764229e6e34477cd04

Merge "Allow basic grant holders to autocreate accounts"

Merge "maintenance/populateChangeTagDef.php: Handle missing valid_tags table"

Merge "More clean up of oracle/mssql"