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

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

Use WatchedItemStore::countWatchers in SpecialWatchlist

Bug: T129481
Change-Id: I6da41a80bfc658106862238622f95db087c5bea7

Add countUnreadNotifications to WatchedItemStore

This query / logic has been extracted from
ApiQueryUserInfo.

Unit & Integration tests have also been added.

Relating to the task linked this is the last change
needed in this ApiQueryUserInfo!

Bug: T129482
Change-Id: I91aa109416c16cd1f257c9de46669e35d6fd34d7

Move countVisitingWatchers to WatchedItemStore

Also adds tests

Bug: T129479
Change-Id: I2868c31fc09121de381d822e8f49194e3022bb42

Move counting of watchers to WatchedItemStore

Also adds tests

Bug: T129479
Bug: T129482
Change-Id: I5a465773599cce9f8c9e94847cede6d12282c827

Improve cache assertions in WatchedItemStoreUnitTest

Change-Id: If4b2086cc29fcfc73dee2e9f96591a9e4aa50da9

Merge "Move some LocalFile comments around"

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

Change-Id: Ic0d51a41c50f52e1fbdab58e3b63319caa5c7d1d

Merge "Don't add label-elements for elements that doesn't have one"

Uncache things in WatchedItemStore::updateNotificationTimestamp

When the notification timestamp for a LinkTarget is
updated all items relating to that LinkTarget should
be uncached!..

This also switches this class from a BagOStuff to
a HashBagOStuff as this cache index is only maintained
per process..

Change-Id: I5dc58e018a6a4a15903abc1e0b326b4220abc75e

Merge "Change remaining places to use short array syntax"

Merge "Follow-up Ia16e19985: Fix table sort arrow direction"

Merge "Add support for External Store for unit tests"

Follow-up Ia16e19985: Fix table sort arrow direction

In Ia16e19985 we accidentally fixed the down arrow being
'sort_up', but by not switches the class order we changed
the meaning of the up and down arrows.

An up arrow should be labelled as 'up' but show when the
table is in ascending order (smallest at the top).

Bug: T129775
Change-Id: Ia30d4a26728dd0fe93939d746ce9bf9e90742de6

Merge "Use an example with working digit transformation"

Don't add label-elements for elements that doesn't have one

