Replace some usages of Linker::link with LinkRenderer

Classes: PatrolLogFormatter and ProtectLogFormatter

Bug: T149346
Change-Id: If02094df5dd7acc6efd02e540515f8e472ec3d4d

Merge "Document which EtcdConfig parameters are optional"

Merge "Remove load array indexes from LoadBalancer errors"

Merge "UploadBase::getTitle can return null"

Merge "Lower the amount of jobs pushed into redis at once"

Document which EtcdConfig parameters are optional

Change-Id: Ic6b56ed2f8789ae7920cdfc12f34b00db4a76c3a

ReadOnlyMode: Add a few doc blocks

Follows-up 820f46964f7968a.

Change-Id: I7866eb7c8bb9c45a24a3c567a7befe3505821873

Lower the amount of jobs pushed into redis at once

This further limits how long the server can be tied up by push().

Change-Id: I02d242578dadc19912c9fccfdcf5e15c5eb78e9e

Merge "A service for read-only mode"

Remove load array indexes from LoadBalancer errors

This are not very useful and where not using SPI interpolation either.

Change-Id: Ia3a33da3a4593fbcba59b21f5b5028860752ce09

Localisation updates from https://translatewiki.net.

Change-Id: I6e67503e539268e66f495a594f29231e3ee4b6dd

Merge "media: Remove unused private function in SVGMetadataExtractor"

media: Remove unused private function in SVGMetadataExtractor

Change-Id: I3cd00bb4f460a141966ad01e155a97b8a087a8e8

A service for read-only mode

Introduce a service to represent wfReadOnly() and friends.

It's necessary to have two service instances, one for wfReadOnly() and
one for wfConfiguredReadOnlyReason(), to avoid a circular dependency,
since LoadBalancer needs the configured reason during construction, but
wfReadOnly() needs to query the currently active load balancer.

Not having a cache of the configuration makes it possible to dynamically
change the configuration. Ideally things would not change the
configuration, and I removed such instances in core, but to support
extensions, I added a test ensuring that the configuration can be changed.

Change-Id: I9bbee946c10742526d3423208efd68cb3cc5a7ee

Merge "Remove links list from the 'rclinks' message"

HACK: Temporarily patch OOjs UI locally to work around upstream problem

Applied upstream in Ic11b981e70e3.

Bug: T163358
Change-Id: If8c26ffe5095e4e8c2f206c9dda1981bccba2b62

Merge "Allow editors to exclude navigation items from search indices"

Merge "RC Filters: Detect filters conflicts to by-pass db query"

RC Filters: Detect filters conflicts to by-pass db query

Filters are in conflict when their combination is guaranteed
to return no results. For instance: minor and log entries
is a conflict because major/minor does not apply to
log entries and the field is set to major by default.

Letting conflicts go through result in some very slow
database queries.

Bug: T160220
Change-Id: Ia6b0125c675c4a3cc4e4be4f83d1bd10d23059ba

Merge "qunit: Also clear $.timers when stopping unfinished animations"

Merge "resourceloader: Move mwNow() to after isCompatible()"

Merge "Update OOjs UI to v0.21.1"

qunit: Also clear $.timers when stopping unfinished animations

Stopping the animation is essentially the same as pausing, it remains in the
animation registry and will continue to fail all subsequent tests until one
of those later tests starts the animation queue again and also happens to
wait long enough for this unrelated animation to finish.

Fix the testrunner to actually fully stop the pre-existing animations, which
requires clearing $.timers as well. This matches the logic we have for
pending ajax requests.

Bug: T163211
Change-Id: Ic7d848187bc3c800e8347e0650093b2ffce6dddc

Update OOjs UI to v0.21.1

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.21.1

Depends-On: I5edcf21effee760d9951965a152481f2c603b59b
Change-Id: I5edcf21effee760d9951965a152481f2c603b5b9

build: Sync qunitjs version for karma with version of Special page

