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

Merge "OutputPage: addParserOutput*() family doesn't need to take a reference"

OutputPage: addParserOutput*() family doesn't need to take a reference

We never assign to the variable, only call some (mutating) methods on
the object. With PHP 5 we don't need to pass this by reference.

The functions that evolved into this family were originally added in
r12337, back then we probably still ran on PHP 4 or something.

Change-Id: Ib4ab141ca6d803f9df0351b1f65c7e9955c37d57

Merge "Be consistent about 'TablePager' CSS class usage"

Merge "Split includes/Pager.php"

Merge "Remove unused styles for .imagelist"

Localisation updates from https://translatewiki.net.

Change-Id: I0d1cdf104964c0923cfea01e3c7e18838ed47046

Be consistent about 'TablePager' CSS class usage

* Do not use it when not actually using the TablePager
* Always use it when actually using the TablePager
* Use parent::getTableClass() rather than hardcoding it every time
* Place it before other classes to allow overriding

Change-Id: I042b65be64e2c2fa6c68a7bb972a7a2ea7f55b4e

Split includes/Pager.php

Change-Id: I67fcffca4e3de081a895deb1a285a5545940ece9

Remove unused styles for .imagelist

Added in r16083 for Special:ImageList, which has been renamed to
Special:ListFiles in r45357, with class names being changed in the PHP
code without adjusting the CSS.

It seems no one has missed these styles over the years, so let's just
delete them.

Change-Id: I1f7775b3c7733cbfdd3f404527063e973fb80ec7

Merge "Add more tests for GlobalVarConfig"

Merge "Add a test for ApiFormatNone"

Mark ConfigFactory::destroyDefaultInstance() with @codeCoverageIgnore

Change-Id: Icd4a5724aefab58dad1af07874ef7ee415922ead

Add a test for ApiFormatNone

Because lolz.

Change-Id: I9c472131746a722737300b7d2d2291c70f80bb2e

Merge "ResourceLoaderFileModule: Do not separately cache .less files"

Fix documentation for $wgResourceLoaderLESSVars

* These aren't string values, but LESS code snippets. This rarely
  matters, but it matters very much when it does.
* Changes to the values do not actually trigger cache invalidation.
  Hashed contents are only used for "second-level" cache keys and do
  not make it possible to invalidate module cache.

Change-Id: I373dfedac1e43db4d0aa709384eee7f7e7f1808c

Merge "Use config instead of globals for ImageGallery"

Merge "Use getConfig for ChangesList"

Merge "Fix name of SpecialListFilesTest > ImageListPagerTest"

Merge "Remove unused private var and fix phpdoc"

Merge "Rename variable in RequestContext, $t => $title"

Merge "tests: avoid sql queries in TitleTest"

Merge "Change @return to start with type"

tests: avoid sql queries in TitleTest

testIsAlwaysKnown was invoked on an interwiki link. To have the test
pass, we used addDBData() to inject the interwiki in the database.  This
end up being execute on each test when it is only needed for a single
assertion.

Instead of creating a Title, use Title::makeTitle() to populate the
interwiki, thus letting us avoid the database hits.

Random timing: 1.20s -> 150ms.

Change-Id: I63a4e7b9af5eacb7dc1de4b33b8e631e6c3f1fa6

Merge "Rename variable in Title: $parser -> $titleParser"

Merge "Pass config to ResourceLoader from OutputPage"

Merge "Use getConfig in HistoryAction for global"

Avoid sleeper connections in LoadMonitor::getLagTimes()

* Also improved the cache fallback logic

Change-Id: I65a2c46b49a2d79e014b910a7f7357aa64964690

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

Use config instead of globals for ImageGallery

Have to pass a context to the constructor to acutally use it for
settings.

Also adds a RequestContext::getMainAndWarn to get a default warning,
when using the main request, but it would be better to pass one.

Change-Id: I1628a1790c45d44aa4239701486b8b1b7c59a0e6

Change css rule order

:invalid is incorrectly overriding a :focus:invalid element
correct the order and add a note about this.

Change-Id: I4ee1a4634840b2251e294dfb704c1f5e3b730f7d

Localisation updates from https://translatewiki.net.

Change-Id: Ie07b1bd0e3696af417e3c131f8c6d2506c881b0e

Pass config to ResourceLoader from OutputPage

Change-Id: Iae99ab65a254fc3fe3acac764a068470f99af1c4

