Set ThrottledError's response code to 429

ThrottledError currently returns a 503, which in turn results into
badly-written spambots occasionally flooding our 5xx logs and graphs.

There is no reason, however, for ThrottledError to return a 5xx in the
first place: it's a user-generated error (user hitting a rate limit and
being throttled), not a server error. 5xx error codes in general have
many other implications, such as frontend caches treating this as a
backend failure and potentially retrying the same request, so they are
unsuitable and undesirable for the ThrottledError exception.

RFC 6585 (April 2012, updates: 2616) has added a special 4xx code
specifically for rate-limiting, 429 Too Many Requests. As the
description of that code matches exactly what ThrottledError was meant
for, switch it to using 429 instead.

Note that there is a chance 429 might be mistreated and not showed by
older or badly-written user agents as it's fairly new and not part of
RFC 2616, the original HTTP/1.1 spec. However, the last paragraph of
section 6.1.1 of RFC 2616, specifically covers the issue of UAs &
unknown status codes: it dictates that applications MUST understand the
class of any status code and treat them as the "x00 status code of that
class" (here: 400), MUST NOT be cached, and "SHOULD present to the user
the entity returned with the response, since that entity is likely to
include human-readable information which will explain the unusual
status".

Change-Id: I46335a76096ec800ee8ce5471bacffd41d2dc4f6

HttpStatus: add RFC 6585 status codes

Add newer (Apr 2012) HTTP status codes & their messages to HttpStatus'
getMessage. Adds 428, 429, 431, 511 per RFC 6585, "Additional HTTP
Status Codes".

Change-Id: Ia055880c4b2f3ac43cf5d94dbdd45a3690a58569

Merge "Adding function documentation for Skin::footerLink()"

Merge "Fix documentation of wfDebugLog()"

Merge "Set up node-jscs via Grunt (and pass it)"

Merge "Update formatting for includes/specials/"

Merge "Autoloader incorrect case warnings fixed"

Update OOjs UI to v0.1.0

New changes:
4f4fc2c Localisation updates from https://translatewiki.net.
ea88cce Localisation updates from https://translatewiki.net.
37af34f Add the ability to clear the SearchWidget
1cd4b32 Make PopupWidget clippable.
81f2edc Localisation updates from https://translatewiki.net.
fa6e8e5 Make popup toolgroup labels a little closer to their icons

Change-Id: I63fb1f9edb24a86cc882b204075540427ac4d11b

Set up node-jscs via Grunt (and pass it)

* Set up Grunt via package.json (run `npm install` in mediawiki-core)
* Add grunt task for node-jscs (NEW)
  This is a style checker (as opposed to jshint, which is for
  code quality). There are a few small style-related things that
  JSHint can check (camelcase, onevar etc.) but those are being
  deprecated in JSHint v3, people should use more sophisticated
  tools like node-jscs for this instead. As such this commit
  removes moves of those options from our jshint configuration.
  See: http://jshint.com/blog/jshint-3-plans/
* Add grunt task for jshint
  This will use the same jshint configuration as we use on
  Jenkins but makes it easier to run locally from the command
  line by being part of the same `$ grunt test` task list.

Also:
* Changed jshintignore to use "dir/**"" instead of "/dir" or "dir"
  because the latter is not compatible with Grunt for some reason.
  See also https://github.com/gruntjs/grunt-contrib-jshint/issues/126.

Examples of coding style rules that were being violated that we
can now catch in node-jscs:
* Operator "," should stick to preceding expression
* Missing space after "if" keyword
* Multiple line break
* Empty block (in jquery.textSelection and mediawiki.language)

Bug: 54218
Change-Id: Ib9d7eab9f0d5cea5fb33f0b9f82e5554897fdfe0

Localisation updates from https://translatewiki.net.

Change-Id: Icad3f57cf182107ce7e813297f00473e75e4ad6f

Adding function documentation for Skin::footerLink()

Change-Id: Ib38173ed7bdc434274ff17dfd8c85e5ec95bfb29

Followup af4085a: reset $prev interwiki to null in between result sets

Change-Id: Ie2ba7b061e532e0ab749b0519a7e72bc018a1b5a

Merge "Automatically add a new line at the end of wfLogDBError()"

Allow interwiki searches to return arrays of interwiki results

This allows them to group by interwiki in more sane ways.
Also remove no longer used $terms paramter

Change-Id: I5d420595edfd35a1ce85662d2722ebbe2d357ce3

