Default installations to using RemexHtml for tidying

This combines two changes – defaulting tidying to on, previously off, and
defaulting the tidying library to RemexHtml, previously the tidy binary.
Config options are going to be a bit of a mess until we drop support for
the old tidy binary config route.

Bug: T185753
Depends-On: I0a8973f508fbf65160177b003260831639828eea
Change-Id: I6879a77a78d780c7c056d807dde20682c6097d1a

Merge "mw.special.apisandbox: Use a real button to insert values in multi fields"

Merge "rdbms: make Database query error handling more strict"

mw.special.apisandbox: Use a real button to insert values in multi fields

Stuffing an indicator inside a normal widget is weird.

Also, always add it, even when we also handle Enter presses on the input.

Change-Id: I7191b4f31bfe4b42a524c786994150f318fd8cd9

Merge "Swap isset by null check in ApiSetNotificationTimestamp/ApiPurge"

Merge "Change default of $wgLogAutopatrol from true to false"

Merge "SizeFilterWidget: Reduce width"

Change default of $wgLogAutopatrol from true to false

"$wgLogAutopatrol = true" basically just spams the logging table
and isn't useful in most cases (or ever?)

Bug: T184485
Change-Id: I418cc2fcc7017e5d2cbc0c6159b4c459b9c77ac7

Swap isset by null check in ApiSetNotificationTimestamp/ApiPurge

Change-Id: I1ebcbacf476a716d23b183dcbf379555ee1519dd

Api requests can throw a ApiUsageException, and that should be expected

Change-Id: I4aa6e7511747c0d811e2e617b88e681a6eda875d

rdbms: make Database query error handling more strict

Handle all errors in query() that might have caused rollback by
putting the Database handle into an error state that can only be
resolved by cancelAtomic() or rollback(). Other queries will be
rejected until then.

This results in more immediate exceptions in some cases where
atomic section mismatch errors would have been thrown, such as a
an error bubbling up from a child atomic section. Most cases were
a try/catch block assumes that only the statement was rolled back
now result in an error and rollback.

Callers using try/catch to handle key conflicts should instead use
SELECT FOR UPDATE to find conflicts beforehand, or use IGNORE, or
the upsert()/replace() methods. The try/catch pattern is unsafe and
no longer allowed, except for some common errors known to just
rollback the statement. Even then, such statements can come from
child atomic sections, so committing would be unsafe. Luckily, in
such cases, there will be a mismatch detected on endAtomic() or a
dangling section detected in close(), resulting in rollback.

Remove caching from DatabaseMyslBase::getServerVariableSettings
in case some SET query changes the values.

Bug: T189999
Change-Id: I532bc5201681a915d0c8aa7a3b1c143b040b142e

Merge "User: Remove deprecated, unused auth functions"

User: Remove deprecated, unused auth functions

According to a github search they're not used anywhere anymore and
already deprecated since 1.27:

 ::getEditTokenTimestamp
 ::getPasswordFactory
 ::passwordChangeInputAttribs

Change-Id: I904e46e7343baee28dc960f8fd5437ae9d17fc4b

Merge "CSSMin: Don't match empty string as remappable url"

Merge "Convert HTTP links to MediaWiki.org to HTTPS"

Merge "rdbms: rename and clarify getTransactionLagStatus method regarding begin()"

CSSMin: Don't match empty string as remappable url

The empty string being matched causes an undefined array index
notice in production, seen from various random gadgets, but spiked
after a change in MonoBook from last week that introduced a
broken background-image rule with empty string as url.

