Merge "mediawiki.requestIdleCallback: Re-enable use of native requestIdleCallback"

Merge "Fix @covers for (non-integration) parser tests"

Merge "Fix @covers for BalancerTest"

Merge "Follow-up 1e9c3619: Correct @T1234 accidental cruft"

Merge "/*.php: Replace implicit Bugzilla bug numbers with Phab ones"

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

Merge "includes/libs: Replace implicit Bugzilla bug numbers with Phab ones"

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

Merge "includes/api: Replace implicitly-Bugzilla bug numbers with Phab ones"

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

Merge "includes/specials: Replace implicit Bugzilla bug numbers with Phab ones"

languages: 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: Id2f9d229d17b8eee66b2ca4e3927f3f66ac62988

includes/specials: 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: Ic878feaea72893a1c47ea79ac284e581209e1731

Localisation updates from https://translatewiki.net.

Change-Id: I5bf398e4a74a8e06d96236004e7ab0169c838952

Merge "Tweak mw-ui padding to match oo-ui for buttons and text inputs"

Merge "RCFilters UI: Limit filter popup height"

Merge "Add more Czech special page aliases"

Fix @covers for (non-integration) parser tests

You have to allow tests to cover private Parser methods that they
execute. Private methods will never have separate tests.

Change-Id: Ic842e2be4675f505dc26d1d3e1dd9000401df46c

Fix @covers for BalancerTest

This test is intended to cover the whole file, not just one method.

Change-Id: Ice800ce467e030e8264db96e19feadf9b68afb9a

Localisation updates from https://translatewiki.net.

Change-Id: I3575803833e078c706053138ad10473b56d1a016

Add maintenance script to add sites to sites table

Also: Add language code validation to Site::setLanguageCode().

Bug: T132937
Change-Id: I763ec65cb06d5250a3886a66eefdde8701b2299c

Tweak mw-ui padding to match oo-ui for buttons and text inputs

Change-Id: I594f42d1e13d1c5e9bf44363a137fc2fe9f937e7

Merge "Database: Remove weird commented-out block"

Merge "FeedUtils: Remove weird commented-out block"

Database: Remove weird commented-out block

Commented out in 033b6b9646da6b71c5bb9dd6803a662bcf510a99 (r20329).
After ten years, I think it's safe to bet this won't ever be getting
un-commented.

Change-Id: Ibb1f3e2969b2d81f6f2a17fff57e9b05cc17d58b

FeedUtils: Remove weird commented-out block

This has been added in 58f7bb30a04d6c1f137cc1659b8998c5349bf13e (r36376),
already commented-out, and was moved and amended several times until
it became completely out of place.

Change-Id: I11b6e3e2fbb2d0a325a6723bf5874133a7f53851

Localisation updates from https://translatewiki.net.

Change-Id: I93fcfdcd97dae44a3f7f3bd702175d646e10ea4d

Merge "Move wpSummary from shared.css to mediawiki.action.edit.styles.css"

RCFilters UI: Limit filter popup height

Make sure the popup clips itself after it changes height
after filtering results on search.

Bug: T158118
Change-Id: I2f9e8b775b154635aa8b12f44876279173548b99

RCFilters UI: Scroll the capsule to the top of the screen

When the popup shows up, scroll the capsule widget to the top of
the screen.

Change-Id: I5366a8a0918bda0aabe1c97db252278c36a89347

Merge "Log a backtrace from the culprit location if headers were already sent"

Merge "Use Database::addQuotes instead of hard coded apostrophs"

Localisation updates from https://translatewiki.net.

Change-Id: Ia8e467b3f295184a8abb3737d3949994078bbba3

includes/api: Replace implicitly-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: I49e2a10350a328a8572fcedd44012751a29e1068

Follow-up 1e9c3619: Correct @T1234 accidental cruft

Change-Id: I94e50ef3356e4e53f9a9082484dff97e49a0e2e2

Merge "Better punctuation in apiwarn-toomanyvalues"

Merge "Improve logging of exceptions which are not thrown but attached to context"

Merge "Move remaining LoadBalancer classes to Rdbms"

Move remaining LoadBalancer classes to Rdbms

The old names are left as aliases.

Change-Id: I52a327f2463a2ba7437324047b5b00d28cd1d758

installer: Fix "relation 'user' does not exist" error for Postgres

On Travis CI, the Postgres build has been failing very early on
in the installer (before phpunit) due to a database error:

> Creating administrator user account.. DBQueryError at Database.php:1059
> Query: SELECT user_id FROM "user" WHERE user_name = 'Admin' LIMIT 1
> Function: User::idForName
> Error: 42P01 ERROR:  relation "user" does not exist
> LINE 1: SELECT /* User::idForName  */ user_id FROM "user" ...

This is because the installer makes its own Database object without
involving ServiceWiring or MWLBFactory, which means wgDBport and
(more importantly) 'keywordTableMap' don't get applied.

While keywordTableMap doesn't matter during the database installation,
after the installer is done updating GlobalVarConfig and resetting
MediaWikiServices, DatabaseInstaller::enableLB takes that homemade
connection and injects it into MediaWikiServices by redefining
the 'DBLoadBalancerFactory' service. Which then affects all use
of wfGetDB(), such as from User::idForName().

Bug: T30162
Bug: T75174
Bug: T75176
Change-Id: I7af58c4beffc4908a93c0c1d8ab1aec9d4ec57c6

build: Simplify Travis configuration

* Remove redundant 'group: edge'. Use the default (stable) instead.
* Use simplified syntax for apt packages. On sudo-enabled VMs, like we
  use now, this translates to the same command. But has the benefit
  of being compatible with the fast non-sudo containers, which we
  want to use in the future. But, for that, djvulibre-bin needs to
  be approved first.
  <https://github.com/travis-ci/apt-package-whitelist/issues/4036>

Bug: T75176
Change-Id: I3b42763ef3f8f08eec08a9008e5cf1e161bb1dff

build: Change Travis postgres user "root" back to "travis"

Follows-up 5168cb60f8f, in which we moved from Precise vms to Trusty vms.
As a side-effect, the undocumented behaviour of the mysql user "travis"
having create-db rights was no longer. As such, we changed it to "root",
per <https://docs.travis-ci.com/user/database-setup/>.

However, this broke Postgres builds since those should still use
"travis". There is no user named "root" for postgres.

* Add 'dbuser' to the matrix environment.
* Improve inline documentation.

Bug: T75176
Change-Id: I09fc0a1da8737e71b3d2b4b88d72b58c150519c4

Merge "RCFilters UI: Add 'direction' property to the wrapper"

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

RCFilters UI: Add 'direction' property to the wrapper

Adds 'direction: ltr' to the entire interface, rather than just
the input, since the entire FilterWrapperWidget is interface-direction
and not content direction.

Bug: T157189
Change-Id: I94ccfea878d83a24e071696b9c4e58115c9295c9

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

Log a backtrace from the culprit location if headers were already sent

Install the backtrace collector very early, so that we can get the
backtrace even if headers were sent from LocalSettings.php.

Bug: T157392
Change-Id: I9bc732b34481c95afb5362e135a87bd4302498e2

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"