Merge "Set the DatabaseDomain in some tests classes for sanity"

Merge "Remove dangling $wgDBmysql5 usage from DatabaseMysqli"

Remove dangling $wgDBmysql5 usage from DatabaseMysqli

Change-Id: Iee21655e12eb3001496123ff8438adc1c39ecdf8

Merge "Check Database::mSessionTempTables in Database::tableExists()"

Check Database::mSessionTempTables in Database::tableExists()

Also make the temp table tracking catch plain "DROP TABLE"
in addition to the stricter "DROP TEMPORARY TABLE" clause.

Bug: T146300
Change-Id: Ia8306ec25e63adcdcf0dcc8f6a700dd01afdc948

Merge "Database: Behave correctly when inserting booleans"

Merge "mw.api.upload: Suppress default API call timeout"

Merge "Make LBFactory::waitForReplication() use "domain" instead of "wiki""

mw.api.upload: Suppress default API call timeout

mw.Api sets a default timeout of 30 seconds, meaning that if the
request takes more than that, it will be aborted. This is reasonable
in most cases, but file uploads can easily take longer than that.

Bug: T117545
Change-Id: I086108421d229c60f5e9d5c9edb70150d875dcb0

Merge "Add sanity check to LoadBalancer::setDomainPrefix()"

Database: Behave correctly when inserting booleans

Pretty much everything seems to assume that PHP booleans should be
converted to 0/1: MySQL does this implicitly thanks to the lack of
strict mode by default, while PostgreSQL and Sqlite (and Mssql) do it
explicitly.

The addition of MySQL strict mode for unit tests in Ib2873913 exposed
the assumption in the case of MySQL by making some extension unit tests
fail. So let's make casting bool to int the default behavior of
Database::addQuotes().

This also cleans up the phpdoc for Database::addQuotes() to properly
reflect all the supported types that can be passed to it.

Change-Id: I13d0e402fa676bc27c345e8ac12f363ebc627f6a

Merge "Remove .mw-help-field-hint and -data CSS classes from mw.legacy/shared"

Remove .mw-help-field-hint and -data CSS classes from mw.legacy/shared

No longer used anywhere.

Also removing the images used in these classes.

Bug: T89981
Change-Id: I665cfee4e10a50789ce2260983ff6f623bb4c9e2

Add sanity check to LoadBalancer::setDomainPrefix()

Fixed some errors that popped up in CI:
* Also cleanup $domain handling in reuseConnection().
* Fix empty string handling in openForeignConnection() where
  the empty string check against $dbName failed since an empty
  string $domain results in $dbName being null.

Change-Id: Ie78fefa1acb401fe4e8bdc96b75053692aa0a925

Add table prefix in new MySQL tableExists()

Bug: T146289
Change-Id: If73907f69471a6a91e1db7131a57439f950c5d8c

Set the DatabaseDomain in some tests classes for sanity

Change-Id: I6531dc6cf89fbe7e5656354bcd4a27369f573752

Make LBFactory::waitForReplication() use "domain" instead of "wiki"

Change-Id: Id7e55fddb96631b3938ee6ddcfcc3ce13ba942d4

Fix @return of LBFactorySimple::getExternalLB()

Change-Id: If5353af7c4ec3781597159c12481444159c463f3

Merge "ChronologyProtector: Remove unused 'use MediaWiki\Logger\LoggerFactory'"

Merge "Use getConnectionRef() in ExternalStoreDB to avoid connection leaks"

ChronologyProtector: Remove unused 'use MediaWiki\Logger\LoggerFactory'

Change-Id: I54a0d17605d3bc25c1d78a3a606e371fa74eaab0

Use getConnectionRef() in ExternalStoreDB to avoid connection leaks

Change-Id: I3df270241653841a4fb2f65e240aae9d2e0b650f

Add some missing reuseConnection() to LoadBalancer

Change-Id: I99099f909e7759c061d44ad4e1ed049e038a4d7d

MultiHttpClient: Don't use wfArrayToCgi()

http_build_query() should be functionally identical.