Improve docs for $wgExtensionCredits (and add 'license-name')

Follows-up 357eb3d488b9d8c which implemented 'license-name'.

Change-Id: I2b2c17965230558fd1650e3db92abafb082737d1

Autoloader incorrect case warnings fixed

Change-Id: I610de60b49840a26ddb7ceeb14264aec77dc8b96

Add getRobotPolicy()

Bug: 57764
Change-Id: I5d53b9b36b8ec5263bf7ebb13c7ef8c102c1ce68

Test against php 5.6 on travis

Change-Id: I7401f1d059437eadd3712c012a93a6a5b6ac4403

Merge "Use correct Creative Commons license abbreviation"

Merge "Removing Wikitravel from the default interwiki list"

Use correct Creative Commons license abbreviation

Change-Id: I7df989de9dca2ee056030ff6f5830d5f0124185e

Tweaked "latest" handling of filebackend stat entries

* Let "latest" stat entries override non "latest" so
  that future getFileStat() calls with the "latest" flag
  can actually have a cache hit.

Change-Id: I1e9391039537d608b89773b4d51575e3b364a751

Merge "[Regression] Delete autoblocks when removing autoblock flag"

Merge "DatabaseMysqlBase: Remove broken check for Percona Server"

Merge "Remove double wfDebug() call in Database.php"

Merge "Add support for specifying headers in FileRepo::quickImport"

Merge "Add Special:ListDuplicatedFiles expensive query special page."

Merge "Add default implementation of getLength to base MediaHandler class"

Add Special:ListDuplicatedFiles expensive query special page.

I saw some comments recently on commons suggesting that
better ways are needed to manage duplicate files (There are tools
for if a specific file is a duplicate, but no backlog of
outstanding duplicate files).

This seems like a fairly easy first step in that direction.

Wasn't sure if this should be an image gallery type
query page, or just a list. I think in this case a plain
list is more useful.

Change-Id: Ibe4b9da71ca6451ec4e6b0050feaf3ca70e1b888

Add support for specifying headers in FileRepo::quickImport

Previously only content-disposition headers were allowed.

This is needed so that TimedMediaHandler can put
X-Content-Duration headers on ogg transcodes.

Bug: 62928
Change-Id: Ic053be63b3edf348da95a88fc494fcf334430265

Add default implementation of getLength to base MediaHandler class

It was being called in File::getLength, but was only implemented
in a subclass, that's not even in core. Any method of a handler
class called from a method in File class should work on any
subclass of MediaHandler.

Change-Id: I94eda90ec3dd64b6ff2e3a5732aa539366cec521

Localisation updates from https://translatewiki.net.

Change-Id: I30f333c40b241dfeefea11e617f3f80c70a84bb1

Merge "Button group adjustments"

[Regression] Delete autoblocks when removing autoblock flag

Since I5403f6e1d7f8f07273cd5da8068b928fdddcdfc9 on reblock the block
will be updated and not deleted/inserted and that update did not trigger
a delete of existing autoblocks.

Bug: 62748
Change-Id: I50384a41d819689944907e653e2495391253831b

Merge "Misc fixes to UploadSourceField"

Update formatting for includes/specials/

Change-Id: I478dedd0b692eae002a0ca9b0ec15cb2c357411f

Link to bugzilla.wikimedia.org in README for consistency

Change-Id: I27a97cb89ba1981c931fe5fe2ae1422c832c4405

Remove double wfDebug() call in Database.php

Fix for I12adfb4fcb (54664be).

Change-Id: I6acfd35aae18af9d6e5ae29e5554ca7d04342af5

Automatically add a new line at the end of wfLogDBError()

I found two calls to wfLogDBError() that do not add a new line
at the end of the message. So instead of adding them to that
entries, I changed wfLogDBError() to automatically put it on
icoming messages; as for wfDebugLog().

Change-Id: Id014b5827a0aeef6873ebf08d78f0a3d7581d63b

Fix documentation of wfDebugLog()

- $public parameter does not exist anymore
- Put the type before the parameter name

Change-Id: I0f3a893803ab5466570ef45398f797e4714d9d37

Misc fixes to UploadSourceField

Follow-up I7c15fd4250 (00ca365).

- Added 'radio-id' configuration option to set the id of the radio button,
  and changed Special:Upload to use it. Had to keep the old way to set it
  since this class is used by some extensions.