In browsers, that is actually interpreted as valid and "expands"
to the current url and re-fetches as Accept:image/*, silly, but
still broken. The broken icon was fixed in MonoBook, but we still
need to avoid trying to remap empty string as url.

Two changes:

1. Fix regex used by remap() to not match empty string.
   This was already fixed for the 'url()' case without the
   optional quotes, but with quotes, it was being matched as
   non-empty. This is now fixed by using '+' instead of '*'.
   Added tests to confirm they produce output, and PHPUnit
   is configured to also assert no Notices are emitted (which
   it converts to fatal exceptions).

2. Fix processUrlMatch() as sanity check to throw if the key
   is missing.

Bug: T191237
Change-Id: I0ada337b0b4ab73c80236367ff79c31bcd13aa7d

Localisation updates from https://translatewiki.net.

Change-Id: I56c13502176ea25a03148afc4f2256079e76dc82

Merge "Change index on rc_namespace, rc_title to rc_namespace, rc_title, rc_timestamp"

Merge "Fix parsing of <pre> tags generated by extension tag hooks"

Merge "Revert "Work around Firefox 57 attribute parsing regression""

Merge "Special:Preference: Check for changes before enabling Save button"

Convert HTTP links to MediaWiki.org to HTTPS

Bug: T189687
Change-Id: I846d3a74f617b98c8237f2a8babef85c6c128b14

Merge "Update OOUI to v0.26.2"

Update OOUI to v0.26.2

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.26.2

Depends-On: If570668a7fa615b2ffc3474ce5a161515af05f3b
Change-Id: I0131533e91fc0e308f90aea2555ead07740b093d

Merge "Use fallback languages in ResourceLoaderImage"

Merge "rdbms: clean up DBO_TRX behavior for onTransaction* callbacks"

Merge "Special:ProtectedTitles: Use HTMLForm"

Merge "Use new API for descriptions in TitleWidget"

Merge "Remove outdated comment for .background-image-svg"

Merge "RCFilters: Followup Iebc3176c5d: Bidi isolate filter tags"

Use new API for descriptions in TitleWidget

Bug: T191333
Change-Id: I5682f2fe3000d61c3760d22e9a567c762cf36710

Remove outdated comment for .background-image-svg

Since we dropped prefixed versions, we are no longer embedding
multiple times.

Change-Id: I58ad6f732289253c22cf12c74e340cd81c26e943

Merge "Convert Maintenance to the new shell framework"

Merge "Switch Installer to the new execution framework"

Merge "Switch ServiceWiring to the new execution framework"

Convert Maintenance to the new shell framework

Change-Id: Ibec8a84071fdf061f9831b2964143d99464cd4cc

Switch Installer to the new execution framework

Change-Id: Id04ba0751ff94e9614852e351377405f65b6ea1c

Switch ServiceWiring to the new execution framework

To test:

$wgMimeDetectorCommand = 'file -bi';
$services = \MediaWiki\MediaWikiServices::getInstance();
echo $services->getMimeAnalyzer()->guessMimeType('/vagrant/mediawiki/README');

Change-Id: Iaf124901c68292736e5588636f5ec746147f6a54

RCFilters: Followup Iebc3176c5d: Bidi isolate filter tags

Bug: T190718
Change-Id: I8e556624ddbc4e824a3fb9724f6fb721c18d2e85

Let install.php detect and inject extensions [2]

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.

The original patch 53926d9d was broken and got reverted. This is the
proper one.

Bug: T189567
Change-Id: I41820fce12e01224d64ab711e016835ef4d4febc

Change index on rc_namespace, rc_title to rc_namespace, rc_title, rc_timestamp

Bug: T190444
Change-Id: I5c59f1ee9306710505654a8db965f85ebf855873

rdbms: clean up DBO_TRX behavior for onTransaction* callbacks

* Make onTransactionIdle() wait until any transaction round
  is gone, even if there is no SQL transaction active. This
  is what onTransactionPreCommitOrIdle() already does.
* Decouple "transaction round mode" (DBO_TRX) from whether a
  round is active via a 'trxRoundId' LB info field. If rounds
  are enabled, but not is started, then the transaction state
  should be interpreted as "idle".
* Improve related documentation.
* Add more related unit tests.

Change-Id: I3ab18f577ec0375897fcb63f18f4ee2deeb436e9

Update OOjs to v2.2.0

Release notes:
 https://gerrit.wikimedia.org/r/plugins/gitiles/oojs/core/+/v2.2.0/History.md

Change-Id: If2f673da4c2f6c1576620d96ed28ac4c538ba119

Merge "rdbms: clean up session/transaction loss logic in Database"

Merge "OOUIHTMLForm styles: Replace fixed properties with `.transition()` mixin"

Merge "OOUIHTMLForm styles: Unify spacing aligned to new base font-size"

OOUIHTMLForm styles: Replace fixed properties with `.transition()` mixin

Change-Id: I16789cca4d797e3147dfbaac8b50280d0240a1b7

OOUIHTMLForm styles: Unify spacing aligned to new base font-size

Follow-up to I693d168d2ccf2babbcfe8952af3e1c262aa97773

Bug: T177668
Change-Id: I86077eadbf0986868cbaabd41c33b48e333d7973

Make autopatrol third option in rc_patrolled and use it in API

Bug: T184791
Change-Id: If64ba8b845b122a87f20646dddf72ef257b793cb

Merge "Populate ar_rev_id and make it non-nullable"

tests: fix PHPUnit @requires for APCU

@requires is an annotation used by PHPUnit to flag a test as requiring a
PHP function or extension. c214c9a95 changed it to list two functions
separated by a slash:

   @requires function apc_store/apcu_store

However that is not recognized by PHPUnit and the test is always
skipped.

Change the @requires to require the APCU extension.

Repro case:

  phpunit.php --verbose includes/libs/MemoizedCallableTest.php

Follow-up c214c9a95255abf1032d389d0fe8bcbde38b55db

Bug: T140587
Change-Id: I1fdd52c8fc8ca1c7c0c7f914e6da8774799ba5d8

Merge "LanguageConverter tweaks to Pig Latin converter"

Localisation updates from https://translatewiki.net.

Change-Id: I23379ec2eb8320618f5be56618ed5c45587c8825

Merge "Add further test cases to deleteAutoPatrolLogsTest"

Add further test cases to deleteAutoPatrolLogsTest

Also make --batch-size work and make an if check nicer
(makes no functional difference AFAICT).

Bug: T189594
Change-Id: I3ce63386cb35441acfa226f313ec8aac1aa417a6

Merge "Introduce deleteAutoPatrolLogs maintenance script"

Introduce deleteAutoPatrolLogs maintenance script

It's useful to delete old patrol logs that are not useful

Bug: T189594
Change-Id: I605bb85f172eb25df45ed83ce50a3d1044f1c281

Merge "DB: Add img_description_id column"

Merge "build: Updating mediawiki/mediawiki-codesniffer to 17.0.0"

rdbms: rename and clarify getTransactionLagStatus method regarding begin()

Make sure any value from the last transaction is cleared and not used.

For sanity, make getLagFromPtHeartbeat() use this regardless of the age of
the transaction unless it returns null.

Change-Id: I52df6147f99736ad1a389ae70d347ae968e50c7f

Revert "rdbms: make getMasterPos() ignore GTIDs outside of gtid_domain_id"

This had a noticeable increase in LoadBalancer::doWait timeouts.

This reverts commit ceb7d61ee7ef3edc6705abd41ec86b3afcd9c491.

Change-Id: I7004d55a05c20f646f70d778d7b6496123e270a4

rdbms: avoid incorrect warnings in getLagFromPtHeartbeat()

Change-Id: I4c0d42be0ee0c518d7bacdae0f76feb53007861e

Merge "Make LanguageConverter roman-numeral cases consistent"

Populate ar_rev_id and make it non-nullable

Revisions deleted before MediaWiki 1.5 do not have a value in this
field. This is going to be a problem for migration to the MCR schema, so
provide a maintenance script to clean this up.

Then, for good measure, change the schema to make the field
non-nullable.

Bug: T182678
Change-Id: Ie2e11f12a30f379db32c3e074658012c6f93adb0

Merge "Selenium: update WebdriverIO configuration"

Localisation updates from https://translatewiki.net.

Change-Id: I35703b36f31021e7ba272c3b1869808cb95b580f

Special:Preference: Check for changes before enabling Save button

* Changing mousedown and keydown to mouseup and keyup
  …so that isPrefsChanged gets the updated value and can return
  something useful

* Changing one to on
  …so that if a value is updated and then changed back to the
  original, the button can be disabled again

Bug: T180538
Change-Id: I7146f8d3166e09e13b26321e7a6a7c8ac74f9aaf

LanguageConverter tweaks to Pig Latin converter

Add apostrophe to the set of valid word characters for the en-x-piglatin
converter, so that "don't" and "can't" are properly converted to Pig
Latin (eg, to "on'tday" and "an'tcay").

Add an optional `s` before `qu` so that "squish" is properly converted
to "ishsquay".

Change-Id: Ibc5cf2c007a42d9447688b857aa75f9a3d8ae152

Make LanguageConverter roman-numeral cases consistent

Add a look-ahead to ensure that the regex intended to match roman numerals
doesn't also match the empty string.  Tweak the regular expressions slightly
to ensure that Sr/Ku/Crh all have identical regular expressions.

Change-Id: If43bf99a21c42c6c5050f814c0bc99edec353228

Use fallback languages in ResourceLoaderImage

Bug: T191120
Change-Id: Ic72aead466c12d344bac359654e6dcc7f0e5c8cb

Add Hebrew alias for special page ApiSandbox

Change-Id: I09bf2951b5bc18bf9f7080d178ea74466d14e97e

Localisation updates from https://translatewiki.net.

Change-Id: I20bbb93b56df4246c31344f3929742af581c3118

Merge "MessagesBs.php: Update namespaces names"

Merge "rdbms: avoid lag estimates in getLagFromPtHeartbeat ruined by snapshots"

Localisation updates from https://translatewiki.net.

Change-Id: Id315b02132af92cc2d4b61a28da34be32e333e54

rdbms: avoid lag estimates in getLagFromPtHeartbeat ruined by snapshots

Bug: T190960
Change-Id: I57dd8d3d0ca96d6fb2f9e83f062f29b1d53224dd

Merge "rdbms: update IDatabase::getLag comments"

Merge "rdbms: add more error logging to DatabaseMysqlBase::masterPosWait"

MessagesBs.php: Update namespaces names

Bug: T172050
Change-Id: I3eb31ef0785138b14342c27a4232bec2b782c0bf

Selenium: update WebdriverIO configuration

No changes are made to the configuration file. It has just been
regenerated with the latest WebdriverIO version. Minor whitespace
cleanup has been made to the file after it has been generated so it
conforms to our ESLint configuration.

This is the first step in updating configuration so videos of failed
tests are recorded.

Bug: T179188
Change-Id: Id0db3743ae7d828357b1527770bb7dcd2e02f99b

Merge "mediawiki.action.view.rightClickEdit: Remove redundanat dom-ready handler"

rdbms: add more error logging to DatabaseMysqlBase::masterPosWait

Change-Id: Ia687ddeffc9c9427c6126d8ed831fb52b38e8260

Merge "Improve docs for revision, archive, and slots schemas in tables.sql"

Localisation updates from https://translatewiki.net.

Change-Id: I0c57d6f3f632ddd06f36f833c53f97d96e7dd645

rdbms: update IDatabase::getLag comments

Also add missing __METHOD__ to getLagFromPtHeartbeat()

Change-Id: I4257b1d47a88779b47d807a881561c331ff3aa30

Merge "rdbms: remove some dead code from Database::begin"

rdbms: remove some dead code from Database::begin

The "since" filed is computed after getLag(), so this was useless

Change-Id: I252602e85709b1cfc191d4783bd1c53d5b3e8fe8

Merge "Improve docs for archive schema in tables.sql"

Improve docs for revision, archive, and slots schemas in tables.sql

Follows up Id2257827.

Change-Id: Ifd61eaa51a6f7781522cecdbf8c563465e97e885

mediawiki.action.view.rightClickEdit: Remove redundanat dom-ready handler

This file was wrapped in a jQuery() dom-ready callback, but it doesn't
and shouldn't have that because it needlessly delays execution.

I probably added because back then, using the first parameter of jQuery()
was a common way for aliasing it to '$', but we use closures for that
nowadays.

Change-Id: I65768564ed556828fd193dfe99d11370fa745112

rdbms: clean up session/transaction loss logic in Database

* Refactor the code to be less cluttered, moving some logic
  to reconnect(), which is now named replaceLostConnection()
* Handle the case of a second connection loss on query retry
* Make canRecoverFromDisconnect() check for temporary tables
* Do not clear session-level variables on deadlock errors
  since only the transaction is lost, not the whole session
* Make sure any empty transaction is destroyed on deadlock
* Attempt reconnection *before* triggering the transaction
  callbacks since some of them  might want to use the database
* Define wasConnectionError() for postgres
* Remove unused return value from handleSessionLoss()

Change-Id: Ic1dcab03f087ce310637210e8e9bc0771e44f045

Merge "Allow schema overrides to drop tables."

build: Updating mediawiki/mediawiki-codesniffer to 17.0.0

Change-Id: Ib494b47c54fe6354d166055b1e1b31d3583bb992

Allow schema overrides to drop tables.

This allows unit tests to use schema overrides that drop tables,
in addition to overrides that create or modify tables.

Change-Id: I59761c7db7f83698749324ca6b9ffced86ab1249

Merge "Bump avro to 1.8"

Merge "Fix MockMessageLocalizer"

RCFilters UI: Remove CSS customization for help icon

This fixes most of the misalignment after the OOUI v0.26 upgrade.

Bug: T191014
Change-Id: I4623836d134fa52fb6b562e671a5da104b787b7d

Merge "Add protection for User: JSON pages in the same manner as JS & CSS ones"

Add protection for User: JSON pages in the same manner as JS & CSS ones

Also recognise MediaWiki: JSON pages (with the existing protection of
the editinterface right).

Bug: T76554
Change-Id: Idba166d82ee6dd507d7345c9bdbefc8ca78ed7b4