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 "Enable supplying inline interwiki results"

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

Merge "Fix parameter type docs"

Merge "Document type of LogEntry::isPatrollable property"

Merge "mediawiki.special.apisandbox: Avoid bitwise tricks when checking for integers"

Merge "Fix small typos in includes/lib/lockManager/LockManager.php"

Merge "Fix assigning void return value to a variable"

Merge "API: Expose wfIsBadImage() in prop=imageinfo"

Fix parameter type docs

Changes:
 - uses int instead of number as param and return value type,
 - uses stdClass instead of stdObject
 - fixes ResourceLoaderClientHtml constructor's $target param type:
   it is string|null, not an array (previously misspelled as "aray")
 - changes the type of references to XML parser in XMP lib to resource
   instead of not existing XMLParser

Change-Id: I98c363ebc6658d1f4dcabad97a9a92f3fcd7ea8c

Document type of LogEntry::isPatrollable property

Change-Id: I557b92f25f15f57123af329c79ef1752025aedf3

mediawiki.special.apisandbox: Avoid bitwise tricks when checking for integers

In addition to being more obvious than `n | 0`, `Math.floor( n )` also
works correctly for numbers outside of 32-bit int range.

Change-Id: I254457500e141c0dd4e28c707fb4a00c1b04c272

Merge "mediawiki.page.watch: Avoid dynamic message keys"

mediawiki.page.watch: Avoid dynamic message keys

Restructure the code to avoid constructing message names by adding
strings together. Makes it easier to find the usages.

Also, simplify a check for odd/even namespace numbers.

Change-Id: I0b6794cc07ed2a600499632d240b71898715c5b0

Fix assigning void return value to a variable

Change-Id: I1e675d121938c5b6b7f8165681c645154e3a70d0

Merge "Add externallinks.el_index_60 column and indexes"

Merge "Export: Use BCP 47 language code for attribute xml:lang"

Merge "Message: Fix buggy parameter handling in Message::params()"

Merge "The automatic test in MediaWikiTestCase was marked as risky"

Merge ""Image size limit" text should always read left-to-right"

Fix small typos in includes/lib/lockManager/LockManager.php

Change-Id: Ib3e3c6e01dc1ad4604b1cf6c95467f25444f38a2

Merge "Convert legacy bugs ID to Phabricator tasks ID for Parser class"

Merge "Add script to support per-line @suppress annotations in Phan"

API: Expose wfIsBadImage() in prop=imageinfo

If iiprop=badfile is set, expose the wfIsBadImage() status as a boolean.
This is mainly intended for use by Parsoid, but could be useful for
other API users too. The context title can be set by using the
badfilecontextitle parameter.

Also be a little more paranoid and make sure wfIsBadImage() always
returns a boolean.

Bug: T148343
Change-Id: I3432ead92b8d2a17de5243285f2760fc39b501ae

Apply color palette change also to wikitable cell border

In e7a4daf the color of the wikitable border was changed but not the color of
the cells. This leads to a different border color for empty cells and for
non-empty cells.

Change-Id: Ifb691b7f7c388b068e26ccecb94f822432552466

Merge "Add googleweblight to JS blacklist"

Add googleweblight to JS blacklist

Google runs JS on their proxy servers and outputs a page which may
be broken in creative ways (e.g. clicking anywhere triggers the
fundraising banner). Treat them as a non-JS browser to minimize
bugginess.

Currently broken with CentralNotice banner:
https://googleweblight.com/?lite_url=https://en.wikipedia.org

Images below the fold never load:
https://googleweblight.com/?lite_url=https://en.wikipedia.org/wiki/Barack_Obama

Bug: T152602
Change-Id: Ia277ab51634b770a950616a842bedb29fcbf1f12

Localisation updates from https://translatewiki.net.

Change-Id: I37f811dd07e6704c47cc159715f2e24ddaf50ab0

Add script to support per-line @suppress annotations in Phan

Adds the functionality of per-line @suppress annotations. Phan already
supports per-class or per-method annotations, but does not have any
per-line support due to the PHP AST only returning comments that are
class/property/method level docblocks.

This is a bit of a hack, but get's the job done. Removes the
PhanTypeInvalidLeftOperand issue from blacklist and suppresses it
to demonstrate the supression works as expected.

Change-Id: I5066b3b431fb69175a711ee366e95f31c7c47639

Export: Use BCP 47 language code for attribute xml:lang

The patch changes for example
 https://crh.wikipedia.org/wiki/Mahsus:Export/Ba%C5%9F_Saife
from
 xml:lang="crh-latn"
to
 xml:lang="crh-Latn"

Change-Id: I2fb218fe026c5ffee081fb8aaee7b154a8732bdc

Adjust phan to allow for making progress strengthening analysis

With phan set to only report critical errors it will be quite hard
to cleanup the code base to the point where we are running stronger
analysis. Instead of setting minimimum severity to critical, set
minimum severity to low and suppress all existing issues.

