Merge "GallerySlideshow: Always set image height, adjust according to caption size"

Merge "release-notes: Add Moment.js update"

Merge "qunit: Make eslint config pass on qunit test files"

Merge "qunit: Minor clean up in various tests"

GallerySlideshow: Always set image height, adjust according to caption size

When the size of the caption changes (for example one word vs four lines) the
content below jumps because of the space the caption is taking.

We now adjust the height of the image based on the height of the caption to
avoid jumping.

Bug: T140596
Change-Id: I567652ff8b1483cef474493dd5bd790e95288b30

Merge "build: Update HHVM for Travis to 3.12"

Merge "Remove duplicate test"

Merge "qunit: Remove obsolete jshint/jscs options"

release-notes: Add Moment.js update

Follows-up f13a0d952.

Change-Id: Ib4692310c8abee8f6b1f69c664b373f4ea83e2dc

Merge "mediawiki.api: Tweak deprecation logging again"

mediawiki.api: Tweak deprecation logging again

Follow-up to 15b5dc5d8eecb5a1784b0b7165a90a81e071d750,
9a8e2b7124ed10db3db7f7767d532cde00935876.

Change-Id: Idf9083e81312b97e69ec786d99ea3cc46ba80123

Merge "resourceloader: Avoid endless module_deps write for the same value"

Merge "EditPage: Throw exceptions on false contentModel"

resourceloader: Avoid endless module_deps write for the same value

Follows-up 047b60b96d (ref T111481).

The if-condition compared the expanded paths, not the relative paths.
This meant there were two conditions under which the code will perform
a useless write that inserts *literally* the exact same JSON value.

1. The base directory ($IP) changes after a branch upgrade.
2. Paths contain '../', '//' or other unnormalized paths.

The latter caused various Echo and ULS methods to keep writing the
same value because one of their images is referenced in CSS using
'../'. When inserted in the database as relative path and then
expanded again at run-time and compared to the input value, they
don't match ("$IP/foo/../bar.png" != "$IP/bar.png") and cause a write.

Bug: T158813
Change-Id: I223c232d3a8c4337d09ecf7ec6e5cd7cf7effbff

Merge "AutoloadGenerator: Add support for class_alias()"

Merge "Add GENDER to rollback-success message"

Merge "resourceloader: Optimize WikiModule preload for reqs without wiki modules"

Merge "build: Enable indentation stylelint rule"

resourceloader: Optimize WikiModule preload for reqs without wiki modules

Currently it was still going through fetchTitleInfo() with an empty array on
the majority of requests without wiki modules, e.g. load.php?modules=jquery.

Bug: T158813
Change-Id: Ie33a2b4da572bb30b2e7a69db07790724ec2f03f

AutoloadGenerator: Add support for class_alias()

Blob, Field, DatabaseBase are now auto-detected.

Change-Id: Ib8fae2ec3fbb3f5e4aca7965f81631c5f0485ea1

Merge "Collation: Allow uppercase letters in UCA collations' names"

Add GENDER to rollback-success message

Bug: T141250
Change-Id: I99c2b5ad5594b25928ad5bfd3f3a36b19a2c041c

Localisation updates from https://translatewiki.net.

Change-Id: I24babbe571a65242fdef9105b82242729f21f1eb

Collation: Allow uppercase letters in UCA collations' names

We have several such collations defined in IcuCollation:

* bs-Cyrl
* de-AT@collation=phonebook
* fr-CA
* sr-Latn

They couldn't actually be used.

Change-Id: I3a62073583c49d3e90910aa8240fe9fcc0682386

Remove duplicate test

Change-Id: If99b0672c631e0428550a73a2a6116394ef32bb9

qunit: Make eslint config pass on qunit test files

Follows-up c0fb8a8836, I890e6e49b.

* Disable 'qunit' env in general source code. And re-declare
  locally in the few src files that use it properly.

* Create separate eslint config for tests/qunit with various
  rules disabled (e.g. valid-jsdoc and es3-keywords).

Change-Id: I37ccec2019de55edfee92697eb80478df7cb6220

qunit: Minor clean up in various tests

Follows-up 5e602c613.

* jquery.byteLimit: Remove redundant setTimeout().
* jquery.byteLimit: Remove unused $elA, $elB variables.
* jquery.hidpi: Actually call bracketedDevicePixelRatio().

Change-Id: I288af22e081385fca6268a87e7b6fe1b27116706

qunit: Remove obsolete jshint/jscs options

Follows-up c0fb8a8836.

