Directly use Language::factory instead of wfGetLangObj

figureMessage always returns a valid language code.

Change-Id: Ifa11ca278e7a6f49781846bb6d1eec94669999f3

Rename getSlavePos() => getReplicaPos()

All callers updated

Change-Id: Ieb87df1bd134e6b0d8b01be581288e98fe3543ee

Remove unused Database::isView()/clearViewsCache() methods

Also make getLazyMasterHandle() protected.

Change-Id: Id6b48ff976a800052c22e90b572695ab3b8beb7e

Merge "Move FSFilebackend and MemoryFileBackend to /libs"

Merge "Make LoadBalancer return docs more specific about types"

Make LoadBalancer return docs more specific about types

Change-Id: Iab671b66a4a4da33c780b6d78752c484ec329254

Merge "Clean up misuse of wfTimestampOrNull()"

Move FSFilebackend and MemoryFileBackend to /libs

Change-Id: Id38a2ef9e0e22537d0267b536f8a4dfd60c5b41d

Clean up misuse of wfTimestampOrNull()

It's primarily meant for nullable timestamps in the database.
And for the mere purpose of an is_null() check, it's more confusion
than useful to cause this function since in these two cases its return
value is not used as-is but further conditionalized.

* Also fix fragile link between getRegistration()'s false return
  value never being passed to wfTimestamp (to mean "now") because
  of the isLoggedIn() guard. The ternary would now make those
  follow the 'else' branch toward null.

Change-Id: I9a1ee2a56e1767bfb750e27b1f37bdaeb5e6378d

Merge "Remove wf* function usage from FSFileBackend"

Merge "Avoid DatabaseBase type errors in setupDatabaseWithTestPrefix()"

Avoid DatabaseBase type errors in setupDatabaseWithTestPrefix()

Change-Id: I94cc0672a487d547a09c27997fe36274d7d62977

Merge "Add transaction ticket to LinksUpdateComplete hook"

Remove wf* function usage from FSFileBackend

Change-Id: Iad6471724f8cdc596c755e6194da7556158e9203

Merge "Move FileBackendStore and FileOp classes to /libs"

Merge "SpecialRecentChanges: Allow tagfilter=tag as "subpage""

Merge "Move RedisLockManager to /libs"

Merge "Move RedisConnectionPool to /libs/redis"

Move FileBackendStore and FileOp classes to /libs

Change-Id: If490f64bec282e5dfcdaf7feb1cbf46d3dce1064

Remove wfDebugLog() dependency from FileOp classes

Change-Id: Ib8ab9edd1afdb28f7444f5e13cf34a696a066234

Merge "Add HTTPFileStreamer class"

Add HTTPFileStreamer class

* Move most StreamFile code to this new class
* Remove StreamFile depedency from FileBackendStore

Change-Id: I4a272ef49497b589114fc2b37ba800bc26d9161f

Merge "Move FileBackendMultiWrite to /libs"

Merge "Revert "resourceloader: Make cache-eval in mw.loader.work asynchronous""

Merge "mediawiki.js: Refactor definition of mw.log singleton"

Merge "mw.loader: Use native Set where possible instead of string keys"

Revert "resourceloader: Make cache-eval in mw.loader.work asynchronous"

This reverts commit 482ad8d9fb2e8273cefc4a5a557604b093944c8b.

Change-Id: Ic31463313b5a6d9720f0a5e036bb7200d3340021

Merge "Add IMaintainableDatabase for non-OLTP type methods"

Merge "Add ILBFactory interface for completeness"

mediawiki.js: Refactor definition of mw.log singleton

Move this out of the large plain object that defines 'mw' itself
because right now none of the mw object initialisation is able
to call mw.log.

In a future commit, a call to mw.log is introduced in the mw.Map()
constructor, which is also used to set the mw.messages, mw.templates
and mw.user.options fields.

