Merge "Call wfThumbIncrAttemptFailures() in thumb.php on fatals"

Call wfThumbIncrAttemptFailures() in thumb.php on fatals

Change-Id: I3dbe07d12abdc265d377cd05bb87c837df156995

Merge "Support tighter rate limiting for "non-standard" thumbnails"

Make includes/title pass phpcs-strict

Change-Id: Iac03d1f3d4ee622496b3cb0bf8086d2f362b52af

Merge "Deprecate ResourceLoaderGetStartupModules"

Deprecate ResourceLoaderGetStartupModules

In bug 63240, Roan calls it a 'misfeature', which I think is exactly right. It
has been used as a lazy workaround for having to think about load-order,
dependency management, and race conditions.

Bug: 63240
Change-Id: Ic48ad39c6d3d166d6bb8f60dfdd5f95aa7ed16a6

mediawiki.mixins.less: Correctly embed images in .background-image-svg

Using the /* @embed */ annotation twice for the same property
(background-image) confuses the LESS compiler, resulting in one of the
images not being embedded.

Let's use the embeddable() and embed() MediaWiki LESS functions,
like in the .background-image mixin.

And let's extend and clean up the documentation while we're here.

Bug: 61941
Change-Id: I39ea2004b9db34d096eed08b3ffdfcaf8d5fcd7d

Localisation updates from https://translatewiki.net.

Change-Id: I7816dbccd7d162fc934cf36ff7c23d3b12f0fa9b

Merge "Allow additional interwiki prefixes on local interwiki links"

Merge "Fixed some @params documentation (includes/api)"

Merge "Fixed some @params documentation (includes/[db|installer])"

Merge "Fixed some @params documentation (includes/[file...|upload])"

Merge "Clarifying double-redirect messages"

Clarifying double-redirect messages

Change-Id: Ifbe13b4ed98d34ecbb6dc5b583422c2dc4ff307e

Merge "jquery.suggestions: Handle CSS ellipsis when calculating suggestions' widths"

Merge "Fixed some @params documentation (includes/*)"

