Add .list-style-image-svg

Edokter suggested to use this, this way.

Bug: T37338
Change-Id: I09319e9188878cf316dd5a20c76804be9d88999d

Merge "Removed deprecated RefreshLinksJob2 class"

Removed deprecated RefreshLinksJob2 class

Change-Id: Id5659c7fe0c6ebcbee0ef435ee4a95c1033cda47

Merge "Added query/connection expectation support to TransactionProfiler"

Merge "Make Profiler::$instance private and drop double underscores"

Merge "Add Monolog handler for syslog UDP transport"

Make Profiler::$instance private and drop double underscores

Nothing calls it anymore outside of the class

Change-Id: Icc9ebcd63e440f9bafca29647856d36fc493ff39

Localisation updates from https://translatewiki.net.

Change-Id: I6c61d096e9eb1924c0b4f61f812d8e52e3c6a9ba

Merge "Made BagOStuff::debug protected"

Merge "mediawiki.api.login: Include second HTTP request in abort()"

Merge "mediawiki.jqueryMsg: Refactor "Match PHP parser" test suite"

Make eval.php try to handle newlines in code

Change-Id: I9f4664bde6bea6f0e1dca0f4623553b9e849c83b

Merge "Use MediaWikiTestCase methods for tempdir in unit tests"

Merge "filerepo: Use standard method for creating temp dir in unit test"

mediawiki.jqueryMsg: Refactor "Match PHP parser" test suite

Previously it was doing quite a few things wrong:

* Tests could potentially run in parallel. This was somewhat
  obscured by the asynchronous getMwLanguage() call being inside
  the synchronous each() loop. As such it basically first fired off
  the requests, and then handled them as they came back. They
  pretty much always come back in order (lucky) so the tests pass,
  but it was fragile.

* By default, jQuery.ajax() appends a cache buster ("&_{random}" query string)
  to requests with dataType 'script'. Disable this by setting cache=true.

Also:
* Store Promise objects instead of a low-level Callbacks list so that we don't
  have to wrap it. Instead, we simply chain from the $.ajax() promise, and use
  then() to feed the value instead of calling resolve() or fire().
* Use hasOwn() as extra sanity check (in case a random browser's non-standard
  Object.prototype methods match one of our non-standard language codes).
* Remove dead code "mw.messages.set( test.message )". This object has no message
  property (or anything else that it could be mistaken for). It just performs
  15 no-op in a loop. Follows-up 261aac63ec0777d6.

While the requests happen to (almost) always arrive in order, them running in
parallel put a lot of pressure on slow VMs using SQLite databases (which are
I/O bound). This causes a lock when queries from different processes overlap.

Change-Id: I0c5064780f80e1ddcc58ce32c390b2639b1f1ab6

Merge "Update OOjs UI to v0.7.0"

SpecialWhatlinkshere: Mark redirects containing templates

Mark redirects containing templates with class "mw-redirect" on
Special:WhatLinksHere.

Bug: T89098
Change-Id: Icd921c25a9c3542acb44e7175195328f283a1d5c

Update OOjs UI to v0.7.0

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

Change-Id: I70c56ce5aa070d1a5e6f51f80fce7cd7750b3007

Fixed FileRepo::newGood handling of $value

bug: T89296
Change-Id: Icc2cc058d99a972bc0b243c94c49326627aa89b5

Merge "Made BagOStuff::merge() take any callable"

Localisation updates from https://translatewiki.net.

Change-Id: Id5664c8bed61f9c2f0260163de04befce964cbd4

Made BagOStuff::merge() take any callable

Change-Id: I5ce99b8d343f1629f888a3b7594976a83839e2b7

Merge "Never treat persistent stat cache as "latest" in FileBackend"

Merge "Made RecentChangesUpdateJob lock wiki specific"

Added query/connection expectation support to TransactionProfiler

* Master connections and writes on read requests will now be logged to
  get visibility into code that reduces performance and site availability.

bug: T88445
bug: T86862
Change-Id: Ic2ae95c33facbb54e062aef2ce67d6182caa044a

Merge "Do not call purgeExpiredRestrictions on simple page views"

Fix doc comment on $wgResponsiveImages

Doc comment referred to older implementation's data attributes.
Updated to refer to the correct srcset attribute as settled on
by browser makers, which was switched to long ago here.

Change-Id: If2bc83dd41dd88d6480957ff4968111396c9fe98

Made RecentChangesUpdateJob lock wiki specific

Change-Id: I566d81affcd3d60cd5e8da090db9ea79495e7722

Merge "Clean up state of libxml on failed import."

Merge "Revert "Add AvailableRightsTest for User::getAllRights completeness""

