build: Bump npm development dependencies to latest

 grunt-contrib-copy     0.8.2  →    1.0.0
 grunt-contrib-watch    0.6.1  →    1.0.0
 karma                0.13.19  →  0.13.22

Change-Id: Idc5c1ac97580e91d9c958c55cdfdda681330c254

Merge "Handle HTTPS when running jobs asynchronously"

ListFiles: Call Skin::setRelevantUser() when applicable

Also validate user name before calling `User::newFromName()`

Bug: T129825
Change-Id: I7481ab0fc1720e5e840f0d552934324f676c0241

Merge "Many more function case mismatches"

Many more function case mismatches

Change-Id: I5d3a5eb8adea1ecbf136415bb9fd7a162633ccca

Merge "MediaWikiTestCase::assertException() was removed"

MediaWikiTestCase::assertException() was removed

Deprecated since 1.22. No uses in core or Gerrit extensions remaining.

Change-Id: Icae4077078c73d766d3c1c9e9e569922151296d9

Update grunt-karma to 0.12.2

Reason because this update will include a fix for incompatibility with
grunt 1.0.0 when it is released.

Change-Id: I39f01e3a68b764a4638c3465a1d116aefd517ef5

Fix numerous class/function casing

Change-Id: I23982bfa0548c9ea3bdb432be7982f1563930715

Merge "API: Allow fetching login token from action=query&meta=tokens on private wikis"

Merge "resetUserEmail: Allow resetting email without scrambling password"

Merge "Actually make $wgNoReplyAddress default to $wgPasswordSender"

Merge "Localisation updates from https://translatewiki.net."

Merge "Try to fix some other broken-looking legacy maintenance script options"

Merge "Register importImages options without arguments as actual maintenance script options so they are usable"

API: Allow fetching login token from action=query&meta=tokens on private wikis

The problem is that ApiQuery requires the 'read' right even though
ApiQueryTokens doesn't.

So, we introduce an exception: if action=query gets only meta=tokens
(and optionally rawcontinue and indexpageids, since they don't affect
anything), no other modules and nothing in the ApiPageSet,
ApiQuery::isReadMode() will return false.

Bug: T130112
Change-Id: I83dafb0305ff0cb1fc3bac668b88b5d2022e5880

Localisation updates from https://translatewiki.net.

Change-Id: I935a1d0bbc9fa5028f2fa1c9b05272ec2e416a7a

Merge "ApiQueryStashImageInfo: Don't throw exceptions when the user is not logged in"

Merge "Use imported namespaces, don't fully qualify"

Merge "Remove $wgAllowMicroDataAttributes and $wgAllowRdfaAttributes"

Remove $wgAllowMicroDataAttributes and $wgAllowRdfaAttributes

$wgAllowMicroDataAttributes and $wgAllowRdfaAttributes have been
introduced in MediaWiki 1.16 and required at this moment $wgHTML5
to be true. This last setting has been removed in MediaWiki 1.22.

To simplify the code maintenance and the configuration complexity,
those settings are removed and the features are always available.

RDFa users must now explicitly set $wgHtml5Version to a RDFa
version. Currently the correct values are:

  - HTML+RDFa 1.0
  - XHTML+RDFa 1.0

Bug: T130040
Change-Id: I17a7bff2cad170e381eabf0aec4e26e4fd0cddc3

Merge "skins: Minor doc and code clean up in getCategories()"

Merge "Change "COMMIT ENQUEUED" warnings to level INFO"

Merge "Make spreadAnyEditBlock() calls respect read-only mode"

Merge "SpecialUploadStash: Convert form to OOUI"

ApiQueryStashImageInfo: Don't throw exceptions when the user is not logged in

Bug: T130253
Change-Id: If70b919c65a21b57b3022d495756bf250b0ba786

Merge "Link to Special:WhatLinksHere in CatWatch bundeled summary"

Actually make $wgNoReplyAddress default to $wgPasswordSender

Bug: T130368
Change-Id: Ifc32f0cf82ea80361822f612b0c6aeb427721606

Use imported namespaces, don't fully qualify

Remove unused import in CachedBagOStuff

Change-Id: I60ca8acc0e34384ce6ad6cde1cfb3a5dc8021e5f

build: Remove redundant jscs rule 'disallowQuotedKeysInObjects'