Change-Id: I890e6e49b4801667b6eb463efec46a380a27d028

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit e23a818554548cd922ee262ea1d8da47ea457248

Change-Id: Ib26b170c51aa4425a54871fa32543b2eef5db41e

Merge "qunit: Remove redundant testCount expectation"

EditPage: Throw exceptions on false contentModel

contentModel should always have a value here.

Change-Id: Iba2538f383de7da94bd0e0e3eb7d7eca4e6cee7c

Merge "Add "redirecció" as translation for the "redirect" keyword"

Add "redirecció" as translation for the "redirect" keyword

So that editors etc. prefer it over the Occitan translation.

While at it, remove old non-working bookstore URL.

Change-Id: Iee2c6844b9e324ee7811f337c02ab06b348b651b

Merge "resourceloader: Fix testMakeModuleResponseError() failure on Travis"

Merge "mediawiki.language: Add unit tests for digit transform table"

resourceloader: Fix testMakeModuleResponseError() failure on Travis

> 1) ResourceLoaderTest::testMakeModuleResponseError
> Failed asserting that '[e08c982d974548127cb5d7ce] Fatal exception of type Exception'
> matches PCRE pattern "/Ferry not found/".
> .../ResourceLoaderTest.php:519

This happened on Travis CI, because ResourceLoader::formatException() behaves
differently based on $wgShowExceptionDetails. Which is enabled in Vagrant
and Jenkins, but disabled by default (and thus in Travis CI builds).

Bug: T75176
Change-Id: If15dd03213703b7b6ff899cad5e5569e2515b378

Merge "API: Fix ApiMainTest::testExceptionErrors and use $wgShowDBErrorBacktrace"

API: Fix ApiMainTest::testExceptionErrors and use $wgShowDBErrorBacktrace

Apparently Jenkins sets $wgShowExceptionDetails to true but travis
doesn't. The test should make sure it's set since it depends on it.

And apparently no one ever added $wgShowDBErrorBacktrace to the API.
Let's do that too.

Change-Id: I0793eda349426b265a47900cdeb87bb920b5c291

qunit: Remove redundant testCount expectation

QUnit used to have bad state management (a few years ago) at which point it
became useful to verify the number of assertions in case an asynchronous
failure happened, as it would likely go unnoticed.

* Errors outside testStart/testEnd weren't caught.
  QUnit now monitors window.onerror.

* Assertions could be attributed to the wrong test.
  QUnit no longer does this since the assert object is associated with
  the current test through lexical scope.

* assert.async()/done() replaced global semaphore (QUnit.start).

* A test could forget to be marked as async and make no assertions.
  QUnit now marks a test as failed if it makes  0 assertions.
  QUnit also has built-in async tracking for promises.

If a test is not reaching all assertions for some reason, this
will cause an error of some sort that is tracked. If in some
specific scenario this isn't the case, assert.expect() can still
be called (e.g. when expecting 0 assertions), but it'd be worthwhile
to file an upstream bug report in that case.

Follows-up 7c363752, which removed 'QUnit.config.requireExpects' from
our test configuration.

Follows-up c4c7007de6 and various other commits that already removed
the test counts from a subset tests. This commit removes the remainder.

Change-Id: Ie58396ba9c83d27220508481cb97c0fa74487756

build: Update HHVM for Travis to 3.12

Travis defaults 'hhvm' to 3.6.6 for back-compat, however newer
versions are available. The latest supported version on Travis is 3.12,
per <https://docs.travis-ci.com/user/languages/php#HHVM-versions>.
This also matches the version currently used in wmf production.

Bug: T75175
Change-Id: I1c562f61acf83aa7f558b2e11a6af81d13523dea

Merge "Mention $wgRateLimitsExcludedIPs semantics change in release notes"

Mention $wgRateLimitsExcludedIPs semantics change in release notes

Follow-up for I727c19214.

Bug: T156983
Change-Id: I845d2acc2284a107d4bacd526e7598bade118d0a

Localisation updates from https://translatewiki.net.

Change-Id: Ice30c7acd43a92ea0c5f21895ffd7d3e35d98083

Merge "API: Add "messagecode" to AuthManager responses"

Merge "User::isPingLimitable(): handle CIDR notation in $wgRateLimitsExcludedIPs"

API: Add "messagecode" to AuthManager responses

Following the same logic that the API uses for assigning codes to
warnings and errors.

Bug: T158609
Change-Id: I4e1a4f252e12e990727e8e9ada4ea00c526e3f8f

ApiCreateAccount was removed in REL1_27