* Rename the other log() function to avoid clashing.
* Update outdated comment about not using mw.log() which was
  there about confusion between log() and mw.log(). However,
  this code was updated long ago to use mw.track() instead of
  log() directly, and log() has now been renamed.

Change-Id: I7d6bd4e3c9753e30278752c59d6cbd4d2fd10d96

Merge "Add DBO_* class constants and defines.php alias the class constants"

Move FileBackendMultiWrite to /libs

Change-Id: I8079693a62db390028cd9f72b2bd7a81ae1164c3

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

Localisation updates from https://translatewiki.net.

Change-Id: Iff900bc05ff000bd0c290d77ad20be6808b12fb0

Add DBO_* class constants and defines.php alias the class constants

Change-Id: If13b23ef849d4cf4c711b0ec2bf2e8a795f90738

Dependency cleanups to FileBackendStore

* Inject a PSR logger in place of wf* logging methods.
* Replace wfTimestamp() call with ConvertableTimestamp.
* Inject the Profiler object.
* Rename $wiki => $domain to avoid "wiki" mentions.
* Remove stray FSFile::extractImageSizeInfo() method.

Change-Id: I3e3a43d36506bec341dc5727b52f5f5a5d32bff5

SpecialRecentChanges: Allow tagfilter=tag as "subpage"

This allows an url like example.com/wiki/Special:RecentChanges/tagfilter=visualeditor
to only show changes made with the VisualEditor (instead of using the long version
like index.php?title=Special:RecentChanges&tagfilter=visualeditor).

This also allows the transclusion of the Special:RecentChanges page with a tagfilter.

Bug: T142878
Change-Id: Iaa749a556ccc422e79fbb2bac709fab45b3cc83d

Merge "Add alternative DB_* class constants to ILoadBalancer"

Add ILBFactory interface for completeness

Also fixed comment indentation.

Change-Id: I535d320b63477302b06fd634b3ab47eef6868ffc

Merge "Make newExternalLB() public to match newMainLB()"

Add alternative DB_* class constants to ILoadBalancer

Change-Id: I5d7e6d9f0f5bd4275e240a94b996f3740ef46330

Merge "Add HTMLFormField class for MWRestrictions and use it for bot passwords"

Merge "Log when Message::__toString has an unexpected format"

Merge "ScopedCallback: Fix a typo in a comment"

ScopedCallback: Fix a typo in a comment

Change-Id: I6179f3b97fdf787c7175e8d7edf06cb2d84f7403

Merge "Handle links with multiple pipes correctly in edit summaries"

Code comment and style cleanups to MysqlLockManager

Change-Id: Idf05c0c44f5f6fbb373625c0aebe015e385700ce

Merge "Move tagging of API deletions to RC save"

Move RedisLockManager to /libs

Change-Id: Ia476f117243cf0f6984e373d460b24db18584704

Move RedisConnectionPool to /libs/redis

Change-Id: Ied4a85d7172ab76b90f6d9ce4d47a83c3fd7d111

Merge "Ensure users are able to edit the page after changing the content model"

Make newExternalLB() public to match newMainLB()

Change-Id: I7e46c947882c5de6e4d25a7110c2a2558df7ad76

Move more LockManager classes to /libs

Change-Id: I4bfa79f430827b8717a57bb61cb7c36bc7fcb489

Ensure users are able to edit the page after changing the content model

It is possible for page restrictions to be dependent upon the content
model a page. The best example of this is user JavaScript and CSS
subpages. This adds a Title::setContentModel() function which allows
mocking a Title's content model for the purpose of permission checks.

EditPage and Special:ChangeContentModel were updated to ensure the user
can edit the page with the newly proposed content model before making
the change.

Title::$mContentModel was made private to make sure nothing else mucks
around with it. There were no uses outside of Title anyways.

Bug: T145316
Change-Id: I28c46f408cadf65ed1868401cd00dc15e5acd2fe

Merge "Inject "srvCache" and local DB connections into LockManagerDB"

