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"

Move tagging of API deletions to RC save

Since the recent change save of logged actions is now deferred, we need
to move tagging of API deletions to the RC save, like other API actions
do already. Otherwise, only the log gets tagged, not the RC.

Bug: T108564
Change-Id: I4e6e18e7f8fb7a6b0932e7579bafddcc1b0a9758

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

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

Dependency inject $wgTmpDirectory into FileBackend classes

Change-Id: I219daffa67ff672bf5bd38921f4b1ca058d96b0f

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

Clean up lock name in lock messages in DatabaseMysqlBase

Change-Id: I9d3eb566f4e4f6d982cbc15df082cf2dd3f9614e

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