{ "allExcept": [ "reserved" ] } found in the wikimedia preset is
the same as the customized "allButReserved" present in MediaWiki,
except that the latter is deprecated as stated at:
http://jscs.info/rule/disallowQuotedKeysInObjects

Change-Id: I5de2a8f7ad42857ccdc793551d08c77e2aae8156

Make spreadAnyEditBlock() calls respect read-only mode

Bug: T129968
Change-Id: Ib391bd880fa9e04777a355634035183065de5b0a

Merge "Log slow diffs"

Change "COMMIT ENQUEUED" warnings to level INFO

These are too unimportant and spammy

Change-Id: I2205d9b42014f138ac523967f91df6d340234256

Merge "Rename ApiRequest to ApiAction"

SpecialUploadStash: Convert form to OOUI

Bug: T130328
Change-Id: I2ebbc783ff93dd47c552fa88230ea5ab2dbefb1e

Merge "SpecialUploadStash: Disable output gzipping when outputting remove scaled thumb"

Rename ApiRequest to ApiAction

Half of the work needed for handling this logging stream was done using
the channel name "ApiRequest" and the other half was done under the name
"ApiAction". The ApiRequest naming is easier to change at this point.

Bug: T108618
Change-Id: I5797731abeba22ef6ced7c8407ee670344d35b3d

SpecialUploadStash: Disable output gzipping when outputting remove scaled thumb

This is the only code path that wasn't calling wfResetOutputBuffers()
already. All other ones end up in outputLocalFile(), which calls
it via StreamFile::prepareForStream().

I can't really tell why this causes Varnish stuff to go bust, but it's
definitely the right thing to do anyway.

Bug: T130204
Change-Id: Ib6bc9c64c6c893fd7f5a5d906da32d9bee509a98

Cast API timeSpentBackend to an int

Fixes {"timeSpentBackend":"Expected integer, but received double"} Avro
encoding error.

Bug: T108618
Change-Id: Iea79d14afe0ef0d1a9ea88096b7bbd90083df757

Localisation updates from https://translatewiki.net.

Change-Id: I610b8736afb5cc84d6a057c521a241c3656b73b8

skins: Minor doc and code clean up in getCategories()

* Clarify @return as being a string of HTML.
* Use Html class instead of inline string concatenation.

Change-Id: If3ff352aca897c3952b1f4d6c1b9416f9db35ae6

Mention that 'APIEditBeforeSave' is also being run on undo

This came up in T126861 where we identified this as desired
behavior.

Change-Id: I37e8f59b4b88cc289709da9802df3a0f123939b1

Add WatchedItemStore::countVisitingWatchersMultiple

This is for batch counting of visiting watchers, following the change
made in I2868c31fc09121de381d822e8f49194e3022bb42.
Query/logic has been extracted from ApiQueryInfo.

Bug: T129482
Change-Id: Ia9a534f5edb7af3cb7bf86be358dddb5d8c259cf

Link to Special:WhatLinksHere in CatWatch bundeled summary

This link could probably also include
&hidelinks=1&hideredirs=1 options
at a later stage

Bug: T119712
Change-Id: I578d550d25d92e20360bcaaf5392ef75a868a483

Add --with-phpunitclass arg to phpunit.php

This would allow to easily use stuff like
https://github.com/fiunchinho/phpunit-randomizer.

Change-Id: I28e8b1d261de0395366b18465a0adc4d7c4fde4a

Merge "Fix ApiQueryInfo response for prop=watchers"

Merge "Fix documentation lies about debug toolbar / cache interaction"

Fix ApiQueryInfo response for prop=watchers

Ie84e6feaa42db1bc7a1f89b56aed37dd7fe95ea4 part of them problem
with incorrect API response but if when no watchers data is
fetched (ie. due to early return in ApiQueryInfo::getWatcherInfo),
response contains "watchers": null instead of skipping "watchers"
key entirely.

Bug: T129482
Change-Id: I9cab120ec4e6a9cf4626678e45ef14ea8efc8cbc

Merge "mediawiki.api.parse: Restore support for stringified objects"

Merge "Action::factory: Don't crash on missing Action classes"

mediawiki.api.parse: Restore support for stringified objects

Follows-up 11e6b3d. Don't assume non-string values are mw.Title objects.
mw.Title is the special case. Restore the previous default of assuming wikitext.

This was presumably avoided earlier because 'instanceof' throws if given undefined
as right-hand expression and don't want a needless dependency on mediawiki.Title.

