Pin eslint version

We don't want tests failing every time upstream publishing a new
version.

Instead we should bump this manually everytime they update to make sure
no failures happen.

Bug: T118941
Change-Id: I1c76dacd0950100825b85a3791f74c1f6d5477d9

Merge "upload: Avoid &$this in hooks"

upload: Avoid &$this in hooks

&$this triggers warnings in PHP 7.1. Simply renaming the variable before
passing it by reference avoids the warning, without breaking backwards
compatibility.

Bug: T153505
Change-Id: I78ea04a01ecce82294837e92c2a05b00ffb6e0f6

Protect -{...}- variant constructs in galleries

This also protects naked external links, which are internally surrounded by
`-{R|...}-` by LanguageConverter::markNoConversion.

Originally found in failed tests in I7fa2d85d6.

Bug: T54190
Change-Id: I9b099273203482ffb570a5654d8ba50c833e526d

Protect -{...}- variant constructs in images.

A protected version of explode is factored out as
`StringUtils::delimiterExplode`, since it will be used in follow-up
patches in this series.  The `delimiterExplode` implementation creates
an intermediate array of the exploded results, which is reasonable as
the number of image options is small; but since an Iterator is
returned the implementation can be upgraded in the future (at the cost
of additional complexity) to avoid this.  The additional code in that
case would be similar to ExplodeIterator.

Bug: T146305
Change-Id: I1327685e9e8c07ef476dceaa6f6dae4ba40989ef

Merge "Pass undone revision id to PageContentSaveComplete hook"

Update date format for Wu language

Bug: T125047
Change-Id: I398d93d372ad0d5345b0e4b27657ad69b0079d94

Merge "Fix RevDel API output for formatversion=2"

Merge "ApiEditPage: Avoid PHP warning"

Merge "Remove unused Phan exception"

Localisation updates from https://translatewiki.net.

Change-Id: I905f195a2d4a159bf4f9cb3357465ce96c59f2ce

Do not exit the run method in case of a missing rev

We should ignore broken/missing titles and not stop the whole process.

Change-Id: Ib4acb739c28c5f3278430b43b6304af3a513085e

Fix RevDel API output for formatversion=2

Use booleans, not the old empty-string-or-undefined.

This bugfix is a breaking change, but chances are it won't affect many
clients.

Bug: T153518
Change-Id: If5fa2cdcb973b186349307b29e4266daceef0a9d

Pass undone revision id to PageContentSaveComplete hook

This passes the id of the revision that was undone to the
PageContentSaveComplete hook, since this hook is now inside a deferred
update so extensions can no longer rely on 'wpUndidRevision' being
present in the request.

Change-Id: I50dcb841cd0616acc2d69c3a685ba3cb339986c3

ApiEditPage: Avoid PHP warning

In cases where $result is non-null but doesn't have the 'spam' key,
constructing the fallback array was raising a PHP warning.

Thinking it over, we should just use a switch instead of making an array
of known fallback messages. It's longer source code, but probably less
real work than constructing an array where only one entry will actually
be used.

Bug: T153590
Change-Id: I50349c56692c4b2f9a8e560a849c36db1082120e

Merge "build: Update phpunit to 4.8.31"

Merge "Move the Ukrainian grammar rules from PHP and JS to JSON"

build: Update phpunit to 4.8.31

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

Change-Id: I00e0b94452b371cacecc4eb10330749c0da1890e

Localisation updates from https://translatewiki.net.

Change-Id: I6f343baa88923fcc11e83a066edb01cab77986ec

HTMLForm: Use 'mw-htmlform' CSS class

Makes it easy to identify all HTMLForms on a page.

Change-Id: I5b9494fc925ac953c14b358331acddfe80c2661d

HTMLFormFieldCloner: Set 'formnovalidate' on add/remove field buttons

Otherwise, after we allow HTML5 form validation attributes, these
buttons wouldn't work for non-JS users (unless the whole form was
otherwise valid).

Change-Id: Ia9c16849774c186b991280ea60fc5608ddd4cf42

Merge "Rewrite importImages.php to use Maintenance infrastructure"

Merge "mw.htmlform: Fields hidden with 'hide-if' should be disabled"

Merge "Turn $wgRunJobsAsync off by default"

Remove unused Phan exception

Change-Id: Ib578336abfed02c8dce9ccb47ab22220ddcee856

Rewrite importImages.php to use Maintenance infrastructure

I even tested it!

Change-Id: I999f5842625c752f01c1eb4b012431c4a24ce7ce

Merge "Move TempFileRepo to a separate file"

Turn $wgRunJobsAsync off by default