Change-Id: I504cbe3f399be2e354d53c7d8c4ac8225d1ffc17

Merge "build: Enable no-duplicate-selectors stylelint rule"

Merge "Fix documentation comments for some WebRequest methods"

Fix an incorrect HTML tag in apiwarn-unclearnowtimestamp

Change-Id: I74de3c0ef6feb33091126df3cc0ec452d7ede43b

build: Enable indentation stylelint rule

Enabling indentation stylelint rule and making it pass.
Also making use of mediawiki.mixins' `box-sizing()` mixin in
StashedFileWidget and change interwikiwidget rules order slightly
(example `:hover` modification after normal selector rule).

Change-Id: Ifa9ccae5518d5426b390e0f8321eb3decb211c18

build: Enable no-duplicate-selectors stylelint rule

Enabling no-duplicate-selectors stylelint rule and
making it pass.

Change-Id: Ib62b463898098eea1301f0ddfcd4837498272dd4

mediawiki.language: Add unit tests for digit transform table

Follows-up e8dd1f43fb169, e2834bacbde6.

Bug: T152800
Change-Id: I9ab824118f179b3b6d672d71b5ac8cf87c3b2fc8

Merge "tests: Replace implicit Bugzilla bug numbers with Phab ones"

Merge "parserTests.txt: Replace implicit Bugzilla bug numbers with Phab ones"

Merge "Copy in various point releases to HISTORY"

tests: Replace implicit Bugzilla bug numbers with Phab ones

It's unreasonable to expect newbies to know that "bug 12345" means "Task T14345"
except where it doesn't, so let's just standardise on the real numbers.

Change-Id: I46261416f7603558dceb76ebe695a5cac274e417

parserTests.txt: Replace implicit Bugzilla bug numbers with Phab ones

It's unreasonable to expect newbies to know that "bug 12345" means "Task T14345"
except where it doesn't, so let's just standardise on the real numbers.

Change-Id: I3eeffe40e0a752e1e3c79e65fa2fb556950d9a24

Merge "resources: Replace implicit Bugzilla bug numbers with Phab ones"

Merge "MWExceptionHandler::handleError: Set log severity based on error level"

resources: Replace implicit Bugzilla bug numbers with Phab ones

It's unreasonable to expect newbies to know that "bug 12345" means "Task T14345"
except where it doesn't, so let's just standardise on the real numbers. (This is
just for resources/src for now; includes/ will be another day. :-))

Change-Id: I5c55bab4f7c8a26dda43de7d169cc7bddc74adc3

Localisation updates from https://translatewiki.net.

Change-Id: I80bd2a411965a3e08ee073ae6a8e62a3468ca768

Merge "build: Enable stylelint rule 'number-leading-zero' and make pass"

Fix documentation comments for some WebRequest methods

WebRequest methods getRawVal(), getVal() and getArray()
can return null.

Change-Id: I555dfd93c7cdebc83aab89f3efe4de3018bc9de0

Merge "build: Enable selector-pseudo-class-parentheses-space-inside stylelint rule"

Merge "build: Enable declaration-block-no-duplicate-properties stylelint rule"

build: Enable selector-pseudo-class-parentheses-space-inside stylelint rule

Enabling selector-pseudo-class-parentheses-space-inside stylelint rule and
making it pass.

Change-Id: I6676d743b4375c250377a5b7582251e83f761d10
Depends-on: I3859e04797203a64d82206a379dbb320e5e113d5

Merge "build: Enable use of stylelint-config-wikimedia and make pass"

build: Enable stylelint rule 'number-leading-zero' and make pass

Change-Id: I7f6d7784b5a415d4fd4fc098800bfdf9ed61790f

build: Enable declaration-block-no-duplicate-properties stylelint rule

Enabling declaration-block-no-duplicate-properties stylelint rule and
making it pass. Amongst others, removing IE 7 specific debug CSS rules.

Depends-on: I3859e04797203a64d82206a379dbb320e5e113d5
Change-Id: I0fa611fff61c13646a81193b7fa079f3427a0080

build: Enable use of stylelint-config-wikimedia and make pass

Enabling stylelint-config-wikimedia for Grunt stylelint,
which was already introduced in 3bf6f429295e, but has not yet
been actively used to enforce Wikimedia CSS Coding Standards.
For this first patch nullifying several options, which will be enabled
in follow-up patches in order to be easier reviewable.

Change-Id: I3859e04797203a64d82206a379dbb320e5e113d5

MWExceptionHandler::handleError: Set log severity based on error level

