Merge "Drop in replacement of eval.php based on psysh"

Merge "RCFilters UI: Style the 'old' RC option fieldset"

Merge "RCFilters: Align trash icon with filter list"

Merge "RCFilters UI: Reassess interaction after resetting filters"

RCFilters UI: Style the 'old' RC option fieldset

Bug: T158006
Change-Id: I230dc9095e41abf32e95adc68c40265b53a5f698

Merge "RCFilters UI: Stop mousedown propagation when capsule item 'x' button is clicked"

Merge "RCFilters UI: Fix mute state styling"

Merge "RCFilters UI: Fix CapsuleItemWidget popup styling"

RCFilters: Align trash icon with filter list

Bug: T149391
Change-Id: Icf867bf572253d288602f00890551909d869a459

RCFilters UI: Fix mute state styling

For some reason it stopped styling muted state in filterItemWidgets
Also adjusted the styling to be more similar to the prototype.

Bug: T156429
Change-Id: Ib043729ab7cfd32253d9424b145794e484ec11b6

RCFilters UI: Stop mousedown propagation when capsule item 'x' button is clicked

We don't want the parent (the CapsuleMultiselectWidget) to receieve
the mousedown event, because it then uses it to focus and open the
popup.

Bug: T158006
Change-Id: I1ae9b58b723a70cc150392224196bdb67ebf30b4

RCFilters UI: Reassess interaction after resetting filters

Both resetting to defaults or resetting to no-filters state should
retrigger the interaction assessment so all filters should get their
included, fullyCovered and conflicted states resetted.

Bug: T158135
Change-Id: Iae23129b7fb378adb712b34e8e64208bbb70ccd7

Merge "mw.loader: Adding comment explaining MODULE_SIZE_MAX"

RCFilters UI: Fix CapsuleItemWidget popup styling

Bug: T158006
Change-Id: I83f72273bc5bf7bd9548d60efd094e427ff0e13a

Merge "Folowup I341c3f7c: Fire wikipage.content with the correct content"

mw.loader: Adding comment explaining MODULE_SIZE_MAX

This informaiton was lost in Iedde8a3.

Change-Id: Ib884f3504c2a98537b919f78a043ae0b64a146cf

Folowup I341c3f7c: Fire wikipage.content with the correct content

The content we are getting from the Ajax request can be either a
jQuery or string, which makes the handler of the event break.

Instead, pass in this.$element which already wrapped the response.

Change-Id: I1c1231b1476fb2cbef10231fdd853401ebf02ed3

Merge "Add CSS classes for userlinks on SpecialPages"

Merge "Add class in diff and history links in ChangesList"

Localisation updates from https://translatewiki.net.

Change-Id: I9ec22e1e404d411a90c52c316519c29322c6c412

Merge "RCFilters UI: Highlight behavior"

Merge "phpunit: Fix AvroFormatterTest failure on PHP 7"

RCFilters UI: Highlight behavior

Let there be highlight! and there were highlights
And RCFilters separated the highlight from the darkness
And it defined highlights as five colors
The lights are called yellow and green, and the darks red and blue
And there were colors and there were circles; one highlight.

This is the commit that adds highlight support for filters both in the backend
and the UI. The backend tags results based on which filter they fit and the
front end paints those results according to the color chosen by the user.
Highlights can be toggled off and on.

Also added circle indicators to the capsule items and each line of results
to indicate whether the line has more than one color affecting it.

Bug: T149467
Bug: T156164
Change-Id: I341c3f7c224271a18d455b9e5f5457ec43de802d

Merge "De-italicise placeholder text"

Merge "SearchFormWidget: Move hidden fields below the ActionFieldLayout"

Merge "build: Enable selector-list-comma-newline-after stylelint rule"

SearchFormWidget: Move hidden fields below the ActionFieldLayout

I think ideally, these would be at the end of the form, but there
are some hooks below which can produce arbitrary HTML and potentially
want to override these with their own fields, so I'm avoiding any
revolutionary changes to the field order here.