The current situation doesn't work in all installations and has being worse
since MediaWiki 1.27. Default settings should be safe in all installations,
and $wgRunJobsAsync wasn't one of them.

Bug: T142751
Change-Id: I8daa38534a0e925a75ba2fb47e3e299f1baee4b1

Move the Ukrainian grammar rules from PHP and JS to JSON

Bug: T115217
Change-Id: I15a06b07e381cc9074e64e746d22ec51e9e638c4

Merge "Make the code for grammar data processing common"

Merge "Optionally collect context in TestLogger"

Move TempFileRepo to a separate file

Change-Id: I6fc0870cc82a7b19859abc00bd4b2ecee6c0fc9e

Merge "Add dotall modifier to EDITSECTION_REGEX"

Merge "mediawiki.special.watchlist: Fix render stampede after "Mark all pages visited""

Localisation updates from https://translatewiki.net.

Change-Id: I649e54c975f5fe8445821e53ef121ec088167ca9

Merge "Fixing mediawiki's য় problem"

Fix deprecation message - use correct method name (defaultPrefixSearch)

Change-Id: Ie4dbb28b4ca11ecfeaccd331db55f92130f6cdf1

Merge "mediawiki.api.watch: Use formatversion=2 for API requests"

Merge "user: Minor clean up of User::isLocallyBlockedProxy()"

Merge "Fix various minor IDEA warnings in LoadBalancer"

user: Minor clean up of User::isLocallyBlockedProxy()

Change-Id: If8bcb2f26f14ce04ee23fb9500be95c8cdf1a967

mediawiki.special.watchlist: Fix render stampede after "Mark all pages visited"

Follow-up 9ea1142fad, f4d70ceedc, 73592aea1c.

The progress bar has a continuous animation that is rather CPU heavy.
While it is set to 'visibility hidden', this is merely a way to make the
contents of the element transparent. It is still interactable (e.g. if it
has hover or click handlers, they would trigger if hovering in that area)
and still rendered with full 60fps paint and layout.

Instead, detach and (re-)append the element as needed. Since the element
has an absolute position, it doesn't cause a content reflow when the element
is detached. This fact is already made use of considering the element is
inserted only after the submit button is clicked - without content reflow.

Change-Id: I06ef0d527044e5a6080febdb618eaffeb6ec85fa

Merge "Change linking Special:PageLanguage on action=info"

Merge "mediawiki.searchSuggest: Remove inline browser compat check"

Merge "Adjust phan script to run sub-projects"

Merge "SearchInputWidget: Support searching a namespaces besides 0"

Merge "mediawiki.api: Add test for api deferred rejection of errorformat!=bc"

Merge "jquery.tipsy: Remove support for 'live' option"

mediawiki.api: Add test for api deferred rejection of errorformat!=bc

Change-Id: Ia4e1d575e07df690919d63502f9fedffff7be0a3

jquery.tipsy: Remove support for 'live' option

Follows-up aa524075ac from 2014.

It was relying on jQuery features that have been deprecated a long time
ago, officially removed in jQuery 1.11, and removed for real in 3.0.
We have to get rid of it at some point. As far as I can tell, nothing
in our repositories uses this option anymore.

Rather than remove the code and diverge further from upstream Tipsy,
I'm just making it throw an error if this is passed. Maybe one day we'll
upgrade Tipsy too to some modern version and this will then disappear.

Bug: T85048
Change-Id: I7dca8334556b1079cd4fee2e6e921232fe3d46ef

mediawiki.searchSuggest: Remove inline browser compat check

Introduced in 1f987fb5eea4.

This is quite dated and obsolete since then. The current code in
the 'mediawiki.searchSuggest' and 'jquery.suggestions' works fine
in all currently supported Grade A browsers (per startup.js).

Change-Id: I6d281459e230fa6a2971a31314f9e963dc83ff2b

Merge "Fix type annotation"

Merge "tests: Fix broken setup/teardown in mediawiki.language.test.js"

Adjust phan script to run sub-projects

Rather than duplicating the phan and postprocess-phan.php scripts
into extensions, this allows passing the path to the extension
(or skin, or whichever) to the phan script and it will run the analysis
for that project.

Individual projects will need a tests/phan/config.php file that defines
how the analysis will run in their project.

Bug: T153040
Change-Id: I58ad44f6a8769088e87d5d6cf1079e633c082f88

SearchInputWidget: Support searching a namespaces besides 0

0 is currently hardcoded in searchSuggest, making it hard to use
SearchInputWidget for other things.

SearchInputWidget already takes a namespace parameter by virtue of
extending TitleInputWidget, so use that.

Change-Id: Iecf93d1e8d4075a5a9aa0f08059c609aa8db4f6d

Fix type annotation

