Merge "mediawiki.api: Refactor getToken and postWithToken methods"

Merge "ApiFeedRecentChanges: Validate param target"

ApiFeedRecentChanges: Validate param target

Change-Id: I6933177d47c25d48b9b55d479afa4dedcc64299d

Merge "mediawiki.html: Add missing @return documentation"

mediawiki.html: Add missing @return documentation

Change-Id: I63f12ee44aace6cf4f7762dd67244648b95d7301

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

Adapt signup form numbers to new Vector typography

The .mw-number-text CSS was less specific than Vector's typography
styling of 'div#content h3'.  Match Vector's serif font stack.

Bug: 63450
Change-Id: I23041c356ad4356dcf085479721cb950a89b33d0

Update OOjs to v1.0.9

Code:
 https://oojs.github.io/releases/oojs-1.0.9.js

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.0.9/History.md

Change-Id: Ic1fea7dd3a07af32a29511191c45da90bc306b17

mediawiki.api: Refactor getToken and postWithToken methods

* getToken now keeps promise objects around so that it can:
  1) Re-use already retrieved tokens instead of always making
     a new request (this was previously done locally inside
     one of the callers, namely #postWithToken).
  2) By storing promises instead of the tokens, we can re-use
     it asynchronously. While it is unlikely that one module
     calls getToken twice in a short period amount of time,
     it is quite likely that two separate modules (both using
     mediawiki.api) request similar tokens from the API at the
     same time (e.g. at page load).

  This cache has to be additionally keyed by API endpoint since
  mw.Api is a generic class that can be (and in practice, is)
  instantied for foreign wiki as well.

* Initialise the cache with pre-resolved promises for values
  from user.tokens from the local wiki (embedded in the page
  output, no point in requesting these again from the API).

* postWithToken no longer has its own token cache, it now
  unconditionally uses getToken.

* postWithToken already took care of clearing the token cache when
  the server responds with a 'badtoken' error. However, it then
  changed the request to delete the 'token' query and try again
  without any token. That seems rather silly. Change this to
  instead have getToken fetch a fresh one and re-submit with that.

* Add unit tests.

Bug: 34733
Change-Id: I5c25ae5ea4bf3336899bc60fd94ec3b0948050e1

Better memory reporting when running in HHVM

Change-Id: If06b7b3ff4dc7ba8b873361f7878f8abb718f571

Visualize profile output in debug toolbar

This adds a new pane for the debug toolbar labeled Profiler.  It groups
together events sent to wfProfileIn/Out and draws a timeline giving a
visual representation of what the page was doing at a particular point
in time. Hovering over a timeline brings up a flyout about the contained
events.

Depends on I08854d80f57286129f19cb14566f84ed3c7fa21c for proper positioning
of the flyout.

Change-Id: I23a0ce4137abcd74ad87967fb263214c08cb9685

Merge "Made BagOStuff fail fast in cas/lock on certain errors"

Merge "mediawiki.ui: Do not depend on divs being present inside .mw-ui-vform"

Merge "Fix language statistics after messages have been moved to JSON"

Respect sizing from user-supplied class

A user-supplied class provided to tipsy that affects the
height or width of the flyout breaks tipsys positioning.
Adjust tipsy to apply the user-supplied class earlier so
its css rules are taken into account during positioning.

Also clean up trailing whitespace.

Change-Id: I08854d80f57286129f19cb14566f84ed3c7fa21c

Merge "Cleaning up function definitions in Language.php"

Cleaning up function definitions in Language.php

Some of these were confusing or incorrect.

Change-Id: I2dd02669a7cf9d8b92e2d5c1b86ab8c5e0cbf6bd

Re-organized PoolCounter files

* No actual changes to the classes

Change-Id: I4e35a2b097e4bc72315688d0390d3f5c4c1ca4ed

Fix language statistics after messages have been moved to JSON

Statistics are now showing always no messages since languages.inc
does not manage to load the JSON messages.

Had to make LocalisationCache::readJSONFile() public so that it can
be used in  languages.inc; since all methods from LocalisationCache
return localisations merged with fallbacks, which is not what we
want here.