This will allow developers to choose an issue type from the list, fix
it, and update the config.php to report on the issue in the future.

Change-Id: I2771a4ff37f43cca215b939ed7d82f137d72cea6

Merge "Block API: Allow blocking/unblocking by user's ID"

Do not lose message parameters in UploadFromChunks::verifyChunk()

This code is gross, and my changes do not really make it better,
but it works more correctly more often.

Bug: T147720
Change-Id: If9ce05045ada1e3f55e031639e4c4ebc2a216de8

Block API: Allow blocking/unblocking by user's ID

Add feature to block/unblock users by their ID. For this,a new
parameter `userid` is added to block & unblock API request.

Bug: T34496
Change-Id: I084a4e275cd937053c505cd388a365b316990ece

Remove "Generated by PHPUnit" comments in 3 files

Change-Id: I8624f8cd0f658a37384a948076e8f6d68f484ce1

Convert legacy bugs ID to Phabricator tasks ID for Parser class

Side edits to comments:
  - update a @fixme as T10068 has been declined
  - fix spelling

Change-Id: I7f9f191ff68bb56de72563dde957ccf4731267e4

Set $wgUserEmailUseReplyTo = true by default

Mainly to work around restrictive DMARC policies (e.g., Yahoo!).

Bug: T66795
Change-Id: I3b34f6820915cc68d46ff6fd38feacd06efd2b69

Merge "Mark pages on watchlist as visited using JS without reload"

Merge "Move Special:Watchlist auto-reload check from PHP to JS"

Always show exceptions of type 'error' in LTR

Bug: T153027
Change-Id: Iad5b9a01f5b8cdaa2ed94b3ece937acc1f6faa60

Localisation updates from https://translatewiki.net.

Change-Id: I1b174ddfd90298e39e68a7495439e8c3b08feb1c

Mark pages on watchlist as visited using JS without reload

When the "mark pages as visited" is clicked, a dialog appears,
asking for confirmation. On confirmation, an API request is sent
to mark all pages as visited, and all unvisited watchlist entries
are changed to appear visited.

Based on a userscript by User:NQ (from English Wikipedia)
https://en.wikipedia.org/wiki/User:NQ/WatchlistResetConfirm.js

Bug: T150045
Change-Id: I45fb02a1edc1b0331925e9a244a2455f86ad3886

Article::fetchContent() was removed

Bug: T145728
Change-Id: Ie8bc12e391f2983452bf472333ea3cfaa72e4918

Merge "WikiPage::getText() was removed"

WikiPage::getText() was removed

Bug: T145728
Change-Id: I9e5ac149ad1780719b5c8f4e27048168cf33c0b9

Merge "ImageHistoryPseudoPager: Don't ignore limit from URL"

Merge "WikiPage::prepareTextForEdit() was removed"

WikiPage::prepareTextForEdit() was removed

Bug: T145728
Change-Id: If15777c04a52bd398092884fab592cfb0930255b

"Image size limit" text should always read left-to-right

Bug: T144386
Change-Id: I10ab5ed71c114bbfefcc841d8a3d4b280bf1acf2

Merge "installer: Restore PHP 5.3 compatibility in index.php"

Localisation updates from https://translatewiki.net.

Change-Id: I204f2ce7bacf1bcf382dc51b2a574e405220d5d3

installer: Restore PHP 5.3 compatibility in index.php

This file should remain compatible with PHP 5.3, so that we can
actually display the error message about old PHP version, rather than
a syntax error.

Change-Id: If12f79f5d375e925291728aac7c9db18ca226d26

The automatic test in MediaWikiTestCase was marked as risky

MediaWikiTestCase::testMediaWikiTestCaseParentSetupCalled is
marked as risky when strict code coverage is activated in
PHPUnit (--coverage-html ../../docs/code-coverage
--strict-coverage --debug -v).

This patch adds the annotation @coversNothing to disable
code coverage on this test. As an automatically-added test,
it must adapt to a number of different environments, so it
is not feasible to add the exhaustive list of @uses classes.

Bug: T152923
Change-Id: Ic670ff2d89f1c5df500c0d10b153d4efd013d7f4

ImageHistoryPseudoPager: Don't ignore limit from URL

I thought the parent class would handle this for us, but no: our setLimit()
call has been overriding the limit from URL set in parent constructor.

Follow-up to f94d5239b528c406c85e090152805a1306e13ded.

Bug: T152813
Change-Id: I93762021f70613bfde72da1eb737e9fa32d7d97c

Move Special:Watchlist auto-reload check from PHP to JS

This makes the watchlist js load all the time, instead of only
if the auto-reload preference is set, so we can put other things
in the same js module.

Bug: T150045
Change-Id: Ib8acca39593fe3d2369dc3187a9a55413553843d