Revert "Add AvailableRightsTest for User::getAllRights completeness"

This change has not been advertised and cause random extensions
to start failing giving little clue to users.

Please have this change discussed on wikitech-l and announced
ahead of time to minimize disruptions. It brokes MathSearch at least:

https://gerrit.wikimedia.org/r/#/c/189442/

This reverts commit d333cd8f7be190d7e296721caa79478ab7ef1b20.

Change-Id: If9a28b1386fca946d8ba351c16134cdf7da8a148

Merge "mediawiki.api.login: Chain then() instead of Deferred wrapping"

Clean up state of libxml on failed import.

Make sure we always call XMLReader::close() to clean up libxml's internal state,
even if import fails with an exception. Otherwise, any subsequent attempt at importing
(or otherwise using an XMLReader) will fail with:

  XMLReader::open(): Unable to open source data

This is particularly annoying for unit tests which should be allowed to fail
without dragging down subsequent tests. Even more importantly, they may
explicitly be testing a failure case, which should not cause subsequent tests
to fail.

NOTE: Wikibase patch Id035ecebebb67 is blocked on this,
please re-check once this is merged.

Change-Id: I31c014df39aa11c11ded70050ef12a8e2c5fefc5

Fix mediawiki.startUp documentation

Follow-up to I41177b730a7370b4c9ca13d758d9f8493ff5192e

Change-Id: I4a7bfd7f4efc719bda2ae8680555c6b27e6ccfe2

Merge "Ignore EXIF data in FormatMetadata::fetchExtendedMetadata()"

Merge "Handle missing width nicely in thumb.php"

Do not call purgeExpiredRestrictions on simple page views

* Also cleaned up the on-the-fly expiry check.

bug: T88445
Change-Id: I0f601df978cc7cdd0a0b7861d39e59dae9bdda00

mediawiki.api.login: Include second HTTP request in abort()

Change-Id: I52e81781b2caf330ddb920b3ce842903808b7df3

mediawiki.api.login: Chain then() instead of Deferred wrapping

No need to wrap a promise and manually forwarding fail().
Using "then()" does this by default.

Remove redundant 'data.login' assertion. API errors are already
handled at this point.

Handle login error before success to allow a clean function exit.

Change-Id: I032226b1e2b362c61eb2c6de0c7f5ebe2ed25b7e

Use MediaWikiTestCase methods for tempdir in unit tests