Fix for I918cfdc46c (0dd91d5).

Change-Id: Ib52287db618b9d072e847130070d165a3e7ae44b

Merge "Adding debugging for bug 62241"

Rename $p to $parts in Special:Search

Change-Id: I794fca2d24fcebb59067d5e619b68d86cbaa2a8f

Fix variable naming in SpecialSearch, $t => $title

Change-Id: I70ec1e147f99a73bc0b12511669ab5a799018695

Rename $t to $title in SpecialSearch

$title is a bit more informative than $t

Change-Id: I66deeb6672dc45161e0fcc30988bcc1efcb93f56

i18n: Fix the parser function name

- Fix the parser function name "#ifexist"
- Put the parser function in code tags

Spotted by Siebrand
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Expensive-parserfunction-category-desc/en

Change-Id: If29585ceda7f409c668804ff13c821b4eece95fd

Add punctuation to "pool-servererror"

Follow-up on eb03cf3f7dda4.

Change-Id: Iee1fd4ca9f4baad0d03998dbc1ea1e24f2210ba7

Merge "Put LESS test back into extension suite in XML"

Update OOjs UI to v0.1.0-pre (8986c46d35)

New changes:
735cebc Filter calls to selectItem and highlightItem on SelectWidget
897b30f Return null in Element#getDocument
b5ede88 Add Element#isElementAttached
68df19f Disable tool groups when all its tools are disabled
1aaa507 Grunt-watch less files
e4dd7b2 build: Update devDependencies
64d6c32 Localisation updates from https://translatewiki.net.
b5c6db5 Localisation updates from https://translatewiki.net.
b9460cb apex/button: Reset border-color of disabled button when not hovered
32c4799 Localisation updates from https://translatewiki.net.
674165e Enable jscs for OOjs UI repo
d98b338 grunt: Add messageDocumentation task to ensure each 'en' message has a 'qqq'
d421dd6 demos/tools: Indentation fix

Change-Id: Ic9b7ff01fe066eb5cba6a0fd0fc515ff3c62b66a

Added a Redis pool counter class

* This should be easier to set up for typical installs

Change-Id: Icb4a7481b944fa0818c4635e3edbe12d08af9924

vector: Remove redundant margin-top reset

There is nothing setting this to anything else. The only other
preceeding relevant rule inherited before this is "h1, h2, h3 .."
which sets "margin: 0;" already.

Change-Id: If6531221891ae3c7eb792c5bdc930ddaaf613f52

Put LESS test back into extension suite in XML

Followup to 13f3e21e9c7ddcef0d59982aeb3bb4b9e240e5f4

Change-Id: I1a15a0d8aba05b774f072673787b712f6ba97fcd

Migrate translations for core to JSON format

Largest patch set in the history of MediaWiki!

Change-Id: I918cfdc46c099017ca3622d1bcbbdd30c4fd00f6

Preparations for migrating core to use JSON based i18n

LocalisationCache and Language have to take the JSON files into account
in deciding if a language is present or not.

Standardizing language validity checking with isSupportedLanguage
and isValidBuiltInCode.

Co-Authored-By: Niklas Laxström <niklas.laxstrom@gmail.com>
Co-Authored-By: Siebrand Mazeland <siebrand@kitano.nl>
Change-Id: I35bbb3a7a145fc48d14fff620407dff5ecfdd4fc

Made BagOStuff fail fast in cas/lock on certain errors

* This avoids wasting time waiting on down servers

bug: 63058
Change-Id: Ia78a2036ffa7a20102ec9e8bf941cacb7fa63435

Send profiled items under the correct name

Fix for Ie8481a2e13 (811a084). The "-total" item
was send under the name "close" since it's ended
by close().

Bug: 63347
Change-Id: Iad91f1d12303e5cfb92c111f79c0ce5a8187f53b

Merge "Include parsed revision ID in parser cache"

New Hook rc/watchlist hook ChangesListInitRows