Change-Id: Ia7d909e97a89d66dbe5187e9a0423e2ecf8561e8

Avoid Wikimedia\Assert in HashBagOStuff

This is the only usage of wikimedia/assert in the planned objectcache
library, so it doesn't make much sense to have a dependency upon the
entire library just for one usage that is trivially replaced.

Change-Id: I8f967fb2b861852c7fc74f7d42e2cbab1df05bc4

Merge "Fix importation of weird file names in importTextFiles.php"

Fix importation of weird file names in importTextFiles.php

When importing a file whose name contains a #, the script would call
e.g. Title::newFromText( '#foo' ), which succeeds (because titles
are allowed to contain fragments) but causes problems when trying
to create the revision.

Also avoid fatals on actual invalid titles.

Bug: T142675
Change-Id: I6b4c8fd8dd09db14c0704c74137e112b292c964a

Merge "Move XMP* classes to includes/libs"

Merge "XMPValidate: Use ConvertibleTimestamp instead of wfTimestamp()"

Merge "mediawiki.requestIdleCallback: Avoid native window.requestIdleCallback"

Move XMP* classes to includes/libs

These no longer have any dependencies upon MediaWiki and can be split
out into a separate library.

Tests were updated for the new location and to no longer depend upon
MediaWikiTestCase.

Bug: T100922
Change-Id: I6ad507fa883d5934b807f9e31c507659d0730b52

XMPValidate: Use ConvertibleTimestamp instead of wfTimestamp()

This removes the last MediaWiki dependency in the XMP* classes, and they
can now be moved to libs/ (in a follow up patch).

Change-Id: Iee509fce0e44ec0697e1cdd4cd711846cf35e1e6

Merge "Toolbox: Mark RecentChangesLinked link as nofollow"

Merge "Improve lag/flag behavior and docs in LoadBalancer"

Toolbox: Mark RecentChangesLinked link as nofollow

Bug: T146233
Change-Id: I2dd0b3a179dc0d1d7210d98748f45e75f953d2bf

Merge "Use Status::isOK() method in FileDeleteForm"

Merge "jsduck: Remove custom @source tag"

Improve lag/flag behavior and docs in LoadBalancer

* Set DBO_DEFAULT if "flags" is not set. MediaWiki config bootstrapping
  does this, but for outside users, it should still be the default too.
* Make "is static" do more than no-oping DatabaseMysqlBase::masterPosWait().
  It now also treats servers with this flag as having 0 lag at all times.
  This makes it far more useful, as the point of the flag is to mark DBs
  that are archival clones of each other and may not even replicate.
* Add more ILoadBalancer usage documentation.

Change-Id: Ib0b44e42e9f4c57a292d6a6ea17beb454db464bc

Merge "Mention the lock name in DatabaseMysqlBase lock() errors"

Merge "Remove Database clone call from DatabaseTests.php"

jsduck: Remove custom @source tag

Only used in external.js to render a simple link. These methods
don't have a regular description and as such the link renders
effectively the same way in the same place without `@source`.

Change-Id: I83cd524b6b53e67c5c2d18258c804ebd49a29b74

Mention the lock name in DatabaseMysqlBase lock() errors

Also use warning() instead of debug()

Change-Id: I9f5b331072caf29985bfb4319786cc2c5cddf5dd

mediawiki.requestIdleCallback: Avoid native window.requestIdleCallback

Follows-up 482ad8d9fb.

This currently has bugs in Chrome that can cause it to not get called
for up to 30 seconds.

https://bugs.chromium.org/p/chromium/issues/detail?id=647870

Change-Id: I221cd6c2bcdbee89bc7401c071f2ab1865a738ed

Remove Database clone call from DatabaseTests.php

Change-Id: I60a22de9ccc7253abc9bbefe2a4f38675655ee4f

Merge "FSFile and TempFSFile cleanups"

Merge "Move some FileBackend related classes to /libs"

Use Status::isOK() method in FileDeleteForm

Change-Id: I3c21ee1f0ecc4077772ddba6e740a67ed143ece0

