Merge "CSSMin: Skip #default#behaviorName when detecting local files"

Add doc for showEditTools method

Change-Id: Ic6931ed6cdecb7b42e2c47b7c8b34b6d1d23be13

Merge "Show legend of Special:SpecialPages more gradually"

Show legend of Special:SpecialPages more gradually

The legend is shown when restricted or cached pages are shown, but for
cached pages the legend does not contains a text, because there is no
default styling of cached special pages.
Break the note into two messages and disable the message for cached
pages by default. Wikis having style for cached pages can use this
message to create a legend.

Change-Id: Iec429d450ba7019bbd2cd2301a8280a562341387

Merge "Article: Remove unused variable"

Merge "UserPager: Use multiple checkboxes instead of multiselect"

Article: Remove unused variable

Follow-up to 97a8fb78faa4ca2d9604140e5547f5deae5a2524.

Change-Id: I8822742f15c3972cec3ac2ec47a2f594453b8672

Merge "WebInstaller: Use <div> instead of <span>"

Merge "RCFilters: Don't empty the changes list on first load"

WebInstaller: Use <div> instead of <span>

The parser output is enclosed by a <div> and must not enclosed into a
<span> because of an HTML validation error.

Bug: T171423
Change-Id: I034d54436a622803fe07d3c93483b9fe705bc925

[SECURITY] Restore ability to suppress pages while deleting

The OOUI refactor in 97a8fb78fa lost the ability to suppress pages while
deleting.

This converts the suppress checkbox into a OOUI one, and then modifies
the detection of whether the checkbox was used to use
WebRequest::getCheck().

Bug: T171405
Change-Id: I8c388808c08b8804689b41876fd2bcdeff7f0e70

Localisation updates from https://translatewiki.net.

Change-Id: I134586d9e08c971826e4a8a019985ac34ddf13e0

Merge "config: Add tests for EtcdConfig::fetchAllFromEtcdServer"

Merge "rdbms: Complete DatabaseDomain code coverage"

Merge "config: Complete coverage for Config class"

Merge "phpunit: Remove skins/ from coverage index"

config: Add tests for EtcdConfig::fetchAllFromEtcdServer

Follows-up 9b459d29e0, 110a21ea18.

Bug: T156924
Change-Id: Ib68c55ea3386a12e707f6d7d732da9fe727ccc2f

build: Update mediawiki/mediawiki-codesniffer to 0.10.1

And auto-fix all errors.

The `<exclude-pattern>` stanzas are now included in the default ruleset
and don't need to be repeated.

Change-Id: I928af549dc88ac2c6cb82058f64c7c7f3111598a

phpcs.xml: Escape . in exclude-pattern

And fix errors in a file that was previously being excluded.

Bug: T129664
Change-Id: I60161b4037ca122ccd2f0730a4b4aa59d878e459

UserPager: Use multiple checkboxes instead of multiselect

The URL from the serialized form was something like:

    &wpoptions%5B%5D=editsOnly

whereas we needed:

    &editsOnly=1

Bug: T171332
Change-Id: Iaec223195e121dab83b5605a3dfaffa8e1f93684

config: Complete coverage for Config class

* Fix @covers for testSalvage and testGetConfigNames.
* Add missing coverage for registering an object directly.

Change-Id: Ib42b8333a1716d2a4747c25ab3f6cd795e546f3e

phpunit: Remove skins/ from coverage index

As of a few releases ago, there are no longer any skins in core
in that directory (base classes and fallback are under includes/).

Removing this makes running phpunit coverage faster locally,
when there are skins installed. This should be excluded, similar
to how extensions/ is not indexed for the core report, either.

Change-Id: Idc01578bf22febc684dbcb7aeb9ec9475e930ae4

CSSMin: Skip #default#behaviorName when detecting local files

Bug: T162973
Change-Id: If76869910f308f8a91c73f287e7e74c214f02e9b

