Merge "shell script fix using shellcheck lint"

Allow use of GENDER in "fileexists" message

Requested by Protnet in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Fileexists/el

Change-Id: I51fdc106938977a5c9abd78de4fd6f297e2ac880

Merge "Reword typo in file header for EditPage.php"

Merge "Capitalise Message in SpecialAllMessages files"

Merge "Rename SpecialAllpages to SpecialAllPages"

Merge "Revert "mediawiki.mixins: Add border-radius mixin""

Localisation updates from https://translatewiki.net.

Change-Id: If9f9760aecc14851052ac63f3cd9da15152f500e

Revert "mediawiki.mixins: Add border-radius mixin"

Prefixes for border-radius haven't been needed in a long time. This
degrades gracefully and isn't worth the overhead.

In addition, only a few weeks ago did we merge commits that removed all
this prefixes (e.g. in Ie5ce3202b4). Adding this mixin would encourage
doing the reverse.

This reverts commit 591ba359f283c5ab4ba1b2a0359a98692a3b204b.

Change-Id: Ic25a001de3fe1516cc4939b3159e362e87cb790b

Followup e48ecbc5: fix fatal error due to undefined variable

Bug: 66669
Change-Id: I195a5ff539f09addb92d2bb07d53bdb3aef0db1f

Minor: more specific parsoid image styling with child selectors

Bug: 51245
Bug: 66616
Change-Id: Iff228e932bbb3d548e9a0430a561020cdccf1b4c

Rename SpecialAllpages to SpecialAllPages

Change-Id: Ic137bc7adbc4a9ed96448ba0fee4807b67b3112d

Capitalise Message in SpecialAllMessages files

No internal changes to Special:Allmessage etc made

Change-Id: I21a164af0b6ec123bf654cd1e4e7085b1192f067

Merge "mediawiki.mixins: Add border-radius mixin"

mediawiki.mixins: Add border-radius mixin

Change-Id: Idb46dee0ac8a29d489ec7fb405f8a3149a729b3b

Split messages for recent change legend and letter tooltips

This makes it possible to use formatting in the legend ones
(e.g. links to help pages) and to use different wording
(the one for tooltips is sometimes too verbose for the legend).

Documented the format of $wgRecentChangesFlags better.

The default text in not modified in this change.

Bug: 58715
Bug: 60235
Change-Id: Ie9872c8fb3e43620877300ee8536bb5ddee5ab46

Reword typo in file header for EditPage.php

Change-Id: Iefad919e09632b11bb06abcab390a756455c2ddd

Merge "Add tests for API's assert={user|bot}"

Revert "Revert "Officially deprecate skin autodiscovery""

The CI issue that made the revert necessary is fixed now.

This reverts commit d672aa3123a612583ef9ec67115992aafce5b2af.

Bug: 66429
Change-Id: Ib2b4a895c5d01e5ccc7e4edfdbfdcaf0a93c267c

Merge "Fix some line mis-wrapping and typos in RELEASE-NOTES-1.24"

Merge "Update jquery.jStorage.js"

Merge "Collapse duplicate $existingExpiry == 'infinity' case into else"

Collapse duplicate $existingExpiry == 'infinity' case into else

Change-Id: I1cc612e9f646dd75f58cc6ae0ae073ebffa7d6d3

API: Allow 'infinity' as a valid protection expiry

Change-Id: If9eb1f3e835579f5c8b8be22297a1eb26beda4e7

Switch API to use Config classes

Only done where globals are config (so not $wgParser, $wgContLang etc)

Change-Id: Ic39cdd858cfb9096a2bc09618f97e64270d76f13

Localisation updates from https://translatewiki.net.

Change-Id: Id7f62d5c54ac1d476f37d238759a28ef803347fb

Update jquery.jStorage.js

Project site

* http://www.jstorage.info/ and https://github.com/andris9/jStorage

jquery.jStorage.js file

* https://github.com/andris9/jStorage/blob/master/jstorage.js

Author and contact

* Andris Reinman and email andris.reinman@gmail.com

README

* https://github.com/andris9/jStorage/blob/master/README.md

Change-Id: I0ca42e909cc08f8f0b67cec0a7c0aedd0ebf91d7

Merge "mw-ui mixins: More colors, more browser helper mixins"

mw-ui mixins: More colors, more browser helper mixins

Change-Id: I895679ffe2f5eb1f002ff110c750e77824004820

Localisation updates from https://translatewiki.net.

Change-Id: Id1349f603b9e9c7d7c039a7fbea03ccf69238d89