Use getConfig for ChangesList

Change-Id: I1f440a2fca2dd8f1973987d635c70954938b6ea6

Use getConfig in HistoryAction for global

Change-Id: I4a3bb4149c3c1da75215e0e6336c89e4a00fc8e4

Made getCachedRevisionObject() use MapCacheLRU

* Previously the cache size was unbounded and leaky
* Also made MapCacheLRU handle null values properly

Change-Id: Ia02258cf051e1ccf11457c758741b8c7922afe89

Merge "Ensure that expiry times are given as integers"

Merge "Fixed bogus mysql error code 0 on mid-transaction disconnect"

Protect MathML from Tidy

MediaWiki installations that use the setting
$wgUseTidy = true; are unable to output
MathML since the well defined MathML elements
are filtered out by Tidy. This was reported as
  http://sourceforge.net/p/tidy/patches/84/ .

This change hides MathML blocks from
Tidy.

Bug: 66516
Change-Id: Ib48b91238c3eddd6a86b62f6ce57801d7058f0d8

Merge "Drop "left in" debugging var_dump from WikiImporter"

Ensure that expiry times are given as integers

Fixes "Fatal error: Argument 4 passed to Memcached::cas() must be an
instance of int, float given".

Change-Id: Ibf1ea638ec1a4dcf009cdaea8aa66008c74ff30b

Merge "Make ApiModuleManagerTest::testAddModules un-risky"

Rename variable in Title: $parser -> $titleParser

so not to confuse this with Parser

Change-Id: I7e06baa0a924310e9491f125d0cb851bde2702ab

Drop "left in" debugging var_dump from WikiImporter

I found this on accident when searching for a var_dump I forgot
somewhere in my own code. We are maintaining production code here,
right? Debugging and testing should be somewhere else.

Also note the stray print before the var_dump.

Change-Id: I98725b277039f55db9ff95399e9559a477b43c26

Merge "API: Add iwprop=url, deprecate iwurl for prop=iwlinks"

Make ApiModuleManagerTest::testAddModules un-risky

Change-Id: Idbea51cf45fc95676759e5ac9abb342701e5b551

Check for existence of iw_local in Interwiki::getAllPrefixes

Interwiki::getAllPrefixes only sets the iw_local member in a response
row if $local is set to a non-null value. Under certain error
reporting conditions, the unqualified access to the array key leads to
warnings such as "message repeated 1471 times: [ #012Notice: Undefined
index: iw_trans in ...".

Change-Id: Ibf434dc9c362984cad76ae6bdb29de826974d81e

Merge "mediawiki.language: Expose language fallback chain"

resourceloader: Only conditional-wrap script responses with only=scripts

Follows-up 9272bc6c4721225. The shouldIncludeScripts method returns
true for only=scripts, but also for other responses (except for
only=styles, naturally).

Regular load.php responses that load both scripts and styles don't
need the conditional wrap because those requests should only be
made from the mw.loader client which inherently means the
environment has been provided already.

It's merely unnecessary and shouldn't have caused any issues since
it simply evaluates to true always. It was already correctly being
excluded from raw modules that run before the environment is
provided (such as startup, jquery and mediawiki).

Change-Id: I375a7a8039f9b3ad4909e76005725f7d975d8a5e

mediawiki.language: Expose language fallback chain

The data can now be accessed via mw.language#getFallbackLanguages or
mw.language#getFallbackLanguageChain.

Bug: 69285
Change-Id: Id0c3267f01d5c4da0e4000a0fb318a9dfb538ef3

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

New changes:
7205b2a build: Check for files' existence before building using grunt-file-exists
d600838 Make window managers destroyable

Change-Id: I32e4f5bd41a5401b817cc6e37c6325df624a9e0e

Add more tests for GlobalVarConfig

Change-Id: I31d50a0636792104e32e3f25694f45942c5c0217

ResourcesTest: Add variable type annotations

Change-Id: I07ec986830dcec739b0553395b9a220b44bba77d

Merge "Add more tests for SkinFactory"

Delete skins/common/images/button_template.png

Unused in core, as well as extensions and skins.

I have removed the two usages on Wikimedia wikis (found by mwgrep)
which must have been the cause of the 97 requests for this file per
bug 69277 comment 11:

