Don't call localisation functions when initializing pageset

Language::commaList() gets called even though in 99% of cases it's not
description that's needed but other parameter information, resulting in
0.6% of overall cluster CPU time being wasted per
https://performance.wikimedia.org/xenon/svgs/daily/2015-01-02.svgz

Change-Id: Ic77e4a03d1bbd1aa5f86dc250d9f903d7eb25618

Merge "API: Avoid unstubbing User for language pref when not needed"

Merge "Proper namespace handling for WikiImporter"

Localisation updates from https://translatewiki.net.

Change-Id: I360f162ccf794d3709c0d64d617d6ffbdf85b23f

Merge "Add applicable protection types to the prop=info API"

Add applicable protection types to the prop=info API

Disabled by default, enabled with inprop=protections,
returned as 'restrictiontypes', also for non-existing titles.

Change-Id: I98604b60626b84bd8cb601a4149b80db1b6787b7

Merge "Remove silly reference ampersands from database functions in maint"

exception: Implement error-json log

Follows-up 399ba2f.

Bug: T75619
Bug: T45086
Change-Id: Ia7ba355d5925a6268ffa321ffc13cc6906b487f8

Merge "Add batch lookup for user groups and titles on Special:ActiveUsers"

Merge "Use content language for edit summary on upload overwrite"

Remove silly reference ampersands from database functions in maint

This is 2015 and we use PHP5 I think.

Change-Id: I012de0df540fc91577585de939bb929c68664222

SpecialJavaScriptTest: Add export feature

Add an 'export' subpage to SpecialJavaScriptTest which allows
one to request a self-sufficient JavaScript payload that will
bootstrap a ResourceLoader client and load the test suites.

This is needed for using Karma (which only loads JavaScript,
no full html pages). As such elements from the Skin and OutputPage
will not exist. While all QUnit tests in MediaWiki core and
most extensions I've seen already use #qunit-fixture, this is
now required. This to prevent leakage of elements from one
test to another, but it also prevents tests from depending
on elements provided by the server.

While the Karma setup is still in the pipeline (might land before
this commit loses WIP status), for now this can be tested via
the 'Special:JavaScriptTest/qunit/plain' subpage.

Refactor:
* Use HTTP status code 404 in the response for "noframework".
* Simplify HTML footprint by using <div id="qunit"> instead of
  hardcoding the full structure. This feature was added to QUnit
  since v1.3.0 (Feb 2012), we're using v1.14.0 (Jan 2014).
  QUnit's header is automatically derived from document.title.
* Remove redundant addModules() for 'test.mediawiki.qunit.testrunner'.
  This is already added by default.
* Move allowClickjacking() call so that it applies to other modes
  as well. The exported javascript needs to have wgBreakFrame set
  to false so that test runners can frame it.
* Change mediawiki.special.javaScriptTest to not depend on QUnit.
  It caused QUnit to load on error pages. And in theory the page
  is suited for other frameworks and shouldn't load QUnit this way.

Bug: T74063
Change-Id: I3d4d0df43bb426d9579eb0349b8b5477281a7cfc

API: Avoid unstubbing User for language pref when not needed

It's fairly common that the API doesn't need to load the user
preferences, except to implement the unfortunate default uselang=user.

So let's move the handling of uselang=user to
RequestContext::getLanguage(), and have the API just assume that its
parent context will eventually fall back to that for uselang=user.

Bug: T85635
Change-Id: I947348d87b31808d331055dac6feb0cc2e1dd15d

Revise wording for installed libraries release note

Change-Id: I7afafa1b5b954e4555b10c8fd2a225f0f4754115

Don't display composer installed extensions/skins on Special:Version

Extensions and skins should already be adding their credits through
$wgExtensionCredits, meaning that they would be duplicated in the
"Installed libraries" section.

Anything that has a type that begins with "mediawiki-" will now
be skipped.

Release notes were added for this commit and 528297f8db8dea.

Change-Id: Ie873b16c1a8a696c7de833ae69b263161d9e15a8

Merge "mw.ui: checkbox: Remove margin from checkbox input to fix the pseudo element overlap"

Merge "mw.ui: radio: Add state transition to radio buttons"

Merge "Adjust images in packed gallery on window resize"