Fix some line mis-wrapping and typos in RELEASE-NOTES-1.24

Change-Id: I08c513ed6418f58b1d269456affd02d882ab8e9b

Localisation updates from https://translatewiki.net.

Change-Id: I70982e6a517a7fd472acb8577adde727cabd7a21

Add SpecialEditWatchlist::prefixSearchSubpages

Also add 'clear' to SpecialWatchlist::prefixSearchSubpages

Change-Id: Ibc1ca44a9472cd86a6dc5df04274089b8fcf30a2

Merge "Change MostLinkedTemplates to MostTranscludedPages"

Merge "Made FSFileBackend use closures for doExecuteOpHandlesInternal()"

Change MostLinkedTemplates to MostTranscludedPages

Rename Special:MostLinkedTemplates to Special:MostTranscludedPages, and
allow it to return pages from any namespace.

Change-Id: I27c2fb50f77aef760f8c7006ae48297451230c6f

Merge RELEASE-NOTES-1.23 into HISTORY

Used the REL1_23 version of that file, and moved the note added in
5e48ad2e78ce to RELEASE-NOTES-1.24, where it belongs.

Change-Id: Id5f4726152fc2f4c90117762f6dc4904f5fa9f94

Localisation updates from https://translatewiki.net.

Change-Id: Ib84e5a1bc0740a2948a2a902e6916fa3f9c1627a

Merge "Remove $wgDBClusterTimeout"

Made FSFileBackend use closures for doExecuteOpHandlesInternal()

Change-Id: I8401a92384cb454eb79a4607b350c50d432a9510

Merge "Apparently for certain (API) requests $this->getTitle() doesn't return a valid Title."

Remove $wgDBClusterTimeout

Also added release notes that I forgot to include in
change I4b409ef

Change-Id: I9416c0dc7dea1c32c3e1f6d7177ee9bc4e34e1de

Merge "Deprecate $wgMasterWaitTimeout"

Use type hinting in ApiBase

For ideological reasons this was not included in Ie6bf1915.

Change-Id: I5f7119665746eb6fcf86c3a403caa2dcb67904eb

Autocomplete special pages' subpages in search suggestions

Includes basic "example" implementations for:
* Special:Watchlist (predefined subpage list)
* Special:Log (based on config variable)

Bug: 20210
Change-Id: I2b0505df244b9d23bd23d5fb4d3405dc28d57739

Merge "Vector: Workaround for legacy IE font-sizing issues"

Merge "Create PostLoginRedirect Hook for changing the redirect behavior"

Vector: Workaround for legacy IE font-sizing issues

In IE <= 7, 'Ctrl' + '+' scales the font size in far too large steps if the
root element's font size is given in the 'em' unit. Everything becomes normal
if the root element's font-size is given in '%'. This should not have any
effect in other browsers.

Bug: 24515
Change-Id: I5cc864d73935a02713075840b7ff80000b3ae88a

Deprecate $wgMasterWaitTimeout

Not only was this global unused, it also caused the LoadBalancer
to look for the wrong array key, which would always set
$mWaitTimeout to 10 in the conditional. There is also
a manual function LoadBalancer::waitTimeout() for this.

Change-Id: I4b409ef890bec5aac7f666a2ed3dff9ea8bc14cc

resourceloader: Implement "skip function" feature

A module can be registered with a skip function. Such function,
if provided, will be invoked by the client when a module is
queued for loading. If the function returns true, the client will
bypass any further loading action and mark the module as 'ready'.

This can be used to implement a feature test for a module
providing a shim or polyfill.

* Change visibility of method ResourceLoader::filter to public.

So that it can be invoked by ResourceLoaderStartupModule.

* Add option to suppress the cache key report in ResourceLoader::filter.

We usually only call the minifier once on an entire request
reponse (because it's all concatenated javascript or embedded
javascript in various different closures, still valid as one
large script) and only add a little bottom line for the cache
key. When embedding the skip function we have to run the minifier
on them separately as they're output as strings (not actual
functions). These strings are typically quite small and blowing
up the response with loads of cache keys is not desirable in
production.

* Add method to clear the static cache of ResourceLoader::inDebugMode.

Global static state is evil but, as long as we have it, we at
least need to clear it after switching contexts in the test suite.

Also:
* Remove obsolete setting of 'debug=true' in the FauxRequest in
  ResourceLoaderTestCase. It already sets global wgResourceLoaderDebug
  in the setUp() method.

Bug: 66390
Change-Id: I87a0ea888d791ad39f114380c42e2daeca470961

