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

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"

Merge "Remove dangling $wgDBmysql5 usage from DatabaseMysqli"

Remove dangling $wgDBmysql5 usage from DatabaseMysqli

Change-Id: Iee21655e12eb3001496123ff8438adc1c39ecdf8

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

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

Merge "Improve Database::__destruct() and add a __clone() method too"

Merge "Remove OutputPage->mDebugtext"

Merge "Fix variable name in release notes"

Merge "Emit deprecation warnings from Article::fetchContent"

Improve Database::__destruct() and add a __clone() method too

* Close dangling connections in LoadBalancer/Database destructors.
* When DBs are cloned, create new connection resources for the clone
  so the two do Database objects don't clobber each other.

Change-Id: I3adb57cbb1fdc2a17e6d95389d0562ef22701576

Fix variable name in release notes

Change-Id: I7da512f4aa68c80dd08d93910888be1b49783912

Remove OutputPage->mDebugtext

Deprecated since 1.20, unused anywhere.

Change-Id: I4316b77c48e65faf788de60be28fc9cf65c785f1

API: Remove deprecated methods

All deprecated ApiResult methods are removed. These have been deprecated
since 1.24 or 1.25, and the only users remaining in Gerrit are wrapped
in backwards-compatibility checks and so should not be being called.

ApiBase, ApiFormatBase, ApiMain, and ApiQuery methods for generating the
pre-Ib14c00df help text are removed. Nothing has called these for a long
time, and only Flow implemented them in any way. Deprecated methods for
providing the text for such help, such as getDescription(), haven't been
removed yet, though, since some extensions still call some of them.

Change-Id: I3ca7c98174b4a3f6f67f2b023e0f4446637e7a84

Merge "Pass User to SearchEngine::getProfiles"

Pass User to SearchEngine::getProfiles

Useful for search engines that allow users to customize search profiles.

Depends-On: Icd577c8ebc6e162befe30bde4fe276e633d2e434
Change-Id: I471cd090730d2a25cb70d622ec3bebbe9583118c

Localisation updates from https://translatewiki.net.

Change-Id: I83c8f60a9022018fae1996910242f4514cf8cd3f

Merge "Fix minor SqliteInstaller IDEA warning"

Merge "Set an initial dummy domain in Database::__construct()"

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

Set an initial dummy domain in Database::__construct()

This avoids errors when LoadBalancer calls getWikiId() in
reallyOpenConnection() after a failed connection.

Change-Id: Idc3d93ae39be09f2da277df57cdc5a564cb3408d