Add IMaintainableDatabase for non-OLTP type methods

Change-Id: I4cbdfd7a40af17269c527df6e1c3c5644672b8a7

Merge "Call setTransactionTicket() on DeferredUpdates sub-queue items too"

Reduce queries in CategoryViewer via addGoodLinkObjFromRow()

Change-Id: Id782b50f166efbee6c9f8b9b263f09fc5fb5e3f2

Add transaction ticket to LinksUpdateComplete hook

This lets callers use commitAndWaitForReplication() more easily.

Change-Id: I743bd1f989b8fb3b7ba3e5cc8ce1bb44c00f99af

Merge "Remove see comment"

Merge "Rename includes/Services to includes/services for consistency"

Merge "Revert "Move wfEscapeWikiText() to Parser::escapeWikitext()""

Revert "Move wfEscapeWikiText() to Parser::escapeWikitext()"

Apparently it is possible for Parser::mParserOptions
to not be set in some cases. I'll try again later.

This reverts commit bda74bff6e180b5a6bb2cc68e12ae391d53789a3.

Bug: T146433
Change-Id: Idb6d1b20995d5f86b712abb386ab987356c4f560

Remove see comment

Change-Id: I58fcd682dba6c778c0975baf67a6b87f36b1aff2

Rename includes/Services to includes/services for consistency

Change-Id: I900fab26a7cf5a339233f55c31168f8c2963bc8c

Call setTransactionTicket() on DeferredUpdates sub-queue items too

This should lower the rate of "does not have outer scope" log
warnings in DBPerformance.

Also add traces to such logs.

Change-Id: I7d21ea745cae07e0fbbe4cd8de82e93f1d10e0a5

Merge "Support multiple limits and arbitrary periods in account creation throttle"

mw.loader: Use native Set where possible instead of string keys

Total time spent in sortDependencies
(Chrome Canary, unthrottled, MacBook Pro, MediaWiki-Vagrant)
* Before: 0.53ms
* After: 0.3ms

Bug: T146432
Change-Id: I65e2b0efbd080adab003b5c9605405ea0ae5380d

Merge "Fix doc typo for wasConnectionError()"

Merge "Split out new ObjectCache::newWANCacheFromParams() method"

Merge "Simplify and clean up FileBackend exceptions"

Simplify and clean up FileBackend exceptions

Use standard exceptions for unexpected errors and remove
FileBackendException class, leaving FileBackendError. The
later is actually intended to be caught in some cases.

Change-Id: I735a525e0b14e518b2da5f18762e0f293064dfc2

Add DBConnRef sanity check to LoadBalancer::reuseConnection()

Change-Id: I6b079b994f29c5f7953efe73982ef9e6946a21f8

Fix doc typo for wasConnectionError()

Change-Id: I70eaeb630e2a4bb8f2940c2dcc938f7305014d48

Merge "Make MigrateFileRepoLayout use getErrors()"

Merge "Move wfEscapeWikiText() to Parser::escapeWikitext()"

Generalize core schema docs in DatabasePostgres

Change-Id: I52e64867c1533c50406623d5bc121c1e4da2459a

Log when Message::__toString has an unexpected format

Message formatting methods have a side effect on how string conversion
will work, which is a security problem waiting to happen:

    $msg = new Message( 'foo' );
    echo $msg; // parsed
    echo $msg->plain();
    echo $msg; // not parsed

This change logs Message -> string transformations which are
affected by a prior call in this way. The behavior will be removed
in a later patch (possibly replaced by something more explicit
if it turns out that something depends on it).

Bug: T146416
Change-Id: Id51cf6a5a937bc41a914f317e980ef42e4d385fb

Merge "Break long lines and generalize some comments in /rdbms"

Merge "Inject schema vars into DatabaseMysql"

Merge "IcuCollation: Update comments on $tailoringFirstLetters"

Merge "Correct error handling for exceptions in 'user' module"