* Use MediaWikiTestCase::getNewTempFile and getNewTempDirectory
  instead of wfTempDir().

  The upload api tests wrote a tempnam() file directly (where
  wfTempDir() is typically shared with other systems and concurrent
  runs). Use MediaWikiTestCase::getNewTempFile and
  getNewTempDirectory instead.

  This also ensures its removal by the teardown handler without
  needing manual unlink() calls. And it doesn't rely on the test
  passing. (Many unlink calls where at the bottom of tests,
  which wouldn't be reached in case of failure).

* For the upload test, the presistent storing of
  'Oberaargletscher_from_Oberaar.jpg' (downloaded from Commons)
  was removed. Note that this didn't work for Jenkins builds anyway
  as Jenkins builds set $wgTmpDirectory to a unique directory
  in tmpfs associated with an individual build.

* For filebackend tests, moved directory creation from the dataProvider
  to the main test.

  Implemented addTmpFiles() to allow subclasses to register
  additional files (created by other means) to be cleaned up also.

  Removed unused $tmpName and $toPath parameters in data
  provider for FileBackendTest::testStore. And fixed weird double
  $op2 variable name to be called $op3.

* Skipped parserTest.inc, MockFileBackend.php, and
  UploadFromUrlTestSuite.php as those don't use MediaWikiTestCase.

Change-Id: Ic7feb06ef0c1006eb99485470a1a59419f972545

Merge "registration: Apply ResourceFileModulePaths to all modules"

Merge "registration: Throw an Exception if a JSON file is invalid"

registration: Apply ResourceFileModulePaths to all modules

Previously any module definition that specified a "class" was skipped. This
is problematic for custom modules that extend ResourceLoaderFileModule and
should be using the default paths.

Since we can't determine at initialization time whether a module class is a
ResourceLoaderFileModule or not, just add the default paths to all modules,
and leave it up to the individual classes to use the extra arguments or not.

Change-Id: Iee5d178343326030968755983f62629b81ef3c88

filerepo: Use standard method for creating temp dir in unit test

Previously, the unittests-thumb directory (and its hashed subdirectories,
and the .htaccess and index.html files) were being left behind.

While that may be a bug in the FileBackend class, the unit test
shouldn't rely on that for its teardown. If we want to assert that,
that can be a separate test.

Bug: T89085
Change-Id: Ided46b71ace82159864145b76d98bb10da5d0311

Merge "registration: Remove unused ExtensionRegistry::loadInfoFromFile()"

registration: Throw an Exception if a JSON file is invalid

Currently you get confusing fatals like:
 Argument 2 passed to ExtensionRegistry::processAutoLoader() must
 be an array, null given

Change-Id: Icc7198db152e071c5a42e06547268fbaee916107

Make UserRightsProxy::addGroup/removeGroup return bool

Required as of e0094618b577a82209b9cc52cd15dda5e1deb784

Change-Id: I0cf9d8ee3ef9796d47442baff12a6cf8ae425d37

registration: Remove unused ExtensionRegistry::loadInfoFromFile()

Nothing calls this, I think it was left over from an earlier implementation
that did.

Change-Id: I6dd41066e040d90c8f2baa1eaf3ac3aef14b6385

Never treat persistent stat cache as "latest" in FileBackend

bug: T89184
Change-Id: I035c0f85a641efa0e4e3209b2bcf9847c08ab0d2

Merge "Remove last profiling data from unit tests"

Merge "Allow adding of context-based LESS Variables"

Merge "Moved RecentChange::purgeExpiredChanges to a job"

Allow adding of context-based LESS Variables

Add new hook (ResourceLoaderGetLessVars) called in ResourceLoader::getLessVars to
allow context-based less variables. Cache the resulting array to avoid multiple runs
of this hook.

Change-Id: I5a73bbd0ab58f8fe34519931c4f26c90998e3451

Remove last profiling data from unit tests

Change-Id: I66255ef5364f4eebafadbed663e0945e13f31969

Localisation updates from https://translatewiki.net.

Change-Id: I0af6985fe915f16fc887f4da140057cdca7d788a

Only support "daemonized" mode for redis job queues to avoid code duplication

Change-Id: I83bf61734fc48feea8a2f93adfe70f7d3a066b9e

Handle missing width nicely in thumb.php

Bug: T88508
Change-Id: I2cbe4ab914a9edba71461b194151938feeafeb11

Merge "Remove phpunit profiling. It doesn't make any sense"

Remove phpunit profiling. It doesn't make any sense

Change-Id: I7aa98ddc0a80f931a685e2946e61aa4a7dfdabff

Made BagOStuff::cas protected, which is just one merge() implementation

Change-Id: I3ef82226231f7e03f7493ae042cad22339f4c869

Avoiding writing sessions for no reason

bug: T88635
Change-Id: I4afaecd8dc29390b1ee2a6a77f9ca0cba333ef92

Merge "Add AvailableRightsTest for User::getAllRights completeness"

Merge "mediawiki.user: Simplify getRegistration() code branches"

Merge "mediawiki.util: Document optional first parameter of getUrl()"

Merge "mediawiki.util: Clean up getUrl() test cases"

mediawiki.user: Simplify getRegistration() code branches

Change-Id: Iec3663dca276f16e107192a75193b91da230b502

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

Merge "resources: Move QUnit to resources/lib/qunitjs"

Add AvailableRightsTest for User::getAllRights completeness

Because extensions often don't add their rights to $wgAvailableRights
or via the 'UserGetAllRights' hook, thus User::getAllRights is incomplete.

Change-Id: Id9ae9eff71e822ec5c038c1c1f990ac36f05cea8

Merge "Common interface for ImportStreamSource and ImportStringSource."

Merge "Separate startUp() call from main script of mediawiki module"

Common interface for ImportStreamSource and ImportStringSource.

ImportStringSource is handy for testing, but was unusable due to type hints
against ImportStreamSource. Introducing a common interface implemented by both
fixes this.

Change-Id: I820ffd8312789c26f55c18b6c46be191a550870a

Merge "Make thumb.php respond with HTTP 400 (not 500) on invalid parameters"

Update QUnit from v1.16.0 to v1.17.1

* Code
  https://code.jquery.com/qunit/qunit-1.17.1.js
  https://code.jquery.com/qunit/qunit-1.17.1.css

* Changes
  https://github.com/jquery/qunit/blob/1.17.1/History.md

Change-Id: I8078f88dcb92ee4626de80a8f0fe6596b76affd1

Merge "composer.json: Specify a specific version of PHPUnit to use (~4.5)"

resources: Move QUnit to resources/lib/qunitjs

This never belonged in resources/lib/jquery as it isn't a jQuery plugin.
It's its own package and library.

In fact, while it is a jQuery Foundation project, it does not even
depend on jQuery core (e.g. like Sizzle, and PEP).

Change-Id: Ibb00f47f4c9444870832ba8d327853f335708ae7

Workaround for HHVM bug in readline()

https://github.com/facebook/hhvm/issues/4776

Change-Id: I9d3e6817cee2fa2c11e02989a4b11915f0c708f5

Add Monolog handler for syslog UDP transport

The stock Monolog\Handler\SyslogUdpHandler only works with a specifically
tailored Formatter class and it's output cannot be fed directly into
Logstash's syslog input. This handler implements RFC 3164 in a way that
can be used with any Formatter and parsed directly by Logstash.

Bug: T88870
Change-Id: Ib098d5cb8fe9643742360bf51b54fc0e27996c0f

composer.json: Specify a specific version of PHPUnit to use (~4.5)

Using * will use the latest compatible version, which could potentially
introduce breaking changes. ~4.5 is equivalent to >=4.5,<5.0

Change-Id: I65bf0998cc028707f607ce7adaf45e27b1e6bca1

mediawiki.util: Document optional first parameter of getUrl()

There was already a test for it, but the documentation didn't
reflect it. Elaborate test to also cover case of default title
with additional query parameters.

Change-Id: I147054fc89a1a79c4aa8d3cfa861926515080a94

mediawiki.util: Clean up getUrl() test cases

Change-Id: I87e2ef7c45de349060ae671778fc7dcddd1bb2c2

Merge "Generate valid HTML code on error pages"

Merge "build: Bump various devDependencies"

build: Bump various devDependencies

grunt 0.4.2 -> 0.4.5
grunt-contrib-jshint 0.10.0 -> 0.11.0
grunt-jscs 0.8.1 -> 1.5.0
grunt-karma 0.9.0 -> 0.10.1
karma-firefox-launcher 0.1.3 -> 0.1.4
qunitjs 1.15.0 -> 1.17.1

Some minor fixes to pass new jscs rules; big ones postponed for now.

Change-Id: I5f2348c3cf96cf5aa20eeb9083300155b3920c5b

Split StatusValue out of Status class and put it in /libs

* Deprecated useless FileRepoStatus class

Change-Id: I015635a9bf080ef6d98b2cff49b949c4378a859f

Merge "Trival: Comment speling"

Merge "registration: Add resource template-like functionality to extension.json"

Trival: Comment speling

Change-Id: I17f50b5f26f5e8c1c9687c040c64699720ea6167

registration: Add resource template-like functionality to extension.json

This allows for extensions to specify common arguments to ResourceLoader
module definitions by only specifying them once.

The only supported values are "localBasePath" (which is relative to the
current directory), "remoteSkinPath", and "remoteExtPath". If a module
is using a custom class or is already specifying paths, the default paths
will not be added.

Tests are included to cover existing functionality and newly added
functionality. The convertExtensionToRegistration.php script was also
extended to try and guess what the default paths should be.

Bug: T88786
Change-Id: I802461796e8d8584dacf3b0c811b5ba97a4a8f7f

Localisation updates from https://translatewiki.net.

Change-Id: I3e3b21c3c18a8a6ae1862c263134f637acef9719

Merge "HTMLForm: Support hide-if for HTMLCheckMatrix"

Generate valid HTML code on error pages

For img_auth.php see
http://validator.w3.org/check?uri=https%3A%2F%2Fwww.mediawiki.org%2Fw%2Fimg_auth.php;No200=1

For profileinfo.php consistently use "/>" for self-closing elements.

Change-Id: I7e6aad0e7a2a745836adc430e3b2f3be5f31b6bb

Merge "registration: Fix regression in ExtensionMessagesFiles handling"

Localisation updates from https://translatewiki.net.

Change-Id: Ie8bca0b8e5832b189f299ee1534e468df969fe36

Merge "Documentation of compressOld.php"

Localisation updates from https://translatewiki.net.

Change-Id: I2ca7ae2e6545647817f47b96aa037f4c85ae848e

Resources.php: Fix phpcs-strict

Make line 1076 multiline to shorten it and
pass the phpcs-strict test.

Change-Id: I9a26d8b562cfa33881cbc6fb7447bdd53e4a71c5

HTMLForm: Support hide-if for HTMLCheckMatrix

It outputs multiple table rows, and each one needs the class and data
attribute.

Change-Id: I6667c3ea0e10167d941cb214d76889b389af79a3

registration: Fix regression in ExtensionMessagesFiles handling

In f2daeaa7494b404, the handling of MessageDirs was fixed to properly
handle arrays, except that broke ExtensionMessagesFiles which doesn't
support arrays. Since both settings are pretty much different, I split
them into separate functions.

Bug: T88858
Change-Id: I48765af2489d2ef1e30adb1d7c5a5a8f23bf522d

Merge "Fix type hint in HistoryAction::getArticle"