Bug: T45086
Change-Id: I240b75f5a4e0075c7a357609aa26834708f93b0b

Merge "Fixed documentation on Parser::getVariableValue"

build: Fix stylelint glob pattern

Change-Id: I2f37668d32d70d72ed493e81af6b521a3bfaedba

Localisation updates from https://translatewiki.net.

Change-Id: I300a922345f6e7e4fb7109b0bc758bec1667ccbf

Replace reference to deprecated module deletedrevs in API documentation

Replaced reference to 'deletedrevs' in documentation of undelete
module by 'prop=deletedrevisions'

Bug: T109823
Change-Id: I82a2e5ff2f180dae2e11ef61790a20999b1a5802

Fixed documentation on Parser::getVariableValue

$index is definitely not a int here, see the big switch( $index )-case
statement below. It switches for strings, not numbers. Also, note that
this is lowercase, one might expect it to be uppercase as this is how
magic words are written in wikitext.

Bug: T96633
Change-Id: Iea93c3796fdee4ed7abbb7608e89b627ca95aead

Localisation updates from https://translatewiki.net.

Change-Id: I76204d933123d39082efbdebe625b80e45eb4b2a

Merge "Add "Unknown user" to $wgReservedUsernames"

Add "Unknown user" to $wgReservedUsernames

Used in WikiImporter when importing revisions with no author
(e.g. when the author of the exported edit was revision-deleted).
Follow-up to changes related to T121338.

Bug: T158474
Change-Id: I49f3d5b9a23a0535c51a733d8837a681f6e51959

Copy in various point releases to HISTORY

Change-Id: If0e32d89b26b5f8a38639871d9c907d9b69cd487

Merge "Split RCFilter's RL modules"

Merge "Move ORAResult to /db"

Merge "Move Field classes to Rdbms namespace"

Split RCFilter's RL modules

Create 'dm' / 'ui' and 'controller' modules for ResourceLoader,
make sure that Special:RecentChanges loads 'ui' module (that
depends on the other two) and yet the qunit tests only load
the dm module.

Bug: T156532
Change-Id: If53a735458703f0bd2c094349edf86f38f05ccd7

Merge "resourceloader: Consistent stylesheet order for debug and non-debug"

Merge "Fix timestamp format in maintenance/manageJobs.php"

build: Bump stylelint and make pass

 grunt-stylelint              0.6.0  →   0.7.0
 stylelint-config-wikimedia   0.3.0  →   0.4.1

Change-Id: Id87b8df04d415e1f1058a4042a31408236402037

resourceloader: Consistent stylesheet order for debug and non-debug

Previously, style modules were only in a predictable order for production mode.
In debug mode, the order was determined by order in which modules were added
to queue at run time. This made it sometimes hard to debug, especially when
dealing with gadgets that apply in a different order among each other.

Change-Id: I4bff0c91d127e4ad8015cd8c1775220fe460cbc3

Merge "Don't bail on single-line definition list due to excess close tags."

Merge "Protect -{...}- variant constructs in definition lists."

Don't bail on single-line definition list due to excess close tags.

When parsing a single line definition list, we track nested tags so that:

; <b>foo:bar</b>: baz

breaks before `baz`, not between `foo` and `bar`.  But we currently bail
out of this algorithm entirely if we see a mismatched close tag.  We should
just ignore the unmatched tag, like Parsoid does.

Change-Id: I6306dcad6347abeb6ab001d35562f1ab9f374bd1

Merge "resourceloader: Limit module_deps write lock to unique index"

Merge "resourceloader: Use upsert() instead of replace() for module_deps write"

Localisation updates from https://translatewiki.net.

Change-Id: Ie3c31443ad7baae08af90f15eba74443257dcb7e

Protect -{...}- variant constructs in definition lists.

Given the wikitext:

;-{zh-cn:AAA;zh-tw:BBB}-

Prevent `doBlockLevels` from trying to split the definition list at the
embedded colon and using `AAA;zh-tw:BBB}-` as the `<dd>` portion.

Bug: T153135
Change-Id: I3a4d02f1fbd0d0fe8278d6b7c66005f0dd3dd36b

Merge "Demote "throttler data not found" logs to info level"

Merge "Demote "Unverified user provided and no metadata to auth it" log message to info"

Merge "Add fr (French) language fallback for br, nrm, oc"

User::isPingLimitable(): handle CIDR notation in $wgRateLimitsExcludedIPs

Bug: T156983
Change-Id: I727c19214cb3f9fad558d433bb38fbcf25d8497a