Merge "Only try to show character difference if it isn't empty"

Merge "mediawiki.action.edit.preview: Remove dependency on module 'jquery.form'"

Merge "Fix typehint for multiple dbs in addCallableUpdate"

Localisation updates from https://translatewiki.net.

Change-Id: I5a54edceb1c256ae591518b788a6f9a93dd74abb

Fix typehint for multiple dbs in addCallableUpdate

Change-Id: Ic4fe6c529d9394cfd0c8c8652a28f47973342096
Follow-up: I0371ecd20101c5e622497e5912676300b040865e

mediawiki.action.edit.preview: Remove dependency on module 'jquery.form'

The dependency on jQuery.fn.formToArray from module 'jquery.form' was
added in e7505417 and removed in 30a2b712.

The module 'jquery.form' is now unused in core.

Change-Id: Ic257b426584d8aff3a3bad233074f628077250eb

Merge "Add SPARQL client to core"

Merge "Tag redirects from moves as new redirects"

Merge "Search for Hooks::runWithoutAbort in maintenance/findHooks.php"

Merge "Add clearUserWatchedItems methods to WatchedItemStoreInterface"

Add clearUserWatchedItems methods to WatchedItemStoreInterface

Follows up to 6cf900ffb0bec584e64202bc3.
The main implementation of these methods was merged, but the
method signatures were not added to the interface, nor to the
other implementations of that interface (NoWrite*).

I would have expected phan to spot this, if we were not
skipping so many of its checks.

Change-Id: I169d0312e9022d0d3b31d2424ec62dd9978f03bb

Merge "Improve ExtensionRegistry test coverage"

Improve ExtensionRegistry test coverage

Change-Id: I7f7445952f057995a3e3215145803affa5aceede

Migrage callers away from SiteStatsUpdate::__construct()

Change-Id: I4d81f5f87c93865e8fe1b7986074070376eab8f9

Various cleanup to SiteStats classes

* Mark method visibility
* Remove unused code/methods
* Use WAN cache process cache in pagesInNs()
* Remove "m" prefix from SiteStatsInit fields
* Avoid use of $wg variables

Change-Id: Iab4001f02c9b2e8667ca4bac033fd4f6ef272148

Merge "resourceloader: Removed tests for deprecated 'position' parameter"

resourceloader: Removed tests for deprecated 'position' parameter

This parameter is a no-op for these tests, as such, nothing was being
tested by specifying it in tests. The behaviour 'test' and 'test.top'
was identical.

Bug: T184257
Change-Id: Ia2bb731f00d4b4175f7b75174aeafaca9412329e

registration: Remove unused ExtensionRegistry::markLoaded

This is unused, and it would make more sense for whatever wants to add
credits to directly modify $this->loaded instead of incurring the
overhead of an extra function call.

Change-Id: Icce1a87d2dc8ce61cb05eace6e0b65d6cea4c58d

Move SiteStatsInit to its own file

Change-Id: Ief4e5f83d8dce93641623c2fa9c2195a564ee656

Merge "Make SiteStatsInit::doPlaceholderInit() use 1 for ss_row_id"

Make SiteStatsInit::doPlaceholderInit() use 1 for ss_row_id

This makes it consistent with refresh() and avoids having two rows
on new wikis. Also make the SELECT explicitly look for row 1.

Bug: T186947
Change-Id: I4f952888bf8fecc791366a9698e46d61a4ad4ff3

Merge "Add tests for ExtensionJsonValidator"

Merge "Complete VersionChecker test coverage"

Merge "SpecialLog: replace unnecessary strict comparison"

Merge "mediawiki.special.upload: Reorder functions"

Localisation updates from https://translatewiki.net.

Change-Id: I706d41286e7e63e42a7cce5413ed3b3b81c70dd3

Merge "mediawiki.Title: Reorder function declaration"

mediawiki.Title: Reorder function declaration

This avoids the eslint rule exceptions:
* no-use-before-define
* vars-on-top

Change-Id: Ie9c9145052fe8629b1aa50ac3e2ac62d6a10ec0f

Merge "Refactor Skin::buildSidebar() code style"

Merge "Update suppressWarning()/restoreWarning() calls"

Localisation updates from https://translatewiki.net.

Change-Id: I2a2c6ff85ebe6c1bf93c74e58dcdbf2e3afbee33

SpecialLog: replace unnecessary strict comparison

As suggested in 409541 replace unnecessary strict comparison
introduced in e6b2491736a9

Change-Id: I20a963be9aeac4274984cbcd2d7e97e040ceea9a

ParserOutput: Add 'deduplicateStyles' post-cache transformation

This transformation will find <style> tag with a "data-mw-deduplicate"
attribute. For each value of the attribute, the first instance will be
kept as-is, while any subsequent tags with the same value will be
replaced by a <link rel="mw-deduplicated-inline-style"> with its href
referring to the "data-mw-deduplicate" value using a custom scheme.

This also adds an $attribs parameter to Html::inlineStyle() so the
data-mw-deduplicate attribute can be added.

Note this doesn't actually depend on Ib931e25c, but action=mobileview
will break if it starts being used without that patch.

Bug: T160563
Change-Id: I055abdf4d73ec65771eaa4fe0999ec907c831568
Depends-On: Ib931e25ce85072000e62c486bbe5907f03372494

Search for Hooks::runWithoutAbort in maintenance/findHooks.php

Follows-up I94c7ab656bd1a04

Change-Id: I63faf3071e55bb0ba5aba1536bc98a969fb564af

Add mediawiki.widgets.visibleByteLimit to JSDuck index

Change-Id: Iaad0e4aad3e78294667e82b4fbd009e241f765e8

mediawiki.special.upload: Reorder functions

This avoids forward references and disabling of the eslint rule
no-use-before-define can removed.

Change-Id: I7bc229f4d95862f39a5a408218135dc6eae6155d

SpecialLog: Fix results when no offender is specified

With 467ee1e82f15 offender is not ignored when it's a
nonexistent username and a empty list is showed.
Ignore offender search when no offender is specified.

Bug: T186950
Change-Id: I93c9dafc9299d0ba1d01090471c5b49dcc904fe8

Update MediaWiki\quietCall() -> Wikimedia\quietCall()

Bug: T182273
Change-Id: Id7d8e176fcd93040e30e46cb64fc6a3d36bc8230

Update suppressWarning()/restoreWarning() calls

Bug: T182273
Change-Id: I9e1b628fe5949ca54258424c2e45b2fb6d491d0f

Localisation updates from https://translatewiki.net.

Change-Id: Iab9baefdc22d9652659a4bc2b976a677d080a962

Add tests for ExtensionJsonValidator

Change-Id: I883a502fc3ed6cd7b8651d9e5e78dba7177ead9c

Complete VersionChecker test coverage

And fix a few typos in the error messages and the accidental inclusion
of whitespace.

Change-Id: I4ea90c5543e394c3f30b1da1456e4676377188f7

registration: Simplify extracting path based globals

The code for ParserTestFiles and ServiceWiringFiles was identical, so
unify it into one function. For most extensions, this should save two
unnecessary function calls as a bonus.

Change-Id: If5ed189f97b7d3b9cd7b550f3ad2e90abe30005d

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

Tag redirects from moves as new redirects

Bug: T184117
Change-Id: Ic9a0abcb530787f652e7d439ecc7cac5f603be56

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

Refactor Skin::buildSidebar() code style

Change-Id: I054452ba6a83b3fc3c717a825acae6824d84bb49

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)"