API: Warn about POST without Content-Type

HHVM sniffs the content type and so probably correctly guesses
application/x-www-form-urlencoded and fills in $_POST, while PHP 7 does
not sniff and so doesn't fill in $_POST.

Since there are clients that have been expecting the HHVM behavior, give
a warning both to let them know and so we can have some idea of how many
such clients there are.

This also adds any warnings and errors as an HTML comment at the top of
the auto-generated help page, to hopefully make it easier for people
getting that unexpectedly to find out why.

Bug: T230735
Change-Id: I017b7afe808844d74d376f6436894a5a2f525a9f

Merge "Use injected services instead of static method"

Merge "Convert MessageCache to service"

Merge "Add recursion check to createService()"

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()"

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

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"

Merge "Clear test user registry between tests"

Introduce TitleParser::makeTitleValueSafe()

This works exactly the same as Title::makeTitleSafe(), but for
TitleValues. 100% test coverage of splitTitleString(). Also added
identical parallel tests for Title::makeTitleSafe() to verify that
behavior is the same.

Along the way, I discovered that TitleValue doesn't allow an empty page
name even if there's an interwiki prefix, which is wrong, so I fixed it.

Bug: T220966
Change-Id: I4b915244ceee4c1857178dd68dcdf57f1ee32200

Use ExtensionRegistry instead of class name

Use the ExtensionRegistry instead of checking for the main class
name of the Cite extension. - I'm not entirely sure if this is much
better. A hook would still be the better option. But at least Cite
could be refactored and the class name changed.

Bug: T89151
Change-Id: I35e5aa9955141b575de68a5be2c0d5b87585eb77

Merge "Set @return-taint of Sanitizer::stripAllTags to tainted"

maintenance/populateChangeTagDef.php: Handle missing valid_tags table

The valid_tags table was added in MW 1.15 and removed in 1.33. If
someone is trying to upgrade from a really old version of MediaWiki,
valid_tags may not exist even though change_tags.ct_tag does.

Bug: T230317
Change-Id: Ib94542878b8d301c1d7f806405ee39838ebc6d4a

More clean up of oracle/mssql

Bug: T230418
Change-Id: I1ff01548e55f01f9419668e31d5aa7dd03d23258

Merge "Tests: Allow vendor/bin/phpunit from within extension directory"

Merge "DefaultSettings: Document wgTmpDirectory guarantees and expectations"

Drop Oracle and Mssql

After approval of RFC T191231, we are going to drop oracle and mssql
and it will be possible to bring back the support using the abstract schema

Adding to release notes will be done in a follow-up

Bug: T230418
Change-Id: I90bd5cfcc3e18011b193c965fdb1fa54675040b5

Tests: Allow vendor/bin/phpunit from within extension directory

Prior to this patch, one could not run this command from within an extension
directory: `../../vendor/bin/phpunit -c ../../phpunit.xml.dist tests/phpunit`,
because our code in bootstrap.php loaded "LocalSettings.php" instead of
$IP/LocalSettings.php.

Note that making this change does not result in loading LocalSettings.php when
running unit tests.

Bug: T87781
Change-Id: Iee8440f11a2bb255567646289a1986d636c12765

Clear test user registry between tests

Otherwise User members will wind up retaining stale references to
services. The more things move to services, the more state we need to
reset between tests to avoid subtle and confusing test failures!

TestUsers can't be created if the DB prefix is not either unittest_ or
ut_, which caused failures in RCFeedIntegrationTest.php with this change
now that it was trying to create a new TestUser. Fix is to set the
prefix to one of those two instead of empty.

Change-Id: I41f87e1acffe94361748ef4ab69c290de587e6be

Localisation updates from https://translatewiki.net.

Change-Id: I6bf6f5827c4cb7527902f380e342bfb1263e83c3

Set @return-taint of Sanitizer::stripAllTags to tainted

phan-taint-check (aka SecurityCheckPlugin) doesn't recognize
Sanitizer::stripAllTags' output as tainted in certain situations.
Adding a @return-taint of tainted to ensure that it does, which
may result in the reporting of more issues.

