Remove hard deprecation of PasswordPolicyChecks::checkPopularPasswordBlacklist

It's causing quite a bit of WMF log spam because it's still used in wmf-config

Partial revert of I4c90e361b7538efeb0c5efa7ca61aed2e468591c

Change-Id: I7d1282e4260e7a42bee35b81d77470eb40dea818

Merge "LockManagerGroupFactory to replace singletons"

Merge "Use correct metadata when adding exact query match"

Merge "mail: Avoid use of global state and use MediaWiki services DI"

Merge "Use UserIdentity::isRegistered instead of User::isAnon in BlockManager"

Merge "Use ->getTitle() instead of ->mTitle to get the title from Parser"

mail: Avoid use of global state and use MediaWiki services DI

Change-Id: If335cb6a220ebc5eb7487e51ee097f9d2afa0d2d

Merge "wdio: Switch to dots reporter for brevity"

Use correct metadata when adding exact query match

Bug: T230819
Change-Id: If85fa1cbad1f3ef9ab2cb0da966a98aafffad05e

Use ->getTitle() instead of ->mTitle to get the title from Parser

This allows to make mTitle a private property.

Change-Id: If953546cf4bb0324e4ae41d2195254c37c0ad3b4

LockManagerGroupFactory to replace singletons

100% test coverage of code that appears to be working and used, in both
LockManagerGroupFactory and also LockManagerGroup. Where possible I
wrote it as unit tests. One preexisting code path seems to be broken and
I marked the test as skipped. Two methods look unused and perhaps not
especially helpful, so I didn't write tests for them yet in case we want
to just get rid of them instead.

Change-Id: Iaa7354f31c451b87773468609c674a3bf1d4382f

Localisation updates from https://translatewiki.net.

Change-Id: I52497a0a7f1d15ca6643556e3967d9302572a40f

Use UserIdentity::isRegistered instead of User::isAnon in BlockManager

BlockManager::getBlockFromCookieValue accepts a UserIdentity, so should
only call methods defined in that interface.

Change-Id: If30df15b800cceee0ad052a2e0524df8b28d9901

Merge "TablePager: put parent construct call back at end"

TablePager: put parent construct call back at end

Restores position of parent constructor call, changed in commit
I082152b64141f1a.

The parent constructor calls getIndexField(), which depends on the
mSort value already being set.

Bug: T231261
Change-Id: If07f10075a51fbbe9de24464cb6844faaad94780

Merge "Add more information to exception thrown"

Add more information to exception thrown

It's not just about the preference name, because current user's properties
are used for validation too.

Bug: T231029
Change-Id: I268b959017bb0dce2b4295d5302a544bfa3513eb

Merge "Add help link to Special:NewSection"

Merge "Add SearchResultTrait"

Add help link to Special:NewSection

Bug: T231260
Change-Id: I856f5256b45c3961186f27c3b0b190998e297c1a

Merge "Setup: Remove scopedProfileIn() calls"

Merge "REST: call MediaWiki::preOutputCommit and doPostOutputShutdown"

Merge "SpecialNewSection: Fix handling of 'preloadparams'"

Merge "Clean up LinkRenderer usage in IndexPager and extensions"

Merge "Revert "Make LocalisationCache a service""

Merge "objectcache: SqlBagOStuff cleanups for sqlite and postgres"

Merge "rdbms: simplify comment in LoadBalancer::reuseConnection()"

Merge "rdbms: rename $db => $conn in remaining places in LoadBalancer"

Merge "Setup: Move wgRCLinkDays logic from Setup to ChangesListSpecialPage"

Merge "Use local variabe $services instead of MediaWikiServices::getInstance()"

Merge "Disambiguate confusing $fromReplica variable in BlockManager"

SpecialNewSection: Fix handling of 'preloadparams'

The parent class RedirectSpecialPage handles array parameters
automatically, and we need to specify the name without the '[]',
same as in RedirectSpecialArticle.

Bug: T231088
Change-Id: I52838a39f3311fe612c466903d7eda95fecedfa7

Revert "Make LocalisationCache a service"

This reverts commits:
 - 76a940350d36c323ebedb4ab45cc81ed1c6b6c92
 - b78b8804d076618e967c7b31ec15a1bd9e35d1d0
 - 2e52f48c2ed8dcf480843e2186f685a86810e2ac
 - e4468a1d6b6b9fdc5b64800febdc8748d21f213d

Bug: T231200
Bug: T231198
Change-Id: I1a7e46a979ae5c9c8130dd3927f6663a216ba753

objectcache: SqlBagOStuff cleanups for sqlite and postgres

