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

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

Localisation updates from https://translatewiki.net.

Change-Id: I6c1534c40494dff1a9866fa5ec478c4646477688

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"

Ensure logo preload transforms urls if needed

Follows-up 5f55e9c9c2a24.

If the logo url is from within /w, then ResourceLoaderSkinModule
will (as it should) apply a file hash query to it.

The preloader didn't do that, so it specified the wrong url.

Refactored SkinModule to make this logic re-usable.

Bug: T100999
Change-Id: I1ba11f7c70d1a725ad72754fee4a3f33c2a4c1be

Merge "mw.widgets.Complex*: Fix setDisabled"

Localisation updates from https://translatewiki.net.

Change-Id: I9d431f5c949ec909d85949436b90ffde42139962

mw.widgets.Complex*: Fix setDisabled

* Check if elements exists before trying to disable them
* Return `this`, as the method is marked as chainable

Follows-up I6747f4e64dd81197bb3e37c09876399f1cb3be07

Bug: T162667
Change-Id: I17665e57537dbed0821bc3226377849eb8388a32

Merge "Convert EditPage buttons, checkboxes and summary input to OOUI"

Merge "Use delete_and_move_reason in content language on move over redirect"

Merge "Move Database and subclasses to Rdbms namespace"

Merge "mw.loader: Don't assume var 'loading' assigns before Promise resolves"

Move Database and subclasses to Rdbms namespace

Change-Id: I52bef87512f9ddd155d1f4cc0052f6b7a0db5b42

mw.loader: Don't assume var 'loading' assigns before Promise resolves

In rare situations the variable loading already assigned with a promise and
the call to loading.map fails with

  TypeError: loading.map is not a function

Change-Id: Ie50bdda229e48b159702fc2a83e641a35d7c850c

Remove multiple spaces in PHP files

Change-Id: Id9c26ec5ca730a9536f8fdccb8853eb03cbe87a4

Merge "Remove some ancient upgrade information from release notes"

Merge "Preload the logo using link rel="preload" http header"

Merge "Prevent PHPDBG from issuing notices in unit tests"

phpunit: Avoid use of wmf-production host names

* Remove used of 'eqiad' and 'wmnet' in unit tests.

Change-Id: I5bf19b63876e4dc8deaca8cd9907dca0bfa15455

Preload the logo using link rel="preload" http header

This greatly increases the priority of loading
the logo on browsers that support rel="preload".

Bug: T100999
Change-Id: I0738fcc0a575153dab65016fa87faaa9b8b97a9d

Convert EditPage buttons, checkboxes and summary input to OOUI

Several methods now have a new implementation using OOjs UI widgets
(ButtonInputWidget/ButtonWidget, CheckboxInputWidget, TextInputWidget).
The existing (public) methods are unchanged. The OOjs UI version is
used by default.

Because this change can cause problems for extensions and on-wiki
scripts depending on the exact HTML, the old version is still available
and can be used by setting $wgOOUIEditPage = false; in LocalSettings.php.
This will be removed later and OOjs UI will become the only option.
To make testing easier, users can also force either mode by adding
&ooui=true or &ooui=false to the action=edit URL.

* EditPage::getSummaryInput() and EditPage::getSummaryInputOOUI()
* EditPage::getCheckboxes() and EditPage::getCheckboxesOOUI()
* EditPage::getCancelLink()
* EditPage::getEditButtons()

Bug: T111088
Co-Authored-By: Amir Sarabadani <ladsgroup@gmail.com>
Co-Authored-By: Florian Schmidt <florian.schmidt.welzow@t-online.de>
Change-Id: I25aa78ac59082789938ecfb5878eb16614392995

Merge "Remove "editusercssjs" user right"

Merge "RCFilters UI: Adjust styles to OOUI's newest release"

RCFilters UI: Adjust styles to OOUI's newest release

Slight modificationsfor OOUI 0.21.0

Change-Id: I8b1db97fd7d52a57a7e3bdd13cc7dc4e364b8976

Update OOjs UI to v0.21.0

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

Change-Id: I94d5b7a89551e7d34d53223872ad3005f0ac4a04

Merge "objectcache: Complete coverage for newAnything()"

Merge "wikibits: Remove methods deprecated since 1.17 and 1.18"

resourceloader: Fold legacy modules into base modules request

Follows-up 0ac4f998 (restore "blocking" legacy modules).

After d790562, legacy modules in the top queue were no longer consistently
loaded before the bottom queue due to the top queue being async.

The implied dependency was made explicit by 0ac4f998 by forcing all modules
to wait for legacy modules before executing.

This had the negative side-effect of putting an extra HTTP request between
the startup module request, base modules request, and actual execution
of page modules.

(Indentation aligns with when a request is triggered.)

Before:
 1. Request: Startup module.
   2. Request: Base modules
   3. Request: Legacy modules
     4. Page module request (or local store hit) and execution

After:
 1. Request: Startup module.
   2. Request: Base+legacy modules
     3. Page module request (or local store hit) and execution

This could alternatively be fixed by moving the top queue to be before
the embedded modules and enforcing the embed in a different way.
It could also be fixed by debouncing module load calls so they naturally
end up in the same request as page modules.

However for now I'm addressing this by adding legacy modules to the
list of modules in the initial load request from the startup module.

This was not possible before because the legacy wikibits had dependencies
and base modules cannot have dependencies. Fixed in I7f9f61ea81ad1ef.

Bug: T159911
Change-Id: I54f087655e1cde1b8ff1ca5fe56e82f7f7d80965

Merge "interwiki: Remove inappropiate use of wfMemcKey()"

interwiki: Remove inappropiate use of wfMemcKey()

This is used for lookup in a CDB file or PHP static array.
In neither case is the key created by wfMemcKey() or any other
implementation of BagOStuff::makeKey().

This is already broken if:
* An interwiki prefix were to contain characters not supported by
  Memcached.
* An interwiki prefix and wikiid together are too long for Memcached.
* If the site has wgCachePrefix configured, which overrides the
  wfWikiID() namespace makeKey() normally uses.

dumpInterwiki.php does not use wfMemcKey() either (and should not).

This was simply here as leftover from many rewrites ago, its
only purpose is to create wikiid + prefix joined by colon.

Ref T148958.

Change-Id: I45682133ed593fbb0d66af5a67751f77f15a4a14

Merge "resourceloader: Add unit tests for ResourceLoaderImage"

resourceloader: Add unit tests for ResourceLoaderImage

Follows-up I5b14d65a and I5a563c59.

Change-Id: Id42e1b868c9fe97cdb14b4bc7328947820a7fd94