Update OOjs UI to v0.16.4

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

Change-Id: Ic0f8cf4fc956feff29189c4d3e1d83e071daa8f7

Localisation updates from https://translatewiki.net.

Change-Id: Iba1610b61c1658b22216e57ed5d60d131073feab

Merge "Revert "Add link to anon's user page; remove "Not logged in"""

Merge "Message: Use RequestContext instead of $wgLang"

Revert "Add link to anon's user page; remove "Not logged in""

This reverts change I049d0671a7050.

This change was reverted in the wmf/1.27.0-wmf.17. Since there is no
clear consensus, revert in master before branching wmf/1.27.0-wmf.18.

Bug: T121793
Change-Id: I2dc0f2562c908d4e419d34e80a64065843778f3d

Names.php: Use ʻ instead of ` as ʻokina in Hawaiian

https://en.wikipedia.org/wiki/%CA%BBOkina:
In the Unicode standard, the ʻokina is encoded as U+02BB ʻ MODIFIER LETTER TURNED COMMA

Change-Id: I030f3b46382d5c5ed5ba81471f7c4156c660b244

Localisation updates from https://translatewiki.net.

Change-Id: I9e4277291fc529f86496f46c8eab1c2cd4590903

Merge "mw.widgets.CategoryCapsuleItemWidget: Pass the right 'this'"

mw.widgets.CategoryCapsuleItemWidget: Pass the right 'this'

Follow-up to 7b57752e65c3cac62d973f43d1060b6bd0ab2a2f.

Bug: T130544
Change-Id: I89c18be911bcc492f3a70e1c1da3d2fd2bf65a05

Merge "Use WatchedItemStore in ApiQueryInfo::getWatchedInfo"

Use WatchedItemStore in ApiQueryInfo::getWatchedInfo

Adds a method for getting watchlist's notification timestamps
for a batch of LinkTargets.

Bug: T129482
Change-Id: I1f84212e7879a84b34bb3b53859069fcea282bba

Localisation updates from https://translatewiki.net.

Change-Id: I3d9b14ae3a5d77fea9694ef113b0180e5677c39e

Merge "Update phpunit to 4.8.24"

Merge "Call Skin::setRelevantUser on Special:EmailUser"

Update phpunit to 4.8.24

See changelog at
https://github.com/sebastianbergmann/phpunit/blob/4.8/ChangeLog-4.8.md

Change-Id: I604f10917e015788acf49cdcd6389957da89fbdb

UploadBase: Set mFileSize, if given, even if mTempPath is unknown

When uploading a file from stash using the action=upload API, with
async=1, UploadFromStash is initialized without initializing a temp
file. dcb5ec5cbf92b9a07f0776b9c194183a13400193 accidentally made it so
that the given file size is ignored if there's no path. This caused
validity checks to fail (because mFileSize is null) and action=upload
to also fail with cryptic 'emptyfile' warning.

This made it impossible to upload files bigger than 10 MB using
UploadWizard, as it uses the async mode for them.

Bug: T130238
Change-Id: Ie35a66a565a370fe9adc66d5fee0866c4d51470e

Localisation updates from https://translatewiki.net.

Change-Id: I67c4a49a16f5cee599db767b1414cac75bc19fd5

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

Call Skin::setRelevantUser on Special:EmailUser

Adding the relevant user to the skin will add user releated links to the
navigation bar. User related links are: contributions, logs and blocks

Change-Id: I37126a228717addc9879fd69fcbe89057ca94cb7

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

Message: Use RequestContext instead of $wgLang

This cuts the $wgLang and StubUserLang which is a bit cleaner.
It also makes it more reliable when load.php sets its
RequestContext user interface language.

Bug: T127920
Change-Id: I05302feb9b3ce9e4c29541e07a4260effc4b55b2

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"