Merge "jquery-ui: Remove dark color from links inside dialogs"

jquery-ui: Remove dark color from links inside dialogs

Bug: T85857
Change-Id: I29aafd41ce142b05c1bd9aafb096f38273ca747e

CleanUpTest: Consolidate individual "@todo Document" comments into one

This avoids PHPUnit marking nearly every test as risky.

Change-Id: I7cb5fc3eca2e9496a61d223dcae7576e7ddb4cad

Merge "Update QUnit from v1.14.0 to v1.16.0"

Update QUnit from v1.14.0 to v1.16.0

- Release notes
  https://github.com/jquery/qunit/blob/1.16.0/History.md

- Source of imported files
  https://code.jquery.com/qunit/qunit-1.16.0.js
  https://code.jquery.com/qunit/qunit-1.16.0.css

- Unit tests still pass.
  Tested on [[Special:JavaScriptTest/qunit]] with $wgEnableJavaScriptTest = true

Bug: T85994
Change-Id: Ic2d5e91d4f1e16a87618b18c3a83d31aaa0ab91c

Merge "Remove the 'top' definition from .mw-ui-icon-element:before"

Remove the 'top' definition from .mw-ui-icon-element:before

This was making the icon stick to the top of the button it was
inside. We couldn't find an example of the issue that the earlier
patch was trying to fix (on #wikimedia-mobile).

Bug: T85778
Change-Id: Ifea204ee9b86b01f61f3c48372619dadce64da9b
Reverts: Iadd741f02155a9b5ddeb6cd8ff64fa13ae5506d7

Merge "Escape unescaped messages in Special:Preferences"

Cleanup ParsoidVirtualRESTService

* Remove unused global
* Throw \Exception instead of \MWException since this is in includes/libs/

Change-Id: I2e51d7e25a755b79d14d1814951449f6ce14c68e

Merge "Make a VirtualRESTService class for Parsoid"

Make a VirtualRESTService class for Parsoid

So that code to deal with Parsoid can be shared across different extensions.

Relies on I1d57ff24

Bug: T1218
Change-Id: Id658d925b722b885fd425c27f7c4fd2cd02ad0b4

Escape unescaped messages in Special:Preferences

Bug: T85864
Change-Id: Ie516549bce72a0052570ab38bde4c03e2bc1b69b

Trigger onResponses on non-replacement request too in VirtualRESTServiceClient

Change-Id: I1e550812f3c021c2e2f4246345df2f9dd47d15b8

Localisation updates from https://translatewiki.net.

Change-Id: I8a6dcdacfbe59bd9b34c314c957ce05a15c0e7a5

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

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

Actually on second thought, having the parameter disappear is
a better option than keeping it around with only one possible
value.

This reverts commit 7a79b017a03c78ddc0b8bf8762346957bef71936.

Change-Id: Ia67fa1425e0aa1ee95e0a1733f89d29f94f1eeee

Merge "Remove double escaping of group member name on Special:ListUsers"

Merge "grunt: Add Karma task for automated QUnit testing in browsers"

Remove double escaping of group member name on Special:ListUsers

Similar to T31340 the group name gets double escaped

Bug: T85864
Change-Id: I9202fc12c2d72baf1f61cb69624b37411b0edacc

Merge "Always use the canonical extension name to link the license file"

Use also $wgAutoloadClasses in tests for module names

Extensions using $wgAutoloadClasses where this test will fail

Follow-Up: If1125cd5fa4ed836fe15fc79480d78ebd899be4e
Change-Id: Ic2024605d7d59890c527cc0580a49da73f8516c8

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

grunt: Add Karma task for automated QUnit testing in browsers

To use, ensure npm modules are up to date by running 'npm install'
in tests/frontend. Then run 'grunt qunit' to run it in Chrome.

To run it in both Firefox and Chrome (locally), run

 grunt karma:more

Moved fixture from hardcoded HTML to the testrunner so that our
tests don't depend on arbitrary HTML and thus also pass in browser
contexts where the page only loads QUnit + test suites (e.g. Karma).

Change-Id: I4e96da137340a28789b38940e75d4b6b8bc5d76a

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

Always use the canonical extension name to link the license file

Items in $wgExtensionCredits can expose a 'namemsg' attribute,
but it should only be used for localized strings.
SpecialVersion->execute() correctly uses 'name', while commit
326f3f450ce56420090e3bb30eb9a9cccfe1816e inadvertently made
getCreditsForExtension() use 'namemsg'.

Also reduced code duplication.

Change-Id: I029ea17c55191d74bc6a803fad6cf092105f5b3c

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