- Always set the "for" attribute of the label to the field and the not the
  radio, since the text it contains does not describes the radio button.

Change-Id: I524f8743fd03546024f8c97852a2bcb3c23c6019

Merge "AutoLoader: Remove "Class ... not found" debug message"

Merge "Remove some unneeded local vars from EditPage.php"

Localisation updates from https://translatewiki.net.

Change-Id: I93194841aebc13bb2e6e147ae6170a3acf2186e7

Merge "Give name for oojs-ui i18n files"

Add a few @since to mediawiki.api

Change-Id: I0717d1171a7179beb4cee6a922d682864e600e9c

Give name for oojs-ui i18n files

Even though this shouldn't cause problems with array_merge etc.
everything else has a name, so why shouldn't this.

Change-Id: Id40d9036c2f3cf89343bcfd14d37a14d4c306d01

DatabaseMysqlBase: Remove broken check for Percona Server

Percona Server does not include its name in its version suffix.
It would be possible to check for precompiled versions of the
product using the version_comment variable, though not third-
party builds from its source code (which are merely indicated
as "Source distribution").

Given that even Percona itself does not distinguish between
Percona Server and MySQL in their version of the mysql command-
line client, I have chosen simply to remove the broken check.

I have also made the check for MariaDB consistent with that
in its version of mysql (by also checking for the old suffix
"-maria-").

Follows-up 0eab1ace67e7.

Change-Id: I62c3949b775a38f28f09568ebb28af2adb9be69b

Swap wfProfileIn/Out calls for ProfileSection

Change-Id: Iece1532241dda4297dd4ca2c93287c13462eb2e9

Merge "Avoid showing crazy staleness times at ActiveUsers"

Merge "Remove completely useless implementation of getInfo()"

MWException: Expand {{SITENAME}} in pagetitle with Message::text()

Bug: 58447
Change-Id: I5f4fcb6d6e044f708cd6adb1bc7312d592fc74dd

Localisation updates from https://translatewiki.net.

Change-Id: I18970accbce3b383e4ad89df2b5cd0d638861e26

Update OOjs UI to v0.1.0-pre (3b434d5388)

New changes:
076b85e Split autoAdd into catchall and group specific settings

Change-Id: Idb99c0af20f966ead553c7a7eded421ad8b44382

Removing Wikitravel from the default interwiki list

It has already been removed from the WMF list.

Note that this will have no effect on existing wikis, only new wikis.

Change-Id: I201bc28ab17fdae3b7c5d46f1c65d70e03380ca9

AutoLoader: Remove "Class ... not found" debug message

Per existing FIXME comment: "This is not very polite. Assume we
do not manage the class."

Reverts the remaining portion of r44296 / 2f584f68a95a while
clarifying the "Give up" comment.

Change-Id: I4471f9376cae6b3c3e8df99e960770863ebf37ca

Merge "Updating link protocols for WMF wikis in the interwiki map"

Merge "Adding Wikivoyage to the default interwiki map"

Merge "ResourceLoaderStartUpModule: Improve comment about modifiedTime hack"

Merge "ResourceLoaderStartUpModule: Remove no-op "delete isCompatible""

Merge "Fix possible error list of action=revisiondelete"

Exclude prop=uploadwarning from allimages and stashimageinfo

Was added with I4a0af8986f924cd127a73828e72da6998f28536c,
but looks only useful on prop=imageinfo

Change-Id: I59c5f11f83be7e59f317686ab7fa16ad6fda008b

Make Special:Whatlinkshere.php transcludable

This commit makes Special:Whatlinkshere transcludable by extending
SpecialWhatLinksHere with IncludableSpecialPage. It also adds a condition
in the showIndirectLinks function to avoid displaying the form when the
page is being transcluded.

Bug: 35486
Change-Id: I49d6be57a9a780f9f959cb4356c5ac026db81775

Updating link protocols for WMF wikis in the interwiki map

Change-Id: I18d281104c1c670c3b2dce6b2970930aba26bef0

Adding Wikivoyage to the default interwiki map

Note that this has already been added to the WMF interwiki map, so
it will have no effect on WMF wikis, only 3rd party wikis.

I'll update the link protocols in a follow-up commit.

Change-Id: I78fbd3ec32c02e2b080c3ee64c6bb36218930a84

Merge "Add link to PD help translatable pages from sidebar"

Merge "Get rid of Internet Explorer for Mac fixes in Monobook's main.css."

Fix possible error list of action=revisiondelete

