Clean up "bad parameter" error messages in Message handler

Change-Id: I64b54b1991d17ae97a8dd9414d6807daab982a3a

Merge "Fix up replica/pos logging"

Merge "Improve connection logging a little more"

More specific warning when RC page fails to load

- Provide more specific message when the page fails to load
(when the connection drops for any reason), replacing the current message,
which says misleadingly that no results could be found.
- Center warning messages horizontally.

Bug: T177413
Change-Id: Ie2bc4fe4bd19103b513e4753e59f4adb295fbafa

Improve connection logging a little more

Don't include the trace in the actual message, that keeps us from
grouping these messages by type. Instead inject the trace as an
extra parameter we'll pick up in logstash

Change-Id: I48184c1af2560827ef50baff4fc0443f00697504

Localisation updates from https://translatewiki.net.

Change-Id: I48e20dc76cf2d7ae9e54c428f7896014b36b7ab1

Make filter area togglable

- Close and open the dropdown filter menu when
a user clicks on the active filter area.
- Fix inaccessible menu when user clicks on browser
scrollbar while dropdown filter menu is visible.

Bug: T176954
Change-Id: Id1c0c5078d80064a432b3db0f42775d0c00cdd85

Fix up replica/pos logging

Change-Id: I8670aef4a684c1c7f9bb510e7f2cc1530bdcfed0

DifferenceEngine: Fix wikidiff2 version check

The feature was actually released in 1.5.0.

Change-Id: Ic1a8818e34327510a75a4252f98b1f67aeeacb04

Merge "resourceloader: Remove outdated readyState handler for base modules request"

Merge "Put page_is_new back into ContribsPager query"

Add render moved paragraphs marker in diff view

This overwrites the generic marker from the wikidiff2 lib and adds
simple RTL aware arrows that indicate movement of the paragraphs.

Bug:T176486
Change-Id: I3554785772a091c3813e41071efcb7bb9a26a951

Merge "RCFilters: Move label prefixes from dm.ItemModel to ui.TagItemWidget"

Merge "Remove Media namespace from the list of RCFilters"

Merge "RCFilters: HTML-escape tag names in filter capsules"

Add deprecation notices for selectFields() methods deprecated in Idcfd1556

Now that WMF-deployed extensions have been updated to no longer call
them.

Change-Id: I04942ca1b95baa2126f2dcf4d0975536f4dd07c7
Depends-On: I300130c7b952a353ac28989d39d7f01366da2a37
Depends-On: Ia81bf6a655af800ce8ac19940d851e65746e1f77
Depends-On: I0d869aacaaad85cdd34361a611ac8348bdbb757d
Depends-On: I7b05a99e23da296a673eb0bd34f18344618c8be3
Depends-On: I42237e8e29497bbc56606f6ad01de3d525bf8b2a
Depends-On: If75d2e76c2f166bc40a544dd502da43171ce1e7b
Depends-On: I401809d2638b11e6c77a318fc8fbbc41fa639083
Depends-On: I5d62ad76fdb64a9c6efd228f27e9b5f512f17d5e
Depends-On: I488afaa991e3d26b638a4f588f70db455959eadf
Depends-On: Ibe82dadb0f2d1c5dbc38b96731c6e443c5469ff0
Depends-On: Id25b86dd415e2e3c6190a91faee2a3b815e50f61

Put page_is_new back into ContribsPager query

Accidentally removed in Idcfd15568.

Bug: T179467
Change-Id: I0a26da37638352d4a4dca08c0ef3eb9f99c7100a

Remove Media namespace from the list of RCFilters

Bug: T176990
Change-Id: I3c5b65244fd6b2782dea0404fe3671407fbb99e6

Merge "RCFilters: Change main entry point text"

RCFilters: Spell out Tags and Namespaces labels

- Provide Tags and Namespaces menus with labels where the icons now are,
in order to increase discoverability.