Make createTables() work for Postgres. Also add and use new
initSqliteDatabase() method for SQLite.

Also use ConvertibleTimestamp instead of wfTimestamp

Change-Id: I8e4505775e57fc6cd7380afa015fc0901a1bfc1b

rdbms: rename $db => $conn in remaining places in LoadBalancer

Change-Id: I7e419cdc97d03616d5ee011a97fca658d0d79cad

Merge "language: Move ConverterRule to includes/ and add test case"

Use local variabe $services instead of MediaWikiServices::getInstance()

Change-Id: Ieb6230231d9b916fd1366e7b473db00c87ac4428

language: Move ConverterRule to includes/ and add test case

Add a very basic test that creates an instance, invokes it
with an empty string, and asserts the types and values returned
by the public methods used in LanguageConverter::applyManualConv.

Bug: T225756
Change-Id: I3600260e712f019243ba5f00890011e694444e43

Pass correct store to rebuildLocalisationCache.php

e4468a1d6b6 completely broke rebuildLocalisationCache.php by
unconditionally passing in LCStoreDB( [] ) instead of constructing the
correct object.

Bug: T231183
Change-Id: I0d52662e8745cf0e10091169b3b08eff48ef2b8f

Correctly register storeDirectory in l10n cache

e4468a1d6b6 made LocalisationCache a service and refactored a bunch of
setup code. In doing so, when processing 'storeDirectory' from
$wgLocalisationCacheConf, it accidentally started treating empty
non-null values (such as the default "false") as storage paths instead
of meaning "fall back to $wgCacheDirectory". This would have broken all
config that used file store for LocalisationCache and did not specify
'storeDirectory'.

Bug: T231183
Change-Id: I9ff16be628996b202599e3bb2feed088af03775f

Localisation updates from https://translatewiki.net.

Change-Id: I85f8b577290b04d60af1fa4f9151a6756a658aa8

rdbms: simplify comment in LoadBalancer::reuseConnection()

Change-Id: Ie6ea6ac7617b1f026b0ee7b8b76c287ec0da9e34

Merge "title: Fix flaky TitlePermissionTest::testUserBlock"

REST: call MediaWiki::preOutputCommit and doPostOutputShutdown

As in api.php. Among other things, this enables profiling.

Move EntryPoint test out of unit/ so that it passes.

Use ob_start()/ob_end_clean() instead of assuming an output buffer is
open, so that EntryPoint::execute() can be run from CLI mode.

Change-Id: I38162a9eac6fd5acfed2035b87cac4a97ffd50d6

Setup: Move wgRCLinkDays logic from Setup to ChangesListSpecialPage

This is only needed when viewing SpecialRecentChanges (or a related
page). Move the consuming logic to that class, where it can be
maintained as part of the rest of the code. This also makes it
more testable and makes config easier to work with or load from
elsewhere in the future. Aside from some dynamic default values,
configuration should generally not mutate. If there is some domain-
specific way to consume it, the relevant component should be
responsible for doing so. This also means we defer such logic to
where it is needed, instead of unconditionally for all possible
features that might be used.

Bug: T189966
Change-Id: If17608909711d98ac560b6d64f72ba7913a561a9

Merge "resourceloader: Add coverage for internal Context::getReqBase()"

Merge "rdbms: refactor caching in LoadBalancer::getReadOnlyReason()"

rdbms: refactor caching in LoadBalancer::getReadOnlyReason()

Avoid nesting of the same getWithSetCallback() cache updates.
Also favor accuracy over initial cache use for the case where
there is already a master connection. Add missing "lockTSE"
flag to protect against stampedes updating stale values.

Change serverIsReadOnly() to use SELECT in mysql instead of
SHOW to avoid internal temporary tables.

Bug: T227838
Change-Id: I2b0d680c9c3bdc7aaa1d1e1d6beb2dd203a815f1

rdbms: make IDatabase comments more up-to-date and stylistically consistent

Keep the first comment line of methods short, mention "expected" exceptions,
and break long method signatures into separate lines.

Change-Id: I64b36210f2451d46f8f2412b48dc846b56989e82

Setup: Remove scopedProfileIn() calls

To my knowledge this never goes anywhere. These are left overs from
when we used profileIn/profileOut still, which we no longer use
given things like Tideways and XHProf are now recommended for
profiling, instead.

Bug: T189966
Change-Id: Id1bacc951f267dc1d3872aed935afd306adc37b3

Clean up LinkRenderer usage in IndexPager and extensions

The property needs to be private because pagers in extensions
may declare the same property with stronger visibility which
would crash.

