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

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

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"

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

Remove unused DatabaseBase::debug() method

This just trivially wrapped getFlags()/setFlags() anyway

Change-Id: Iab5b508f6746098b66da00c8bf038215cb2359ef

Merge "Remove strictIPs(), cascadingDeletes(), and cleanupTriggers()"

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

Remove strictIPs(), cascadingDeletes(), and cleanupTriggers()

These DatabaseBase methods have no callers are were a mistake to
begin with. They only encouraged code forking and incompatibilities
among different backing DB types.

Also removed stray realTimestamps() method that was left over
from prior removals.

Change-Id: I37bef8585151e2782d2c88b5ae1e6752109c6ac7

Localisation updates from https://translatewiki.net.

Change-Id: Ia226031f53b3b32abf5cec62bda2c90548ee940b

Merge "Fix SQLite $wgObjectCaches definition + b/c handling"

Fix SQLite $wgObjectCaches definition + b/c handling

Follow-up to 47e816c69c197fec86c344256d81d7c8f781b880

Change-Id: Id94857a7c8726871cf826a34b5cf10cb465b7125

Merge "Clarify comment in traditional gallery"

Merge "Avoid using cascadingDeletes()/cleanupTriggers()"

Clarify comment in traditional gallery

Change-Id: I13aa2132d64a1dca8a5ab867fdffc48aca2cf9d1

Merge "Avoid calling awful DatabaseBase::strictIPs() method"

Merge "Fix SqliteInstaller DatabaseSqlite creation"

Fix SqliteInstaller DatabaseSqlite creation

Follow-up to 47e816c69c197fec86c344256d81d7c8f781b880

Change-Id: I1612b0a187ea1df272bdd304ffb2bafbc006b2e4

Fix "MWExceptionRenderer::getText() should not be called statically" errors

Change-Id: Icfb65795b1455217ca9bc2c1f7cd453144ad5106

Merge "Html: Update list of $voidElements"

Merge "Cleanup ServiceWiring/LBFactoryMW interaction"

Make MWExceptionRenderer work with PHP7's exception changes

See https://wiki.php.net/rfc/throwable-interface

Change-Id: Ib65f912580f5d84997c5926e7f602d5f5ef3faba

Html: Update list of $voidElements

`<command>` is nowhere to be found in current HTML5 specification.
Scarce documentation on the internet hints that it has been removed.

Change-Id: I2a704194c7e8f8ca307f9d97c7f47a47cfaf00a6

Cleanup ServiceWiring/LBFactoryMW interaction

* Move almost all the code to LBFactoryMW and inject the main config.
* Make LBFactoryMW no longer extend anything, which is now pointless.
* Let site admins explicitly set "servers" and "externalServers" arrays.
* Pass in the $wgDBschema field regardless of $wgDBtype. It defaults to
  null, so no one would bother setting it if they did not want to use it.

Change-Id: I51f15c0f5d98a73907c51958bdb82dea76b3e38c

Merge "Various small cleanups to ResultWrapper"

Merge "Move LBFactorySingle to /libs/rdbms"

OOjs UI: FieldsetLayout: Styling fixes for `<legend>` labels

[Backport from OOjs UI master as this fixes a significant regression.]

* Make the `<legend>` 'display: inline-table' rather than 'table'.
  A block-level table would push down elements that follow it,
  including the "help" icon, which in turn messed up the fields.
  Fixes T145682, T145678 and first half of T145680.
* Only style .oo-ui-labelElement-label with margin and padding
  if we have the .oo-ui-labelElement class (label is not empty).
  Fixes second half of T145680.

Follow-up to 9cf8b941a9c618f1d7ba2512508f576f70116e1e.

Bug: T145682
Bug: T145678
Bug: T145680
Change-Id: Id95f4167afa6c000d6d2fa2ad7a62b65b6d4d53b

Merge "Add Localisation to the links, add the link to Localisation  in Languages/Language.php"

Stub out DatabaseBase::getSearchEngine() as a dummy method

SearchEngineFactory::getSearchEngineClass() should be used instead

Change-Id: I984b80815633a8dc38a76d7de8a46ec249ae729f

Extract replacePrefixes into a static method

Useful for some search engines that have a keyword that wants
to reuse this logic without building a new SearchEngine object.

Change-Id: Iee5bfd1da70b8339a98555ba062bd33b21f0b761

Merge "Make ApiMain use isBot() to catch global bots in checkReadOnly()"

Various small cleanups to ResultWrapper

* Make FakeResultWrapper call super to avoid IDEA warnings
* Lots of documentation cleanups

Change-Id: Ifcd7163890fa8c7718e88c4244cc38371ed624fc

Merge "Move LIST_ constaints to IDatabase class constants"

Merge "Move DatabaseSqlite to /libs/rdbms"

Merge "Fixed method name casing and IDEA warnings in FileRepo"

Move LIST_ constaints to IDatabase class constants

Leave aliases in Defines.php for backwards compatibility.

Change-Id: I88b45e0943cbfe97d863c2e0a4911fc0f81e5bb5

Fixed method name casing and IDEA warnings in FileRepo

Change-Id: I463885a9f425491cd040cb1cdbc4c16925522848

Move DatabaseSqlite to /libs/rdbms

Change-Id: Ibda20041803deb922b7e3803cba6fc8d15214169

Merge "Remove wf* function and global variable usage from DatabaseSqlite"

Remove wf* function and global variable usage from DatabaseSqlite