An extension hooking into recent changes may need to load additional
data that did not fit into the recentchanges table.  This hook gives
extensions an opportunity to see the full result prior to rendering
and batch load where approprite rather than loading piecemeal during
the render process.

This is implemented as an optional method called by the ChangesList
consumer, since the ChangesList never sees the full result set. Hook
implementers must be able to work regardless of the hook being called,
they just have the oportunity to be more efficient when it is called.

Change-Id: If74ae600ffba949364dd381dd3d466cbbaa27286

Include parsed revision ID in parser cache

One theory for what's behind bug 46014 is that the vandal submits the
edit, then someone (maybe the vandal) gets into the branch of
Article::view that uses PoolWorkArticleView, then ClueBot comes along
and reverts before the PoolWorkArticleView actually executes. Once that
PoolWorkArticleView actually does execute, it overwrites the parser
cache entry from ClueBot's revert with the one from the old edit.

To detect this sort of thing, let's include the revision id in the
parser cache entry and consider it expired if that doesn't match. Which
makes sense to do anyway.

And for good measure, let's have PoolWorkArticleView not save to the
parser cache if !$isCurrent.

Bug: 46014
Change-Id: Ifcc4d2f67f3b77f990eb2fa45417a25bd6c7b790

Merge "i18n: add "Special:MyLanguage/""

Merge "Fix order of expected, actual in some tests"

Fix order of expected, actual in some tests

Change-Id: Iabd431517586c06d411a0894e5d3621a4c4ddf05

i18n: add "Special:MyLanguage/"

Change-Id: I791399e1a9bdff0bc672fc6ae69f4c2374690636

Add newline at the end of converted json files

1) Some text editors add missing newline at the end of the file
automatically.
2) It's generally a nice thing to do to avoid messing up display
for example when cat'ing a file

Change-Id: Icf636721c4edcf4ae706c9a346fe36233928cae8

Speed up LocalFile locking behavior

* Lowered the lock() timeout
* Wrap the destination DB write with memcached locks for move
  ops since it is easier to control that lock timeout than the
  DB lock timeouts.
* Cleanup locks in the destructor (e.g. on exceptions). Other
  process can pile up more if locks have to expire.

bug: 63058
Change-Id: I38b28d81ec96daa80ece2354db284a614289ba0b

Adjust debug log for caches being used

I always found out the debug message listing the cache being in use to
be slightly confusing:

    MemcachedPeclBagOStuff[main] MemcachedPeclBagOStuff[message]
    MemcachedPhpBagOStuff[parser]

Swap the class name and identifier to output:

    main: MemcachedPeclBagOStuff, message: MemcachedPeclBagOStuff,
    parser: MemcachedPhpBagOStuff

Change-Id: If069dec70eb31ed2dafe0e88ec4d4653005ff7e3

Merge "Add a LESS test suite"

Add a LESS test suite

Add the LessFileCompilationTest test case class, which represents the
validation of a LESS file by compilation.

Add the LessTestSuite test suite, which tests all LESS files registered
with the ResourceLoader, and use this to rewrite the checkLess.php
maintenance script.

Bug: 54665
Change-Id: Iedb8dc31e4817d8b4e40b655cf9b8fb092979e90

Merge "mediawiki.log: Fix unknown method "apply" error in IE9 and below"

mediawiki.log: Fix unknown method "apply" error in IE9 and below

In IE < 10, the console.log function is implemented as a non-standard
host function (not inheriting the regular Function prototype) which
doesn't provide an "apply" method.

Change-Id: I6ecf933589ec430ff4d1748a5f3c511b692e3b9a

Adding debugging for bug 62241

In the event of a bad filename warning record the relavent variables
so that we can figure out what is causing the failure.

See also change I532f8ee7c7455e43862b0204a7fff23d6792412e

Bug: 62241
Change-Id: If5d8b9e7f34c0f9bd09684d7dada8fafd7f5f7c2

Merge "MWException: Fix sitename appearing twice in <title>"

Localisation updates from https://translatewiki.net.

Change-Id: I54a8e52634c1e36fadd7a84f236747765fc98684