Merge "Check headers_sent() before sending cpPosTime headers"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Idaf7df307423797d7bed4f911abcb8da9ddba271

Merge "mediawiki.hlist: Add sensible defaults"

Check headers_sent() before sending cpPosTime headers

Some entry points stream output and flush their own headers.
This avoids "headers already send" warnings in some cases.

Change-Id: Ifb232d4575486749bbbccba88f3f688972fe9c20

rdbms: Complete DatabaseDomain code coverage

Add missing coverage for:
* DatabaseDomain::equals()
* DatabaseDomain::newUnspecified()
* DatabaseDomain::__toString()

Change-Id: I38863c5652ec395e7194bfb85b3485e3343a3b7e

mediawiki.hlist: Add sensible defaults

This allows skins to define how hlists appear (e.g. which pseudo
elements to use or whether to use them at all)  while
providing some sensible defaults.

Bug: T42062
Change-Id: I61b5f077d8b4a4c7fe845b7b6d1df98bb2dbafc8

Merge "RCFilters: Add 'boolean' group"

RCFilters: Add 'boolean' group

The group allows filters to be represented 'directly' with their
selected values corresponding to their parameter "1" or "0" value.

Change-Id: I56e9b52ff79a46227de71c905b2ecd97a3823624

RCFilters: Don't empty the changes list on first load

jQuery.empty() apparently also unattaches event handlers.
Only empty before we re-append if we are not in the first
load of the changes list.

Bug: T171085
Change-Id: I68754e99a6c5c821be47b1b1e8e22f23db85746c

Merge "RCFilters: Change loading animation"

Merge "API: Add deprecation warnings for Iae0e2ce3"

RCFilters: Change loading animation

Bug: T165286
Change-Id: If816dd1e3a2677c36195546c51f87505ba883e04

Localisation updates from https://translatewiki.net.

Change-Id: I9687475e5593765b7922a91838b78ba6b18c583c

Merge "resourceloader: Add ResourceLoaderModule::shouldEmbedModule and use it"

resourceloader: Add ResourceLoaderModule::shouldEmbedModule and use it

Rather than only the 'private' group triggering embedding, allow modules
to explicitly specify if they should be embedded.

The default is still to only embed when the group is 'private', and the
'private' group is still special in that ResourceLoader::respond() will
still refuse to serve it from load.php.

Change-Id: Ib9a043c566822e278baecc15e87f9c5cebc2eb98

RCFilters: Make the pulsating animation on the live update button not block clicks

Change-Id: I37b06b1464ae9ca70f5f85277cc46ac102aa0ec3

Merge "Remove auto-generated "Constructor" documentation on constructors"

Remove auto-generated "Constructor" documentation on constructors

Having such comments is worse than not having them. They add zero
information. But you must read the text to understand there is
nothing you don't already know from the class and the method name.

This is similar to I994d11e. Even more trivial, because this here is
about comments that don't say anything but "constructor".

Change-Id: I474dcdb5997bea3aafd11c0760ee072dfaff124c

Merge "rdbms: Refactor DatabaseTest"

Merge "rdbms: Move DatabaseMysqlBaseTest to libs tests"

rdbms: Refactor DatabaseTest

* Move DatabaseTest and DatabaseSQLTest to libs,
  and remove MediaWikiTestCase dependency.

* Refactor DatabaseTest to be a test of the Database abstract class,
  not of whatever current DB backend is configured by LocalSettings.

  - Remove most switches/conditionals and other tests for specific
    database backends. Move those to individual test classes for
    those backends instead.
  - Some tests appear to have been integration tests for the PHP driver
    and/or the db backend itself. Moved to a new DatabaseIntegrationTest.
  - Now that only the abstract Database is invoked, the test runs a bit
    faster (no real connections/queries).

* Add missing @covers tags, and remove or fix broken ones
  (follows-up 26e52f0c491a4).