Adjust images in packed gallery on window resize

Original element sizes are stored in data(), and on window resize,
they are restored, so elements can be justified as on just loaded
page.

Bug: T55664
Change-Id: I286e0a4c8230c11619ca30f8f3b66778de835a33

Merge "mw.ui: button: Update usage instructions for neutral and quiet buttons"

Merge "Make jquery.suggestions obey maxRows setting"

Merge "Expand error message when parser tests found no hook"

Expand error message when parser tests found no hook

The existing message is hard to understand and does not mention, that
this is a problem in the parser test itself and not with phpunit.

Before:
1) Warning
The data provider specified for ParserTest_<file>::testParserTest is
invalid.
Problem running hook

After:
1) Warning
The data provider specified for ParserTest_<file>::testParser
Problem running requested parser hook from the test file

Change-Id: I0b4225cc9ab95e8dd048515315c789113dacf39e

Update OOjs UI to v0.6.1

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.1/History.md

Change-Id: Iafe5770c74b671dc04a92b9dfab6f121e7fe718d

mw.ui: button: Update usage instructions for neutral and quiet buttons

Bug: T75252
Change-Id: If18d4773f491d465309662a7decd8466673002fe

mw.config: Show deprecation notices when accessing globals

The mw.config.value object is no longer an alias to the global object
when $wgLegacyJavaScriptGlobals is true.

Instead, set() is made to copy to property to the global object. This
matches behaviour of other deprecated properties in that changes to the
deprecated property directly are ignored.

Bug: T58550
Change-Id: I703f7c12b59bc3207b2a291eacc393a8ae92df6f

Make jquery.suggestions obey maxRows setting

The maxRows setting was previously unused and had no effect. This patch
passes the property in the fetch() call, and makes it enforce the setting.

Also updates mediawiki.userSuggest.js to use response()

The previously unused setting of 7 is changed to 10 to match current
behaviour.

Bug: T39316
Change-Id: I2873963aba9fd751607d11904b94bfcec8eb2d32

Localisation updates from https://translatewiki.net.

Change-Id: I3f72d9e7d620c0fe2070a7da08d704965a61e00c

Make use of DatabaseBase::selectRowCount in Title

Change-Id: I686f4f785ffa6323ac94b679794ba50539d951f0

resourceloader: Make arguments to mw.loader.implement optional

This will allow the server to trim any trailing parameters with
empty objects from invocations.

'templates' was the only parameter added after the initial ResourceLoader
release, the other properties have always been required.

Change-Id: Ie32e7d6a3c09f86a52d60394c474a62cb1b4e1d6

Merge "Fix documentation of mw.loader.getState"

Fix documentation of mw.loader.getState

The misleading comment was copied from mw.loader.getVersion
in commit 6a1ec17e79.

Change-Id: I73ddb67901da808fd402742232b4b4dd2ad31ebc

Merge "Database::makeList() : Handle NULL when building 'IN' clause"

Merge "mediawiki.action.edit.preview: Use .textSelection( 'getContents' )"

Localisation updates from https://translatewiki.net.

Change-Id: I3cd9917fd67fe31a829e38108bb0220bc5c41842

Database::makeList() : Handle NULL when building 'IN' clause

Amends to Database::makeList() to check if 'NULL' is present in array while
building IN clause. Presently a NULL in array intended to build IN clause, is
treated as a usual value. This creates
problem:https://dev.mysql.com/doc/refman/5.0/en/comparison-operators.html#function_in
Therefore null is appended at the end of the IN clause as " $field IS NULL"
rather than within the IN clause.
DatabaseSQLTest.php: Some test cases added for the modified behaviour.

Bug: T50853
Change-Id: Ic255c37e9702fd44d8173027ab8036ddc48f9d2e

Merge "Add SQL tests for Database::makeList"

mw.ui: checkbox: Remove margin from checkbox input to fix the pseudo element overlap

The target for the checbox was extending unusually due to extra
margins. See https://phabricator.wikimedia.org/M27/51/

Change-Id: I93fb29cfed5e0fd0ceb1f89977abd2991629fe34

Add namespace translations for nan

