rdbms: add IDatabase::wasConnectionLoss() method

This takes the logic from wasErrorReissuable(), but puts it under
better name. The way that method was used, as well its comments,
were only about connection loss.

Make wasErrorReissuable() check if there was any error that
does not preclude the ability to retry. This matches the actual
name of the method.

Also improve some other related comments.

Change-Id: I68455d803afb2370897fecab0e79aadbb5d1a740

Localisation updates from https://translatewiki.net.

Change-Id: I484a2f4f3606d6a6aab12011a8ce7ab7379ea3e6

Merge "Rewrite passwordtoopopular message"

Merge "Pass revision being reverted to edit code"

Merge "Add default for revision.rev_text_id where missing"

Rewrite passwordtoopopular message

Bug: T177018
Change-Id: Ie0b6a6f26de0271935514fc0764d4894ef0975f4

Add default for revision.rev_text_id where missing

This adds a default of 0 for:
 - Mssql
 - Mysql
 - Sqlite

The field is already NULLable in:
 - Oracle
 - Postgres

Bug: T188741
Change-Id: Ibe1cc363d21fb96f38c263181bf30a429a1c7b45

Merge "mediawiki.util: Document isIPAddress() allowBlock as optional"

Merge "Fix WikiPageDbTestBase::testNewFromId_returnsNullOnNonExistingId"

mediawiki.util: Document isIPAddress() allowBlock as optional

This was already the case in the code, but the documentation
was not reflecting this accurately. Passing 'false' as second
parameter is not desirable in most cases.

Change-Id: Idbbd7de795b444e401a16292dfb8c9a80f4d07be

Merge "Move image_comment_temp entries when the file is moved"

Merge "Remove @dataProvider and @covers from non-test functions"

Merge "Remove unused HashRing::newWithoutLocation() method"

Remove @dataProvider and @covers from non-test functions

Only functions beginning with test* should have that annotation

Change-Id: I85cd795970205943aebe692d85434145b3cf3f5b

Merge "Set visiblity of some HashRing methods"

Use SPDX 3.0 license identifier

SPDX released version 3 of their license list (<https://spdx.org/licenses/>),
which changed the FSF licenses to explicitly end in -only or -or-later
instead of relying on an easy to miss + symbol.

Bug: T183858
Change-Id: I4cbb98ea2bbacf46a57401dbba2bdb076fca1d3a

Move image_comment_temp entries when the file is moved

Bug: T189985
Change-Id: I437102d62cb94fd3195ff06ee8185ce5a2dc941e

Remove unused HashRing::newWithoutLocation() method

Change-Id: I6f149b2ef2e7e9fe2e7ddbda6c0c7d795c24720e

Set visiblity of some HashRing methods

* getLiveRing() is now protected
* getLocation() is now final

Change-Id: I16002fe7187d8bbb6e36d4dd9a1302ee1d46a4bb

Merge "Normalize PHPDoc attributes"

Merge "Don't use deprecated NS_IMAGE*"

Merge "RevisionStore: Properly encode timestamp"

Merge "Remove internal use of deprecated $wgRequestTime"

Merge "Remove @group from ParserTestRunner"

Remove internal use of deprecated $wgRequestTime

* Use $_SERVER['REQUEST_TIME_FLOAT'] unconditionally in WebRequest.php
  and libs/Timing.php. WebStart.php was doing this already without issue.
  The key existst since PHP 5.4, for both Web and CLI (we require 5.5).

* In wfDebug() and wfReportTime(), use $_SERVER['REQUEST_TIME_FLOAT'] instead.

* In ApiFormatBase and MWDebug, use WebRequest::getElapsedTime() instead.

* In Maintenance.php, remove setting of $wgRequestTime.

* In rebuildFileCache.php, update mocking to $_SERVER['REQUEST_TIME_FLOAT']
  so that we avoid re-introducing bug T24852.

Change-Id: I1b647da2862f815029caa533b592ec8a05b33806

Merge "Rename ApiTestCaseUpload"

Merge "Make IPTest::isNotIPAddress a test function"

Fix WikiPageDbTestBase::testNewFromId_returnsNullOnNonExistingId

This test is passing an "integer" that is beyond what even 64-bit PHP
can properly represent as an integer, so it likely winds up passing it
as '7.3574757437438E+19' to the database. MySQL, being MySQL, doesn't
care much, but other DBs do.

Change it to 2**31-1 instead.

Change-Id: Ieba51f034eaa1c3eabb2ada6b1ce876beefd3bff

RevisionStore: Properly encode timestamp

Timestamps in database queries need to be passed through
IDatabase::timestamp().

Change-Id: I1c163d80b2111a78bb10de49fe06d61f02c21d07

Normalize PHPDoc attributes

Change-Id: I83e686d099de0ff0aacda7e332972e1c7ee49f04

Don't use deprecated NS_IMAGE*

Change-Id: I3df00c8e55a79baa3f3727daeea6c9441113aebc

api: Restore fractions for default siprop=dbreplag response

Without this fix, the situation is that a query for 'siprop=dbreplag'
shows a db server with lag=0, while the siprop=dbreplag&sishowalldb=1
query shows the same server having non-zero lag. That's confusing.

Regression from r47865 (88f35cdf6e).

Change-Id: Id4854c8453206a6f1f8c369dc89c3a8bd910f83f

Merge "Remove #toc and #toctitle style definitions"

Remove #toc and #toctitle style definitions

Styling by class is preferred over styling by id as a rule, and
this #toc .toc doubling up was in place since 2005.
(28c9a747797dae738cd6fe1a97c29e1591564112)

13 years of cache protection seems adequate :)