Bug: T230234
Change-Id: I357c168417a26882c7c460df20f36ec2be401096

Merge "ApiFeedContributions: Throw if the username is invalid"

ApiFeedContributions: Throw if the username is invalid

Bug: T230239
Change-Id: I4141047c8f1ff73665b79a27a7c5eb995c52ea88

Merge "Don't use new MCR schema without using DB"

Merge "Allow spaces in TitleValue constructor"

Don't use new MCR schema without using DB

If $wgContentHandlerUseDB is false and
$wgMultiContentRevisionSchemaMigrationStage is not SCHEMA_COMPAT_OLD,
RevisionStoreFactory::getRevisionStore() throws. This is coming up in
some seemingly unrelated code changes, perhaps due to access of stale
service objects, but I'm not sure because I can't reproduce locally. So
this is a shot in the dark to fix it.

Change-Id: Id29a62e1f537fa1b2016aac396773b728e238cda

Localisation updates from https://translatewiki.net.

Change-Id: I71d2db842bf204a483e3c9b4695cbf744bca2cf2

Fix typos in code

Bug: T201491
Change-Id: I4e236ed2dc7a4d0c4aae43167a01e710983d9e27

Add Parser::resetOutput() and getStripState()

For Parsoid's convenience, add to Parser resetOutput(), getStripState()
and a $revId parameter to startExternalParse().

Parser::resetOutput() allows Parsoid to get the ParserOutput of each
template without resetting the DOM cache.

Change-Id: I084676acdc8db185b8efd9b97b7d5326afd2a997

Merge "objectcache: clean up MemcachedBagOStuff expiry handling"

Merge "Add german alias for displaytitle_noerror/_noreplace"

RedirectSpecialArticle: Avoid double redirect for action=edit&redlink=1

If you visit `Special:MyPage?action=edit&redlink=1` and your user page
exists, this would redirect to `User:Yourname?action=edit&redlink=1`,
which then would redirect further to `User:Yourname`. Just redirect
there directly.

Additionally, the double-redirect seems to somehow conflict with the
fake-redirect code for $wgHideIdentifiableRedirects. I couldn't figure
out why this happens, but you end up on a blank page and no redirect
of either kind occurs.

Bug: T229794
Change-Id: I952fc9df91b1243db22f956ec09bee457c8a21bf

Allow spaces in TitleValue constructor

Change-Id: I809731508036409fda72b271aac8a37f3ac1ef2d

Add german alias for displaytitle_noerror/_noreplace

Reuse the values from defaultsort_noerror/_noreplace

Change-Id: I8bef5920040d10128d94e5131bd7480e762ca928

Merge "Add german alias for special page NewSection"

Fix Special:NewSection showNoRedirectPage form handling (3)

Handle invalid titles by using `newFromTextThrow` and
returning a fatal status if the title is invalid.

Bug: T207577
Change-Id: I2bdfcdbcb7ee41f96122e7f8aa561f6eee34be7b

Add german alias for special page NewSection

Change-Id: Ibdd838d53f4b323dce9c1438b7d13e7289ce570f

Localisation updates from https://translatewiki.net.

Change-Id: I60e49dd26845b81e64b20bb386ee23bea73713c1

Merge "Inject LoadBalancer into GenderCache"

objectcache: clean up MemcachedBagOStuff expiry handling

Partly a follow-up to 88640fd902900.

Use real time in changeTTL() tests to fix all remaining
failures for BagOStuff sub-classes.

Change-Id: I537d665d6c8770a68a5a79233a913f1714881dfb

Merge "rdbms: remove DatabaseSqlite::checkForEnabledSearch() in favor of explicit queries"

Localisation updates from https://translatewiki.net.

Change-Id: I34b028ec7073f2b146654992d0eff6a221bd1247

Merge "resourceloader: Rewrite the minify.php maintenance script"

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

Bug: T226785
Change-Id: Ib306cdcbc939553ea318612cb5273d6b77c5e21c