Aliases added from fallback language cdo. Per request in
https://translatewiki.net/wiki/Thread:User_talk:Nike/outo_p%C3%A4tk%C3%A4_osoitteissa

Change-Id: Ic73d81cd9a477a9818aa0f0c8b596ab49850460a

Localisation updates from https://translatewiki.net.

Change-Id: I7e5267badb3467b3f7b5e000efe6c37f8fd0c7d7

Merge "jQuery.footHovzer: Fix scrolling issues"

jQuery.footHovzer: Fix scrolling issues

Fix scrollbar appearing if the page does not fill the screen, and the
debug toolbar is enabled.

This patch also fixes an issue where the page will have a height of
~1000px if the CSS is loaded after the JS executes (display:none
isn't triggered before the outerHeight() calculation). I've only been
able to fix this by adding a setTimeout, which is a little bit hacky..
but can't figure out how to solve it any other way.

Bug: T38273
Change-Id: I1f92cb0681b7a82a5936267c120f85481a95a962

Fix @covers tags for includes/libs/composer

Change-Id: I90a39bce22f304eb06dd595eb845bb953e6937c3

Merge "Don't replace user text with suggestion in searchbox"

mw.ui: radio: Add state transition to radio buttons

Bug: T78370
Change-Id: Icaac128c7d1e2424a8bed30e28f148bb1e720c5a

Merge "Clarifies the meaning of the function which tests the bug T34712"

Don't replace user text with suggestion in searchbox

On Enter press, there was highlight() called, which also replaced the text.
Since it seems to have no use here (highlight() highlights the selected
suggestion in the box, but the box is immediately hidden, and there's
probably no case when the suggestion is chosen, but the text in searchbox
does not match it), it was removed.

Bug: T53900
Change-Id: I9fc2e954ae429ba166ddc7c713f9790a25a837c2

Clarifies the meaning of the function which tests the bug T34712

Change-Id: I17922b9bc6ac2ce5fef7aebc27020874d349f507

mediawiki.action.edit.preview: Use .textSelection( 'getContents' )

Replace .val() by .textSelection( 'getContents' )
for '#wpTextbox1' and '#wpSummary'.
This is already supported by extension CodeEditor.

$( mw ).trigger( 'LivePreviewPrepare' ) is not needed anymore.

Bug: T78838
Change-Id: I30bd709d8b4778f45f3d5f395201e0ca6ce18891

Document OutputPage::$mPreventClickjacking

Change-Id: I3b4f967743c810c5cbef094b3e30807cf0a668ca

Implement support for a minimum log level in $wgDebugLogGroups

For some log groups, we only want to log them if they meet a certain
level of severity. An example of this is the current 'memcached-serious'
log group, which can be merged with the normal 'memcached' group
in the future, and report at a severity of ERROR.

This adds a 'level' parameter to the $wgDebugLogGroups, for example:
    $wgDebugLogGroups['memcached'] = array(
        'destination' => '/var/log/mw/memcached.log',
        'level' => \Psr\Log\LogLevel::ERROR,
    );

Bug: T85073
Change-Id: Ic53bc4c8e318ed188fe6f4e838e6789b3c3fd574

Merge "Only return CORS headers in the response as required"

Only return CORS headers in the response as required

- Split out responses of preflight and actual CORS requests
- If the request is not CORS valid, don't set the CORS response headers

Note that invalid CORS requests should not actually throw error
responses, the client should simply not handle the response because the
response does not have the right headers (it's a client side policy
error not an http error). We do throw a 403 for a mismatch with the
queryparam, but since that is 'outside' of the spec, that might be
appropriate.

Bug: T76701
Change-Id: Ib296c68babe5c0b380268ee7793b3d6d35b9c3e3

Keep backend param to search API as long as there's a backend

Originally this was designed to only show up when there was
more than one backend. That's more likely to break for clients
though if you want to go back to only having one backend again.

Change-Id: I9009f2599f5bc6fa8f9c49370fd0ffb714e31755

Merge "Use preview content when it transcludes itself"

Merge "Use PHPUnit_Framework_TestCase in a few utils/ tests"

Remove normalizeAttributeValue()

Sanitizer::normalizeAttributeValue is completely unused and private, and
as such, should be removed

Change-Id: I655ef90fe97a775011f9d16c0b51feb2612aec26

Moved getTitleInvalidRegex() from Title to MediaWikiTitleCodec

Deprecated it in Title, and updated all current references to use the
non-deprecated version in MediaWikiTitleCodec

Change-Id: I2b9c36992028c97f695f2b95ba027fbb11904b57

Merge "Documented the Classes ImportStringSource and ImportStreamSource"

Documented the Classes ImportStringSource and ImportStreamSource

Added short descriptions for each class

Change-Id: I28d3dea76ab70326a1e16b7c41b1f3758f8648b8

Add JSON to docs/contenthandler.txt

Bug: T85580
Change-Id: I70edfbe4e50fec0701484d5004a5c673c0dd69b7

Format multiple autocomments in edit summaries

Before r39373, all autocomments in an edit summary were formatted. In
fixing a bug with page titles containing "/*" this was accidentally
broken.

To use a single preg_replace_callback call to replace multiple
autocomments, we need to make sure that the match of one autocomment
doesn't overlap the match of another, which means we can't have "(.*)"
before and after. But we do still need to detect whether there is
anything before or after. "(?=(.?))" and "(?<=(.?))" would do nicely,
except the latter isn't actually supported. "(?=(.))?" and "(?<=(.))?"
work too, but older versions of PCRE don't support that. They do,
however, support "(?:(?=(.)))?" and "(?:(?<=(.)))?", so that's what
we'll go with.

This change does change the values for $pre and $post passed to the
FormatAutocomments hook; extensions need to be updated to accept (and
not prepend/append) booleans for these parameters.

Bug: T18530
Bug: T70361
Change-Id: I36c3a9e548a4ef72f93974bb35f9add8c29e9287

Merge "mediawiki.action.edit.preview: Restore trigger LivePreviewPrepare"

Localisation updates from https://translatewiki.net.

Change-Id: I5d944ab07707d9a1a886ff6336f8136f9fa237ca

Merge "Set mw.config wgFileExtensions only on Upload instead of site-wide"

Merge "Make TitleMethodsTest a LangTestCase"

Use preview content when it transcludes itself

When a page transcludes itself, such as <noinclude>foo
{{:{{FULLPAGENAME}}}}</noinclude><includeonly>bar</includeonly>, use the
preview content in its own transclusions. This code was basically ripped
straight from Extension:TemplateSandbox.

Bug: T85408
Bug: T7278
Change-Id: I1aa091a395a4f7b7b744e09e0bed59bc2e1176d0

Correct apihelp for show parameter for some modules

They mention "!redirects", but the value is called "!redirect"

Bug: T85431
Change-Id: I1034b56b659db0c1e0aaede92a373919a7878dfe

Make TitleMethodsTest a LangTestCase

With I40a51c14ce07c301c15c6c2128cc18bb12e71103 a compare of namespaces
was added which fails when content language is not en.

Change-Id: I21466231e10e6cd5969c8c9bfd4824bf9c47e6d8

Use PHPUnit_Framework_TestCase in a few utils/ tests

IPTest, StringUtilsTest and ZipDirectoryReaderTest do not
need the extra overhead

Change-Id: Ic05afb1a3a094a57383d483f4e50bd1ed9a183e2

Convert most libs/ tests to use PHPUnit_Framework_TestCase

They don't actually need any of the mess that MediaWikiTestCase provides

Change-Id: Ibd067480fc294096d1249132cb800e09889efb18

Merge "Implemented hasRules()"

Expose installed external libraries on Special:Version

Change-Id: Ic767a06ccfd938fbe519333bbd683f6116aaa262

Add checkComposerLockUpToDate.php script

Checks whether your composer.lock file is up to date
with the current composer.json file.

Bug: T77388
Change-Id: I528d63172c238cf1ea9bc02e8eb39b93225865de

Merge "Apply time adjust on merge log"

Merge "Allow querying for multiple MIME types in the allimages API"

Allow querying for multiple MIME types in the allimages API

The 'aimime' parameter can now be a pipe-separated list of
MIME types. Also tweaked the English apihelp and added an
example for the new functionality.

Bug: T78690
Change-Id: I775ecae53c81eccb0a898081715d48589b53af15

Add batch lookup for user groups and titles on Special:ActiveUsers

Call parent of doBatchLookups to get the LinkBatch for the userpages.
Use than the populated user group cache to get the groups from to
display them.

Follow-Up: I4a945f83ad28edf5cc040139943cf743cb3d133c
Change-Id: Ibbabf40d60dd9f8e3667cf17455e7582b4ee472a

Merge "Fix warning on Special:ListGroupRights for group with invalid group page"

Add SQL tests for Database::makeList

Change-Id: I5a5be6769c7d0e5f2a97d3fbefd62df3c59f0716

Escape word-separator in LogFormatter

Change-Id: I9428f5ff5b95d09932b3588f0118d6992a384390

API: Clean up uncaught exception backtrace output

The existing response seems to be designed to look good with
format=xmlfm, which doesn't make much sense now that format=jsonfm is
the default.

Change-Id: Ic3993aaf7e401d6233bbbe4522e77004099e3c40

Fix escaping of specialList and clarify comments

Change-Id: I4bead5f5f310dd35e8dfee738f35a070e7bf869f

Merge "Include log id in api error response"

Fix warning on Special:ListGroupRights for group with invalid group page

Avoids
Notice: Linker::link: Requires $target to be a Title object. [Called
from SpecialListGroupRights::execute in
\includes\specials\SpecialListgrouprights.php at line 97]

Change-Id: Ie90ea33b0f57861985a296380e4ee7296dac79af

Remove over/underescaping detected in Special:UserRights

Bug: T31340
Change-Id: I99823cd56e0a6f501101cb85be832d2925ce9779

Shorten lines to pass phpcs test

Change-Id: I5588e1f16f1a23d77160cd180058bd2000a93ab6

Implemented hasRules()

hasRules() looks at $mRules and returns true for a non-empty string, and
false for an empty one

Change-Id: Ia7a1316498dc3f2753f61134705c9bd80fc9caeb

Merge "Parser: Add <bdi> to the whitelist for TOC links"

Localisation updates from https://translatewiki.net.

Change-Id: I82520452e4635c34348089fdf7cc605b32d95881

Apply time adjust on merge log

The merge log params contains a revision timestamp in UTC, this needs
time adjust in case of users with a timezone in the settings.

Change-Id: I1612a33307667fa767e80a6061c013ea0f10b0a6

Merge "Use a more generic DB Error wording"

Merge "Sort languages by code in Language::fetchLanguageNamesUncached"

Use content language for edit summary on upload overwrite

The edit summary for the null revision on upload overwrite was in the
user language and not in content language

Possible a regression from Idf281898d8a5a023a0b9ce3bc90b3ca55c1a6376

Bug: T72293
Change-Id: Ie70a3874e170b3849e91265e0c9c61da838f2d20

Sort languages by code in Language::fetchLanguageNamesUncached

This is being used in API output in ApiQuerySiteinfo so having
it sorted is nicer.

Bug: T85406
Change-Id: I796112187d17337a1cc12d6752839263b53f40e1

mediawiki.action.edit.preview: Restore trigger LivePreviewPrepare

CodeEditor needs this trigger.

Bug: T78838
Change-Id: Ibf355384b821ad98616671797f82dfa7410407e3

Use a more generic DB Error wording

Not everytime there is a problem to _contact_ the database server (e.g. a false server),
sometimes (e.g. a false password or username) it's a problem of the user credentials
or that the database does not exist. So use a more generic wording, just saying,
that the database could not be accessed.

Bug: T76317
Change-Id: I502d5894acb855ac51c6f45ea91e5446b4b3aeb3

Include log id in api error response

For internal api error it is helpful for reporting when having the log
id under the exception was logged in the exception log.

Also use the redacted trace in the api error response instead of the
normal one to avoid leakage of function values if exception details are shown.

Change-Id: I7d917e532da1b1c7979a6599425f07d2f4f35aed

Parser: Add <bdi> to the whitelist for TOC links

Bug: 72884
Change-Id: Id5aa9a4eb32fb185881141e55de700ae36f806c5

Localisation updates from https://translatewiki.net.

Change-Id: I4731cb83ad392d361cd9bfca154aa27a8278d9c1