Change-Id: I9dc4a558e701d00e95789e7eb8e02926783b65ad

Merge "RCFilters: Make sure single_option always has 1 item selected"

rdbms: Move DatabaseMysqlBaseTest to libs tests

* Remove MediaWikiTestCase dependency.
* Add missing @covers tags.
* Add missing visibility/static declarations.

Change-Id: I2d81a9265016174e7b4ff810f140f7d031ce9f14

RCFilters: Make sure single_option always has 1 item selected

Bug: T171081
Change-Id: I82b6cf9660dfd5ee006197504557be366e148244

RCFilters: Exclude 'hidden' filters from 'areCurrentFiltersEmpty' query

Bug: T171129
Change-Id: I831692efbc5562d0f1d7433b9933118c7a9f6f14

Merge "qunit: Prepare testrunner for QUnit 2"

qunit: Prepare testrunner for QUnit 2

* Nested modules:
  - Support for Sinon extension was fixed by Ib17bbbef45b2bd.
  - Support for Fixture extension was still broken, masked by the use
    of a local variable that made the handler not fail when setup ran twice
    in a row. Fixed using the same moduleStack.length check.
  - Add regression test.

* beforeEach/afterEach:
  - Added in 1.16, with compat for setup/teardown.
    Our wrapper adds its own setup/teardown, and preserves any original one.
    However, it didn't account for beforeEach/afterEach, so it ends up
    sending both but only one is used.
  - Fix to support both on the incoming localEnv object, and also switch
    our wrapper to use beforeEach/afterEach in prep for QUnit 2.0.
  - Fix our wrappers to preserve return value since QUnit 2 allows beforeEach
    and afterEach hooks to be asynchronous by returning a Promise, similar
    to how one can do from QUnit.test().
  - Add regression test.

* Centralise makeSafeEnv logic
  - We always create our own env object to pass to orgModule().
    Document why this is (to avoid recursion).
  - Add regression test.

* Custom assertion methods:
  - Use this.pushResult instead of the deprecated QUnit.push() method.
    This also improves the in-browser reporting of errors by properly
    supporting 'negative' results for notHtmlEqual reporter.

Bug: T170515
Change-Id: If4141df10eae55cbe8a5ca7a26707be1cd7b9217

Merge "Refactor LoadBalancer::setDomainPrefix() to use forEachOpen* method"

Refactor LoadBalancer::setDomainPrefix() to use forEachOpen* method

This makes the code more consistent with the other class methods
that iterate over connection handles and less fragile.

Change-Id: Ie2d8efb80a66a5c87b5b3edd64cb6bc4268fa911

Localisation updates from https://translatewiki.net.

Change-Id: I2cb7798cbaa72ddbb752f70483c55d38e20c8bff

Use constants for LoadBalancer connection array keys

Also renamed "foreignUsed" to "foreignInUse" for clarity.

Change-Id: Ida35042979a87417b38d256610b726611555d7b2

Merge "Fix docs for LoadBalancer::openConnection()"

Merge "mw.widgets.visibleByteLimit: Temporarily disable whilst OOjs UI label bug is fixed"

Merge "Adjust hidden element rules for printed view"

Adjust hidden element rules for printed view

This change updates the hidden element list
for the printed view.

Elements hidden:
* Indicators
* Category links
* Redirect header hidden from all namespaces

Remove selectors that are not inside core. Due to
a miscommunication with Volker these were unintentionally
added in I9aa92a3e70991126c56df0a6db408d1c6f84f172:
* `#coordinates` is added by templates and should be in
MediaWiki:Print.css
* `.play-btn-large` is added by TimedMediaHandler and should
be removed in that extension
* `.noexcerpt` should not be used in this context. It's meant for
marking content up that should be hidden from extracts not for
content that should be hidden from print view - that's what `.noprint`
is for

Bug: T169823
Change-Id: I8b37a6b00618abda7267f92e4712169b22c20b94