Bug: T178539
Change-Id: I2101898d10043ef45d7636f6676d8f1f51b9db58

RCFilters: Change main entry point text

Bug: T178069
Change-Id: I52c41a795ca76af9c645cb35266ccebe32a8f1fa

Merge "resources: Remove deprecated 'mediawiki.widgets.CategorySelector' module alias"

Merge "resources: Remove deprecated 'jquery.appear' module"

Merge "resources: Remove deprecated 'jquery.placeholder' module"

Merge "Add statsd metric support to WANObjectCache"

Merge "resources: Remove deprecated 'jquery.autoEllipsis' module"

search: Hyphen to dash for range in results message

Change-Id: I0ace65b08a21b4024f7c9825173dc17b7f408d3d

resources: Remove deprecated 'mediawiki.widgets.CategorySelector' module alias

Change-Id: I28557186a6cfe45b2a378f06ca59848be3f8e781

resources: Remove deprecated 'jquery.appear' module

Change-Id: Ifce63eb64c94203acb049d3f588e75f18123f12e

resources: Remove deprecated 'jquery.placeholder' module

Change-Id: I1f14561e54df477067460411ff155e3f2eeb167a

resources: Remove deprecated 'jquery.autoEllipsis' module

Change-Id: Ib181b814953bac0153dead95a25040ed2d4ca7ca

Merge "resources: Remove the deprecated 'jquery.badge' module"

Update OOjs UI to v0.24.1

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.24.1

Depends-On: I9ebdc86488c4eda7c524fb6004ea6a124a63d7ab
Change-Id: Ia6332cc6bd7ae6e99bfe68043b0510a8bd483668

Localisation updates from https://translatewiki.net.

Change-Id: I522e124758e4aecddc5dc36024b3df1ca8891067

Merge "EditPage: Extract some edit conflict code into EditConflictHelper"

Merge "Prefer using plaintextParams instead of rawParams where possible"

resourceloader: Remove outdated readyState handler for base modules request

There is a bug in IE10 where under certain conditions[1], the browser will
fire 'onreadystatechange' on script tags before the script has executed.

To avoid this bug, simply remove this outdated compatibility code. It was
originally placed in startup.js in 2011 for compatibility with IE6-8, and
was inspired by the internal source code behind jQuery's ajax transport for
script tags.

jQuery no longer does this either and uses 'script.onload' in all browsers,
which is supported by IE 9 and above. MediaWiki currently requires IE 10
for its Grade A experience.

Looking further, it actually seems that Microsoft has now retroactively
documented that readyState="loaded" in IE6-10, for script tags, represents
the completion of the download, not the execution. [2] Presumably the reason
we did not find this bug earlier is because of the rare conditions [1], and
because even then, on most page views, readyState="loaded" is never fired.
Instead, usually the event is skipped by IE and the first and only readyState
event is readyState="complete".

[1] The conditions are: A relatively short HTML page, with a script from
<script async href=".."> that in turn appends a dynamic script tag, where
the <script async> is followed by a <link rel=stylesheet> that is relatively
large/complex (possibly due to use of cross-domain @imports, not sure).

[2] https://connect.microsoft.com/IE/feedback/details/729164/ie10-dynamic-script-element-fires-loaded-readystate-prematurely

Bug: T178943
Change-Id: Id3de61515e4701a7ef59e4a4d8643a0e09f8f589

Merge "build: Upgrade grunt-stylelint"

Replace selectFields() methods with getQueryInfo()

Several classes have a "selectFields()" static method to tell callers
which fields to select from the database. With the recent comment table
change and the upcoming actor table change, this pattern has become too
simplistic as a SELECT will need to join several tables to be able to
retrieve all the needed fields.

Thus, we deprecate the selectFields() methods in favor of getQueryInfo()
methods that return tables and join conditions in addition to the
fields.

Change-Id: Idcfd15568489d9f03a7ba4460e96610d33bc4089

build: Upgrade grunt-stylelint

