Complete ExtensionProcessor test coverage

Change-Id: I80103e9b7eaba0bb3efa3b736f3f188a536f2919

Merge "editTests: Use the correct list of parser test files"

Merge "Add a few more @covers to ParserIntegrationTest"

Merge "rdbms: make safeWaitForMasterPos() respect "waitTimeout""

rdbms: make safeWaitForMasterPos() respect "waitTimeout"

Also remove the "m" prefix from the member variable.

Change-Id: I695f520a69980214da5e9b7fbd2c80924b3a4038

Merge "Fix ParserOutput::getText 'unwrap' flag for end-of-doc comment"

Fix ParserOutput::getText 'unwrap' flag for end-of-doc comment

The closing div might be followed by debug information in HTML comments.

Bug: T186927
Change-Id: I72d4079dfe9ca9b3a14476ace1364eb5efdee846

editTests: Use the correct list of parser test files

$wgParserTestFiles is deprecated, so this wasn't running the core parser
tests. Using ParserTestRunner::getParserTestFiles() includes everything,
including autodiscovered extension parser tests.

Change-Id: Ie3b02565c184e8e06931ab52a39ca8ae0877aab9

Merge "Use WatchedItemStore clearing in SpecialEditWatchlist"

Merge "Add clearWatchedItems to WatchedItemStore"

Merge "Avoid pointless DB_MASTER connections in User::clearSharedCache()"

Use WatchedItemStore clearing in SpecialEditWatchlist

This switches to using the WatchedItemStore that has the ability to
delete all watchlist items for a user using the jobqueue.

The job was introduced in:
Icea573a10078ea3f09dc2e4e9fdc737bf639935d
Functionality added to the store in:
I2f8c8be860a03116535dbd1e6a18ce2e8bc546b4
This patch is the first code calling the new functionality.

If the number of watched items for the user is less
than the number that can be completed in a single
batch then they will still be deleted during the
interactive session.

Bug: T68212
Bug: T129481
Change-Id: I2f8c8be860a03116535dbd1e6a18ce2e8bc546b4

Add clearWatchedItems to WatchedItemStore

Change-Id: I67d1057c76ddccece4727f4df701a3ad14c3bbaa

Avoid pointless DB_MASTER connections in User::clearSharedCache()

Change-Id: Ifc4f489cea4460185cb2d8317f174cabd67f5862

Add a few more @covers to ParserIntegrationTest

Change-Id: I638077683d9d3cb40f81b4f4d594f16c349f1519

Catch Error exceptions in MediaWiki::run()

This avoids triggering other exceptions down the road such as
"Got COMMIT while atomic sections FileDeleteForm::doDelete,
LocalFile::lockingTransaction are still open". Those would
happen in LBFactory::__destruct(), when it tries to commit any
dangling transactions (firing attached callbacks too). Just like
with the Exception case, the DBs needs to all be rolled back.

Also make LoadBalancer::rolbackMasterChanges() rollback any
explicit transactions even if they have no pending changes.
This clears up the state to avoid later atomic section errors.

Change-Id: Ic0b6b12c1edc1eec239f4f048359b3bbb497d3ff

Fix CommentStore->createComment() call in LocalFile.php

This was incorrectly changed in I3abb62a5, createComment() is not one of
the methods that was adjusted in that patch.

Change-Id: I9c26e0ad69ffc6fa5349a6975a4134efb3cf0eff

Merge "rdbms: make DOMAIN_ANY ignore bogus MySQL DB names in config"

Localisation updates from https://translatewiki.net.

Change-Id: I79bd58ffc0181305a64d1a8c379b58547e3a745c

rdbms: make DOMAIN_ANY ignore bogus MySQL DB names in config

This regressed in 14ee3f2, trying to select the server config
"dbname" value as the database on connect. If that config is
bogus, then the connection attempt would fail. Instead, always
pass null to the driver's connection function if the DB domain
doesn't matter.

This affected WMF sites during lag checks, on account of the
"serverTemplate" value in $wgLBFactoryConf always using the local
wiki domain (whether the cluster had such a database or not).

Use strlen() as mysqli sees null and "" as the same for $dbname.

Bug: T186764
Change-Id: I6699d17c0125a08415046211fee7906bbaf2c366

jquery.textSelection: Add 'replaceSelection' method

replaceSelection is a little brother of encapsulateSelection.
It can't do as much, but tries really hard to do it well.

Alternative textSelection APIs can implement it in an optimized
way if they can more efficiently replace small fragments of the
textfield's contents than the entire thing.

Additionally:
* encapsulateSelection uses replaceSelection internally.
* setContents now also preserves the scroll position.
* setContents is now chainable.