Fix the OOUI field element implementation to conditionally add
labels to the FieldLayout (only if the label isn't empty). Also
add setShowEmptyLabel( false ) to HTMLButtonField (labels usually
aren't set outside of the button, the button itself should have a label).

Bug: T129821
Change-Id: I0499bb82245273519e77c80e78bc431588875a85

Localisation updates from https://translatewiki.net.

Change-Id: Ib5ad9c2a48f781ae667e27786edbcb4c03c4abc5

Merge "build: Update grunt-jscs to 2.8.0"

Merge "Update Karma's QUnit to v1.22.0"

build: Update grunt-jscs to 2.8.0

Change-Id: Ica4fac1bee551467614a76c76aa699406927c9a4

Localisation updates from https://translatewiki.net.

Change-Id: I8af274b32e9af1553e074c4b29eedd0f7f3803eb

registration: Improve error message for non-array attributes

Change-Id: I7cbf7353ebbbbbbf36500f86164e3b32d7723cca

Update Karma's QUnit to v1.22.0

Follows-up 54bbef467d, which updated the version used by
MediaWiki's Special:JavaScriptTest.

Change-Id: I7876808c3b5ac88976706cdff1cf7abe8075cfdb

Convert array() syntax to [] in Makefile.py

Update Makefile.py for Chinese conversion table to convert array() syntax to [] in languages/data/ZhConversion.php .

Change-Id: I10ba5d16ca619f64458feba51e12c3bd5d18435d

Merge "Chinese Conversion Table Update 2016-1"

Upgrade QUnit to v1.22.0

The v1.19.0 release fixed a major bug in QUnit.test Promise handling

> Test: Don't pass Promise fulfillment value to QUnit.start

Without this fix, passing tests that involve an async Promise
can wrongly be marked as failure when they have a falsey resolution
value due to the resolution value being passed to QUnit.start()
by accident.

Change-Id: Id4b52085529f4cf8996f8053648680026ed62941

Add support for External Store for unit tests

This allow running PHP tests with External Store DB configured.
This means it needs to do the unittest_ prefixing similar to the main
tables.

It could do resetting (truncating the tables) similar to the tablesUsed
functionality.  However, if External Store is truncated unconditionally,
revision, etc. also needs to be.  This can cause performance issues.
I was thinking of making resetting ES optional, but it wasn't needed
to pass any of the core tests, so I just left it out entirely for now.

Bug: T95870
Change-Id: If27435210d5a2165795dd759b17a7e51fe9bba10

Merge "Add description for mw.hook#event-postEdit in JavaScript"

Add description for mw.hook#event-postEdit in JavaScript

Change-Id: I82ae21348b2ee686f80eda454f2aa6cf60ae20c4

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

Improve the personal tools:
* Remove the "Not logged in" text message. It is misleading. Being among the
  personal tool links people click on it expecting it to have some function.
* Instead add a proper link "Unknown user" to the IP user page.

Bug: T121793
Change-Id: I049d0671a7050585659fc8b9dcf21e0f92393bcf

Move some LocalFile comments around

Change-Id: I7d7c5618481245aa6e8328ad58f5e841f010fe01

Localisation updates from https://translatewiki.net.

Change-Id: Ie105d1bced55a141d9c641ab0c6cec53e267d427

Merge "Normalize user provided URL link for Special:LinkSearch page"

Change remaining places to use short array syntax

Change-Id: I0785c73a239c11188ad7411c5cd8ed09a2dbc9b3

Merge "mediawiki.api.parse: Allow parsing pages"

Wikimedia\Assert\Assert is never used

Change-Id: Ie12f158ec01deb2db7c3d515fe16839117380ee8

Merge "i18n of submit button on Special:ChangeContentModel"

Don't return PHPUnit_Framework_MockObject_MockObject when not needed

The callers of this method don't need to know about this additional
interface.

Change-Id: I9e5531e495b7f5c890e8e520d661ccd5b0a644e1

Update and correct documentation in Language class

Change-Id: I1d530bded0da7133f9b5120b515f32fc87507c84

i18n of submit button on Special:ChangeContentModel

This patch adds a message, 'changecontentmodel-submit', for
Special:ChangeContentModel to prevent reuse of the default
"Submit" text from FormSpecialPage.

Bug: T120576
Change-Id: I72466867a6739ba144c5a2984b8ca2589292815c

Merge "OutputPage: Set wgPageContentLanguage to pageViewLang instead of pageLang"

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

Unfortunately some backends (like Cirrus) are not able to do proper crossnamespace
lookup. If the backend returns no result we should still try an exact title match.

Bug:T129575
Change-Id: Ic08ec0aac0fd8a289f21753d74ae62ba509de841

OutputPage: Set wgPageContentLanguage to pageViewLang instead of pageLang

Follows-up 80a090ea7 (r104483).

I'm assuming the difference is that pageLanguage will be the canonical variant
for the page (as authored in wikitext, presumably the wiki's default), whereas
pageViewLanguage will be the variant of the pageLanguage based on the current
user-interface language (if and only if the user interface is related).

Viewing zh by default has:
* userLanguage: zh
* pageLanguage: zh
* pageViewLanguage: zh

Viewing zh-cn by default has:
* userLanguage: zh
* pageLanguage: zh
* pageViewLanguage: zh-cn

Viewing zh-cn with uselang=en has:
* userLanguage: en
* pageLanguage: zh
* pageViewLanguage: zh-cn

Viewing zh-cn with uselang=zh-tw has:
* userLanguage: zh-tw
* pageLanguage: zh
* pageViewLanguage: zh-cn

The main use case behind wgPageContentLanguage in 80a090ea7 was tablesorter
which should match the language and variant of the rendered wikipage content.

The parser output and div#mw-content-text element use pageViewLanguage, so
let's set wgPageContentLanguage to that as well.

Change-Id: Ibf399dfe5ca60fb55e7acd75a5c0238d8b903807

Merge "resourceloader: Create ResourceLoaderContext::msg() method"

Merge "Include user name in Special:BotPassword success messages"

mediawiki.api.parse: Allow parsing pages

Bug: T46925
Change-Id: Ic0d762d02c96e0b39b4983d3b66ebdfb6f88d3a2

Chinese Conversion Table Update 2016-1

Update the Chinese conversion table routinely to fix bugs reported from
https://zh.wikipedia.org/wiki/Wikipedia:%E5%AD%97%E8%AF%8D%E8%BD%AC%E6%8D%A2/%E4%BF%AE%E5%A4%8D%E8%AF%B7%E6%B1%82 .

It is only data changes and only works for Chinese WikiProjects.

Change-Id: I39f9945a38078364a740dbb552f87d4bcf62033c

resourceloader: Create ResourceLoaderContext::msg() method

This makes it easier to fetch messages without having to manually
call inLanguage() on each wfMessage() call, which is currently
causing some code forget this and use plain wfMessage() which
defaults to MediaWiki user language from session.

I've considered setting $wgLang or RequestContext::getMain(),
but that's still bad since modules get passed a context in their
methods and shouldn't be using global context either.

The warnings provided by MW_NO_SESSION are exactly what we want.

Change-Id: I1288fa5622d9f82d21bb66c8eb6518b90e7cddb4

Merge "resourceloader: Inverse hasOwn() check to fix require in debug mode"

Merge "Avoid rebuilding database fixtures for every test run"

resourceloader: Inverse hasOwn() check to fix require in debug mode

Follows-up 94c1162400.

Makes require and module work in debug mode.

Bug: T108655
Change-Id: I68ea05a31d5bde530e5e80aeff076df6b9924bac

Avoid rebuilding database fixtures for every test run

This reduces the runtime of database-bound tests by about 40%
(on my system, from 4:55 to 2:47; results from Jenkins are
inconclusive).

The basic idea is to call addCoreDBData() only once, and have
a addDBDataOnce() that is called once per test class, not for
every test method lie addDBData() is. Most tests could be
trivially be changed to implement addDBDataOnce() instead of
addDBData(). The ones for which this did not work immediately
were left out for now. A closer look at the tests that still
implement addDBData() may reveal additional potential for
improvement.

TODO: Once this is merged, try to change addDBData() to
addDBDataOnce() where possible in extensions.

Change-Id: Iec4ed4c8419fb4ad87e6710de808863ede9998b7

Merge "Replace create_function() with a closure"

Replace create_function() with a closure

As in 266da0fb89ec4ae5 in extensions/Flow.

Change-Id: I0f6a38761cf1fa2956dad0f44fa023e1a8855df4

Convert timing to ms before we've recorded too much data

Change-Id: I0a4e18290cdc662e597484a94205a481415cbb7b

Merge "Respect undeclared command line options"

Merge "HTMLForm: Use OOUI\Element::configFromHtmlAttributes instead of rolling our own"

Merge "HTMLFormFieldWithButton: Don't make the button primary by default"

Merge "Remove some deprecated functions from ApiBase"

Localisation updates from https://translatewiki.net.

Change-Id: Iaea8b19d6e8c2cb24cb41c36f42c833c8cadf85a