Bug: T187808
Change-Id: I7f3a9df7f9fd1178986112b5265b1ae7c991d5b7

Pass revision being reverted to edit code

Otherwise, PageContentSaveComplete hook users are unable to detect
reverts.

Bug: T189819
Change-Id: Ib1dc0c323a9a5083b462421c68575fc22ad4af35

rdbms: small code cleanups to Database

Change-Id: I831b4ea1ced425b5008f3dd2802e16bb898f4b47

Rename ApiTestCaseUpload

Having *TestCase at end of the class name makes it easier to detect it
as TestCase class

Change-Id: Ic7ae7328e729c0f2d97afcf7c5be793d6a8df58f

Selenium: configuration file for beta cluster

Bug: T185011
Change-Id: Icf8588f0e82acf09e6654dd5dbf33479e4ecbbb7

Make IPTest::isNotIPAddress a test function

Only functions starting with test* are running by phpunit

Change-Id: I91061fb3d5132c61e01e368fab0df0e511b1920d

Remove @group from ParserTestRunner

It should be used on phpunit tests only

Change-Id: I0f73690606c8b92ce65ed1324394f5523b8156f5

Merge "Fix User::idFromName() ignoring cache for non-existent users."

Fix User::idFromName() ignoring cache for non-existent users.

This fixes a database time out issue where User::idFromName()
is repeatedly called from Special:Import due to the user not existing.
The response is cached as null, but isset() will return false on a null key.

Bug: T189786
Change-Id: I78705089a25dfec84d3c75bedaf623b1e5ee82c4

Merge "tests: fix Specified key was too long; in a schema"

Fix variable name in NewFilesPager::getQueryInfo

Bug: T189846
Change-Id: I11763ecbfd391deea0494386c0d7c1cb9861aa81

Merge "Add cleanup for handlers cache."

Add cleanup for handlers cache.

This is useful if we want to override certain handler for testing
but have no way to remove it from cache otherwise.

Change-Id: Icb4f9360c6e7684d15e91a7009a5a3a3e2febb37

tests: fix Specified key was too long; in a schema

MediaWikiTestCaseSchemaTest.sql comes with two varchar(255) combined
into a PRIMARY KEY.

That caused the testsuite to fail creating that imagelinks table with:
Error: 1071 Specified key was too long; max key length is 767 bytes