Change-Id: I4c47f1d290802705d061167fe369c130405512e0

resources: Mark MediaWiki.UI as deprecated.

Change-Id: Idd5f898c1b596040be8d14dee0a9893b59d6a532

build: Bump karma devDependencies to latest

 karma                   1.5.0  →   1.7.1
 karma-chrome-launcher   2.0.0  →   2.2.0
 karma-mocha-reporter    2.2.3  →   2.2.5
 nodemw                 0.10.1  →  0.11.0
 qunitjs                 2.4.0  →   2.4.1

Change-Id: I1e06189a12fbbc1d62781a4636f192c5b7078c1a

Merge "build: Bump eslint devDependencies to latest"

Localisation updates from https://translatewiki.net.

Change-Id: I647e1c0b0bf533b1918b8ab7404676e86e9ace74

Add action/user tracking to html cache purge jobs

Change-Id: Ic7155a7303debfaf26b13cb836497ccbc89ea238

Merge "Article: Show moveddeleted on 404s for all pages if the user has a session"

Prefer using plaintextParams instead of rawParams where possible

Using Message::rawParams() is something that should immediately
jump out as scary. While it's perfectly safe to do something like
wfMessage( 'foo' )->rawParams( $userControlledVar )->text() and
escape later, mixing the safe type of rawParams with the
dangerous type makes it more likely someone will miss something
when checking for security.

To reduce the likelyhood of confusion, prefer ->plaintextParams
for such cases, as it can be seen to be safe at a glance.

Change-Id: Ib067eb08ec934779ae743751306f1a0397bdf71c

Merge "No longer special case 'patrol' in $wgFilterLogTypes"

Article: Show moveddeleted on 404s for all pages if the user has a session

Add a check for user session and show movedelete message for this case as well.

Bug: T178099
Change-Id: Id42f27a19e85b608a734c7fedb4e016d8cdc163e

Support language fallbacks for Special:MyLanguage

Add fallback lookup based on the interface language to
Special:MyLanguage. Lookup order:

  <user lang> -> <explicit lang> -> <fallback lang(s)> -> <base>

Example when the user's lang is arz:

  arz -> <explicit lang> -> ar -> en -> <base>

Bug: T50292
Change-Id: I245cab71fdd4b4585f86dde808493cd69841b09a

Localisation updates from https://translatewiki.net.

Change-Id: I2a9bd114c751283b554f79e576168a78cf700cfd

Localisation updates from https://translatewiki.net.

Change-Id: I248dd76bd1d4ed220a09573916ba99ec73f225d0

No longer special case 'patrol' in $wgFilterLogTypes

Previously, if you did not have the right to patrol/view patrolmarks,
you were not allowed to filter Special:Log to remove autopatrol
entries.

Now if wikiadmins want to disable the filtering, they have to
directly change the config value.

This was stupid because:
* Users without patrol rights are just as likely to not care about
  autopatrol spam as users with the priv.
* Sometimes wikiadmins want to hide old patrol log entries even
  after they disabled the patrol feature.

It should be noted there have been two previously attempts at fixing
this issue that didn't go anywhere:
* I9de17fc197a06402a4999a9fb792b86657641f76
* I88448ca0f09069943fd514a5b8213dfdafa57299

Bug: T44246
Change-Id: I590db72c169f3a9ad96c710f088923419d40e48d

Group XMP parse failures by exception message

Change-Id: I2bba40ded5c90f2e19a32bb5540e3c441b605da6

Merge "rdbms: Group disconnect/reconnect errors by DB server name"

rdbms: Group disconnect/reconnect errors by DB server name

Still allow varying on stacktrace, as those are interesting

Change-Id: I62bc3f68fcbe43532dce849d515a5e0620fba6f2

Localisation updates from https://translatewiki.net.

Change-Id: I5824c8668527b63f942ca56da1c6d886adce2ae7

Merge "build: Update stylelint"