Special:JavaScriptTest loads qunitjs from /resources/lib/qunitjs
which was upgraded in 99aad9397e to 1.23.1.

However, when running QUnit tests from the command-line, QUnit is
loaded by karma-qunit using the npm qunitjs dependency, which was
still at v1.22.0.

Change-Id: Ie24b2b061735fa03a8ed1527d09f65c103506c67

Merge "Do an HTMLCacheUpdate of templatelinks on page creation"

Merge "RCFilters UI: Select tag when filter item is selected"

RCFilters UI: Select tag when filter item is selected

Retain the selection behavior from previous versions, and synchronize
the selection of the filter item in the menu with the tag items
in the MenuTagMultiselectWidget.

Bug: T163138
Change-Id: I548a1fca65372c697e24b77d928d56b18f4df1c5

resourceloader: Move mwNow() to after isCompatible()

Follows-up f2fb4a21af. This is logically still the same point in time.
Only 1 function before it now, isCompatible(), which is fine since it
is small and the variable is not used anyway if it returns false.

Change-Id: I34bbe8edf6e9625f8d80f829707adafcb1b91980

Allow editors to exclude navigation items from search indices

When you perform a particular search, the results can be polluted by
navigation elements that are not supposed to be displayed. This gives
editors the ability to mark sections of the document that should not
be indexed.

Bug: T162905
Change-Id: Iab2b83c3778cd5f7f44736c0da569fd938ae2968

Localisation updates from https://translatewiki.net.

Change-Id: I6c1534c40494dff1a9866fa5ec478c4646477688

UploadBase::getTitle can return null

Change-Id: I5bd94f6233476bda43a01155f6e7d6df420412e2

Remove links list from the 'rclinks' message

In some languages, the 'rclinks' message ends
with '...<br />$3.' where $3 is a list of links.
When $3 is empty, we end up with a lonely period
on a new line.

This change removes $3 and adds it on the page
on its own. Unfortunately, it requires fixing all
the messages. Not sure if it's the proper way to
do this.

Bug: T162844
Change-Id: If17f056c135b2e51747a5a1c8aeaba785cf18ff2

Merge "EditPage: Restore IE 6 compatibility for OOUI-style buttons"

EditPage: Restore IE 6 compatibility for OOUI-style buttons

Follow-up to 97d7de0b84ef65094108fbcf62ffcc1c55d75345.

Bug: T163219
Change-Id: Ifb5145c631ce1a3aa7e51956fab3ec7da8b994d2

EditPage: Restore ability to provide edit summary for OOUI-style input

The 'name' attribute was missing, so the field was not being submitted
with the form.

Change-Id: Ic7238b9bfe9ca254f844fb828cf09ecf44171461

Do an HTMLCacheUpdate of templatelinks on page creation

When a page is created, anything that's trying to transclude that page
needs to have an HTMLCacheUpdate run on it. But that's not currently
happening.

Normal template transclusion happens to get updates only because
transclusion of a nonexistent template is rendered as a link to that
template, which registers a pagelink, and pagelinks *are* being updated
on page creation to cause redlinks to turn blue.

Change-Id: I18b8cfdeedc53f8fdb1e758a6fca1cd23d9574f8

Merge "Use more specific string[] type hint for language variants"

Merge "Avoid overflowing of unbreakable words in interwiki search results"

Fix EditPage button accessors on PageObject

This will change the accessors to the EditPage in the PageObject to
fit the new DOM when using OOUI.

Bug: T163193
Change-Id: Ic6858d458e2f781c20a101105eb03bd483d6af73

Merge "Use .toc instead of #toc as selector for TOC"

Use .toc instead of #toc as selector for TOC

Since 1bf5a652 multiple TOCs are supported.

In CSS the selector #toc is kept for backward compatibility for uses in the
content.

Change-Id: I07036c0d5782420fc127781a15c675fc7e6fafaf

Merge "RCFilters UI: Don't steal focus when clicking highlight button"

Merge "RCFilters UI: Prevent label from stealing focus on click"