Merge "Only send DB query profiling to MWDebug::query"

Merge "Avoid TransactionProfiler notices for TEMPORARY table writes"

Only send DB query profiling to MWDebug::query

This should avoid massive dberror log spam in jenkins.

Change-Id: Ifa60843e78b99921768e7a62109c505ff9330e41

Add GENDER support to email and user rights toolbox links

Also rephrasing the "User rights management" link to an action-based
phrase, using the more correct term "user groups" rather than "user
rights".

I filed T142322 about the inconsistent use of terminology in regards to
Special:UserRights.

Bug: T142316
Change-Id: I43c3fbe55e0065eee23294738745aa69d3f05202

Merge "Improve Database::__destruct() and add a __clone() method too"

Merge "Remove OutputPage->mDebugtext"

Merge "Fix variable name in release notes"

Merge "Emit deprecation warnings from Article::fetchContent"

Improve Database::__destruct() and add a __clone() method too

* Close dangling connections in LoadBalancer/Database destructors.
* When DBs are cloned, create new connection resources for the clone
  so the two do Database objects don't clobber each other.

Change-Id: I3adb57cbb1fdc2a17e6d95389d0562ef22701576

Fix variable name in release notes

Change-Id: I7da512f4aa68c80dd08d93910888be1b49783912

Remove OutputPage->mDebugtext

Deprecated since 1.20, unused anywhere.

Change-Id: I4316b77c48e65faf788de60be28fc9cf65c785f1

API: Remove deprecated methods

All deprecated ApiResult methods are removed. These have been deprecated
since 1.24 or 1.25, and the only users remaining in Gerrit are wrapped
in backwards-compatibility checks and so should not be being called.

ApiBase, ApiFormatBase, ApiMain, and ApiQuery methods for generating the
pre-Ib14c00df help text are removed. Nothing has called these for a long
time, and only Flow implemented them in any way. Deprecated methods for
providing the text for such help, such as getDescription(), haven't been
removed yet, though, since some extensions still call some of them.

Change-Id: I3ca7c98174b4a3f6f67f2b023e0f4446637e7a84

Merge "Pass User to SearchEngine::getProfiles"

Pass User to SearchEngine::getProfiles

Useful for search engines that allow users to customize search profiles.

Depends-On: Icd577c8ebc6e162befe30bde4fe276e633d2e434
Change-Id: I471cd090730d2a25cb70d622ec3bebbe9583118c

Localisation updates from https://translatewiki.net.

Change-Id: I83c8f60a9022018fae1996910242f4514cf8cd3f

Merge "Fix minor SqliteInstaller IDEA warning"

Merge "Set an initial dummy domain in Database::__construct()"

Merge "Deprecfate WikiRevision::getText"

Merge "Clean up redundant defaults in Database::__construct()"

Clean up redundant defaults in Database::__construct()

* This are already (or now) managed by Database::factory().
* Also remove unused "foreign" parameter.

Change-Id: I9161b9f158b656fd10b1506ecb82bf42ab94ea43

Merge "Fix IDEA Ultimate warnings in ChangeTags"

Fix IDEA Ultimate warnings in ChangeTags

Change-Id: Ie0778e5e32be81bda246d231526630db700b8538

Add debugging logging to LoadBalancer connection closes

This matches the logging used for opening new connections and can be
useful for seeing connection leakage.

Change-Id: I8d24d059062f3b698e64751b46542eb1fa78bb36

Set an initial dummy domain in Database::__construct()

This avoids errors when LoadBalancer calls getWikiId() in
reallyOpenConnection() after a failed connection.

Change-Id: Idc3d93ae39be09f2da277df57cdc5a564cb3408d

Merge "Improve tableExists() for MySQL"

Improve tableExists() for MySQL

Use SHOW TABLES to avoid log spam when the table does not exist.
This should make the CLI jenkins log a bit more readable.

Change-Id: I027a9d0af0e42959e9f03cebb295218068a263d3

Emit deprecation warnings from Article::fetchContent

Bug: T145734
Change-Id: Ia70dedc729fc4724b773a29d8789ace344ae0df9

