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

Merge "Make the code for grammar data processing common"

Merge "Optionally collect context in TestLogger"

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

Fix undefined $param

Change-Id: I2b6862284b80c27f1f6189a261428324295699ae

Merge "MessageCache invalidation improvements"

Fix type annotations

Change-Id: Ib72c98358b2f0b3b73b6f105adea47346373d46d

Minor adjustments to align with WikimediaUI color palette

All but one of them affecting border-colors and in one case a :hover
background-color.

 - <hr>
 - catlinks
 - datatable
 - mw-plusminus-null
 - headings h1, h2 and so on

Bug: T152025
Change-Id: I233e42ac399202791dd44e48dc58487be0f9d8ac

Localisation updates from https://translatewiki.net.

Change-Id: Ia6cd0fab7698eda791b80f21ee7b15a259738bd7

Fix two instances of calling function with too few args

At one point SearchIndexFieldDefinition was updated to require the
engine to be passed in, but it seems that update was missed here.

BackupDumper::loadPlugin() requires the second argument, set it to
the empty string to keep current behaviour.

Change-Id: Ifbd8fc4870ff63b2d338f8bb4d251d7a3477b989

mediawiki.special.watchlist: Optimize JavaScript code

* Create progress bar not before submit.
* Reenable the disabled reset button after response.
  Currently a reload of the watch list after reset may lead to a disabled
  reset button.
* Reduce indenting by make an early return.
* Add some line breaks for better readability.
* Add some comments.

Follows-up to 9ea1142fad935999ac64e2920321520c1bbbfb61

Change-Id: Ibfafa140172bae55be01383ef7a9899104d46938

Merge "Correct DatabaseSqlite::unionQueries annotation"

Merge "Enable supplying inline interwiki results"

Correct DatabaseSqlite::unionQueries annotation

The annotation was claiming string, but it's very obvious from
the usage the it should be string[].

Change-Id: I99849a338971590ca68dfb175b2458ad5ebc0df2

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

Instances of subclasses of IcuCollation with customizations for
specific languages probably shouldn't share this cache with instances
of IcuCollation with the same language.

Change-Id: I06d66d199c99448a3375381baef0366c4d99c8c4

Merge "Set $linktrail for MessagesKu_latn.php"

Merge "Add CollationFa"

Add CollationFa

Bug: T139110
Change-Id: Ie15a2ee1c22ff4a1d2b721ed137227fe83dd12ea

Set $linktrail for MessagesKu_latn.php

Bug: T153233
Change-Id: I488129a002a92481412667706dbd2c534aa8d45b

Merge "SpecialUserrights: Use session data instead of URL parameter for success"

Merge "Fix linking Special:ChangeContentModel from action=info"

Merge "SpecialRunJobs: Use wfHttpError to generate error messages"

Merge "Add maintenance/view.php for viewing page contents"

SpecialRunJobs: Use wfHttpError to generate error messages

wfHttpError generates HTML so the content fits to the Content-Type.

Bug: T146546
Change-Id: Ia90f22564ecb3807aff3b787242bc74fe1678def

RELEASE-NOTES: Fixup for dfb50ab9f3

Change-Id: Ibe2efcd11c3f97ca343f43168bde278e44537be9

Merge "Back off from job types longer for DB read-only errors"

Merge "Escape return path extra params to php mail()"

Merge "A full word instead of an acronym for the revid message"

Merge "Always show exceptions of type 'error' in LTR"

Make DifferenceEngine's markPatrolledLink and getRevisionHeader public so that hooked functions can use them where needed

The said hooks were added to core in
https://gerrit.wikimedia.org/r/#/c/298026/ . One of the many intended use
cases is https://gerrit.wikimedia.org/r/#/c/326074/ which currently fatals
due to these two being protected, and obviously we don't want to lose
functionality when hooking into one (or more) of the aforementioned hooks.

Change-Id: I260c8b57c0bb2af3a6982bd7142b112a4a023391
Depends-On: I02de9069854532faec4c0c1798a10f862e6dfd7c

A few more hooks in DifferenceEngine

For mucking with the class member variable mNewContent and optionally
allowing the suppression of the "missing revision" message when revision
data is not found for a requested revision.

Originally implemented as the "GetUserMessagesDiffCurrent" hook (yes,
these three separate hooks in three separate places were essentially the
same) by Wikia for their SiteWideMessages extension.

Change-Id: Ie0c175af2af418d4ed3de28c94df918115312da3

Localisation updates from https://translatewiki.net.

Change-Id: I1b8437a0df98dccdd524ef3a0ff8493a98bf59f1

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 73798df0632e10313b82987d0b99e93c73407ca7

Change-Id: Ia0e511311eb05276617cc7bdff72b07347591ca3