MWException: Fix sitename appearing twice in <title>

The MWException::getPageTitle method is wrapping the page title
in the 'pagetitle' ($1 - sitename) message, but OutputPage does
this as well when outputing <title>.

OutputPage::prepareErrorPage takes the first parameter as page
title (unwrapped, appears in <h1>,) and the second optional one
is for <title>, which (if omitted, like before) results in
pagetitle being applied by OutputPage.

Now that we specify them separately, this also removes the
sitename from the <h1> on the page, which always looked weird.

It also fixes DBQueryError (extends MWException) where previously
no sitename was added, it just returns 'databaseerror' from
its DBQueryError::getPageTitle() method.

Change-Id: I85d65d89e72b4497a050f440524ff6f959518b88

Merge "Adjust margin between h2 and special page table element"

Adjust margin between h2 and special page table element

With the recent typography update, this margin is much too small.

.mw-specialpages-table applies "margin-top: -1em;" style which
doesn't fit with the vector styles. So this provides an override
for vector.

Bug: 63312
Change-Id: I18a711802a18b5976cf749d76138a1a6f59e0a4e

Merge "API: Allow for format modules that cannot handle errors"

Merge "Expose thumbnail file to extensions"

Merge "Improve handling of uncommitted DB txns with "uncaught" exceptions"

Merge "Provide RequestContext from fixDoubleRedirects"

Merge "Send the profiler output to the 'profileoutput' log group"

Merge "Add missing line breaks to wfDebug() calls"

Merge "Update IRCLineURL hook to include RecentChange object"

Follow-up I7d4bb90: Message tweak + add code comment

When I was saying it in my head, "No description available."
sounded better to me than "Description not available", although
perhaps that's just me.

Add a code comment to Parser::addTrackingCategory telling people
to register their tracking categories.

Change-Id: I17bb0cf4b3118dda8647e0607588685c2b5cdb86

Merge "Implement listing for tracking categories"

Localisation updates from https://translatewiki.net.

Change-Id: Ifc65d7c3844bfaabd97c803b2c72dc8a49cdcc65

Merge "vector: Style .mw-editsection-like in addition to .mw-editsection"

Localisation updates from https://translatewiki.net.

Change-Id: I6ac7ce50b6204e9e9d63695c9532d33796d4cc7d

Merge "commonElements: Move sup, sub styling from Vector's CSS to common styles"

Send the localisation store debug message to the "caches" log group

Rationale is the same as for Ifb6dc2666f (96b04ce): ability to
separate it form the main log group, as it always has a predicitible
value.

Change-Id: I0ac5baf5ab8fbe4d2025ccf6439b883dc82e9d5b

commonElements: Move sup, sub styling from Vector's CSS to common styles

This really should be a basic style, not limited to one skin. All
Wikimedia wikis which used this hack seem to have it on their
common.css, not vector.css.

Follow-up to Ic5ba8363.

Bug: 49965
Change-Id: Ie98f8b57a71b703fbba19c201c13edef3f6adc55

vector: Style .mw-editsection-like in addition to .mw-editsection

Follow-up to Ic5ba8363.

Change-Id: Ia60281595b5903474b660b776bbbe8f2c3819d3f

mediawiki.ui: Do not depend on divs being present inside .mw-ui-vform

This breaks forms that try to use semantic elements like <section>,
e.g. the user creation form here in core.

Bug: 63233
Change-Id: Icb4a6d1d74490808b2fd1777d7dbe812e66bd206

Add missing line breaks to wfDebug() calls

Also removed true as second parameter to it from CloneDatabase.php
since it is the default value of that parameter.

Change-Id: I727ebae2bd4df0e26019985ce8c7ce73381c5642

Implement listing for tracking categories

Special page to implement list of Tracking Categories. Global
$wgTrackingCategories added containing list of tracking categories

Bug: 60333
Change-Id: I7d4bb90622a6bae60845942ef93cfe64f229d2d2

i18n: Give grep a chance to find the usages

Change-Id: Iac0d7cba8633fedd420c6b8e21999e94b4d438a2

