Merge "build: Bump various devDependencies"

build: Bump various devDependencies

grunt 0.4.2 -> 0.4.5
grunt-contrib-jshint 0.10.0 -> 0.11.0
grunt-jscs 0.8.1 -> 1.5.0
grunt-karma 0.9.0 -> 0.10.1
karma-firefox-launcher 0.1.3 -> 0.1.4
qunitjs 1.15.0 -> 1.17.1

Some minor fixes to pass new jscs rules; big ones postponed for now.

Change-Id: I5f2348c3cf96cf5aa20eeb9083300155b3920c5b

Split StatusValue out of Status class and put it in /libs

* Deprecated useless FileRepoStatus class

Change-Id: I015635a9bf080ef6d98b2cff49b949c4378a859f

Merge "Trival: Comment speling"

Merge "registration: Add resource template-like functionality to extension.json"

Trival: Comment speling

Change-Id: I17f50b5f26f5e8c1c9687c040c64699720ea6167

registration: Add resource template-like functionality to extension.json

This allows for extensions to specify common arguments to ResourceLoader
module definitions by only specifying them once.

The only supported values are "localBasePath" (which is relative to the
current directory), "remoteSkinPath", and "remoteExtPath". If a module
is using a custom class or is already specifying paths, the default paths
will not be added.

Tests are included to cover existing functionality and newly added
functionality. The convertExtensionToRegistration.php script was also
extended to try and guess what the default paths should be.

Bug: T88786
Change-Id: I802461796e8d8584dacf3b0c811b5ba97a4a8f7f

Localisation updates from https://translatewiki.net.

Change-Id: I3e3b21c3c18a8a6ae1862c263134f637acef9719

Merge "HTMLForm: Support hide-if for HTMLCheckMatrix"

Merge "registration: Fix regression in ExtensionMessagesFiles handling"

Localisation updates from https://translatewiki.net.

Change-Id: Ie8bca0b8e5832b189f299ee1534e468df969fe36

Merge "Documentation of compressOld.php"

Localisation updates from https://translatewiki.net.

Change-Id: I2ca7ae2e6545647817f47b96aa037f4c85ae848e

Resources.php: Fix phpcs-strict

Make line 1076 multiline to shorten it and
pass the phpcs-strict test.

Change-Id: I9a26d8b562cfa33881cbc6fb7447bdd53e4a71c5

HTMLForm: Support hide-if for HTMLCheckMatrix

It outputs multiple table rows, and each one needs the class and data
attribute.

Change-Id: I6667c3ea0e10167d941cb214d76889b389af79a3

registration: Fix regression in ExtensionMessagesFiles handling

In f2daeaa7494b404, the handling of MessageDirs was fixed to properly
handle arrays, except that broke ExtensionMessagesFiles which doesn't
support arrays. Since both settings are pretty much different, I split
them into separate functions.

Bug: T88858
Change-Id: I48765af2489d2ef1e30adb1d7c5a5a8f23bf522d

Merge "Fix type hint in HistoryAction::getArticle"

Localisation updates from https://translatewiki.net.

Change-Id: Ic8663ea8b5ad5b4e56b349de0c6d35c51cb5da40

Merge "Account for HiDPI variants in thumb.php rate limiting"

Merge "Move wfThumbIsStandard() to GlobalFunctions and add tests"

Fix type hint in HistoryAction::getArticle

See the type hint of the accessed Action::$page variable.

Change-Id: I10e4e27773bcca507c10cf526d193bf0a1cc97ac

Merge "mediawiki.page.gallery: Clean up $galleries variable"

Documentation of compressOld.php

Change-Id: Ib6e4914379568bfae32007f6e76968791798b08f

Merge "Database: Cast to int in estimateRowCount(), selectRowCount()"

Database: Cast to int in estimateRowCount(), selectRowCount()

Doc comments state that these methods return ints. In order to ensure
that, values must be cast to int before they are returned.

With respect to selectRowCount(), follows-up 65f81d284386.

Change-Id: I108221ce4ad1b5b103b015fe875de54e04781741

Removed old SqlBagOStuff alias

Change-Id: I6fdd8791713ce9ed67473065332266c217284974

Merge "Properly escape the messages in CategoryViewer.php"

Localisation updates from https://translatewiki.net.

Change-Id: I3b2be6f19b721e842fda17f0e4ae9ddb7269f08a

Properly escape the messages in CategoryViewer.php

This mostly reverts commit 8fe8035da20d0f5861b51f196c7443dda06301ba,
and redoes it without escaping the parser output.

Change-Id: I87da10e63b55c1a6c83dd497e30f65ad224e052b

Merge "Check $auth parameter in Title::isValidMoveOperation()"