Merge "RCFilters UI: Display 'no results found' if... no results were found!"

Merge "RCFilters UI: Fix scrolling to item on tag click"

RCFilters UI: Display 'no results found' if... no results were found!

The "normal" operation of an OO.ui.MenuSelectWidget is to hide itself
if no results are found in the filtering process. However, we want
the menu to remain visible but display a 'no filters found' message
like it did before the migration to a MenuTagMultiselectWidget.

Bug: T163170
Change-Id: Iee7b07961b90ac514a4aa1f7c56019ecd7b4c3cc

Merge "Split up LoadBalancer::getReaderIndex() and simplify the code a bit"

Split up LoadBalancer::getReaderIndex() and simplify the code a bit

Change-Id: I4e0b5318ad2c987b2a059c4ef2bda3de14486687

RCFilters UI: Fix scrolling to item on tag click

Bug: T163138
Change-Id: I1f3a5b29eb3d9125faf1d4d27c84b8711fd4fdda

RCFilters UI: Don't steal focus when clicking highlight button

Change-Id: I6c41803221fc09ae2676bd09dd734d36f284cc09

RCFilters UI: Fix input direction in RTL again

The style rule for this was in a block for &-search,
which doesn't exist any more.

Change-Id: I1d60cd42df173f7f71085c2a3e5d46bd4bf23ef1

Merge "Move DB errors to Rdbms namespace"

RCFilters UI: Prevent label from stealing focus on click

The filter options' labels are set as "label for" the checkbox
which means that by default they "give" the focus away to the
checkbox when clicked. However, we don't want that behavior so
that we can continue arrow movement through the filter option
items rather than the DOM elements. Also, the checkbox prevents
defaults on click as well as it is.

Change-Id: I9758b0dacd8cd647ec5e3e623edc66e1d7f7420a

Merge "Clarify getLaggedReplicaMode()/laggedReplicaUsed() in LoadBalancer"

Use more specific string[] type hint for language variants

This patch only touches PHPDoc documentation, nothing else.

Change-Id: Ia79d06425a3b8629c171cd68ae435c64dac86f46

Localisation updates from https://translatewiki.net.

Change-Id: I6518eb7aa39e3933714d2df77f881a0929196d71

Merge "RCFilters: Remove 'overflow:hidden' from filter header"

Merge "Support GENDER in the message "sp-contributions-userrights""

Changed "modified" to "edited"

There were two messages that said "This page was last modified etc.".
They are now "This page was last edited". The word "modify" is used
elsewhere in MediaWiki messages for things other than editing a page,
like uploaded files, databases, and configuration, whereas actual
pages are edited and not modified.

Change-Id: I79b68bbec35f7747cd2a27c1a29e154139e3ab76

Localisation updates from https://translatewiki.net.

Change-Id: Id03227c4e4d8963db6cd44ee0bde953dbe2128c6

Merge "EditPage: Make input and button widgets infusable"

Merge "Convert various FormActions to OOUI"

Localisation updates from https://translatewiki.net.

Change-Id: Ib5b863d0778647034f75e65c45e4a10986ec1a3d

EditPage: Make input and button widgets infusable

This will allow gadget developers greater control, e.g. disabling.

Change-Id: I9a21cd2dd3fc7695fc09632b844cc1afd0db0355

Change File namespace name

Set File namespace name for Estonian to literal translation "Fail" ('file') and set previous name "Pilt" ('image') as a namespace alias.

Bug: T162744
Change-Id: Id9616825f8335f512ccfeca167ddeef42d0220c3

Move DB errors to Rdbms namespace

Change-Id: I463bd86123501abc68fdb78b4cda6110f7af2549

Support GENDER in the message "sp-contributions-userrights"

Bug: T130960
Change-Id: Iab6418c6c60c94a0c77e25243e83d1a13b7ca6e0

Localisation updates from https://translatewiki.net.