Inject schema vars into DatabaseMysql

Also fix broken patchSql.php maintenance script

Change-Id: I45bccb0e2e10bd06651a551fa08ed0c66d11eb6a

Merge "Remove old remapping hacks from Database::indexName()"

Break long lines and generalize some comments in /rdbms

Change-Id: Ibf80c67b59df748e98d74eeab492b1f33d198024

Correct error handling for exceptions in 'user' module

Rearrange code so that the try...catch which is supposed to catch
exceptions when evalling code actually catches them. Evaluation of
'user' module was wrapped in `mw.loader.using( 'site' ).always( ... )`,
so it could be executed asynchronously, so try...catch never caught
exceptions from it; they bubbled up to all kinds of weird places and
broke things in confusing ways.

I think the same issue could occur for any module when waiting for
legacy modules to load ('wgResourceLoaderLegacyModules').

Bug: T145970
Change-Id: I91e7d0b4e50c786f7302e30a2b7ed43c3cd0da6c

IcuCollation: Update comments on $tailoringFirstLetters

A few more languages marked as "Verified by native speakers",
based on which collations we've been using in production
on Wikimedia wikis.

(I'm not sure if this makes sense now that we're fairly confident
that these are good in general, but since it's already here...)

Change-Id: I8e1f31fa61509eca8c76a2df4e18638005e68b77

Localisation updates from https://translatewiki.net.

Change-Id: I3796071d3b7f94e65c50a0444fd345a8eeb84534

Add HTMLFormField class for MWRestrictions and use it for bot passwords

Change-Id: Ib50238e3be5eec63eb5df97154b60dc4ca33d581

Merge "Make Database::__clone() a bit more robust using handleSessionLoss()"

Merge "Split up DBError classes into their own files"

Make Database::__clone() a bit more robust using handleSessionLoss()

Change-Id: I6c16abceea612a7b1167b9cca70a174fa6b77020

Split up DBError classes into their own files

Change-Id: I671f1a88f0192a5199cfc9c6d7fbde1bff5c4ac8

Merge "Follow-up 764cd6a1: unbreak quiet buttons"

Merge "Remove MediaWiki mentions from /rdbms error strings"

Remove MediaWiki mentions from /rdbms error strings

Change-Id: Ia330d8e465641d04e5d22a24490d792913f1ccb8

Follow-up 764cd6a1: unbreak quiet buttons

Quiet buttons are supposed to be initially gray,
not initially colored.

Bug: T146401
Change-Id: I8c3020591c82f7b2c1f7043892e73019886478af

Add category name in ID property for extension row in Special:Version page

Extensions can belong to multiple different categories. When listing them in
the Special:Version page each entry is surrounded with HTML element with
ID property "mw-version-ext-{NAME}". This causes generation of duplicate
ID values, which is against W3C rules.
To preserve valid and compliant HTML while retaining ID values, change the
ID pattern to "mw-version-ext-{CATEGORY}-{NAME}".

Bug: T99025
Change-Id: Ifdde0704631f1ef41e4c83e7e8116003983e0808

tests: Fix invalid @covers value in StatusTest.php

Uusally, phpunit issues a warning and continues to generate the
report but in this case it caused an early exit with no output.
Follows-up 3706dcb5c.

> Trying to @cover not existing method Status::getErrorsOnlyStatus.

Also:
* Increase coverage by adding @covers where missing.
* Use setOK() internally.
* Add test for 'ok' setter.

Change-Id: If6db634079c857f02b2594be199e6910ec49a52a

Support multiple limits and arbitrary periods in account creation throttle

This adds support for multiple count-per-period limits and arbitrary
period durations in the AuthManager account creation throttle in the
wiki settings. The $wgAccountCreationThrottle config variable becomes
an array like $wgPasswordAttemptThrottle.

Bug: T146290
Change-Id: Iea182a92a1199b0ce7103ab9ae24f1c87b01985c