Change-Id: I8996a7617c4723c9c4ff8f91d0f3767c9e67dd28

tests: Fix broken setup/teardown in mediawiki.language.test.js

Ref T152476.

Change-Id: Ie871d36dadefd69d0a5dc85c275b55d4d477dc78

Merge "registration: Always specify version number in extension.schema.json file"

Merge "Remove deprecated function User::isPasswordReminderThrottled()"

Merge "Remove 2 deprecated functions from class User"

Localisation updates from https://translatewiki.net.

Change-Id: I586676d3574c23e6746b7673664734ae66eb7984

Remove deprecated function User::isPasswordReminderThrottled()

Deprecated and stubbed in 1.27, only throws exceptions these days.
The only user in core or extensions is AjaxLogin which is completely
broken anyway (T153385).

Change-Id: I298fbc3e65d98b3af2f3cfef3d9884e277e6717c

Remove 2 deprecated functions from class User

They've been deprecated in 1.27 and only throw exceptions
no callers in core or extensions.

Change-Id: I1ccfa39c287b640b435a075449e4674daf8d24e7

Merge "WatchedItemStore::setNotificationTimestampsForUser(): Allow clearing timestamp"

Merge "Minor adjustments to align with WikimediaUI color palette"

Change linking Special:PageLanguage on action=info

Change from linking of description to extra link behind the data. Also
reorder the code and language name to match the order on
Special:PageLanguage

Change-Id: I859df5d04c5d937fba55c309034a0350574b5af7

registration: Always specify version number in extension.schema.json file

Instead of having the current version of the schema live at
extension.schema.json, always include a version number in it. This
simplifies both the code, and makes merging, cherry-picks, rebasing,
etc. easier when the current schema version changes.

Change-Id: I1b5ef4ced1eb34b8bdda7ba9b3f086b1997b704c

WatchedItemStore::setNotificationTimestampsForUser(): Allow clearing timestamp

ApiSetNotificationTimestamp expects to be able to clear the timestamp by
passing null. Allow that to work as expected.

Bug: T153482
Change-Id: Ibf4ba56f0abd3b72283f7a33e4665d5999a70b82

Fixing mediawiki's য় problem

Bengali $magicWords translations with য় (U+09DF) does not working because the character য় (U+09DF) was interpreted by MediaWiki as য (U+09AF) and nukta (U+09BC).

See also: https://gerrit.wikimedia.org/r/#/c/326433/

Bug: T153132
Change-Id: I6ef18acd7c1eb0ce87e81163ef0130620d48d200

Merge "TitleWidget: Cache siteinfo API response"

TitleWidget: Cache siteinfo API response

This information rarely changes, so cache it for a day client-side.

Bug: T153388
Change-Id: I375a17737855eaa661c85e0dde2b7e58b67bc710

Add the concept of "system blocks"

Blocks made for configured proxies, dnsbls, or the configured range
soft-blocks being added in I6c11a6b9 aren't real blocks stored in the
database. Let's actually flag these blocks as such and use a more
appropriate message when displaying them to the user.

Change-Id: I697e3eec2520792e98c193200c2b1c28c35bf382

Merge "Decrease the number of 'function says it should return something' errors"

Merge "Fix type annotations"

Merge "More properly fix error message"

More properly fix error message

I2b686228 was a quick bandaid, but resulted in the error message not
actually reporting the incorrect list type.

Change-Id: I2b2bd6ee66a78fadb31b3524dfe04bf9e1f45535

Make the code for grammar data processing common

This makes the code for processing JSON files with
grammar transformations reusable by different languages
and applies the same logic to Russian and Hebrew.
It will be done to other languages in further patches.