Change-Id: I81753621ecd1e7e3b099243b4c4d0f3428bf9ae2

Cleanups and IDEA warning fixes to FileRepo classes

Change-Id: I98247b299c31b93df183981adc517187cb4aefe5

Make LockManager use StatusValue and move classes to /libs

Change-Id: Ifa41fc2939f3515d4a056746b0fcbff79786d25b

Merge "Remove wf* function dependencies from FSLockManager"

Make ApiMain use isBot() to catch global bots in checkReadOnly()

Change-Id: I2090b1aad8a5858965e4ec261f8e88c37414e804

Remove wf* function dependencies from FSLockManager

Change-Id: I52f08c6e7372ddbbcc1b5f82d505e435b01ff138

Localisation updates from https://translatewiki.net.

Change-Id: Id7a10255e2d68819e0de0c005a4442a43f426502

Avoid calling awful DatabaseBase::strictIPs() method

That method should be removed since it encourages incompatibility

Change-Id: Id8a0174266bcd988c85943507f1edb3518fe087e

Move LBFactorySingle to /libs/rdbms

Also add newFromConnection() methods for cleaner looking construction
since the IDatabase is fundamental and required.

Change-Id: Ife478d045cee92a30c91e9004833a557488bd512

Merge "Remove long-since unused cruft methods from DatabaseBase"

Merge "Fix file/class name mismatch from e2adcc89bbe"

Avoid using cascadingDeletes()/cleanupTriggers()

Simplify DB callers by just having one code path.
All but some very old code paths bothered with these.

Change-Id: Iaf7a2f83146a0ed15995f9cfc74edcf16ae5a448

Merge "Avoid MWDebug usage in DatabaseBase"

Fix file/class name mismatch from e2adcc89bbe

Change-Id: Ieac65f29a13612cecd421b62e1cb3a3f47552bb6

Avoid MWDebug usage in DatabaseBase

This class is in /libs and cannot depend on all of MediaWiki.
Replace the call with a simple debug() call instead.

Also, make the legacy logger route/format errors from the two
new DB log types (DBConnection, DBQuery) to the old wfLogDBError
locations, including MWDebug::debugMsg().

Change-Id: I64895d3f5b9a000d8186ab6a6ffb4b76a7e9ff40

Merge "Move LoadBalancerSingle to /libs/rdbms"

Move LoadBalancerSingle to /libs/rdbms

Change-Id: I2e8177ca8fa6b9fcdec87e44f0cfeb11bb71f2bc

Merge "Move LBFactoryMulti to /libs/rdbms"

Remove long-since unused cruft methods from DatabaseBase

Change-Id: I4b28c15a8589504bdbbe79ba0d3aed1e52dbe65b

Move LBFactoryMulti to /libs/rdbms

Also rename $wiki => $domain and lastWiki => lastDomain.

Change-Id: I2a30f42721715c56f5cddf9f07c10e8de70addd1

Merge "Move mysql database classes to /libs/rdbms"

Merge "Clean up $wgDBmwschema handling and only apply it to mssql"

Clean up $wgDBmwschema handling and only apply it to mssql

Follow-up to 61762bf3f8f904bcc5a0f90985e7f787756032a7

Change-Id: Ib8dc247ba17d7cbf4cf0f46dbe82983a715f2c8d

Move mysql database classes to /libs/rdbms

Change-Id: I1ba1799deec10e5a8ab8829b69e0708626e723e2

Merge "Make Status extend StatusValue and start FileBackend update"

Move LBFactorySimple to /libs/rdbms

* Refactored LBFactory a bit to make this possible.
* Move newChronologyProtector() up to LBFactory and
  make a lazy-loading method instead.
* Move appendPreShutdownTimeAsQuery() up to LBFactory.
* Inject the web request values for LBFactory from Setup.php.
* Remove unused laggedSlaveUsed() method.

Change-Id: Ie8a38a6f4d6359680eb6a5be24a34e30b9816479

Merge "Inject Profiler into Database objects"

Make Status extend StatusValue and start FileBackend update

* This avoids code duplication.
* Callers can safely start type-hinting StatusValue as well.
* Also moved the wrap() logic out of Status::__construct(), which
  only wrap() was (and should have been) using. Use "static" as
  well, so subclass behave properly.
* The docs and type-hints in /filebackend are updated as an example.
* A migration pattern is to inject a StatusValue wrapper
  into the backend and use it on all returned statuses, so MediaWiki
  will still get Status for message methods.

Change-Id: Iff9255f34870ea6b0c4b91f6ddc69eea95186aba

Merge "registration: Support 'ServiceWiringFiles' in extension.json"

Merge "Make database classes handle hyphens in $wgDBname"

Inject Profiler into Database objects

* This removes the Profiler::instance() dependency from
  a class in /libs.
* Also removed unused setTransactionProfiler() method

Change-Id: Iec735a130914090dc51aed30a283565a66aabee6

Make database classes handle hyphens in $wgDBname

* Add DatabaseDomain class to handle passing domains around.
It also can be cast to and from strings, which are of the same
format as wfWikiId() except with hyphens escaped.
* Make IDatabase::getDomainID() use these IDs so they can be
passed into LoadBalancer::getConnection() and friends without
breaking on sites with a hyphen in the DB name.
* Add more LBFactory unit tests for domains.

Bug: T145840
Change-Id: Icfed62b251af8cef706a899197c3ccdb730ef4d1

Merge "LoginSignupSpecialPage: Reduce hackiness of "You are already logged in" warning"