* https://de.wikipedia.org/wiki/MediaWiki:Gadget-Vorlagenmeister.css
* https://hu.wikipedia.org/wiki/MediaWiki:Gadget-sablonmester.css

Bug: 69277
Change-Id: I9d151e1fbc2da948b5265da731496d4178f51039

Made showJobs.php show queues with just abandoned jobs too

Change-Id: I7fbf000c9b9edf433ac6723843a96fcaf937b5c7

Merge "Sync up with Parsoid parserTests."

Update OOjs UI to v0.1.0-pre (55b861b167)

New changes:
466a876 [BREAKING CHANGE] Make content isolation optional
23f26c3 [BREAKING CHANGE] Set active and pressed on the buttoned element, not the button
55b861b build: Switch from grunt-recess to grunt-contrib-less

Change-Id: Ie4454376df9492e3b77b6cc174ce7b38548cc56d

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 13c31fc815749a6239dd357672d39de0a51d51ce

Change-Id: I929e1c195801a9055ad2847994e050836262a896

Fixed bogus mysql error code 0 on mid-transaction disconnect

* Now the proper error (e.g. "server has gone away") and code is shown.

Change-Id: If0255101151441c7d05141ffc7cb416ad79e6bd8

Merge "Set default setting for thumbsize to 300px (index 5) not 180px"

Merge "Never cache pages with the fallback skin"

Merge "Made LocalFile move/delete/restore handle network partitions better"

Merge "Title::getContentModel(): load from DB if necessary"

Update OOjs UI to v0.1.0-pre (944c47c5fe)

New changes:
36ae086 Use a pollNode for every transplanted stylesheet, not just external ones
a1771ed Support nested style transplantation
8c60d3c Create a combo box widget
944c47c Update oojs to v1.0.12

Change-Id: I800bb36b80d04e165e9fa9b00ed8a5223f2d954b

Title::getContentModel(): load from DB if necessary

Also don't cast $model to int in LinkCache::addGoodLinkObj(); content
model IDs are non-numeric strings, not integers, so that field was
always populated with the value 0. Because 0 is a falsy value, this
caused subsequent calls to Title::getContentModel() to return the
default model rather than the correct one.

Also (hopefully) fixed every single query that could cause a
LinkCache entry to be added without the content model.

Bug: 69789
Change-Id: I94f06baf406afa538cd2b10139598442f9fc6759

Update OOjs to v1.0.12

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

Change-Id: I5542d1cf5aad4d4fd78a04f6bea7a27cbabf055a

Set default setting for thumbsize to 300px (index 5) not 180px

Bug: 67703
Change-Id: I172ccc0128747090a199695d3bce04388b3bef57

Add support for adding deferred update within a deferred update

Under some configurations all Echo notifications are inserted
via a DeferredUpdate.  When an extension delays its own notification
trigger via DeferredUpdate as well the Echo notification will not
be run.

Change-Id: Ib40fcd4939ede5b0ff986ab109432630efd1be74

Revert "Load mediawiki.action.view.redirectToFragment in mobile"

This reverts commit 21ec580f066566d7da97cca0ef40c112b6a0b7a1.

Change-Id: I49788d6b56efdd2fed77ff7657e11554b9927076

Merge "Load mediawiki.action.view.redirectToFragment in mobile"

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

Localisation updates from https://translatewiki.net.

Change-Id: I01ae5ce97f4b82c10bc8184a5d5db0517ca6f7fe

On redirects update the URL to that of the target page with JavaScript