That is the limit for innodb and when using UTF8 it allows up to 3 bytes
per characters. Hence the key can be (255+255)*3 = 1530.

One can tune MySQL to bump the limit to ~ 3k with:
    innodb_large_prefix            = 1

Seems easier to just use smaller fields.

Change-Id: Ic4965b9eddc7ad9105c896e678ab9048ce0be8ef

Localisation updates from https://translatewiki.net.

Change-Id: Ief5597ccece86f3f249cf7db7511be269572986c

Merge "Fix field names and behavior in SlotRecord."

Remove `-webkit` vendor extension for browsers earlier than 2012

Removing `-webkit-gradient` value as it was just around for
Safari 4-5 and Chrome 2-9 both in released release range earlier
than 2012.
There's still the fallback color, which should be chosen wisely
anyways without cluttering our CSS output.

Change-Id: Iffb23fac44fc87cfb44f1056dfe1e4e1add444b4

Replace abandoned text color with WikimediaUI color palette one

Replacing leftover `#252525` with `#222` for code sanity.

Change-Id: I0406845950c65babe34672a6a59a390f56687e46

Merge "resourceloader: Fix mw.loader to compute combined version in packed order"

Merge "Convert MarkpatrolledAction to use OOUI"

Merge "Revert "Let install.php detect and inject extensions""

Merge "resourceloader: Make doRequest() an inline helper for batchRequest()"

Revert "Let install.php detect and inject extensions"

It is blatantly broken using non existing variables:
$installer and $options

This reverts commit 53926d9d6940eaa7f40086c9cf89af7bc726e73f.

Bug: T189567
Change-Id: Idd1f002bbd5d474f9fe386cd0e326e1fbfdec097

Merge "resourceloader: Fix mw.loader to compute version for current request only"

Localisation updates from https://translatewiki.net.

Change-Id: Id3b91ddc41a35c360ca3acb8fc2cf1c35321c967

Merge "resourceloader: Fix broken code in load.php mock used to make a test fail"

Merge "resourceloader: Simplify load.mock.php by removing includes dependency"

Merge "WikiTextStructure: Exclude <style> tags"

Merge "Correct PHP version in maintenance/dev/README"

Merge "Fix typo accessable --> accessible"

Merge "rdbms: allow construction of Database objects without connecting"

Merge "Let install.php detect and inject extensions"

Merge "Add Russian grammar forms to support Wikiversity"

Fix typo accessable --> accessible

Change-Id: I239a15e1bc3a311d9284285a04094afae331111d

Merge "rdbms: Add $join_conds to IDatabase::estimateRowCount()"

Merge "rdbms: inject the mysql index name aliases into Database"

Merge "resourceloader: Clean up and better document module list (un)packing"

rdbms: avoid throwing exceptions in Database::close() on reconnect

The check caused problems in reconnect() calls from rollback()
triggered by LBFactory::rollbackMasterChanges(). Since callbacks
are suppressed at that time, handleSessionLoss() does not consume
all of them, so the the call to open() triggered a close() call
that would error out since the callbacks are still there.

Only do that check if a connection was present beforehand.
Check for callback suppression before trying commit() too.

Also make writesOrCallbacksPending() check trxEndCallbacks.

Bug: T188875
Change-Id: Ia46d30d75132358a0b4f60e847937013781c1daa

Localisation updates from https://translatewiki.net.

Change-Id: I627c60bb104653ae4203b4a89b28fc2fdee3f96b

Avoid dumping big stack traces from unit tests.

Using assertFalse( $exception ) results in potentially very large stack
trace dumps, when $exception is an exception from deep inside MediaWiki.

Stack trace dumps include a dump of all parameters, resulting in console
output of (in my case) over 160 MB.

Change-Id: Ib78493d86531844a892586c071b2fa631ad765eb

WikiTextStructure: Exclude <style> tags

They aren't part of the article content, so they shouldn't be indexed
for search.