Merge "MediaWiki\Shell: log stderr"

Html escape the timestamp in Special:Undelete

Change-Id: I377effe4050d961ec6295a9fe684e4ff6a71bc77

build: Update stylelint

Change-Id: I5a0ccb26a54109e641fd121c6427a71e82e05c74

RCFilters: Move label prefixes from dm.ItemModel to ui.TagItemWidget

Change-Id: I0d463b60ffaa7ab099cb46eeac94b709af325933

RCFilters: HTML-escape tag names in filter capsules

Bug: T178975
Change-Id: I9544a675fa2801bdb5d7de3ebd162a4214de740f

MediaWiki\Shell: log stderr

Change-Id: I1495fe2aba10102d7e36c3a3e5fdabf97f14546b

Merge "mw.widgets.DateInputWidget: Fix unexpected MMMMM or ddddd date format"

mw.widgets.DateInputWidget: Fix unexpected MMMMM or ddddd date format

In some locale data in moment.js such as zh, llll date format already
uses MMMM and dddd. The original attempt to expand MMM to MMMM and ddd
to dddd inadvertently expands MMMM to MMMMM and dddd to ddddd, which is
then interpreted as MMMM or dddd followed by an unexpected single M or d.

Change-Id: I2634dfbaaf9615a13dce7b8f4ba3c3bea6863a91

Merge "CheckBlocksSecondaryAuthenticationProvider: Avoid user language during auto-creation"

build: Bump eslint devDependencies to latest

 eslint                 3.12.2  →   4.9.0
 grunt-eslint           20.0.0  →  20.1.0

Change-Id: I7eb9964fb5d163c5bbcef850f62383cbceb59287

Localisation updates from https://translatewiki.net.

Change-Id: I1314aff11d814fe14dd72987ea4c67f273a801fb

Merge "Refactor some Installer code into ExecutableFinder"

Merge "Add Unicode to ICU mappings for versions 58 and 59"

Refactor some Installer code into ExecutableFinder

Refactor Installer::locateExecutableInDefaultPaths() into a separate
utility class, ExecutableFinder. This class is already used in plenty of
places outside of the installer, so it's ripe for being extracted.

This class is located in utils/ due to the dependency upon
Shell::command(). Once that no longer has a dependence upon MediaWiki,
this class can be moved to libs/ too.

Change-Id: I175465acc0d64f990445ce05fabcee8b88a0b259

Group Swift metadata errors by making $path a parameter

Change-Id: I3beb3743e1d13b06a9110c44f84fdc8d1b193137

Merge "Add test for Revision::insertOn"

Merge "Refactor BlockTest"

Remove Article static methods deprecated in 1.24

The Article::selectFields(), Article::onArticleCreate(),
Article::onArticleDelete(), and Article::onArticleEdit() methods have
been deprecated in favor of the correponding WikiPage methods since
1.24. Remove them.

Change-Id: I30f3d88f8d28ccaebf5a858058e183112d29a1a7

Add test for Revision::insertOn

The success test indicates a change in behaviour
of the insertOn method once
I4f24e7fbb683cb51f3fd8b250732bae9c7541ba2
is applied.

Change-Id: I987d64c5bc3788b7d6b0a858ff39e97fae38de3c

Merge "RawAction: Add json to the allowed content types"

EditPage: Extract some edit conflict code into EditConflictHelper

Extensions with custom content models (e.g. ProofreadPage) will extend
EditPage to provide a customized editing experience. However when
another extension like TwoColConflict wants to change just how edit
conflicts look, this causes problems since only one class can take the
place of EditPage.

So instead, split most of the frontend code for edit conflicts into
TextConflictHelper, and call it from edit page. Extensions can override
with the instance created by calling
EditPage::setEditConflictHelperFactory().

And to make that split possible also move EditPage::addNewLineAtEnd()
and ::buildTextboxAttribs() into a separate TextboxBuilder class that
both EditPage and TextConflictHelper can use.