Update OOjs UI to v0.1.0-pre (c5ae888f67)

New changes:
9eec766 Create 'next' and 'previous' indicators

Change-Id: I3f091f25be73fe35a55b113630c253c9a3cf5b38

Update OOjs UI to v0.1.0-pre (9f4f250f88)

New changes:
bf4ba74 Localisation updates from https://translatewiki.net.
a99acae Remove use of imaginary config option in OutlineControlsWidget
8db7499 Localisation updates from https://translatewiki.net.
7d7b9ce [BREAKING CHANGE] Window process cleanup

Change-Id: Ib9ae3fdfe14bbeb1f687b26f552cf4fb49db2d69

Merge "Remove $wgProxyKey"

Merge "RCFeed: Add log_id property (exposes rc_logid attribute)"

RCFeed: Add log_id property (exposes rc_logid attribute)

Bug: 20394
Change-Id: I005be8e41a5cbea934d4dac27184f03e9af7c648

Remove $wgProxyKey

Change-Id: I1748164f108fab701c79532ede13ae387b12769d

Localisation updates from https://translatewiki.net.

Change-Id: Ib84999f8508bc7bd8843eae28a8db1a35b1628be

Apparently for certain (API) requests $this->getTitle() doesn't return a valid Title.

Usually these are API siteinfo requests (api.php?action=query&meta=siteinfo). In that case, this code used to produce a fatal ("Call to a member function getPageLanguage() on a non-object").

Change-Id: Ifb6f99fe554890ff2719220f58d1b6c1a7a95ddc

Reduce deadlocks adding log rows to the RC table on delete

bug: 56776
Change-Id: Ic75530b25f3fa5668019c76ce59a1dda2768f389

Merge "Fix misnamed SVGHandler::canAnimateThumb()"

Merge "Remove isset() check in Sanitizer::setupAttributeWhitelist()"

Merge "Cleanups to WebRequest::getIP logic"

Merge "Add a maintenance script to list variants and languages with then"

Merge "MimeMagic: Don't seek before BOF"

Cleanups to WebRequest::getIP logic

* Throw an error if there is no immediate server IP, otherwise the
  XFF logic is wonky.
* Refactored the loop to be a bit easier to read.
* Better handle "unknown" entries in the XFF chain.

Change-Id: I9541afa408d895c3fd337a883ecfe4ce0ba57090

MimeMagic: Don't seek before BOF

This has weird side effects like only extracting the tail of the
file partially or not at all.

Bug: 66428
Change-Id: I182128c6958244f1515227ee742c3206a7484aee

History: Simplify checkboxes script on History pages

This should reduce the amount of variables constructed and also switches
to a class based implementation, which allows us to do the same with
fewer calls touching the DOM.

Not supported by IE6.

Bug: 51561
Change-Id: I4596b3667f990ef1d8c7ca753e1d80fe285d5614

Remove obsolete RC_MOVE and RC_MOVE_OVER_REDIRECT rc types

RC_MOVE and RC_MOVE_OVER_REDIRECT are obsolete, since at least 2006.
So it is considered safe to remove.

Bug: 63755
Change-Id: I0f17c4d164585a48fb9f0d40b90a7d3b975c7ab8

Merge "Final page of search results sometimes having erroneous "next" link"

Fix misnamed SVGHandler::canAnimateThumb()

Caused the animation warning to not be present on
image description pages

Also add unit tests. Since I'm adding an animated svg file for
this unit test, also add a metadata extraction test for animated
svgs, since we're missing one.

Change-Id: Id03e1a0e1c151d3c575a695a42c54b709187d10a
follow-up: 876128f8c9f0ce6a8424336f

Merge "LoginForm: Pass username as first parameter to error message if aborted"

Merge "Fetch string to pass through writeTemporary() in DatabaseOracle.php"

Merge "Removed undeclared var from wfDebug()"

Merge "Revert "Fix "CSSMin url() value remapping not working in certain obscure cases"""

Revert "Fix "CSSMin url() value remapping not working in certain obscure cases""

This reverts commit fa1053a16a5b3ff2004f6c06c69e87f8570a35d7.

Breaks CSS output, making the skin and extensions look unstyled.

Change-Id: I297150f775dd5559f91f2c9c25c64b134459c6b6

LoginForm: Pass username as first parameter to error message if aborted

If a hook is aborted, pass the username as the first parameter to allow
for more customized messages.

Change-Id: Ifb55a2e180c619f7d06f728dd8eea2c9863b8f82