Change-Id: I794ed4105d116e63ed505a17a237f289b80d3b15

Action::factory: Don't crash on missing Action classes

Show the "no such action" error message instead.

Normally I wouldn't favor putting this in here, but it's way
too easy to register a missing class by accident, see my rant
on T107921#2128080.

Bug: T107921
Change-Id: I56da8a5810cae3547a0c8f2d3f435dfd790fbeaf

Merge "Support multiple extension-dir paths to be passed to mergeMessageFileList"

Localisation updates from https://translatewiki.net.

Change-Id: I2cd76699393ba63879babd31f435387593d3a2aa

Support multiple extension-dir paths to be passed to mergeMessageFileList

If scap is modified to pass the path to both extensions/ and skins/ then
the extension-list file in wmf-config will no longer be needed, eliminating
many headaches. (refs T125678)

Bug: T125678

Change-Id: I4fd0c99d68fa32bf2378691955850a1be2c022df

Update OOjs UI to v0.16.3

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.16.3/History.md

Change-Id: Idb6d1250bea3041bf6c4d198f21544b1f5bc4893

Merge "Make Job::toString() handle array parameters better"

Upgrade Sinon.JS to 1.17.3

No major changes for us other than a bugfix that makes the
following work as expected:

  var clock = this.sandbox.useFakeTimers();
  setTimeout(function () {
   clock.tick(10);
  });
  click.tick(1);
  // Expected: 11ms is recorded
  // Actual: 1ms is record

This was broken up until Sinon 1.16.0 (Lolex 1.3.0) because it saved the
"clock.now" value before running any pending timeout callbacks. Then it
unconditionally assigned the "clock.now" value to 'oldNow + ticked', which
discarded ticks made by a callback.

Change-Id: I2775f4c6353aef12ced6dc45e73fe8d4e49c2b68

Merge "Improve translation for bs namespaces"

Merge "mediawiki.api.test: Refactor to embrace async"

Improve translation for bs namespaces

NS_USER_TALK: "Razgovor sa korisnikom" → "Razgovor s korisnikom"

Translation author: Srđan M. (Srdjan m)

Bug: T115812
Change-Id: Ia674baa1476e5977adb77f4b783c20f90925948f

Drop deprecated $wgPreloadJavaScriptMwUtil

This was always false since introduction in 3c72b527 released in MediaWiki
1.19, and deprecated in 0ac4f998, released as part of MediaWiki 1.26. Any
code that still needs the mediawiki.util module can continue to use it via
expressing this dependency through their ResourceLoader manifest as usual.

Bug: T111077
Change-Id: Ic838af8727476c047f01ef0dbbeb952c85e263e1

Make Job::toString() handle array parameters better

Previously it would result in array(x) in fairly simple cases.

Change-Id: I6bfe67faa45835babb2c7e259b5cd158e0f0a649

Track the # of times remnant refreshLinks jobs divide

Change-Id: I32ec09cf1fc6d3a14d99b30ad465fcf6912ce9e0

Merge "Add RecursiveLinkPurge log for API requests"

Merge "Adding number of changes info for grouped category changes"

Add RecursiveLinkPurge log for API requests

Change-Id: I29636c045f7b561b2da0100f5e69050808d1aec7

Adding number of changes info for grouped category changes

This patch addes the summarized number of category changes
for grouped category changes on the EnhancedChangesList.

Bug: T126849
Change-Id: I3c8ccb2ad6310dac19a724b2ce923c1e67e588be

Localisation updates from https://translatewiki.net.

Change-Id: I4830f0116910a4e4bddfd7b0d44e7caea70f1f5c

Add REL NOTES about watchlist wl_id field

Change-Id: Ib3bfe0d0011d96da6d6cfad5930645d343f7cbb6

Merge "Add id field to watchlist db table"

Merge "Use WatchedItemStore::removeWatch in SpecialEditWatchlist"

Merge "Add WatchedItemStore::countWatchedItems"

Merge "WatchedItemStore add db connection helper methods"

Merge "Reset WatchedItemStore default instance after tests"

Use WatchedItemStore::removeWatch in SpecialEditWatchlist

Bug: T129481
Change-Id: I12bf46244532195ba19516ba571a1728bd365db9

Fix ApiQueryInfo break from use of WatchedItemStore

Fixes an issue introduced in:
I5a465773599cce9f8c9e94847cede6d12282c827