Redirects to sections will now update the URL in browser's address bar
using the HTML5 History API. When [[Dog]] redirects to [[Animals#Dog]],
the user will now see "Animals#Dog" in their browser instead of "Dog#Dog".

The target URL is generated server-side to avoid pulling in dependencies
in the client-side RL module, which is loaded in the top queue.

Browsers that do not support the History API are still supported, the
way they always were.

Given the following three pages:
* A: #REDIRECT [[C]]
* B: #REDIRECT [[C#Section]]
* C: ==Section== \n ==Other section== \n

The following links should behave as follows:
* A                →  C
* B                →  C#Section
* A#Other_section  →  C#Other_section
* B#Other_section  →  C#Other_section

The code also supports forwarding query parameters like 'debug=1'.

Bug: 35045
Bug: 39328
Change-Id: I9d8d834762e19b836b7e35122b6c4cef0f91b7f0

Merge "Fix @param of Language::dateTimeObjFormat"

Merge "Split RevisionDeleteAbstracts.php to separate files"

Change @return to start with type

MediaWiki default is "@return type Description", so set a type after
return and start the description with a capital letter. Also use the
more common spelling of boolean.

See http://phpdoc.org/docs/latest/references/phpdoc/tags/return.html for
more about @return

Change-Id: I4e5198822fe92836f9cef9918a9fc1a1a1e0a043

ResourceLoaderFileModule: Do not separately cache .less files

We used to have a second level of caching for them – when a module's
cache was invalidated, individual .less files that comprise it would
only be recompiled (with all their @imports) if their mktime changed
(or if $wgResourceLoaderLESSVars changed).

Given that practice has shown that RL modules including .less files
usually just include one file that @imports tons of stuff, and given
that RL already is very careful about invalidating caches, I think
this is unnecessary at best and harmful at worst.

The behavior proved problematic when I tried to implement support for
per-module LESS variables, values of which were language-dependent –
files would not be recompiled for different languages, and
getLessCacheKey() is called several levels deep in code that doesn't
have access to the context in which the module is being compiled.

Change-Id: I667f063507583e4cd3a669e32398d440f1431f7e

Merge "API: Adjust feature usage log format"

Fix @param of Language::dateTimeObjFormat

Follow-Up: Ib26407bc55dff7969d8a3b1e2ae51751b202d8fb
Change-Id: I078ffe3ff385f4541d91b2fb8ef9564138f78393

Split RevisionDeleteAbstracts.php to separate files

Same as I244a66fe19c089df4bcabcf745536d99b2103bc4

Change-Id: I90cea070a8e24d82f0675f122c79326988351e08

Consistently use 'Less' rather than 'LESS' in function names

PHP function names are case-insensitive, so this is a fully
backwards-compatible change.

Change-Id: Ide04ad542ac5c3a26b6064093ae272cf9aeef2d1

EditPage: Remove unused classic toolbar accesskeys data

Clearly this is not going to ever be used. Half of these conflict with
something else anyway.

Change-Id: I21ada83eba4638b3eb820b62c0c1eb90951e6f9b

API: Adjust feature usage log format

On the advice of the Research Analyst, quote all data fields and add
slashes to escape quotes.

Change-Id: I3471475c0ae7a11398a774b1a14d023676259c04
Follows-up: Ifd640883e46f3e6c8179c63d122c704b046674ca

API: Add iwprop=url, deprecate iwurl for prop=iwlinks

Someone objected to the possibility of removing the deprecated llurl
because prop=iwlinks still had an iwurl parameter, and they wanted
parallelism. So let's deprecate iwurl too.

Change-Id: I675c36868d6ce531a9de632203ca7b9742d63be1

Add more tests for SkinFactory

Change-Id: I0a8f93abe597d6b2393f1c813e7bf76093f3ec89

Merge "API: Add a "submodule" param type"

Merge "User adjust git timestamp for extensions on Special:Version"

Merge "Remove wrong type from @param on ContentHandler::merge3"

Merge "Fix doc of PPFrame_Hash::cachedExpand"

Merge "Remove WikiPage::estimateRevisionCount (deprecated since 1.19)"

Merge "Remove Xml::namespaceSelector() (deprecated since 1.19)"

Merge "Add missing @param to function docs"

Remove WikiPage::estimateRevisionCount (deprecated since 1.19)

Change-Id: Icc44f67dd6479553b44f5566d1158638e4a09e60

Remove Xml::namespaceSelector() (deprecated since 1.19)

Change-Id: If4eaadda2f2533cdcebc692f90c43bdddab3d82b

Merge "Remove wrong null type from @param of Content::fillParserOutput"

Merge "Remove wrong type from @param of IORMTable::getPrefixedFields"

Correct type of @param of Linker::getRevDeleteLink

Change-Id: Ie382e2f7b68e27f56de469bfead1588b97b5f9c3

Merge "Use FormSpecialPage for Special:RandomInCategory"

Use FormSpecialPage for Special:RandomInCategory

This commit moves [[Special:RandomInCategory]] from using the SpecialPage class
to using the FormSpecialPage class, in the process moving from a simple input
box in the middle of a sentence to a "proper" form.

Bug: 67581
Bug: 67253
Change-Id: Ibf8694c0643f7cd3964ff9f62ffc6049f0978416