Change-Id: I077d94de0c0681f31bac6aab7257151eb8b76fec

Clarify getLaggedReplicaMode()/laggedReplicaUsed() in LoadBalancer

Change-Id: I4ea9033d2d643c6758c49f34a69c9837598eea33

media: Capture stderr when running 'convert --version'

At Wikimedia, $wgImageMagickConvertCommand points to a shell wrapper
that invokes ImageMagick convert with:

  firejail --profile=/etc/firejail/mediawiki-converters.profile

firejail emits to stderr an information message:

  Reading profile /etc/firejail/mediawiki-converters.profile

That ends up in HHVM stderr and is populated up to logstash as an error.

MediaWiki does check imagemagick version by running `convert -version`,
switch from wfShellExec() to wfShellExecWithStderr().

Bug: T158649
Change-Id: I78d1ef59533c605f59b42e10556bb595d6c2cc15

build: Update karma to 1.5.0, karma-firefox-launcher to 1.0.1

karma:
 Changelog at https://github.com/karma-runner/karma/releases
 https://github.com/karma-runner/karma/compare/v1.1.0...v1.5.0

karma-firefox-launcher:
 Changelog at https://github.com/karma-runner/karma-firefox-launcher/blob/master/CHANGELOG.md#101-2017-03-04
 https://github.com/karma-runner/karma-firefox-launcher/compare/v1.0.0...v1.0.1

Karma v1.1.1 added use of `<script crossorigin="anonymous">` for
the urls it initially loads. While most requests will be done by
MW itself directly (and served locally via the proxy at /w/) the
initial request cannot come from the proxy and is loaded from wgServer
directly. Therefore we need to set crossOriginAttribute=false to override
the new default that v1.1.1 introduced.

Change-Id: I9f5067b949c0a77e46f47c2a110a7b5df5953fda

Merge "phpunit: Make LogFormatterTest less slow"

Merge "qunit: Improve testrunner logging for pending ajax"

Merge "Add grep infos to Parser::getImageParams"

qunit: Improve testrunner logging for pending ajax

* Move `restoreWarnings()` in tearDown() to the mirrored location
  of related code in setUp().

* Ensure that accidentally calling `suppressWarnings()` twice will
  not wipe out the original reference indefinitely. If it was
  already set, subsequent calls should do nothing instead of
  overwriting them again so that recovery is still possible.

* Log all ajax requests logged during the test, not just the
  one currently still pending. This should avoid situations
  where we throw "Pending ajax requests" but no information
  is logged about which requests those might be.

Change-Id: I900ad98c4c8520bdd6ae00a24ac82272f3becfee

phpunit: Make LogFormatterTest less slow

Right now it's quite slow because it reloads LocalisationCache
for every test case.

Change the reload to only happen once for the entire class.

Also use RequestContext instead of wgLang to match logic in
the parent class (MediaWikiLangTestCase).

$ phpunit includes/logging/LogFormatterTest.php
Before
- run 1) Time: 23.56 seconds, Memory: 48.00MB
- run 2) Time: 22.41 seconds, Memory: 48.75MB
After
- run 1) Time: 14.27 seconds, Memory: 47.50MB
- run 2) Time: 13.45 seconds, Memory: 47.25MB

Change-Id: Ia791563ca9094c36782acfe9826def45f2a1d621

Add grep infos to Parser::getImageParams

Comments for grep makes searching easier

Change-Id: I98e93baf6bd89df36185d535d6e63c51c6f65bc9

Avoid overflowing of unbreakable words in interwiki search results

The interwiki search results can be pretty narrow, and longer
unbreakable words or titles in the results might easily overflow the
containing box. break-word fixes all that.

Change-Id: I97e2b101433bde50a0aa48b772b2fe5a7157aed4

Fix some LoadBalancer::waitFor*() inconsistencies

* Make sure waitFor() always handles laggedReplicaMode updates,
  even if the actual waiting was deferred until a connection was
  needed.