Bug: T185917
Change-Id: I05086ddce2ddd857ff88eba75cfdb6bf877bd924

Merge "Make it possible to not propagate errors to PHP"

Merge "Fix sessionfailure i18n message during authentication"

Merge "jquery.textSelection: Implement 'encapsulateSelection' in terms of the other commands"

Make it possible to not propagate errors to PHP

Add a $wgPropagateErrors configuration variable which can be used
to prevent passing handled errors to PHP (and thus logging them twice).

Bug: T45086
Change-Id: I64ab09762a04de2007b7d7864e3c504a1d6f8aee

Merge "Update mediawiki/at-ease"

Merge "ApiFormatBase: Encode filenames in Content-Disposition"

Merge "Add @covers tags for parser tests"

Merge "Fix phpdoc description of RevisionStore::insertRevisionOn"

Merge "DI for CommentStore in RevisionStore"

Merge "Remove unused code from LegacyHookPreAuthenticationProvider"

Fix some broken @var comments and add missing uses

Change-Id: I2091f4107b6c23d9c3b698999147ce3a2ec38b66

Fix phpdoc description of RevisionStore::insertRevisionOn

Change-Id: Iae0ac4cf98b1cc93eb5a9f8bb97e64b5ea1d9f0f

DI for CommentStore in RevisionStore

Change-Id: I527388514489e79c53b6016a8bd3119ee1750c83

Remove unused code from LegacyHookPreAuthenticationProvider

Change-Id: I699947fe02f649e24c957eb9790fcff91cb2c14a

Merge "rdbms: apply minimum sanity timeout for all cases of LoadBalancer::doWait()"

Merge "objectcache: remove unused constant and expand on a comment"

Add @covers tags for parser tests

Change-Id: I24d3550a07be7a5699475047eb78806f36caec2e

Merge "Revert legend when results are grouped"

objectcache: remove unused constant and expand on a comment

Change-Id: I3342497d657291d20325a3d87cc634f53d7776c4

Merge "Drop XCache support"

Merge "Fix for warning in ForeignAPIFile when no thumbnails"

Drop XCache support

It was never super popular anyway, APC was always the best option.
The project has no plans to move to PHP7, so it's quickly reaching
its end of life. Oh, and Fedora dropped it from their repos 2 years
ago.

Change-Id: Ia3257e86a6323d8943f04a5c05c72c0bd4c4b0a9

Localisation updates from https://translatewiki.net.

Change-Id: I7fa8feca5bdc1394bc24f3d93aaee1f7be2c2f9e

ApiFormatBase: Encode filenames in Content-Disposition

The return value for ApiFormatBase::getFilename() was formerly
documented as "must be encoded for inclusion in a Content-Disposition
header's filename parameter." While this is ok for the common use case
where the module is returning a constant string or can assume whatever
it gets back from getExtensionsForType() is ok, it's not in general a
good idea to make all callers handle that.

Further, it's not possible to represent characters outside of the
ISO-8859-1 character set in a 'filename' parameter. You have to use
'filename*' to do that (see RFC 5987 and RFC 6266).

So, this patch changes the definition of getFilename() to remove the
encoding requirement, and adds code to properly convert and escape the
value for the 'filename' and (if necessary) 'filename*' parameters.

Note this may give unexpected results (double encoding) if any module
actually is returning an encoded filename. I don't see any such cases in
core or in extensions in Gerrit.

Change-Id: I0c2749a847b639f228efff4e1917a61612a1f7d1

jquery.textSelection: Implement 'encapsulateSelection' in terms of the other commands

Any users of jquery.textSelection that add an alternative API using
'register' no longer have to override 'encapsulateSelection' (which is
a big beast), as long as the other commands are implemented.

Bug: T185917
Change-Id: I39bbbce20336a1286a07bf8032acdfb8a7e0cdb6

jquery.textSelection: Remove incorrect browser hack for old Opera

This code appears to be incorrect, at least on Opera 12.18 on Windows 10,
where I just tested it. It causes the selection to be misaligned, rather
than correct it.

For example, the following code:

    $textbox.textSelection( 'encapsulateSelection', { peri: 'foo\nfoo' } );

Causes the selection to extend one character past the inserted text with
the hack, and to be shown correctly without the hack.

It is theoretically possible that this hack was necessary for some other
operating system, but then it should have also been applied to 'insertText'.
Also, Opera 12 only has a usage share (barely) large enough to show up on
<https://analytics.wikimedia.org/dashboards/browsers/> on Windows (XP).

Change-Id: Ifee877c6202bcfc0fecb65220e34b4b39659dbeb

jquery.textSelection: Remove unnecessary checks