Bug: T149346
Change-Id: I082152b64141f1a9a4085bba23fe81a99ec8d886

resourceloader: Add coverage for internal Context::getReqBase()

Change-Id: I068000f55f2100a6791470896d7f1050bdcb19a4

Remove more Oracle and Mssql leftovers

Follows-up 4d10bb14e81 and 807d793ab9079.

According to codesearch [0], these were the last usages. Note that this
patch leaves two constants in places, IDatabase::DBO_SYSDBA and
DBO_DDLMODE. These are public constants used "mostly for oracle" according
to the docs, but maybe we could find other use cases in the future (?).

[0] - https://codesearch.wmflabs.org/core/?q=oracle%7Cmssql&i=fosho&files=%5C.%5B%5Ej%5Cd%5D%7Cen%5C.json&repos=

Bug: T230418
Change-Id: Ibfb748b4b23b885a77f4de161af4bf2ab9649a89

Merge "Revert "rdbms: make LoadBalancer::reallyOpenConnection() handle setting DBO_TRX""

Merge "objectcache: make MediumSpecificBagOStuff::mergeViaCas() handle negative TTLs"

Revert "rdbms: make LoadBalancer::reallyOpenConnection() handle setting DBO_TRX"

This reverts commit 45831e619c5e667ae1201bcdacfb8d4dcce10b41.

Reason for revert: Caused beta not work at all.

Bug: T231162
Change-Id: Icc5c1fa0dc01082a622641ad96c22c939cd56d48

Merge "Migrate BagOStuff::incr() calls to incrWithInit()"

title: Fix flaky TitlePermissionTest::testUserBlock

The mock timestamp is asserted via $prev, which is set a few lines
up.

The input timestamp is set as 0, which usually passes (if these two
lines execute in the same second), but when the two lines of code
are on opposite sides of a clock second boundary, the test would
fail.

Instead of fixing the setTimestamp() call, remove it in favour of
fixing the 'timestamp' option instead, which was needlessly
being set to one thing and then overwritten by setTimestamp.

Bug: T231166
Change-Id: If6a1e4ec7b91efee0a43e6dd86794f15c5a440b3

Merge "rdbms: add setTempTablesOnlyMode() to suppress CONN_TRX_AUTOCOMMIT during tests"

Merge "Make the JobRunner flushReplicaSnapshots() call cover the first job"

Merge "Make NameTableStore use LoadBalancer::getConnectionRef()"

Make the JobRunner flushReplicaSnapshots() call cover the first job

If JobRunner is called when replica transactions exists, the first job
would previously use the stale REPEATABLE-READ snapshot data.

Also clear any master connection snapshots via commitMasterChanges().
This makes the code more similar to DeferredUpdates::attemptUpdate().

Change-Id: I2157a91fb01ea8c233f964b1f3164e8c3b1a07ca

Merge "parsercache: use WRITE_ALLOW_SEGMENTS for cached ParserOutput values"

Localisation updates from https://translatewiki.net.

Change-Id: I89aab65db771aaded5fb8a3517cea3545a6efa01

Make NameTableStore use LoadBalancer::getConnectionRef()

This method handles reuseConnection() calls, disallows writes to
DB_REPLICA role handles, is more consistent with wfGetDB(), and
is more efficient with I540b08920997c5.

Change-Id: I8edaef0ea4718414e2004fad04afca8ec5797669

Migrate BagOStuff::incr() calls to incrWithInit()

Change-Id: I722d6fe3d50c594858e3e7960fb99ef2fc1664b3

parsercache: use WRITE_ALLOW_SEGMENTS for cached ParserOutput values

This lets large output entries fit into memcached via key segmentation.

Follows b09b3980f99 which applied the feature to PageEditStash.

Bug: T204742
Change-Id: I33a60f5d718cd9033ea12d1d16046d2bede87b5b

Merge "rdbms: make LBFactory close/rollback dangling handles like LoadBalancer"

objectcache: make WANObjectCache::relayPurge() actually use $holdoff

Fixes a regression introduced by 70547f3fa304 in 2016.

Change-Id: I5996b63c31ac3b382e838a6858b8585bdc8c585c

Merge "resourceloader: Map group names to integers internally"

Merge "resourceloader: Add tests for mw.loader not storing private/user response"

Merge "resourceloader: Add tests for disallowing access to private modules"

Merge "objectcache: rename getDB() to getConnection() in SqlBagOStuff"

Merge "rdbms: make LoadBalancer::reallyOpenConnection() handle setting DBO_TRX"

Merge "filebackend: update code comments and break some long lines"

Merge "objectcache: improve BagOStuff arithmetic method signatures"