* Restore the old mWaitForPos in waitForOne()/waitForAll() since
  this do not care about the generic reader index or the sort of
  ChronologyProtector logic related to it.

Change-Id: I0767e9831b8fd7fd115a472354977e3c1e12114a

RCFilters: Remove 'overflow:hidden' from filter header

This will make it possible to z-index it on top of the list
for other elements (like the guided tour 'pulsing dot') to appear
on top of the list.

Bug: T162971
Change-Id: Iac3b8eabdeaee009b65c7b0c104179c760705306

Merge "RCFilters: Followup I42be06913: Destroy tag widget when removing"

RCFilters: Followup I42be06913: Destroy tag widget when removing

Change-Id: Ifa051dd88dd36fa8ab14eacbeaea49f9aa0dbcdc

jquery.tablesorter: Disable a flaky isoDate test case

This one varies by local timezone. To be fixed up later.

Change-Id: I42f3684b721d4c8a9a687adbaf0b4f770c1f057d

qunit: Remove test for deprecated jquery.placeholder

It's not very useful and needlessly emits deprecation warnings
which unfortunately cannot be surpressed at runtime.

The test just asserts that a no-op function exists.

Change-Id: I2768ba40191a3c0cc7be6202ffa5bed529eabfee

Merge "RCFilters: Adjust to use MenuTagMultiselectWidget"

Localisation updates from https://translatewiki.net.

Change-Id: I69e7808271ea3f7afc65989ab13250d4022a603b

RCFilters: Adjust to use MenuTagMultiselectWidget

The new widget in OOUI is more stable and easier to manage,
and gives us a few features that we were missing, like
arrow behavior in the menu.

Depends on OOUI release 0.21.0

Bug: T162829
Bug: T159768
Bug: T162709
Bug: T162917
Change-Id: I42be0691304b1e93b4e9c02eba2e3a724a5ffd67
Depends-On: Ic216769f48e4677da5b7274f491aa08a95aa8076

Merge "Localize namespaces in Doteli"

Merge "Make LoadBalancer::waitFor() and friends  more robust via try/finally"

Localize namespaces in Doteli

Translation authors:
  - @Rameshti (Ram Prasad Joshi)
  - @nirajan_pant (Nirajan Pant)
  - @Janak_bhatta (Janak Bhatta)

Bug: T162872
Change-Id: I3f28ebd5e5486e6e5f3bde7c3cbc42975e26af40

Merge "rewording for cascadeprotectedwarning"

Make LoadBalancer::waitFor() and friends  more robust via try/finally

Change-Id: I50742752bb94ca17997138b440b911849bcbadf3

Upgrade to jQuery v3

Source code:
 https://code.jquery.com/jquery-3.2.1.js
 https://code.jquery.com/jquery-migrate-3.0.0.js

Documentation:
 https://blog.jquery.com/2016/06/09/jquery-3-0-final-released/
 https://jquery.com/upgrade-guide/3.0/

This is not a breaking change because jQuery Migrate covers
all breaking changes.

However some extensions (especially unit tests) may've relied
on undocumented behaviour. For that reason, and due to unresolved
upsteam issues this is still behind a feature flag for now.

It is true by default to ensure this has wide exposure to discover
issues as quickly as possible. If this is not resolved before
the end of the 1.29 release cycle it should be turned off again.

Bug: T124742
Change-Id: I3c3dedaa9a9d449eaa2b7e5d24b4540e7fa421c0

Merge "resourceloader: Fold legacy modules into base modules request"

Merge "Add comments to ApiBase::isWriteMode()"

Add comments to ApiBase::isWriteMode()

Change-Id: I3581f5e3055c425367d265a5c57831bb34dd5a2f

rewording for cascadeprotectedwarning

Bug: T60228
Change-Id: I2ec4cbe4c552dc095dce0fae0f0b8e4a52f0c6a9

Merge "Ensure logo preload transforms urls if needed"

Merge "RC Filters: Disable defaults for legacy filters in structured UI"