Some of the commands were wrapped in unnecessary checks (which were
also inconsistent between the different commands):

* Checking if this element is hidden. I'm guessing this was a
  workaround for some IE 6 bugs where doing things to the selection
  of hidden elements would break horribly. In any case, this should
  not be this code's responsibility.

* Checking for undefined this.selectionStart. I probably should have
  removed these in 1430ca85dcf9cb3f77a92bee6a915473e465f649. The only
  remaining case for them would be if .textSelection() is called on
  something that is not a text input element, which should not happen.

Change-Id: Ib09e64399921fe3861efd4877c6ecb2f900aef09

Merge "rdbms: remove obsolete getReplicaPos() code"

Fix for warning in ForeignAPIFile when no thumbnails

Remote audio files have no thumbnails, so this check in
ForeignAPIFile for thumbnails by iterating over the result
from FileBackend::getFileList() could throw a warning about
invalid input to foreach, as it may return null.

Change-Id: I07f37ed5a299e1a3c9a1e6bddcbc5582f99e5274

Merge "Add @covers for BlockLevelPass"

Merge "jquery.textSelection: Rewrite 'scrollToCaretPosition'"

Update mediawiki/at-ease

Updated
* mediawiki/at-ease v1.1.0 -> v1.2.0

https://github.com/wikimedia/at-ease/compare/v1.1.0...v1.2.0

Bug: T182273
Change-Id: Idd686bf84a44f2106ae13a31516872545946aa03
Depends-On: I1966eaec7e4105f876206ab4cd606cefdaeb0add

Add support for Kumyk (kum)

Bug: T186647
Change-Id: I704e493d743d54071fc6f5341ea04e8a48ebaad7

Merge "Add support from Guianan Creole (gcr)"

Use new TagMultiselectWidget in ApiSandbox

It seems to be generally a 1:1 replacement.

Bug: T183299
Change-Id: I191a99bd373b0fb320770e9089278a16964d80d2

rdbms: remove obsolete getReplicaPos() code

Also fixed incorrect comment for getGtidCoordinates()

Change-Id: Ie2ce8a8bba230ceb6d068957e5ffeb360c3fe473

Update OOUI to v0.25.2

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

Depends-on: Ib00f16ef5e74732068822dc10d43314098b8fa59
Change-Id: Ie327a10622b64b7f969283497822c494c513585b

rdbms: apply minimum sanity timeout for all cases of LoadBalancer::doWait()

Change-Id: I61b4da6429b390e5aae65d1a30adc6df59687d8c

Merge "MWHttpRequest: Restore ability to pass null for $options"

jquery.textSelection: Rewrite 'scrollToCaretPosition'

The previous version tried to assume some platform-dependent
things about text rendering and guess the position based on
that. It did not work very well.

Change-Id: I9f4c465692cf128b7758de4ca75f977e8f8be83e

Merge "Hard deprecate UtfNormalUtil"

Hard deprecate UtfNormalUtil

Deprecated in 1.25, last vestiges have been removed.

Inline one last function that's not supposed to be public anyway.

Change-Id: If44c6a9cce7d2d95dc63e6f135b73b8c969f1c07

Merge "jquery.textSelection: Document methods and parameters"

Revert legend when results are grouped

Since Ifecaaff572, legend was using position: absolute when results are
grouped by page. Most users found such behavior annoying and unexpected,
so positioning is being reverted to previous state.

Only change besides reverting is preventing collapsed legend to push
away the surrounding results. Without that, we would have scenarios
where legend is expanded and collapsed, but results are pushed away as
if legend is still expanded.

Bug: T182156
Bug: T184165
Change-Id: I5adb7e1cb793ec30fba9d5d9674a7fa294d9a3a5

Merge "jquery.textSelection: Move copyright comments to the top of the file"

Merge "Fix localization of time in Turkish"

Add support from Guianan Creole (gcr)

Bug: T186635
Change-Id: I13334bb83562fc943fd69ef1f0fc3cd85aceffe1

Localisation updates from https://translatewiki.net.

Change-Id: I70882c277fb173e316e0ed90712fbe93628ab4d6

MWHttpRequest: Restore ability to pass null for $options

Prior to 3de744597e34, it was possible to pass `null` for $options.
Restore that by setting the default type to be null, and explicitly
document that null is a supported value in the doc block.

Also update the signature of MWHttpRequestTester to match.

Change-Id: I74d586afa5527e0a30ea99f3989f4dd12fa9fea1

Fix sessionfailure i18n message during authentication

Confusing message about session failure, telling the user to
resubmit the form is a better feedback message as most of the
time, it could be a session timeout causing the failure.