The new code now returns all targets even when 0
watcher have been found.
This patch adjusts the api to expect that.

Bug: T129482
Change-Id: Ie84e6feaa42db1bc7a1f89b56aed37dd7fe95ea4

Add WatchedItemStore::countWatchedItems

This method counts the number of individual items
that a user is watching.
A subject and corresponding talk page would mean a
count of 2

Unit and integration tests are also added here

Bug: T129481
Change-Id: I4b55318dc1d1c5abab1c5da16cebf1a43ddf9248

WatchedItemStore add db connection helper methods

As suggested in:
https://gerrit.wikimedia.org/r/#/c/277276/2/includes/WatchedItemStore.php

Change-Id: Id82d987eee025e21efc381bd441cd07fbc50aea0

Reset WatchedItemStore default instance after tests

Prior to this change in tests the overridden store
would remain in the instance static and thus could
be used in other places.

This patch introduces the used of ScopedCallbacks
in the override methods in WatchedItemStore.
This means that any instance of WatchedItemStore
should return to a regular state after each test.

This is better than requiring the tests to reset
the value back to the origional as this would likely
be forgotten and result in long hunts for failing
tests.

This was found while writing more tests...

Change-Id: I9aa71425642174ae9ea2c6d4f85dcd07d724af11

Merge "Use WatchedItemStore::countWatchers in SpecialWatchlist"

Merge "Add countUnreadNotifications to WatchedItemStore"

Merge "Move countVisitingWatchers to WatchedItemStore"

Merge "Move counting of watchers to WatchedItemStore"

Merge "Improve cache assertions in WatchedItemStoreUnitTest"

Merge "Uncache things in WatchedItemStore::updateNotificationTimestamp"

Merge "Make unused variable optional in ChangesList::insertDiffHist"

Merge "Move dbSetup=true; follow-up d920237"

Move dbSetup=true; follow-up d920237

See https://gerrit.wikimedia.org/r/#/c/274643/8/tests/phpunit/MediaWikiTestCase.php

I unintentionally changed it so dbSetup wasn't always set to true
(it was in the early return case in the old code, but not the new)

Bug: T95870
Change-Id: I2119b0233330b88a61d15fe4dc55fe855fce4b6a

Fix undefined $modelId in MWUnknownContentModelException

Change-Id: I13b93cc1e9d7c050765fab5d8de4e17ca4319dc1

Merge "Add a structure test to validate all extension.json files"

Localisation updates from https://translatewiki.net.

Change-Id: Ifab2110151643c82611927a45c0a0da9259f07d6

Add a structure test to validate all extension.json files

Adds a structure test to validate that all loaded extension.json and
skin.json files validate against the schema at
docs/extension.schema.json.

Bug: T128307
Change-Id: I2fd1caaa50c288821ab6847dc29d60e6554d9df5

SpecialVersion: Just use HTTPS links to mw.org instead of protocol-relative

No need to make people redirect if they're on HTTP :)

Change-Id: I273be400e3889add553a2f066dfec008c170e6c0

Merge "Remove support for non-Composer PHPUnit"

Remove support for non-Composer PHPUnit

Composer is now a standard way to install MediaWiki's PHP dependencies,
no reason for it not to be the same for PHPUnit.

Change-Id: Ibd977eb3480dafaf270ff63abc43c413d7b72144

resourceloader: Remove 'require' and 'module' from debug mode

Partially reverts 94c116240 and 7fd977b242 as it was incorrectly
split up. This should've been part of Idbd05488 instead of 94c116240.

Bug: T108655
Bug: T129776
Change-Id: Ia5e6e57c9bd651ec913a2543e2df8951ff345ea2

Merge "Make use of …::class feature in ContentHandler/DatabaseUpdater"

Merge "completionSearch: try an exact match even if the backend returns no result"

mediawiki.api.test: Refactor to embrace async

This was using lots of synchronous hacks but those don't work in newer
versions of Sinon.JS (and only worked by accident up until now).

The test will still run pretty quick, but it will yield to some extend because
mw.Api and jQuery.ajax both create XHR objects with async=true.

* Remove redundant QUnit.expect() calls (deprecated).
* Make use of QUnit's new Promise-return feature.
  This simplifies some tests by no longer having to call async() or start()
  manually when only dealing with a single promise.

This blocks upgrade of Sinon.JS.

Change-Id: I08b08ea3c3e8b41f4aea50d86b2c9dc0f579ba67