Fixed some @params documentation (includes/*)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: Iced714bca004756b461b66067a49a925a7e3b877

Localisation updates from https://translatewiki.net.

Change-Id: I58e3db9db609509bd6c4de229c13524bc262cc88

Fixed typo of encyclopedic in qqq.json

Change-Id: I78861c584d224e915c015172dbc1430749583753

Fixed some @params documentation (includes/[file...|upload])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I8804ebe0922d3a414863b162a2110e0b9e49b80f

Fixed some @params documentation (includes/[db|installer])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I3595e9aac69ac42fbb74b0783fae0ad9bcc831aa

Merge "Fixed some @params documentation (includes/[specialpage|specials])"

Fixed some @params documentation (includes/[specialpage|specials])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: Icf6f36bb53322b39cd5c89523dbd0e4ab10b9ec9

docs: closure -> Closure; callback -> callable

Changed closure to capital word Closure in doc and type hint,
also changed callback in docs to callable

Change-Id: I52c8e8f13d38a837052101c38b9986be780ca057

Merge "Purge removed OOjs UI files"

update-oojs-ui.sh: Replace target dir instead of adding file copies

* A command like "cp dist/* $TARGET_DIR" does not remove files
  that were deleted in the release dist. Using rsync instead.

* Use bash -n or -z instead of "" and "x$VAR" == "x" tricks.

* The "git commit <dir>" doesn't properly take care of staging
  deletion of files removed by the update script, nor does it
  reliably (?) stage new files.
  Instead using a combination of "git add -u <dir>" (The -u
  stages deletion and modifcation of tracked files), and
  "git add <dir>" (stages modification and creation of files in
  that directory).

* Use "grunt test" instead of "grunt <default>" to make more
  explicit what is going on (no change in behaviour, default=test).
  And document why we don't use plain grunt-build without tests.

Change-Id: Id97a66df64d9e43e3c05388486b9f959108363c1

Purge removed OOjs UI files

The old update script didn't remove files in git if there were absent in
a newer version and left these behind.

Change-Id: I087657183c29ff7d9e2b636171961db62687b32f

oojs-update.sh: Replace target dir instead of adding file copies

* A command like "cp dist/* $TARGET_DIR" does not remove files
  that were deleted in the release dist. Using rsync instead.

  Though addition can happen, deletion should never happen as we
  refer to these files in Resources.php. But we also shouldn't
  leave behind old files mixed with new files, so delete them.
  Our structure unit tests for resources take care of catching
  references to files that don't (or no longer) exist.

* Use bash -n instead of != "".

* Refactor slightly to make it re-usable in other projects without
  too much modification (only REPO_DIR and TARGET_DIR need updating,
  the rest is generic now).

* The "git commit <dir>" doesn't properly take care of staging
  deletion of files removed by the update script, nor does it
  reliably (?) stage new files.
  Instead using a combination of "git add -u <dir>" (The -u
  stages deletion and modifcation of tracked files), and
  "git add <dir>" (stages modification and creation of files in
  that directory).

Change-Id: Ieb20978c887c5a141f31cab704b8d239f1572af0

Merge "Fixed some @params documentation (includes/[cache|objectcache])"

Merge "vector: Do styles the ResourceLoader way"

Fixed some @params documentation (includes/[cache|objectcache])

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I86fd10e3f2d4bb80e7432533038d124693acfb3c

Improved HashRing usage in JobQueueFederated

* Added proper ejection and caching in HashRing to avoid rebuilding
  the hash all the time (or doing manual caching).
* Made JobQueueFederated blacklist failing servers for a few seconds.
* Also made the JobQueueFederated root job methods properly respect
  the weights when they fail over.

Change-Id: Ifa4c03272c1777cfff2523ab21f780074ddcf359

Add forgotten OOjs UI i18n files

Change-Id: I8387150a9caf09c16de2ad816ba5dcaa8b56af23

jquery.suggestions: Handle CSS ellipsis when calculating suggestions' widths

This is so nasty :(

Bug: 64093
Change-Id: Ia6b1376dddf69a0faa214d5c9a756bdcd9658367

Update OOjs UI to v0.1.0-pre (989950a4db)

New changes:
ff327f2 Localisation updates from https://translatewiki.net.
21bbbdb Add getCurrentItem method
ca0e453 Remove animation of grid layout inside booklets
b1c75fa Add aggregate method
df59f32 Add push/pop pending to Dialog
21c6d92 Make dialog overlays... overlaid?
989950a Localisation updates from https://translatewiki.net.

Change-Id: Iff681390616a2fa9ce5becf846365e7421f28433

Merge "Remove # from dataProvider"

Merge "Remove superfluous # at end of line from languages.inc"

Merge "Don't cache $wgResourceLoaderLESSVars in a static"

Merge "WebInstallerOutput: Fix path to jquery.js"

vector: Do styles the ResourceLoader way

It used to be done like this a few releases ago, but went
through some perturbation when the LESS rewrite happened
and afterwards during beta-testing the typography refresh.

The current way causes enough issues that I think the gain
from getting rid of it is worth losing some brevity.

Bug: 56592
Change-Id: Ibeff450fad789fbc67d95a1a9befd315f7405421

Localisation updates from https://translatewiki.net.

Change-Id: I6945aedd9791855d4ed1cc62738d1b206d7b950e

Don't cache $wgResourceLoaderLESSVars in a static

Caching the value of $wgResourceLoaderLESSVars in a static variable that is
enclosed in a closure makes it harder to reset global state for test runs, and
it does so in the name of a performance benefit that is miniscule.

Change-Id: I0958b03818f56ab73c8c9124daa9e54cc59f2428

Merge "Make HtmlFormatter return removed elements"

Merge "Made runParallelBatches() fully set the status array on abort"

Merge "mwjsduck-gen: Don't fail when running from different directory"

Made runParallelBatches() fully set the status array on abort

Change-Id: Ie35b28b85e734db6910a42ed384374ba3d594a8e

Remove # from dataProvider

Change-Id: Ie5414173b95e846d735827bffa34c73698e48c17

Remove superfluous # at end of line from languages.inc

Change-Id: Ia625e8de5e4d04dd35b537fd905af53b66f4026b

Merge "Remove 2 unused Special:Search hooks"

Cleanup SpecialAllmessages.php according to conventions

Change-Id: I21a6d4359faf9d86ce3b01e423e8c3ab4c02658f

WebInstallerOutput: Fix path to jquery.js

Follow-up to Id5ead145 and I335e4c08.

Bug: 64091
Change-Id: If40f479df7d68071d6ce5d2f3b8251e6bf3eb80e

Remove 2 unused Special:Search hooks

Nothing is listed on mw.org as using these and nothing in Git uses
them either. Less hooks is a good thing, we've still got 11 here :(

Change-Id: I2dd1dbb269820b192f8d9e9a496a161f4ef851bf

Merge "Simplify SpecialVersion:listToText()"

Fixed some @params documentation (includes/api)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I758fa4ad80ac95e2ddd3770bcb9b7d2e57ec34ea

Merge "Make API action=rsd work on private wikis"

Make API action=rsd work on private wikis

Tweak ApiRsd's settings so that it doesn't require read permissions to function.

Bug: 64085
Change-Id: I15288458ec3b0b583eece930d323f80314bb6ee3

Make HtmlFormatter return removed elements

This shouldn't cause much overhead and is useful for getting the text of
the removed tags.

Change-Id: I97cf66014719244b8bb2b0509b419c82202bdb01

Merge "Fixed some @params documentation (maintenance)"

Merge "Update documentation for $wgExtensionMessagesFiles"

Merge "Remove #bodyContent { width: 100%; }"

mwjsduck-gen: Don't fail when running from different directory

* Do not directly use $0 in help output, use basename instead
* Do not use paths relative to the currect directory

(Full disclosure: I can't write bash.)

Change-Id: I4c73155cd691a2ddf134724e9df94e6594b78a02

mediawiki.Title: Correct documentation structure for Title.exist

Apparently the way we were using the documentation, with sub-properties
of properties, only worked due to the way that JSDuck v4.x happened to
be implemented, and was unsupported; as of JSDuck v5.x this no longer
works, so instead, change the form used so as to document the property
as a single item, as suggested in response to our complaint:

  https://github.com/senchalabs/jsduck/issues/551

Change-Id: If0c66e6e8a904769ee857c1ee94377dc13000d41

Apply coding conventions for whitespaces

Change-Id: Ibd32177ac1f608fbb480d74a573c20b6487d28c4

Fixed some @params documentation (maintenance)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I727deec35a712de0f0c676cc87dfa661f1ee965b

Update documentation for $wgExtensionMessagesFiles

Follows-up e0043257c02e.

Change-Id: Id8212fca83fde09a89891ead1e0a107ff0d66398

Merge "Add 'mw-anonuserlink' class for anonymous users"

Merge "includes/Linker.php: Added hook for "Media:" links"

Simplify SpecialVersion:listToText()

Don't special case $list only having one item

Unindent else block

Bug: 64069
Change-Id: I171a64147fe00f00ac0523ccf58fea11e800e2fd

Merge "Fixed some @params documentation (tests)"

Fixed some @params documentation (tests)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: Ic8aaf0a93796b97d0fa4617c1f86ff59f4b36131

Localisation updates from https://translatewiki.net.

Change-Id: If20be037a2feffec2a926bc51a66b4f11235bd4f

Support tighter rate limiting for "non-standard" thumbnails

* Also avoid passing some junk params to File::thumbName()

Change-Id: I7e89f72ade47be7fcc7bf33f0afb970183aac936

Merge "mediawiki.toc.test: Cleanup"

Merge "Removed 'Remember my login' preference"

Merge "Remove deprecated function mw.util.toggleToc"

mediawiki.toc.test: Cleanup

Used consistent wording, fixed some typos ("visible" and "hidden" were
swapped in one place, etc.) and simplified the flow.

Change-Id: I7745aa48484fb1a36f8b2654f2a3df326b43c9c4

Remove deprecated function mw.util.toggleToc

* Remove dependency from mediawiki.util to mediawiki.toc.
* Load module mediawiki.toc only when toc is existent.

Gadgets that use the messages "showtoc" or "hidetoc" should explicitly
load the module mediawiki.toc or use their own messages.

Follows-up I3ca2acb70db98d00e3f1b (implements mediawiki.toc).

Change-Id: If0438b7b6f4649434e2b83133d6f583f2f8eff16

Remove FakeTitle

This doesn't seem to be used anywhere anymore and it's an awful class

Change-Id: Ie9047a346e410099c3082725ced83818846e95c2

Merge "Update moment.js to 2.6.0"

Merge "Use ResourceLoader::makeComment to embed page title in wiki modules"

Merge "Skip loading PHP shims for JSON i18n files"

Update moment.js to 2.6.0

From: https://github.com/moment/moment/commit/37808f3f9dec14e37ad2b5cff45c70f63b021d17

Bug: 63873
Change-Id: I471c0e47243b10b2db9367079e7e9c86995d5ef8

Fixed some @params documentation (languages)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.

Change-Id: I7a4dec6a8de96ee21ef34e52bb755f723aa3b0e6

Fixed some @params documentation (skins)

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.
Also added some missing @param.

Change-Id: I3f6affbf6f58ec83d6290cdc15a83d7956ce653a

Merge "Consistently use '@deprecated since <version>'"

Merge "Properly deprecate the SpecialVersionExtensionTypes hook"

grunt: Update watch file patterns

* ".jscs.json" doesn't exist anymore (was renamed to ".jscsrc").
* ".csslintrc" doesn't exist.
* Use "jscs.all" instead of jshint.all as the latter doesn't have
  the jshintignore pattern applied (no need to monitor the deep
  node_modules/ directory structure with lots of file stats)
* Add "jsonlint.all".

Change-Id: Ie9d681b7c4a6424c069f258774be46d44f6386e9

grunt: Add jsonlint

Change-Id: Icace398eef40556a3fd4865cb763043c086da469

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

New changes:
b0a3738 Localisation updates from https://translatewiki.net.
722ad64 Followup 099f26ec: fix $content in FieldsetLayout
0a75f92 Check for left click and disabled state on ButtonedElements

Change-Id: I3efff5b337e7852ab90fd76473cbce88cd16702a

Merge "Use new banana checker as a linter to avoid lacking qqq messages"

Merge "Output info in update.php for skipped updates due to shared tables"

Merge "Fixed broken filebackend test for "describe""

Output info in update.php for skipped updates due to shared tables

Can be kind of confusing for people to figure out why certain
updates didn't happen. I just had an irc conversation with
someone who didn't realize why their db wasn't fully updated.

Change-Id: I56223e70bdac8baaf079c28cbb1f0e20ef0336b8

Merge "HtmlFormatter fixes"

Localisation updates from https://translatewiki.net.

Change-Id: I81beafe71aa8784b349ff5cd8e7f0fee3ab6cfa3

Merge "Revert "Revert "Removed useless JobQueue return values"""

Revert "Revert "Removed useless JobQueue return values""

Made the obvious update to a caller missed in the change.

This reverts commit c76d5a95c1dd1c3086461feae8bc1c0b688bde1e.

Change-Id: I67400ba5b9fc7de16c9f9d5075c488c5e58cea9e

Merge "Update OOjs UI to v0.1.0-pre (c58b498573)"

Merge "Get rid of Title::indexTitle and Title::getIndexTitle()"

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

New changes:
4bf6a2e Buttons return false on mouse down
c0eeb38 Only blur inputs within booklet pages when moving away
099f26e Make fieldset layouts render correctly

Change-Id: I919b9587242d4f40ffc2943d39bd91695a7c07e7