Remove <div id="mw-js-message">

This is a remnant of the mw.util.jsMessage() utility, replaced by
mw.notify() several versions ago.

Change-Id: Ifd2765581f06dcc2ca057bc30b27b336df2d066b

Merge "Fix "CSSMin url() value remapping not working in certain obscure cases""

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

Localisation updates from https://translatewiki.net.

Change-Id: I8f904f32657c9b380fffe31de30b18b3a164583d

Use anonymous function in Linker::formatAutocomments/formatLinksInComment

A anonym function has no own scope, so it is not possible to call the
private callback, so moved the code of the callback into the function.

This avoids passing parameters with static variables to the callback.

Change-Id: I41ad85545301f6699817135b7e8ceba0f8b26851

Merge "LinkerTest: A bunch of tests for Linker::formatComment and friends"

LinkerTest: A bunch of tests for Linker::formatComment and friends

Change-Id: Ie805d2945ddc1596545ef3f22d32b429caca8a27

Merge "Change 'delete' to 'rollback' in action=rollback params description"

Change 'delete' to 'rollback' in action=rollback params description

Bug: 66406
Change-Id: Iab874f45b5baaa89a6345e351cc45ed16f5af6e8

Remove isset() check in Sanitizer::setupAttributeWhitelist()

Declared variables have a default value of null, so there is
no point to use isset() to check only if it has the value null.

Also changed the location of the line break to group variable
declarations.

Change-Id: Ic36fc95db86909d8b5075954a72afa479fa20d0d

Remove $dbr from ChangeTags::addTags()

$dbr (DB_SLAVE) is completely unused now.

Change-Id: I02bbde8366480b841d8028403e9a0efe88fa46fd

Merge "Make Minerva skin opt out of parsoid style module"

Make Minerva skin opt out of parsoid style module

These styles are mostly Vectorish, so shouldn't be applied to the
Minerva skin that is provided in MobileFrontend.

We set our own parsoid styles in
VisualEditorOverlay.less.

Change-Id: I11a75c5552fa14ad119ad60bb5fa3f6185582429
Dependency: I5de12c97108dcb6e41ac8c34b17ff993d2e65249

Update OOjs UI to v0.1.0-pre (99068355d6)

New changes:
9906835 Make menu toolgroup take up less space and look more like a menu

Change-Id: Ie5e5f3d8f3f6bccdbacd85f1ea8423bfeaa8e2f2

Merge "Bug 51245: Add auto-numbered link CSS to Parsoid styles"

Merge "Revert "Officially deprecate skin autodiscovery""

Revert "Officially deprecate skin autodiscovery"

The warnings interfere with Jenkins and stop code from being merged
where Jenkins is voting. See bug 66429 - we'll need to fix this before
merging this. Sorry!

This reverts commit 7fe6e4422467fa0c701f3ffb395e66de9cd0d67f.

Bug: 66429
Change-Id: I649462b0ccc22954ba4fbad8087dd55a7ad910bd

Bug 51245: Add auto-numbered link CSS to Parsoid styles

Also use the mw-body-content class to scope styles to the content area.

Change-Id: Id765da7bfb60726eead86e67693b86c82bdb5610

Removed undeclared var from wfDebug()

Change-Id: Ie8a0ff2f9347c6ed2a04f7c99de6c43958dfd1f7

Merge "Vector: Update comments in vector.js"

Officially deprecate skin autodiscovery

Corresponding change in REL1_23 (different, because core skins still use
autodiscovery in that version): If9125ef587ce11f0d69f12eacd00ae7236139798.

Bug: 65748
Change-Id: I6fc30f97d4b35820c132c4892abf6d2c86f9482e

Don't use autodiscovery for core skins, move them to separate directories

Step one on the way to killing the autodiscovery mechanism and making
the core skins less intertwined with core.

This only moves the files and fixes hardcoded paths throughout core.
Any further changes will be done in separate patch(es).

Moved files:
* skins/MonoBook.php → skins/monobook/MonoBook.php
* skins/Vector.php   → skins/vector/Vector.php

Bug: 65748
Change-Id: Ib4bdda5ed3c133fce0113eb17fa39950aa812f87

Merge "Remove unused variable"

Remove unused variable

Follows up Ie47649ef45f9eebdfe

Change-Id: I2c6f2a0b4a8f15056ad0b314da7f8086168b1863

Do not show page navigation on included Special:Contributions page

Bug: 65780
Change-Id: Ie1f96d62e0f7b8c1066aaecc5e8ba528fb94cbf7