Bug: T181960
Change-Id: I8a644ac8adb89ec83a8e3b5a5ad4a2d8dc5c1d8c

jquery.textSelection: Document methods and parameters

Change-Id: I9e4112a75ae7abc8c479efc39a438c3b59933418

jquery.textSelection: Move copyright comments to the top of the file

Functions' doc comments should only document the function.

Change-Id: I56f21b49a35bf191a930d39af2f2020cdeb6ca30

DI for CommentStore in WatchedItemQueryService

Change-Id: I93d9ed5f66297da7009d5b99aa9ed48e1b10582f

Merge "Mock CommentStore in some RevisionTest tests"

Merge "Pass $key into CommentStore methods and use MediawikiServices"

Merge "Add @covers for RemexStripTagHandler"

Mock CommentStore in some RevisionTest tests

Change-Id: I16b00a5514547d93a2308393098e3b363505374e

Merge "[MCR] RevisionStore::getTitle final logged fallback to master"

Add @covers for BlockLevelPass

This class was split out of Parser, so it now needs separate covers tags.

Change-Id: I06c4a6a4fac3d6ff13924e3ca45ee134f7eeab20

Add @covers for RemexStripTagHandler

This internal class is only used by Sanitizer::stripAllTags().

Change-Id: Ib913ee14524539216305da7e3183c07ab7d72cb5

Merge "Use HTTPS instead of HTTP for interwiki"

Merge "installer messages: Move to https: links when available"

Merge "Follow-up ed76b6736: Also remove the now-unused $mExtStyles"

installer messages: Move to https: links when available

Change links used in the installer from http:// to https://
when the linked content is available at the same hostname
using HTTPS.

Change-Id: Ib5afcf5ba4aa6da360c926794e6a021e85008d6c

Localisation updates from https://translatewiki.net.

Change-Id: Ia4e5683f7d850983a5fd76097f2cd3c82daca667

Remove superfluous spaces and semicolons in comments

Change-Id: Ib4b452f1843ec250c8c1fcc2a738d80726b6135d

Follow-up ed76b6736: Also remove the now-unused $mExtStyles

Change-Id: I7622ba15b03df97213acd82e4a3fa78267a30d55

Merge "jquery.textSelection: Fix typo"

Merge "jquery.localize: Fix rendering of generated documentation"

jquery.textSelection: Fix typo

Probably nothing is using this event.

Change-Id: I5e4a8a741b46868f8a1d55257a980aa0853fbb6c

Merge "Force CommentMigration MIGRATION_OLD in 2 RevisionTest tests"

Pass $key into CommentStore methods and use MediawikiServices

This allows CommentStore to be added to MediaWikiServices
without the need of an aditional Factory.

This change includes a compatability layer to allow the behaviour
from 1.30 to continue to be used while deprecated.

CommentStore::newKey has been deprecated.
Keys are now passed into the public methods of CommentStore
where needed.
The following CommentStore methods have had their signatures changed
to introduced a $key parameter, but when used in conjunction with
CommentStore::newKey behaviour will remain unchanged:
  * CommentStore::getFields
  * CommentStore::getJoin
  * CommentStore::getComment
  * CommentStore::getCommentLegacy
  * CommentStore::insert
  * CommentStore::insertWithTemplate

Change-Id: I3abb62a5cfb0dcd456da9f4eb35583476ae41cfb

Use HTTPS instead of HTTP for interwiki

Switch from HTTP to HTTPS where the HTTP URL redirects to the HTTPS URL
or where the HTTP URL and the HTTPS URL deliveres the same content.

Change-Id: I51947ae93cc9e6a30bb31eee7ff13561fc94ece4

jquery.localize: Fix rendering of generated documentation

JSDuck expects an empty line before and after code samples.

Change-Id: I45cf3a8e7c88476e61a681cfc44fc497e8f1c67e

Merge "Preload module 'mediawiki.notification'"

Merge "Fixing typos for ucfirst magic word translation into Ukrainian: ПЕША_ДІТЕРА → ПЕРША_ЛІТЕРА."

convertExtensionToRegistration: Set requires key in extension.json

The highest manifest version is not supported from the start of
extension.json
For extensions converted to this highest version the mininum core
version must be specified to avoid load problems in the extension.

Using 1.29.0 for manifest version 2 due to T149757 /
Id1071fc0647892438e5cd0e3ee621fbdaaa64014

Change-Id: Iea5ba589c70958db7500cf3587b5ebd738532026

Fix back-compat method Article::updateRedirectOn

No reason to set $lastRevIsRedirect to null before pass it as argument

Change-Id: I5e3c54d828a381c682725a43e322e660186b38cb

Merge "Use wikimedia/object-factory 1.0.0"