Merge "maintenance: Cleanup db code and bogus RevisionRecord constants in NamespaceDupes"

Merge "maintenance: Fix bogus IDatabase method calls in sqlite.php"

Merge "maintenance: Fix bad method call in RebuildTextIndex"

Merge "maintenance: Sanity check that --replicadb isn't using the master in sql.php"

objectcache: rename getDB() to getConnection() in SqlBagOStuff

Change-Id: I7ec33e2622f5b418712d76bedcbb4be65376cf91

rdbms: make LoadBalancer::reallyOpenConnection() handle setting DBO_TRX

Make LoadBalancer::reallyOpenConnection() handle initializing DBO_TRX
instead of Database::__construct().

Also:
* Avoid having the "catch" block appear like it returns a
  half-constructed Database.
* Use the variable name $conn instead of $db to be consistent
  throughout the class. Only send Database::__construct() parameters
  that it recognizes instead of mixing in setLBInfo() data.

Change-Id: Iffc3d1d0713051a164adb51a4c4ee12e4ac887c3

maintenance: Cleanup db code and bogus RevisionRecord constants in NamespaceDupes

Change-Id: Ie31beb183d32663b368a32e4926a3ed55456719c

maintenance: Fix bogus IDatabase method calls in sqlite.php

Use getConnection() to directly get a DatabaseSqlite instance.

Change-Id: Ic349d14614b6bf145ae2e3ccdac68319fa2bbf9c

maintenance: Fix bad method call in RebuildTextIndex

This method was removed earlier this month with 2eb5bdbac6b7.

SearchSqlite already checks if the proper searchindex tables was created.
Avoid call to bogus IDatabase::checkForEnabledSearch() method.

Also make use of Maintenance::getDB()

Change-Id: If662e17093c1aef71ee9841259ca76e2ac31a7a9

objectcache: improve BagOStuff arithmetic method signatures

Make the default $init value for incrWithInit() be $value.
This is far less suprising and also makes the operation
easier to replicate without conflicts.

Make decr() definitions more explicit since various cache
drivers do not handle negative incr() values (e.g. memcached).

Change-Id: I2b8d642656cc91c841abbd7a55d97eba101b027a

maintenance: Sanity check that --replicadb isn't using the master in sql.php

Change-Id: I0b7fd896c88b06cbc17b31b1ae2b9cb8e375da1b

resourceloader: Map group names to integers internally

This makes startup response smaller.

Change-Id: Id0ae3cd7cd22163f42b92b629c38b0a806d3ca50

resourceloader: Add tests for mw.loader not storing private/user response

This is in preparation for changing the way group indexes are transmitted
between the server and client, to confirm that the behaviour doesn't break.

Change-Id: I1f84c4c407a4eb69b5b9659d768c643d4a83df0d

resourceloader: Add tests for disallowing access to private modules

* Add a test to confirm that the ResourceLoader::respond() logic
  works as intended.

* Remove the client code for preventing it from being loaded.
  This can never happen in production unless there is a bug.
  Instead of optimising to avoid a pointless request that only
  happens when the software is broken, instead optimise for when
  the software is not broken by just letting it happen. The server
  already handles it just fine.

  This was originally added in 2015 with 1dd73903726 to reduce
  logspam, but that was instead fixed in 6d6b037e122 by making the
  log message debug-only (because it's not a software problem,
  it's a client-error, e.g. a broken user script or a third
  party trying out different things on the load.php entry point).

  Removing this makes the client a bit smaller, too :)

Change-Id: Ic5420d9329a73514f4fc27baa46ae58d94addafb

filebackend: update code comments and break some long lines

Change-Id: I072e15c7c51490888284561260958e405fb061ca

Merge "API: Warn about POST without Content-Type"

Merge "filebackend: Remove private handleWarning in favour of local closure"

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

Merge "Skin: Add new hook, UndeletePageToolLinks, for Special:Undelete views"

Localisation updates from https://translatewiki.net.

Change-Id: Ifdfe6ec25954eb7c61a85b6494bc0d2a18bae9a7

Merge "rdbms: various field name and style cleanups to Database"

Merge "Short-circuit WebRequest::getGPCVal() for printable ASCII strings"

Skin: Add new hook, UndeletePageToolLinks, for Special:Undelete views

This allows extensions to add further links; the particular use case
in mind is for the AbuseFilter extension, but others may want this
too.

Bug: T231055
Change-Id: I671a0479e877e6c37606b688064cb9c893717709

wdio: Switch to dots reporter for brevity

Change-Id: Ica6cc3e4fc445f39ba911db14c58a486f32f7f89