AuthManager phpdoc cleanup

Change-Id: Ib6fa897e76ae479de68acaeb16708c2bdf15ae5f

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

Rename includes/Services to includes/services for consistency

Change-Id: I900fab26a7cf5a339233f55c31168f8c2963bc8c

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

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

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

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

Merge "Remove screen layout references in passwordreset-email* messages"

Merge "Use ConvertibleTimestamp::convert() in a few places"

Actually set errorLogger in Database

Change-Id: Icc2e49bcedf5172c1c1fdf8202acc6c1abdbac5b

Use ConvertibleTimestamp::convert() in a few places

Change-Id: I9da31cd96fa7afa5cfccbbc204fb446fdb1a3c65

Rename Convert*a*bleTimestamp to Convert*i*bleTimestamp

Per the dictionary.

Change-Id: I3dcef9fb020c8c6764ee073111ed8c032e4c5a63

Merge "time: Implement ConvertableTimestamp::now()"

Merge "Add deprecated comment to Article::doEdit()"

time: Implement ConvertableTimestamp::now()

Shortcut for ConvertableTimestamp::convert() for the current time.

Change-Id: I1c34c0a59bb57652c7a2e959836c8ba400a25f49

Merge "time: Implement ConvertableTimestamp::convert()"

time: Implement ConvertableTimestamp::convert()

* This method is analogous to wfTimestamp(). Optimise for the common
idiom of just converting a timestamp without having the caller hold
on to any object.

* Make wfTimestamp() use this (it could already since it didn't
use any MWTimestamp methods). Use via MWTimestamp. While this is
the same as direct access, it allows future changes.

* Add tests covering this new method.

Change-Id: I7f9104f1701d92fe25d72c7943581c64e1d093fa

Merge "Move IP class to libs/"

Merge "Remove MimeMagic depedency from FSFile and move it to /libs"

Merge "Move IP::isConfigured/TrustedProxy() to ProxyLookup service"

Move IP class to libs/

Also fix some misplaced parenthesis in IPTest.

Change-Id: I84d6120c49f733ec45e7e0005259871808b7568b

Remove MimeMagic depedency from FSFile and move it to /libs

Change-Id: Ieaae909b49c798b6e9bed65d15961cefbdaec49b

Merge "Move SamplingStatsdClient into stats/ subdirectory"

Move IP::isConfigured/TrustedProxy() to ProxyLookup service

This creates a new ProxyLookup service to house the
IP::isConfiguredProxy() and IP::isTrustedProxy() functions. The main
purpose of this refactoring is to make the IP class entirely independent
from MediaWiki, so it can be split into a separate library.

Change-Id: I60434a5f3d99880352bc0f72349c33b7d029ae09

Merge "refreshLinks: Use vslow group for DB_REPLICA connections"

Merge "Migrate callers to new MWFileProps::getPropsFromPath() method"

Merge "Dependency inject $wgTmpDirectory into FileBackend classes"

Migrate callers to new MWFileProps::getPropsFromPath() method

* FSFile should not be responsible for handling this much logic.
* Make more MediaHandler classes aware of the fact that an object
  other than File might be passed in. Use the FSFile instead of a
  useless empty stdClass object.
* Also added more fields to FSFile::placeholderProps to make it
  more complete.

Change-Id: I9fe764b2a7261af507c6555e6a57273cf7d00d36

Make MigrateFileRepoLayout use getErrors()

Change-Id: I5281c4ded9eba42eccaf5a67e809a0175fa057ce

Split out new ObjectCache::newWANCacheFromParams() method

Change-Id: Ib2e409dd129bd1e2871fe239e71a4eb8fb42944c

Use standard exceptions in ObjectCache and always handle CACHE_NONE/"hash"

* If cache CACHE_NONE or "hash" is not set yet or was unset,
  return the proper new object anyway for sanity.
* Also fixed $params typo in newFromParams(). This should not
  have any effect since SqlBagOStuff ignores the key in this case.
* Removed excess "use" statement.

Change-Id: I36108a865273b9c209145828e5e1216b545ef431

Merge "Cleanup method visibilites in Database"

Merge "Set more FakeDatabaseMysqlBase fields in __construct()"

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

Localisation updates from https://translatewiki.net.

Change-Id: Ib08086b1ad6f1c9409464178b552525bb74e41e4

Merge "Clean up lock name in lock messages in DatabaseMysqlBase"

Merge "Fix IDEA warnings in CloneDatabase"

Set more FakeDatabaseMysqlBase fields in __construct()

Change-Id: I37a1e4aae186e19637817bd10be0d8fad06187d4

Fix to incorrect calls of header () breaking saving some pages and login

This is to fix incorrect calls of header () with null as its third parametre
in WebResponse::header ().

Under HHVM 3.14.3 this causes warnings in error.log and breaks saving page
and user login.

Bug: T140864
Change-Id: I98291e2746e92e22672de077bccfb36ae91d2c62

Merge "Set more fields in fake DB subclasses to avoid errors"

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

Add deprecated comment to Article::doEdit()

Already there on WikiPage, but this brings it to the IDEs
attention rather than @see

Bug: T145736
Change-Id: Ic931f83d15638f2babce22edf14ee8626b8a0266

Merge "Remove dangling $wgDBmysql5 usage from DatabaseMysqli"

Remove dangling $wgDBmysql5 usage from DatabaseMysqli

Change-Id: Iee21655e12eb3001496123ff8438adc1c39ecdf8

Remove old remapping hacks from Database::indexName()

Change-Id: Icab995aa6a442aab133da723363f794ef9128eb8

Cleanup method visibilites in Database

Also removed some unused cruft

Change-Id: I855d0e0d6571ea15c03d2a27bf55ad5e14cd15d4

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

Set more fields in fake DB subclasses to avoid errors

Change-Id: I6b24422e830a8f82ba24383b3425808cb2a28b15

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

Fix IDEA warnings in CloneDatabase

Change-Id: I45231237dd49dfdd8158a68d8f63d769b762f19c

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

Remove screen layout references in passwordreset-email* messages

See https://www.mediawiki.org/wiki/Localisation#Avoid_references_to_visual_layout_and_positions

Bug: T141199
Change-Id: I3bdee543591000b1dac3387116236e5515ad3292

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

Move SamplingStatsdClient into stats/ subdirectory

Change-Id: Ib871aa5cd674d58da3d19bfbb5660539afc863b3

Add some missing reuseConnection() to LoadBalancer

Change-Id: I99099f909e7759c061d44ad4e1ed049e038a4d7d

MultiHttpClient: Don't use wfArrayToCgi()

http_build_query() should be functionally identical.

Change-Id: Ia7d909e97a89d66dbe5187e9a0423e2ecf8561e8