Merge "RC Filters: Tweak i18n for latest/not latest revision filter"

mw.widgets.visibleByteLimit: Temporarily disable whilst OOjs UI label bug is fixed

Bug: T169982
Change-Id: If5d2b3d705b121c450f99d3ef61b535e7fc89c04

Merge "Change OO.ui.isMobile test to detect MF, not minerva"

Change OO.ui.isMobile test to detect MF, not minerva

Minerva can now be used on desktop, so this test is
no longer good enough.

Change-Id: I80d7aa6479809a5a8882cdd232ec0636f7adad77

Fix docs for LoadBalancer::openConnection()

Change-Id: Ib05d98cb3670899a688d9882a2dc364f18e1d4bd

Merge "maintenance: Consistently use hasOption() for boolean options"

Merge "Add link to Special:PrefixIndex in delete dialog"

Merge "Revert "Make mw.widgets.SearchInputWidget extend OO.ui.SearchInputWidget""

Add link to Special:PrefixIndex in delete dialog

This change adds a warning to ?action=delete with a link to a
Special:PrefixIndex of the page being deleted if the page being deleted
has subpages.

If there are less than 50 subpages, the exact count will be mentioned in
the message.

Bug: T32666
Change-Id: Ibd705ed77452a55f782ed855c676d7ee23e9ad61

Revert "Make mw.widgets.SearchInputWidget extend OO.ui.SearchInputWidget"

Typing a search query into the main box on Special:Search
causes the search to be performed for the first autocomplete
result, rather than the typed query. The first autocomplet
result should only be searched for if explicitly selected with
the mouse or arrow key.

This reverts commit 7882e3b66060f7bbd37e15688baf4dd30954b1b7.

Bug: T171112
Change-Id: I1af6ba90542fafe3ed1aeca420e9d6df6612f7d0

Merge "IcuCollation: Fix diacritic characters for Aromanian (rup) and Moldovan (mo) headings"

Localisation updates from https://translatewiki.net.

Change-Id: If2ca75be14878eb12f54f36e6365872b15c5f7e3

Merge "Simplify print styles"

maintenance: Consistently use hasOption() for boolean options

We already do this in many places, but there were also still plenty
of getOption(, false) calls. Change these to hasOption() for consistency.

Change-Id: I74f91a5bf5a8036496ab1f493e5862a04d27ec0f

IcuCollation: Fix diacritic characters for Aromanian (rup) and Moldovan (mo) headings

They should be Ș, Ț (comma-below) and instead they were cedilla-below (Ş, Ţ).
Same as for Romanian (ro) in 486f64f28302ecceed04977180fd21470cb54c81.

Both of these languages are unsupported by libicu and so the collations
are unlikely to have been used in practice.

Bug: T171043
Bug: T171044
Change-Id: Idd0d593e73cd784fbef7b75e8985f988f5555e26

Simplify print styles

Simplifying CSS code by
- merging similar selectors,
- reordering them by CSS specificity and within selectors alphabetically &
- applying simpler `line-height` units dependent on `font-size` instead
of absolute.

Bug: T135022
Bug: T169823
Change-Id: I9aa92a3e70991126c56df0a6db408d1c6f84f172

Merge "Set batch size to 1000 in updateRestrictions"

RC Filters: Tweak i18n for latest/not latest revision filter

Bug: T166600
Change-Id: I5237cb0f87b3a340eb3fb8198a4ba39d679c6fe3

RCFilters: proper group name for user experience level

It's "User registration and experience",
not "Experience registration and experience".

Bug: T165160
Change-Id: Ie28de2ace687acb0cdd2a4d9c5de1158be3647e8

Update FIRST_LETTER_VERSION for rowiki changes

Can't just clear cache on production, as this
now uses per-server apc instance.

Follow-up 486f64f28302ecceed04977

Change-Id: I88df6d5a91c86ef687543d1a6988e0ec050bbfce