This patch is not supposed to change any functionality,
and the tests are intact (except a comment in the test
for Hebrew - the class doesn't exist any longer).

PHP:
* Move the JSON grammar transformation data processing logic
  from LanguageRu.php to convertGrammar() in Language.php.
  By default all these data files are supposed to be
  processed identically, so the code should be common.
  If there is no JSON data file, nothing new happens.
* LanguageRu's own convertGrammar() method is removed.
* The LanguageHe class is removed, now that all its functionality
  is handled by generic JSON data processing in the Language class.
  LanguageHe.php file is removed from the repo and from autoloading.

JavaScript:
* Move the JSON grammar transformation data processing logic
  from ru.js to mediawiki.language.js.
* JavaScript grammar code files he.js and ru.js are removed
  from the repo and from Resources.php, because all the data
  is in JSON, and the default logic in mediawiki.language.js
  works for both languages.

Bug: T115217
Change-Id: I5e75467121c3d791bb84f9e6fdfcf07c1840f81a

Don't show dialog to confirm whether to reset watchlist

People don't seem to like having a dialog to ask for confirmation to set all
pages to visited in the watchlist, and the PHP server-side reset
functionality doesn't ask for a confirmation anyways.

Bug: T153438
Change-Id: I92aa3c0670925efc691d8bdba2c1c503e17ddb8c

Merge "Cleanup some incorrect return annotations"

Merge "Add <!DOCTYPE html> to HTML responses"

Merge "mediawiki.special.watchlist: Optimize JavaScript code"

Merge "Add ILocalizedException interface"

Merge "IcuCollation: Add the current class name to 'first-letters' cache key"

Merge "Remove un-necessary stdClass use"

Remove un-necessary stdClass use

And it causes the following warning when you visit the list of special pages:

 The use statement with non-compound name 'stdClass' has no effect

Change-Id: I3698dab32a572e8c39788ddfc0ae2b3312a2e94e

Fix various minor IDEA warnings in LoadBalancer

Change-Id: Ic65a04ca1b988fe36e088709d7dc4e981670cb62

Merge "Add LoadBalancer::getMaintenanceConnectionRef() method"

Add NS_TEMPLATE to wgNamespacesWithSubpages in DefaultSettings.php

Bug: T147160
Change-Id: I504de3332bbc4b39d8d69fa324949988d56d0b67

Merge "resourceloader: Include backtrace in formatted errors (if enabled)"

Merge "resourceloader: Don't let module exception break startup"

resourceloader: Include backtrace in formatted errors (if enabled)

Make ResourceLoader error formatting the same as everywhere else.
Which means if wgShowExceptionDetails is enabled locally, the
trace will be included as well.

This matches logic in MWExceptionRenderer.

Also move the typical error handling used by respond() to a
utility method to reduce duplication of code and avoid mistakes.

Change-Id: If04ae99618e4a758ed0f9dd2b555496b76da29de

Merge "registration: Improve dependency checking"

Merge "registration: Allow specifying extension dependencies"

Merge "registration: Generalize CoreVersionChecker to VersionChecker"

Merge "Fix PhanTypeInvalidLeftOperand in Language.php"

Merge "Fix undefined $param"

Decrease the number of 'function says it should return something' errors

Change-Id: Ib5115fe5bbaa67d8a6e54cc3ba1ba7020e239e11

Protect language converter markup in the preprocessor.

This ensures that `{{echo|-{R|foo}-}}` is parsed correctly as
a template invocation with a single argument, not as two separate
arguments split by the `|`.

Bug: T146304
Change-Id: I709d007c70a3fd19264790055042c615999b2f67

resourceloader: Don't let module exception break startup

When getScript (or some other method used in a module response)
throws an error, only that module fails (by outputting mw.loader.state
instead of mw.loader.implement). Other modules will work.

This has always been the case and is working fine. For example,
"load.php?modules=foo|bar", where 'foo' throws, will return:

```js
/* exception message: .. */
mw.loader.implement('bar', ..)
mw.loader.state('foo', 'error')
```

The problem, however, is that during the generation of the startup
module, we iterate over all other modules. In 2011, the
getVersionHash method (then: getModifiedTime) was fairly simple
and unlikely to throw errors.

Nowadays, some modules use enableModuleContentVersion which will
involve the same code path as for regular module responses.

The try/catch in ResourceLoader::makeModuleResponse() suffices
for the case of loading modules other than startup. But when
loading the startup module, and an exception happens in getVersionHash,
then the entire startup response is replaced with an exception comment.

Example case:
* A file not existing for a FileModule subclass that uses
  enableModuleContentVersion.
* A database error from a data module, like CiteDataModule or
  CNChoiceData.

Changes:
* Ensure E-Tag is still useful while an error happens in production
  because we respond with 200 OK and one error isn't the same as
  another.
  Fixed by try/catch in getCombinedVersion.
* Ensure start manifest isn't disrupted by one broken module.
  Fixed by try/catch in StartupModule::getModuleRegistrations().

Tests:
* testMakeModuleResponseError: The case that already worked fined.
* testMakeModuleResponseStartupError: The case fixed in this commit.
* testGetCombinedVersion: The case fixed in this commit for E-Tag.

Bug: T152266
Change-Id: Ice4ede5ea594bf3fa591134bc9382bd9c24e2f39

registration: Improve dependency checking

* Pass $coreVersion to VersionChecker's constructor, don't require a
setter.
* Bump ExtensionRegistry::CACHE_VERSION
* Return single strings from handle* functions, avoid array_merge calls
* Improve invalid version error message
* Fix naming of VersionCheckerTest class

Change-Id: Id4f66b815aa41dbbc4b966095d6b99e542e548b4