Bug: T189528
Change-Id: I3203f1f415eb821975098057d75c0e535b1fc76c

Correct PHP version in maintenance/dev/README

Change-Id: I1d52f60c20317efb3c36763fd47be09dc6dd57dd

Let install.php detect and inject extensions

For CI, I went with a dirty trick to find extensions and either
wfLoadExtension() or include them. That has since been ported to
Installer::findExtensions() and LocalSettingsGenerator::getText().

The WebInstaller() relies on that to detect extensions and let the user
tick the ones to be installed.

Add --with-extensions to install.php so one can include all extensions
from the command line.

Note: The CliInstaller always wfLoad() any skin it can find.

Bug: T189567
Change-Id: Ic8aa6bea4e7294e54fd5b71df267c9e934f78fce

Localisation updates from https://translatewiki.net.

Change-Id: I2deac4d88a30eab3503ff74147c92f429d554574

rdbms: inject the mysql index name aliases into Database

Also added LBFactory::setTableAlias() for consistency with this

Change-Id: Ie49003ff8fd5b99f75db9fae8fe0a184444254d4

Merge "Switch Block::purgeExpired to use AutoCommitUpdate"

rdbms: Add $join_conds to IDatabase::estimateRowCount()

So queries with joins can be estimated.

Change-Id: I9163cf9005d2c2001a88bb102eb4142f0322b0df

Merge "Fix typo acceptible --> acceptable"

Fix typo acceptible --> acceptable

Change-Id: I9304d830744238e4733240dd65c285fed61d0416

Add Hijri calendar in Urdu

Allows users to select their preferred calendar from Preferences.

Bug: T145266
Change-Id: I7299d0f84f629da781b100c4db6b2a30a7d1bb86

Merge "Remove "dash" case in preprocessToObj"

Merge "Move phpunit @group from file comment to class comment"

Localisation updates from https://translatewiki.net.

Change-Id: Iea4877827a12d5117578b8d1a6f1fefc53e1cdeb

Localisation updates from https://translatewiki.net.

Change-Id: I556a81048fbf024a585f7f586b91762eea568cb3

Convert MarkpatrolledAction to use OOUI

Bug: T189385
Change-Id: I9bd78166be5ab80e58498d2934531185404b17de

Move phpunit @group from file comment to class comment

Remove @group from non tests

Change-Id: Iae9ee3bc5f539a9b4ded8374006ab2993234450e

Merge "Fix typo in word compatibility"

Cleanup some comments related to DB replication

Change-Id: Icaaa5144beaedfebb88530480733a1e916c935fb

Remove "dash" case in preprocessToObj

This was introduced in 2877402 and removed in 186a182

Change-Id: Ibfa1ae1597bfc50ae6ea49402c7966ca042f12e5

Fix typo in word compatibility

Change-Id: Ic6ad14f5e156311cee3021cd427561dfcd68633a

Localisation updates from https://translatewiki.net.

Change-Id: Ib13656ed8c5c45facc4166963cd4a7f32ade30bf

Merge "Add ExternalUserNames::getLocal() to get local part of username"

Merge "API: Update prefixsearch/opensearch docs"

Add ExternalUserNames::getLocal() to get local part of username

Bug: T189320
Change-Id: I99908ced218c45ecbb23f390f065170292f153b4

API: Update prefixsearch/opensearch docs

The psnamespace parameter to action=query&list=prefixsearch is ignored
if pssearch has a valid namespace prefix. This should be documented.

Same for the similar parameters to action=opensearch.

Bug: T189139
Change-Id: I4914e5c915e6f8e5c0d5116f034ca671fe45a3f8

Remove useless use

A use declaration for a non-namespaced class in a non-namespaced context
causes a PHP warning.

Bug: T189302
Change-Id: I023e64c8194dd03cc3a1098e2d60c73f99bb02e3

Remove deprecated function wfBaseConvert()

Bug: T189300
Change-Id: I5f05beb105dca8e8079e770df562c004a689c2ed