mediawiki.page.gallery: Clean up $galleries variable

Follows-up a50c7c4181.

Instead of initialising it to an empty object and executing wasteful
code in the other functions, keep it undefined until it is defined
and use it conditionally.

Also:
* Add comment to justify() documentating what the 'this' context is.
* Return early in resize handle if $galleries hasn't been initialised yet.

Change-Id: Ib93f87e71dacfd1aad8bbb7d5559e999f4f00eb3

Use transparent rather than white background for ImageMagick SVG thumbnails

Default value of $wgSVGConverters['ImageMagick'] now uses transparent
background with white fallback color, rather than just white background.
This is done by using #ffffff00 ("transparent white") as background
color (the PNG file format supports saving color for transparent
pixels).

I could find no historical records for why it was done this way;
possibly the intent was to display white rather than black background
in IE 6 (which doesn't support PNG transparency) or in other contexts
where transparent background is impossible.

Bug: T76475
Change-Id: Ief0c7d71c8cc157e256fc84b1b7be3c23cc5fd9b

Check $auth parameter in Title::isValidMoveOperation()

This also includes the integration test from SemanticMediaWiki that
caught this issue.

Bug: T74870
Change-Id: I699e14958ee36ec5e86278e5dc0caed2a015d9af

Use 'rsvg-convert' binary rather than 'rsvg' everywhere

Update $wgSVGConverters['rsvg'] to something closer to WMF production
configuration (there is a more complicated setup involving two
variants of rsvg for some reason).

Documentation is scarce, but 'rsvg-convert' appears to be the "modern"
way to call rsvg, with 'rsvg' being deprecated or not recommended.

Bug: T76476
Change-Id: I5ed877f3a5a1f1e97ae881c1d03fc977276182b6

Merge "Exclude user interface elements in content from selection"

Merge "SkinApiTemplate: Remove <span dir="auto"> for firstHeading"

Merge "mediawiki.language.init: Ignore case of language code"

thumb.php: Generate valid HTML code on error page

http://validator.w3.org/check?uri=https%3A%2F%2Fwww.mediawiki.org%2Fw%2Fthumb.php;No200=1

criticize

Error Line 1, Column 1: no document type declaration; implying "<!DOCTYPE HTML SYSTEM>"

Change-Id: I8dcb6c7f342957ec473fe481aef2011a193ef8a5

Account for HiDPI variants in thumb.php rate limiting

If $wgResponsiveImages is on, include the 1.5x and 2x multipliers
in the wfThumbIsStandard() standard image size comparisons.

Change-Id: I79d866387730cdd4d7c7a976e691192b2335b7ce

Move wfThumbIsStandard() to GlobalFunctions and add tests

Change-Id: Ife9c011a476a4022cd72d433497944cbd7258e67

Merge "Sync up with Parsoid parserTests."

Merge "Non-word characters shouldn't terminate tag names on the tidy side too"

Merge "Removed unused and poorly supported time argument to BagOStuff::delete"

Moved "large write query" code to TransactionProfiler

Change-Id: Ic05e832eb21545a4e639b52aca7b3a5811a890ce

Merge "Made Revision::newFromPageId avoid master queries like newFromTitle does"

Merge "Convert JobRunner to PSR-3 logger"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 028a41e2c1a52476c0d5607ab5c3c6235efd9e3e

Change-Id: I476508ebce3dc9c783f466d190f26e2ed84c8d96

Localisation updates from https://translatewiki.net.

Change-Id: Ie2283d3f5487405235f10474589b858b59066fb7

Merge "Creation, deletion and improved management of change tags"

Creation, deletion and improved management of change tags

This allows users with the `managechangetags` right to create tags for
use by wiki users. (Currently there is no way for editors to apply tags
to their edits; that's to come in a later patch.)

Extensions can reserve tag names for their own use, even if they do not
define them or mark them as active.

Tag managers can also delete tags with <= 5000 uses. Currently, if a tag is
misspelt ("vandlaism") or no longer wanted (testing of OAuth, etc), the
wiki is stuck with it forever. This change allows users with the
"managechangetags" right to delete change tags from the database,
including removing them from all revisions to which they are applied.

Obviously this is a powerful thing to be able to do, but I view change
tags as a "light" kind of interface, useful for revision patrolling and
spam/vandalism fighting but not something that necessarily needs to hang
around forever. It's not a big deal for this kind of data to be thrown
away without being archived anywhere.

Tags defined by an extension can only be deleted if the extension allows
it.

Changes to tags are logged in the new "tag management" log. There's even
a nice API module, just for fun.

Bug: T20670
Change-Id: I77f476c8d0f32c80f720aa2c5e66869c81faa282

Remove LanguageGetMagic profile point

The hook call is already captured in profiling data and this
function is too.

Change-Id: I69ad54b9a66e24a419f492238f45c46922c112eb

Update OOjs UI to v0.6.6

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.6/History.md

Change-Id: I8e0ea18c23df5fe0530cc841799830c4f0082eff

Merge "Improve documentation of ChannelFeed and AtomFeed class"

Improve documentation of ChannelFeed and AtomFeed class

Change-Id: I1bebaf2d17ac71490aab3ab9cd1f08a4bb1fc3fb

mediawiki.language.init: Ignore case of language code

This allows capitalization suggested by BCP 47 without the risk
of duplicate entries.

Change-Id: I0ebdead5440862c087920fa93e85aeb47464272e

Merge "mediawiki.action.edit.preview: Remove <span dir="auto">"

Merge "Don't double escape in Linker::formatLinksInComment"

Made ArticleRevisionVisibilitySet include the revision IDs

Change-Id: I6a14c7b95041fdd44d7962af6dc139bcc29f7223

Unbreak interwiki user rights granting

UserRightsProxy does not have an equals method, nor does
CentralAuthGroupMembershipProxy.

This reverts the problematic part of 80ca508ed117bbf547 which
caused fatal errors.

Bug: T88505
Change-Id: I6c1dfe14bc08193c33f86482c8f835500c0dc21c

Merge "Remove explicit require of ExternalStoreDB.php"

Localisation updates from https://translatewiki.net.

Change-Id: I9b62abdd26a0f66e449f4f683be83db5c74fb1f5

Non-word characters shouldn't terminate tag names on the tidy side too

 * Follow up to Iceec404f46703065bf080dd2cbfed1f88c204fa5.

 * The accepted charset is changed to match the HTML5 parsing spec at:
   http://dev.w3.org/html5/spec-preview/tokenization.html#tag-open-state

 * Equivalent in parsoid at I462c336f9a00c8ccd11f3220a8738389e8ba7c7c.

Change-Id: I69cb000538fe195dd77273da5f91697fe1e7d283

Remove explicit require of ExternalStoreDB.php

AutoLoader should take care of this.

Change-Id: Iabe335cd14e2739df9c184d6ab2e6ee98205b550

Merge "Remove unused REPORTING_INTERVAL from resolveStubs.php"

No such method Exception::getText()

It looks like a recent change converted this try/catch
statement from MWException to Exception, unfortunatly it
is still calling a method that only exists on MWException.

We arn't doing anything with the exception but exiting.  Lets
just allow the exception to be uncaught and fail naturally. The
uncaught exception will still output an error and return
a non-0 exit code.

Change-Id: I3e0e9d283c255d2aba8139d675943e93d7cb021a

API: Don't dump Title object into API result

The intention here is to return the title as a text string, not to dump
all the internal fields of Title.

Bug: T88397
Change-Id: I43ad3c98bc1fc056a13f57e2e95a122108049334

SkinApiTemplate: Remove <span dir="auto"> for firstHeading

In Ibf408bc6c5d2f8c05667107a547b0b02e06de270 the <span dir="auto">
around the title was removed. Remove it here also.

Change-Id: Ia0b23a3c47ab01faef2cd380c318386cc9372709

mediawiki.action.edit.preview: Remove <span dir="auto">

In Ibf408bc6c5d2f8c05667107a547b0b02e06de270 the <span dir="auto">
around the title was removed. Remove it here also.

Change-Id: Ie774b7746caadc28be11782058deb17dbd96388f

Remove unused REPORTING_INTERVAL from resolveStubs.php

It causes a notice when moveToExternal.php is run

Change-Id: I50429742f87ab13a5428cbc282c2ce7eda37f714

Merge "registration: Fix handling of MessagesDirs array and add tests"

Merge "Update OOjs UI to v0.6.5"

Localisation updates from https://translatewiki.net.

Change-Id: I03277966c3b2a86316dbdae595370b9540f982bb

Merge "Gruntfile: Clean up Karma config"

Merge "Output wikiid/api in meta=siteinfo only when non-empty"

Update OOjs UI to v0.6.5

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.5/History.md

Change-Id: I1278a9d0f3b63977f293a79c73283593c65bf910

Merge "Remove dir=auto from firstHeading"

composer.json: Correct license identifier

According to <https://spdx.org/licenses/>, "GPL-2.0" means "GNU General
Public License v2.0 only", not "GNU General Public License v2.0 or later",
which has the identifier "GPL-2.0+".

Also made the same change in the documentation for $wgExtensionCredits.

Change-Id: If25c15d03a71d7c50c9c85a04c2a589a3d978ad0

Merge "codepointToUtf8(): Throw exception rather than calling die()"

Merge "Pass user to FileRepo::getUploadStash"

Pass user to FileRepo::getUploadStash

This avoids use of $wgUser in UploadStash

Change-Id: I82ca69818317508109b4d5f4823a20de47f29b01

Merge "Revert "CategoryView modified to use css columns""

Revert "CategoryView modified to use css columns"

The new feature does not work very well, and the commit
breaks several unrelated things. Sorry, let's try again.

This reverts commit cb5a0f20a436b90a5cfabfba84554e712b32794a.

Bug: T85496
Change-Id: I018050ec0bf773e6e22e99819fe3cad91171bbd2

codepointToUtf8(): Throw exception rather than calling die()

Change-Id: I0f2c05be95d34e069a3e9965f581d2adfc0a1b9d

Gruntfile: Clean up Karma config

Change-Id: I8197c2b0169983c32a44397faec95ad7eb47ab86

Update OOjs UI to v0.6.4

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.4/History.md

Change-Id: I75746e6f702e2108e5282f9a37827a1a533ab57a

Merge "StringUtils: throw InvalidArgumentException and move into libs/"

Merge "Kill hitcounter remains"

objectcache: Use Psr\Log\LoggerInterface instead of wfDebug

ObjectCache::newFromParams() will handle a 'loggroup' parameter
specially, getting an instance of \Psr\Log\LoggerInterface to
pass to the BagsOStuff.

BagOStuff implementations can use $this->logger which will be
an implementation of \Psr\Log\LoggerInterface. As this is set in
BagOStuff::__construct(), all subclasses must now call the
parent constructor.

The goal of this is to make the logging in BagOStuff non-MediaWiki
specific, in the hopes of separating it out into a separate library
in the future.

Change-Id: I8a8e278e6f028814499d8457d6d5341d03eabc7a

Update karma to latest version

Per Timo, hopefully this should fix our issues with Chromium timing out
after 10 seconds?

Related patch: https://github.com/karma-runner/karma/commit/840ee5f7

Change-Id: Idbac350e2c612290f860ae740ef6ba242c9e0896

Merge "PostgreSQL: Drop unneeded foreign key constraint"

PostgreSQL: Drop unneeded foreign key constraint

Change I1c7f3a84f10df05d6b37dccbad4c8232edf51580 causes
an existing foreign key assumption (under PostgreSQL) to be
violated upon deleting a page.  This foreign key assumption does not
explicitly exist in MySQL, and is not implied via documentation.  So
it was probably never needed in the first place.

Don't create the foreign key constraint in PostgreSQL, and drop it
if it already exists when running update.php.

The constraint was previously created with an implicit name, so
drop the constraint involving the specified column name (rc_cur_id),
rather than hard-coding the name of the constraint itself.

This bug probably exists under Oracle and MSSQL as well, but no attempt
was made to address it there.

Bug: T76254
Change-Id: I2abd650c8ce83c5b725aec0545fff14a927a305a

Merge "Tweaked BagOStuff::lock() retry times slightly to be faster"

registration: Fix handling of MessagesDirs array and add tests

Previously the code was designed to handle:
  "MessagesDirs": {
    "FooBar": "i18n"
  }

However, it can also be an array, and some extensions (VisualEditor)
use it like:

  "MessagesDirs": {
    "FooBar": [
      "i18n",
      "also-i18n"
    ]
  }

This properly handles both strings and arrays and adds tests to verify
the behavior.

Change-Id: Iff1523b86f754cac1f5b8d822d4324c5fbfc1a50

Localisation updates from https://translatewiki.net.

Change-Id: I88b24823903debcef363e6c0f79bbf86e0b7a978

Merge "break up long line that phpcs whinges about in DefaultSettings"

Merge "Fix typo in Release note"

break up long line that phpcs whinges about in DefaultSettings

Change-Id: Ic2cc583c112e8235725dffa78e98f18ba6998b72

Output wikiid/api in meta=siteinfo only when non-empty

When using interwiki data from the database the value for iw_wikiid and
iw_api maybe empty strings, because the database does not allow NULL
here.
Suppress the output of the attribute in that case, because there are not
useful.
This should not break clients, because using .cdb (like WMF wikis)
already missing these attributes in the result.

Change-Id: I60b97b4b58fdeaf699951544d418e5dc06e3830a

Fill the iw_api column of the interwiki table when installing

Was added in r69542 for interwiki-transclusion, but not populated.

Also make link for 'mw' consistent with 'mediawikiwiki' by using HTTPS.

Bug: 46992
Change-Id: Ieb6c4121ae4cd4e1aefbc372bbe82bdb0ce53b5d

Merge "CategoryView modified to use css columns"