Merge "mediawiki.api.category: Deprecate getCategories() 'async' parameter"

Merge "Add getLanguage method to Message"

Merge "Allow to optionally set language in Message constructor"

Merge "jquery.makeCollapsible.test: Use fake timers to advance animations"

Merge "Set ThrottledError's response code to 429"

Merge "jquery.color.test: Use fake timers"

mediawiki.api.category: Deprecate getCategories() 'async' parameter

One should never set it to anything other than the default value.

Change-Id: I6d191916f14aa29a04b2190debe9901032629938

mediawiki.api: Documentation and typo cleanup

Also changed some whitespace.

Change-Id: I307c516f41258c06c0bff4ddb8dc9c1aaca44462

Temporarily remove tests added in I8eef5a165

Although the tests themselves pass, their introduction to the codebase pushed
the total number of tests past some threshold at which Jenkins starts
segfaulting. So remove the tests as a temporary workaround, and revert this
patch once the bug with Wikimedia's Jenkins deployment (bug 43972) is resolved.

Change-Id: I5722faf0e4216e1c1f98ecc1f2dba744b240b7d3

Merge "Add visibility to SpecialCategories methods"

Merge changes Ib1ace2be,Iba1dab27,Ife44ea47

* changes:
  Localisation updates from https://translatewiki.net.
  Localisation updates from https://translatewiki.net.
  Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Add support for Algerian Spoken Arabic (arq).

Change-Id: Ib1ace2be1748772acc02a41fecf82d0a7e6a6011

resourceloader: Separate gathering of registry data from mw.loader call

This will make it easier to debug this code and generally to interact
with the registry data before it gets sent out.

Change-Id: I01808ce5d6a0ac6f7a15719bdd2aa90908cb2fbd

Localisation updates from https://translatewiki.net.

Add support for Riograndenser Hunsrückisch (hrx)

Change-Id: Iba1dab2793a371b99b5a1a99fd472ead342d0110

Localisation updates from https://translatewiki.net.

Change-Id: Ife44ea47332196ed798dff5e015709267b9d8d47

Add visibility to SpecialCategories methods

Change-Id: I57c3bb76f7edc23036805424c492722f0ed77010

Merge "Introducing TitleValue"

Merge "HttpStatus: add RFC 6585 status codes"

Introducing TitleValue

TitleValue is intended to be a light weight alternative to the Title
class. Any operations on TitleValue are implemented in separate
service classes, like TitleFormatter and PageLinkRenderer. See
<https://www.mediawiki.org/wiki/Requests_for_comment/TitleValue>
for more information.

This change updates SpecialCategories and SpecialLinkSearch to use
TitleValue instead of Title, to demonstrate how TitleValue would be
used, and how the necessary services can be injected and applied.

The intention is to improve testability and reusability; these
advantages will however only become apparent with further refactoring
of the respective special pages. This will be done in follow-up
changes.

More work will be needed to migrate essential functionality from
Title and Linker classes into the respective service classes,
MediaWikiTitleCodec and MediaWikiPageLinkRenderer.

Change-Id: I8eef5a165de4ffcacfbc4911fdacdb15d502fff4

Merge "mediawiki.util: Fix jsduck syntax and minor clean up"

Add getLanguage method to Message

This allows to more easily test code where language is set,
Message::inLanguage() is called, etc.

Change-Id: If8f9697480f5d084f755990fdc0f2a1e18f056bc

Allow to optionally set language in Message constructor

This helps with testability to not have to rely on $wgLang
and setting globals in tests.

This also provides convenience, so one does not necessarily
need to call Message::inLanguage() if language is known
at time of constructing the message object.

Also added tests to cover this change.

Change-Id: I14ee98972c7be954e04398ece9e6103f96ab60dd

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

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

Expose thumbnail file to extensions

This is needed by Media Viewer in order to display a placeholder
image as soon as possible

See https://gerrit.wikimedia.org/r/#/c/121613/

Change-Id: I3dfa80b02073984dc2d3d7784cb3744b7e4e2cc8
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/293