Localisation updates from https://translatewiki.net.

Change-Id: I73bab410a14742c1c93b79bceb7de73feed5d752

Escape return path extra params to php mail()

PHP only escapes some dangerous shell characters. This is a hardening
measure, as MW's sanitizeEmail routines should also have prevented
evil characters from being in mail addresses in the first place.

Bug: T152717
Change-Id: I3736d612ed40d257ee3dde8e98eb30ccf432670a

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

Such error are likely to persist longer than other random
exceptions. In that case, it is better to avoid burning
through the job retry count.

Change-Id: I6785bd608856f98d21e0b0b05d3899a7081c38e2

Merge "Fix links problem on Special:Search"

Include numerical namespace in slow-parse.log

This makes it easier to sort by namespace rather than trying to parse it
out of the title.

Change-Id: I946cb00548bcb69bd2be98c15a9f1e02e546fa24

Merge "Special:RC 'hidemajor' filter"

Merge "Make refreshImageMetadata not fail completely if it doesn't like a single file"

Localisation updates from https://translatewiki.net.

Change-Id: Ic25484ba66bfe1291c39d7e12ed08fa7d8b633c6

Fix links problem on Special:Search

Fix bug that most links on Special:Search did not have text rendered
Also restore some changes from commit f736c3709a54be33f808e8d12f85707b90763165
where Linker::link() usage was replaced by LinkRenderer

This reverts commit 791d5319bff0be23c3ea474a84d42ed788355688

Change-Id: Id25d1b1acd0dcafc47511b2db3782e5d441f3731

Follow-up e8dd1f43fb169: fix JS error when $wgTranslateNumerals is false

Bug: T152800
Change-Id: I99c341168dada063876a66689d68e0b9adf7f09a

Make refreshImageMetadata not fail completely if it doesn't like a single file

Change-Id: I1170cf91ba6dea08d35457c9859b8c18482d4aa6

Merge "ApiQueryUsers: Add ability to search by user ID"

Merge "Revert "Replace Linker::link() usage with LinkRenderer""

Revert "Replace Linker::link() usage with LinkRenderer"

Most links on Special:Search no longer have links with text rendered.
Please revert this revert and fix the issue before re-merging.

This reverts commit f736c3709a54be33f808e8d12f85707b90763165.

Change-Id: Id92c484adc1d3339f9412bfb60ffe213cc1315ee

Merge "Remove empty lines from PHP and JavaScript comment blocks"

Remove empty lines from PHP and JavaScript comment blocks

This is a pure documentation change. It mostly removes empty lines from
comments (and entirely empty comments), as well as adds a few missing
documentation blocks and fixes a minor mistake. I hope it's ok to have
this in one patch. I can split it, please tell me.

Change-Id: I9668338602ac77b903ab6b02ff56bd52743c37c4

Merge "Use the phan exit code directly"

Merge "Use 1 processes instead of 4 for phan"

Use the phan exit code directly

Change-Id: Ic99e334576e350d11c57ac4b74085c99313401c4

Use 1 processes instead of 4 for phan

Running against mediawiki core phan needs around 1GB of memory
per process. >4GB is too excessive for a default setting. Instead
set the default to 1 process and add the ability for users to
select more processes on the command line.

Bug: T132636
Change-Id: I3d9fa462916fba519fe6d7f1916f881eefb6a942

Merge "API: More i18n cleanup"

Merge "HtmlArmor: Document that null is an acceptable value"

API: More i18n cleanup

* Use errorformat for action=login Failed responses in non-BC mode.
* We removed 'messageHtml' from action=rollback's response on error, but
  left it for success. Remove it there too, it's even less useful.
* We changed action=watch's reporting of errors, but left the
  mostly-pointless reporting of "success" UI messages. These should be
  handled on the client side.

Change-Id: Ia6c402a4254fbacf4c2c3f125ce8bf0bcc71e509

Localisation updates from https://translatewiki.net.

Change-Id: Iba22019e88d103bf2c7b98309e57e24be3328dff

HtmlArmor: Document that null is an acceptable value

And that you'll get null back.

Change-Id: Ibd74e6af16b51796b46c2e23b7442740f5f69a0f

Merge "Let mw.Api recognize errorformat!=bc"

ApiQueryUsers: Add ability to search by user ID

Added new 'userids' parameter to ApiQueryUsers.php. It accepts user ID numbers.

Bug: T34494
Change-Id: I8021bc8660f19e90f0f7d43ea0e82d5258c586ab

Let mw.Api recognize errorformat!=bc

Iae0e2ce3bd42dd4776a9779664086119ac188412 introduced a new
API error format where, in case of error, there is no
'error' property in the response, but an 'errors'.
mw.Api doesn't recognize those as errors and just resolves
the deferred, instead of rejecting it.

Change-Id: I4c05ef7d7574e2ca538170609fab85d8463b7b68