Bug: 62853
Change-Id: I0efec9e8a6cf761def7673011bcc22a9f11111a9

Remove some unneeded local vars from EditPage.php

Looks like a left over from refactoring in r31475

Change-Id: I6ffd2e1db8a842bebc37ce67e6265714258e79f9

Merge "Fixed spacing"

Fixed spacing

- Added spaces after if/foreach/catch
- Added new line before end of file
- Added or removed spaces before/after parenthesis, comma
- Added spaces around string concat

Change-Id: I0590070f1b3542108e242730e8d9a3ba9831e94f

Localisation updates from https://translatewiki.net.

Change-Id: Idf379fd907b86b191bf863d4dded00e5d9c94f35

Merge "mediawiki.page.watch.ajax: Fail early if updateWatchLink is called wrong"

mediawiki.page.watch.ajax: Fail early if updateWatchLink is called wrong

If the exposed function is called from a gadget or user script but
no watch/unwatch link was found the function fails even if the
first parameter is a valid jQuery object as required by the
documentation. $link.attr(...) returns null and null.match(...)
fails.

A very simple example why this can happen is as follows:
mw.page.watch.updateWatchLink( $( '#ca-watch a' ), 'watch', 'loading' );
mw.page.watch.updateWatchLink( $( '#ca-watch a.loading' ), 'unwatch' );
(starts the spinning loading animation and tries to stop it
afterwards but may fail if something went wrong, e.g. the user
clicked the star).

The action parameter needs to be checked because it is used to
build an ID and a message key. Bad values turn the page in an
unrecoverable state (the watch link gets an empty action=, the
label turns into something <undefined> and no script can recover
that broken state since the ID turned into whatever). While such a
check is not necesarry in most cases it is here, because the
function is exposed.

Change-Id: I6ee9a7ee6b7c0fc7a5444674afd1ed6f8cacc858

Merge "Allow setting default thumb size in parser tests"

Allow setting default thumb size in parser tests