Merge "IcuCollation: Fix diacritic characters for Romanian (ro) headings"

Set batch size to 1000 in updateRestrictions

As far as can be seen on enwiki, very few rows to update

Change-Id: Idf8dee951a5240c4ebaa73fc02c1248a6737b3f8

IcuCollation: Fix diacritic characters for Romanian (ro) headings

They should be Ș, Ț (comma-below) and instead they were cedilla-below (Ş, Ţ).

Bug: T168711
Change-Id: I6dc873c3ce93bca3e425439f70d0fb30aecc9533

Merge "Support custom offsets from SearchResultSet"

Return a typed object from WikiPage::prepareContentForEdit

This makes it easier to figure out what values are available and
includes documentation about each field.

This will also allow us to add deprecation warnings for deprecated
properties via __get() in the future.

Change-Id: I4ecc900372546013253256749563aaa203ff8963

Support custom offsets from SearchResultSet

Work to support interleaved AB testing of search will display
interleaved results of two search configurations on the first page of
results, but standard results on all pages other than the first page.
This means that while 20 results may be requested, the next 'new' result
of the primary query may be at position 10. Allow the SearchResultSet
to declare what the new offset is.

Bug: T150032
Change-Id: I14c0c33249fcdb66f72f5966e2aa72781a34daee

Merge "RCFilters: Emphasize UI when saved query is loaded"

Merge "RCFilters: Increase the initial height of the loading box"

RCFilters: Emphasize UI when saved query is loaded

Bug: T169063
Change-Id: Ia300ca7bb4afb7d8e247e57e53ecff826ddf0d7e

Merge "RCFilters: Add range group filters - limit, days and hours"

RCFilters: Add range group filters - limit, days and hours

- Add 'hidden' groups that have base defaults but are not
  viewed in the filter drop-down.
- Add a UI for days, hours and limit selections, based on their
  group models.
- Clean up the fieldset form to remove redundant line breaks
  and empty objects.
- Add 'hours' as a subset of days, where the UI can split itself
  by picking up values >=1 and <1
- Add the ability to allow 'arbitrary' information from the URL
  values, but also make sure there is a validation method
  and a possibility to re-sort the values that are added in.

Bug: T162784
Bug: T162786
Change-Id: I8068a7cc411eef40ddb8af4eef1d4f1e5f2a2b82

Merge "RCFilters: Don't emit wikipage.content on first load"

Merge "RCFilters: Change `What's this?` i18n based on user testing"

Merge "RCFilters: Only open one 'Whats This' popup at a time"

RCFilters: Don't emit wikipage.content on first load

If the first load uses existing DOM elements, don't emit that
hook. Only emit if we're pulling from the server for new results.

Bug: T165350
Change-Id: Ib17fdd16a908808f99fe8506c50c61d90ad4addc

RCFilters: Change `What's this?` i18n based on user testing

Also minor tweak to the qqq to help translators understand.

Bug: T169053
Change-Id: If5aebae20d2e0037b28090d85cd9b076ac4b3178

Merge "RCFilters: Log filter groupings"

RCFilters: Only open one 'Whats This' popup at a time

Remove 'overlay' and this.$element from the $autoCloseIgnore, since
the entire popup is in the overlay anyways.

Bug: T169047
Change-Id: I57300413297d86eca9bcc3d49766f4e41ab628be

RCFilters: Increase the initial height of the loading box

Bug: T168651
Change-Id: Ia4a079c9d177ad024611112a212a60f725ea1943

Merge "maintenance: Add --dry-run option to deleteDefaultMessages.php"

maintenance: Add --dry-run option to deleteDefaultMessages.php

For back-compat, keep the default as-is, because it is also
used as a post-update script by the installer.

But for convenience and safe manual use, do add a dry-run option.
(Ideally this would be a --delete option and dry-run by default.)

Change-Id: I3ca70f15f50b66f9d19fa705cef9a4aae2dd4e5c