Bug: T176393
Change-Id: Ie415edd84329c02d5762477f8a171fced85b01a2

Add Unicode to ICU mappings for versions 58 and 59

Change-Id: I87a5e6ce3a44a2be1e6bf8adf2f98cd0a4745574

Merge "Remove unused mediawiki.ui 'checked' icons"

resources: Remove the deprecated 'jquery.badge' module

Bug: T178450
Change-Id: I4540e5959ea15323b1f54be51fbeea6ca3f35169
Depends-On: I78543abe412d04d61a05e76702cb7681d5fad61d

Localisation updates from https://translatewiki.net.

Change-Id: Ic741440ded2718acbe29973af639442eb722d0ff

Refactor BlockTest

Something is holding state and interfering with
RevisionUnitTest from within BlockTest
as seen in I4f24e7fbb683cb51f3fd8b250732bae9c7541ba2
PS30

Change-Id: I5b779941385eb88e560dd2c63a8aea356804c806

Merge "HistoryPager: Add missing $tagFilter property"

HistoryPager: Add missing $tagFilter property

Change-Id: Iec2d944d5583eb13fa9383c0a6a183b550f60aab

Article: Drop partial purge code and constants, deprecated in 1.29

Change-Id: I4f20d823e6c9206121212b8a73ecc28f91e68b45

Merge "Rename some cache keys with odd virtual hierarchies"

Merge "Make runJobs.php treat --json value as case insensitive"

Merge "API: Include setupSkinUserCss in prop=modules for useskin mode"

Merge "API: Add tests for useskin parameter of ApiParse"

Remove unused mediawiki.ui 'checked' icons

Removing unused mediawiki.ui 'checked' image, which was in use before
the switch to the unified checkbox with blue and white appearance.

Change-Id: I4ba88583a2ed59b44a1f7bbffe823468af8e3874

Optimize SVGs and align colors to WikimediaUI palette

Aligning SVGs to WikimediaUI color palette.
Additionally optimizing by help of SVGO and aligning them to each other,
among changes:
- removing `standalone="no"` attribute as its default
- removing invisible, application specific metadata
- reduction or removal of `id`s where applicable
- unifying identation
- sorting attributes in similar order

Bug: T152025
Bug: T178867
Change-Id: Ia693b2092bbb730ccaabe75bec563bc3a64cc108

Make runJobs.php treat --json value as case insensitive

The docs mentioned "JSON", which did not actually work.

Change-Id: Ie9802551d05f7d80d1db0fd316e36f4b35bbc521

Rename some cache keys with odd virtual hierarchies

This makes automatic statsd metrics more useful.

Change-Id: I07d87dcb8ce9b42cc2f1c84ac4c06d177d463b1d

Add statsd metric support to WANObjectCache

Bug: T178531
Change-Id: I3037281d09cd5195347789f544deae89711f128b

API: Include setupSkinUserCss in prop=modules for useskin mode

Follows-up 90c95fc7f290f, which included result of Skin::getDefaultModules
in the prop=modules list. All hardcoded modules in OutputPage and Parser
were also subsequently moved into Skin::getDefaultModules.

However, a number of modules cannot be moved there because fundamentally
Skin::getDefaultModules can only load modules via OutputPage::addModules().

For style modules, addModuleStyles() must be used.

Fortunately, there is already a centralised place for that, namely
Skin::setupSkinUserCss(). Include that in the ApiParse return as well.
That should resolve the last bit of inconsistency between ApiParse
and OutputPage when it comes to the module queue.

Bug: T140664
Change-Id: I35e2e3bbdccdd1aa2a259b8e624daa80c609ba8c

API: Add tests for useskin parameter of ApiParse

Change-Id: If3dff7be5ccb6791f95d37c06998fcbadf1f469f

Merge "Setup: Merge PreConfigSetup into Setup.php"

Merge "Add action/user tracking to link refresh jobs"