The mediawiki default thumb size is 180px.  The Parsoid default thumb
size is 220px, to match the default thumb size for most WMF wikipedias
(see https://bugzilla.wikimedia.org/show_bug.cgi?id=43336).

This discrepancy leads to inconsistent image-related test output.
Allow a test to set an explicit default thumb size with the
'thumbsize' option so that it is possible to write consistent tests.

Change-Id: Ib764d1f1660a50caaf8f0ff245822d1d1a1d264e

Merge "Update the documentation at the top of parserTests.txt"

Merge "Fix wording of protect-cascadeon"

Change URLs to mediawiki.org in comments to HTTPS

These are only documentation fixes
http://www.mediawiki.org --> https://www.mediawiki.org

Change-Id: I62ad42be1a3aac410cc53e98ce79389ceddd8988

Fix wording of protect-cascadeon

In some cases, users who are not capable of changing the protection level
will see the protect-cascadeon message. Update it to not imply that they
are able to change the protection level.

Change-Id: I102aea94f4b5ad29fa928e865d2f20ec7def4b30

Update the installer to use JSON i18n

Change-Id: Iedcacbd26385c331cd78b92f7ec55a4ef9c015ef

Merge "mediawiki.inspect: add method for grepping loaded modules"

ResourceLoaderStartUpModule: Improve comment about modifiedTime hack

* Re-ordering code to bring the modifiedTime hack and the
  loop it serves closer together.
* Separating the fact that it needs a value and the mtime of
  startup.js since that is just one of the three factors we use.
  This way it's clearer that the startup.js mtime is not just a
  bogus value, but not more or less important than wgCacheEpoch
  and modules mtime either.
* Remove duplicate '/* Methods */' comment, we already have this
  marker a few methods higher up.

Change-Id: Id3a07f02566c0f04b612b81f8353f70fa4ab3977

ResourceLoaderStartUpModule: Remove no-op "delete isCompatible"

Due to the way this function being defined as a function or var
declaration instead of a function expression assigned to a property
it can't be deleted.

JavaScript doesn't throw an error when deletion is not permitted
though, the operator returns false instead.

> delete isCompatible;
  false

We already removed 'delete startUp' from mediawiki.js in favour
of startUp = undefined; (r107402, r74325).

Change-Id: I7aa02e3f4deb3a4f00177b70978bfcb83c80988a

mediawiki.inspect: add method for grepping loaded modules

Example usage:

>>> mw.inspect.grep(':hover .settings-text')
["ext.uls.init"]
>>> mw.inspect.grep('options.expire')
["jquery.cookie", "mediawiki.user"]

Use cases:
* You're debugging a layout bug and want to trace a CSS rule to the module
  which inserts it.
* A JavaScript error contains a distinct and plausibly unique substring (such
  as a variable name), and you want to identify the module that is causing the
  error.

You can re-load the page with debug=1, but you still need to decipher load.php
URLs for non-file modules, and you might not be able to reproduce the issue in
debug mode. Alternately, you can grep a full clone of the production branch
that the wiki is running (assuming you have one handy), but that won't help you
if the module is a gadget.

Change-Id: Ie5f0e7d1c7022f8d399e895f157db8acefc8abee

Update OOjs UI to v0.1.0-pre (0f50cdc46c)

New changes:
6c014fd Revert "Use super to call parent methods"

Change-Id: Iccbb2a1b86af8dc10568855eecb7d31e67b86756

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

Merge "Preferences.php: avoid "Undefined index" if key 'realname' don't exist in $formdata"

Update OOjs UI to v0.1.0-pre (53f2410336)

New changes:
16a7989 Localisation updates from https://translatewiki.net.
c5339e7 Localisation updates from https://translatewiki.net.
48b5403 Fix documentation of toolFactory
6fc6d7b Use super to call parent constructor
b81b5f1 demo: Add ToggleSwitchWidget (disabled, checked)
a72a6e3 Use super to call parent methods
53f2410 Localisation updates from https://translatewiki.net.

Change-Id: I4aeb49dc9a139936c48f1cd790da9f39dc39c1c6

Add link to PD help translatable pages from sidebar

Reuses the existing helppage default message and
adds it to ignored messages as Special:MyLanguage
is used and translations have to be removed:
this does not bother customised local links.

Now using HTTPS URL to mediawiki.org per Brion
on bug 54835. This slightly increases the footprint
of the change but should keep everything functionally
equivalent. The sidebar is not bothered by full URLs,
except in self-defeating tests which are also fixed here.

Bug: 53887
Change-Id: I999b97729536dbab4a3a5efd8d6f86527f031948

Localisation updates from https://translatewiki.net.

Change-Id: If3a8ff71e98aab98c39991147c715ec388ef91aa

Remove completely useless implementation of getInfo()

Embedding random strings in Special:Search's comments doesn't
help debugging and just increases page size.

Bug: 62768
Change-Id: I51270aa3f2cba921841e2d8ebbd4fa665542f8a9

Break up SearchEngine.php into a couple of other files

Change-Id: I753eec1d0cf78f2ea08897c7fd3b83f2c88be18d

Merge "Use square brackets instead of curly braces for char of a string"

Remove bogus entry

Change-Id: I7955931c054e3970cd7bfc1e0df275d9d4bf1880

Use square brackets instead of curly braces for char of a string

The use of curly braces is deprecated in php 4.0, so changing it to
square brackets

Change-Id: Ia820cea34d2bd92fe1c3bc8fd773942a425ec90a

Update the documentation at the top of parserTests.txt

The meaning of the php/parsoid options changed slightly with
change Ie4e68960ca7c352af495ebb59ba83488935a44c4.  Update the
documentation comment at the top of the parser test file to match.

Change-Id: If0caac128704a15b824ccbcfbfb3f49812510f1d

Get rid of Internet Explorer for Mac fixes in Monobook's main.css.

Wikipedia says that IEMac's initial release was 17 years ago and final
release 10 years ago.

IEMacFixes.css, which was basically unmaintained after r2909 (which was
done to ensure HTML validation compliance), was deleted in r61787.

Moved some rules which apparently still apply to IE6 to IE6Fixes.css, as
per code review.

These fixes were originally added in the following revisions (incomplete
list):
* r2909
* r3165
* r3181
* r3658
* r3765

Change-Id: I2352be07964060bc5ed7b3bd82007755b3168654

shared.css: Canonicalize and simplify background image rules

* Use the same form as in the LESS mixin:
  * Don't embed the PNG version
  * Do embed the SVG version
  * Use the -webkit- form too
* Add a descriptive comment
* Don't use quotes inside url() syntax

Change-Id: I044a04a321f0dbba7687c2ce1681028ed5cf0ea4

Make use of strong consistency with Ceph RGW

Change-Id: Ic3b764363e91c3279d3dc017ddce630dae26bec5