Merge "Extract replacePrefixes into a static method"

Deprecfate WikiRevision::getText

Bug: T146162
Change-Id: I60249d687ac74baca146ed28d57c9a8ebcd79fba

Merge "WikiPage::getDeletionUpdates returns DeferrableUpdates"

Merge "phpdoc Title::inNamespaces also allows array"

phpdoc Title::inNamespaces also allows array

Change-Id: I201dbaf9c0dee5b47c4edde73ff68b88560e8a0f

WikiPage::getDeletionUpdates returns DeferrableUpdates

This means the hook also only needs to add
DeferrableUpdate object rather than DataUpdate objects

Change-Id: Ic19c93860aa364d4e1b69a20ed6bbcd61f6f4979

Merge "Replace $wiki with $domain in ILoadBalancer"

Merge "Make rebuildFileCache cover ?action=history"

Make rebuildFileCache cover ?action=history

Also simplified the logic slightly

Change-Id: I6145d52b6b701735fa4bd8e41e07fb2bf6fdcee3

Replace $wiki with $domain in ILoadBalancer

Change-Id: I1752d3358a8fc4314dfd497f3e08d3d5a6bfdd39

Merge "Include setTableAliases/setDomainPrefix in ILoadBalancer"

Merge "Database class parameter and documentation cleanups"

Include setTableAliases/setDomainPrefix in ILoadBalancer

Also remove redundant LoadBalancer documentation.

Change-Id: Ib9a678349cf850f1cec498a8534e546bf9f41386

Database class parameter and documentation cleanups

* Document various parameter arrays.
* Fix $user = false loophole in Database::__construct().
* Set the Postgres port *before* calling super, as it is
  needed by open().
* Remove 'chronProt' parameter as it is lazy-loaded.

Change-Id: Icc1037efa1eee7ae6fdd2919f60001e6e29ae55c

Merge "mediawiki.UI: Make colors' contrast compliant to WCAG 2.0 level AA"

Avoid TransactionProfiler notices for TEMPORARY table writes

* Make Database detect and track temporary tables for the session.
  Creating, dropping, and updating them do not count as "write" queries.
  Even a read-only mysql slave can have these operations performed on it
  and it can be useful for complex read queries that need temporary results.
* Rename handleTransactionLoss() to handleSessionLoss() and cover named locks too.

Bug: T145947
Change-Id: I826439e9e9f550f32a9c46b3dd60e8e8015aa274

Merge "Fix undefined getDelimiter() call in sql.php"

Change DatabaseBase::factory call to Database::factory in LoadBalancer

Change-Id: Id4bbc08b5aa913e2e3e47a7873db600fe957c985

Merge "Remove unused DatabaseBase::debug() method"

Merge "Fix broken $wgExternalServers example setting"

Fix undefined getDelimiter() call in sql.php

* Fixes regression from 4ef4438739.
* Also fixed numerous IDEA warnings.

Change-Id: Id41faff835ca25399ce1ceb10a19d19ffc914689

Merge "Fix "externalCluster" LBFactorySimple setting in LBFactoryMW"

Fix broken $wgExternalServers example setting

Just reference $wgDBservers instead, which uses the same format.

Change-Id: Iaa32cb067f9487b459925310ac4c52efe3f4a3f1

Fix "externalCluster" LBFactorySimple setting in LBFactoryMW

Fixes regression from d175b391ae2a

Change-Id: I4bbc875e2fcd021e342959a37475ec356cb58cce

mediawiki.UI: Make colors' contrast compliant to WCAG 2.0 level AA

Establishing Web Accessibility Guidelines level AA color contrast
compliance by aligning to improved color palette, similar to
I6fdb90af8b9dc5e5e026eb0c1bd13138c73da4cd

Bug: T109915
Change-Id: I191f8ec731eeef0827124741200c51cb42ba99f8

Move DatabasePostgres to /libs/rdbms

Fixed all IDEA warnings in the postgres classes too.

Change-Id: I06b4c5b2c581fb65552d980cc106aa10fed40285