Bug: T158856
Change-Id: I377c0061a365930e11454a86c1e0926853789b55

Merge "Make generic placeholder pseudo-class browser-prefix mixin"

De-italicise placeholder text

For consistency with OOUI & the search bar, and to avoid
issues with CJK languages.

Bug: T147808
Change-Id: I81555ceb52535f68f057ec35fc1257206a5aa8eb

Make generic placeholder pseudo-class browser-prefix mixin

Change-Id: I96686bb53cee1a833db8f4d0b691d5e4a1b82885

Merge "Revert "Temporary hax to hide cawiki's hacked in search sidebar""

Merge "Follow-up I96d2225: Adding white background color to framed images in Parsoid"

build: Enable selector-list-comma-newline-after stylelint rule

Enabling selector-list-comma-newline-after stylelint rule and
making it pass.

Change-Id: I8fa3fbe03fcce57d555c47f39361c2a8d9828c3c

Follow-up I96d2225: Adding white background color to framed images in Parsoid

Bug: T154077
Change-Id: Ie6fd4f5b995f79d6c48ff95f091c88b9c67f13e8

phpunit: Fix AvroFormatterTest failure on PHP 7

Fix the following error on PHP 7.

> MediaWiki\Logger\Monolog\AvroFormatterTest::testDoesSomethingWhenSchemaAvailable
> Only variables should be passed by reference
> includes/debug/logger/monolog/AvroFormatter.php:143

Per https://github.com/researchgate/avro-php/blob/1.8.0/lib/avro/schema.php#L311-L314
the default for &$schemata is null, which is filled with a plain AvroNamedSchemata
instance. So this parameter is obsolete.

Either it needs to be assigned here and then passed. But since we don't use
it anywhere and don't pass it any constructor arguments, the default
should suffice.

Bug: T75176
Bug: T141588
Change-Id: I144bed8a78eb267a97e41f379b89c5faaae30625

build: Change Travis image from precise to trusty (Fix HHVM build)

Follows-up 67737a267f.

Per <https://docs.travis-ci.com/user/languages/php#HHVM-versions-on-Trusty>
using hhvm-3.12 requires a trusty image. The upstream source at
<http://dl.hhvm.com/ubuntu/dists/> doesn't provide it for precise.

In addition to changing to trusty, we also need to:
* Set 'sudo: required' and 'group: edge'.
  Because lightweight "sudo: false" containers are still
  precise-only. There is a beta test and I tried it, but it
  doesn't support out apt-get install commands yet.

* Change mysql user from 'travis' to 'root'.
  The fact that 'travis' supports creating databases on precise
  is undocumented. One is supposed to use 'root' for this,
  which is required on Trusty.
  <https://docs.travis-ci.com/user/database-setup/#MySQL>
  Without this, install.php fails.

More details at <https://github.com/travis-ci/travis-ci/issues/7368>.

This config was tested and passed at
<https://travis-ci.org/Krinkle/mediawiki/jobs/204463664>.

Bug: T75175
Change-Id: Ic59936734211b68d9701119a551d3bd1b83c845e

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"

Add CSS classes for userlinks on SpecialPages

On Special:Watchlist, Special:Contributions, Special:Recentchanges etc.
there are links to (talk | contribs | block) for the user who did the
contribution. Add CSS class for them. Introduce the following css
classes:
- mw-usertoollinks-contribs
- mw-usertoollinks-talk
- mw-usertoollinks-block
- mw-usertoollinks-mail

Bug: T156879
Change-Id: I85a3b0987a016ff25026f1c047214a31170b0452

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"

Revert "Temporary hax to hide cawiki's hacked in search sidebar"

This reverts commit e77dbd78ab869b0db672801bca9be445c302d262.

Bug: T149806
Change-Id: Iffdd1b86c0873f3359d7134053cc09dfc19bdc56

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