ResourceLoaderImageModule: Synchronize CSS with .background-image-svg LESS mixin

Bug: T87504
Change-Id: I790bd953fabbd4ff03f15c94fe101e2f65891520

Merge "Use ksort() in getDeduplicationInfo for sanity"

Merge "RELEASE-NOTES: Increase minimum MySQL version to 5.0.3"

RELEASE-NOTES: Increase minimum MySQL version to 5.0.3

Follows-up 5d9b67e09ed1.

Also updated the list of supported database servers in INSTALL
to match the list from RELEASE-NOTES-1.25.

Change-Id: I6881b1163455d834b058f4b4a7839a6aea44cdab

Remove unused class DBObject

This was left over from a previous attempt at Oracle support,
which was removed in 1.10 (r19196 / ccf91e827a48).

Change-Id: I5f9b82fffc0b4f84286b697be8d504020fe22547

Localisation updates from https://translatewiki.net.

Change-Id: I029a05e24e48f9b5e3893f0e964aed5e8c53280c

Merge "Fixed Style/StringLiterals RuboCop offense"

Merge "Fixed Style/HashSyntax RuboCop offense"

Merge "resourceloader: Add template compiler for Mustache JS"

Fixed Style/StringLiterals RuboCop offense

$ bundle exec rubocop --auto-correct
Inspecting 25 files
CC....C.......CCCCCCCCC.C

Offenses:

Gemfile:4:8: C: [Corrected] Prefer single-quoted strings when you don't
need string interpolation or special symbols.
source "https://rubygems.org"
^^^^^^^^^^^^^^^^^^^^^^
…
25 files inspected, 81 offenses detected, 81 offenses corrected

Bug: T91485
Change-Id: Ic6f1337fa465e8c90e5aa7ea3540eb6bfad15af8

Merge "Fixed Style/PerlBackrefs RuboCop offense"

Fixed Style/HashSyntax RuboCop offense

$ bundle exec rubocop --auto-correct
Inspecting 25 files
.CCC.....................

Offenses:

maintenance/jsduck/custom_tags.rb:16:16: C: [Corrected] Use the new
Ruby 1.9 hash syntax.
      return { :tagname => @tagname, :doc => :multiline }
               ^^^^^^^^^^^
...
25 files inspected, 8 offenses detected, 8 offenses corrected

Bug: T91485
Change-Id: Ic2aff32e106c5b5eaa45839ad3310b97fcec6ddf

Fixed Style/PerlBackrefs RuboCop offense

Refactored to use `Regexp#match`, avoiding use of match globals
entirely.

Bug: T91485
Change-Id: I4b1a40e153c18d92ef9e78291bc4e14781cc6978

resourceloader: Add template compiler for Mustache JS

Compatible library for PHP backend added by I281acc49c1.
This adds the frontend compiler.

Change-Id: I2190276a629ca882f3f5535e4e73635d2494e944

Merge "Fixed Lint/UnusedMethodArgument RuboCop offense"

Merge "Fixed Style/FileName RuboCop offense"

Merge "resourceloader: Add @covers and minor clean up of test suites"

Merge "resourceloader: Use parent getDefinitionSummary() in ResourceLoaderFileModule"

resourceloader: Add @covers and minor clean up of test suites

* Move testMixedCssAnnotations to ResourceLoaderFileModuleTest.
* Re-order data providers before test methods.
* Add relevant @covers annotations in resourceloader/ tests.
* Make test helper function private.
* Add a few @covers for methods called from OutputPage::makeResourceLoaderLink
  (only one level deep, we should have separate unit tests for
  the more internal helpers).

Change-Id: I2cc1757126214ed28059d4566ca813a86bcd95a7

resourceloader: Use parent getDefinitionSummary() in ResourceLoaderFileModule

Right now ResourceLoaderModule::getDefinitionSummary just returns an array
with the class name, but may be extended in the future. Either way, it shouldn't
be repeated in the subclass.

Change-Id: Ic1d58f8d330fb2176daffc7ad39bc0751c196181

TemplateParser: Use ObjectCache::newAccelerator() with fallback set

Change-Id: I202192f6bf509068941e9490d02a11bdedecd872

TemplateParser: make most functions protected, only expose processTemplate()

All of the other functions expose internal implementation details, which no
external caller should ever need. In fact, no external caller does use these
functions directly.

The TemplateParser::compile() tests were removed as they're simply just
checking LightnCandy functionality, which is something the library should be
doing.

Change-Id: If9003d40315e0e5aa361c174b764b799e3b88c34

TemplateParser: Don't fatal on cache misses

Also add regression test, and coverage for more methods.

Was trying to eval the code which had the hmac integrity check in front of it,
which causes a syntax error in valid PHP code.

Follows-up db1866da4, 50c50bea2e.

Bug: T93436
Bug: T93511
Change-Id: Ie90074e4885de7340e53f59fdd479f5384b5eac6

Localisation updates from https://translatewiki.net.

Change-Id: I8313b4e6f3fddaf9ee468e10972a5923bb1201d5

Merge "mediawiki.action.edit.preview: Disable if there is no #wpTextbox1"

mediawiki.action.edit.preview: Disable if there is no #wpTextbox1

ProofreadPage has custom editforms for Index pages, which are not
supported by the API. These editforms have no #wpTextbox1, so disable
live preview in that case.

Bug: T78709
Change-Id: I90cba79e5a4a756454741c4070ebdf620520ab55

Bidi-isolate numbers of added/removed bytes in changes lists

Bug: T93484
Change-Id: I26d3c46a843b2a2114ca50ec0a366fa6b76904a9

Merge "mediawiki.helplink: Use a SVG+PNG icon"

Merge "Set wgAutoloadAttemptLowercase = true for some AutoLoaderTests"

Set wgAutoloadAttemptLowercase = true for some AutoLoaderTests

Avoids:
1) AutoLoaderTest::testWrongCaseClass
Failed asserting that false is true.

2) AutoLoaderTest::testWrongCaseSerializedClass
unserialize() can load classes case-insensitively.
Failed asserting that true is false.

Change-Id: Ic5c271906fb9b2e27639aa8ecfc04ffe6788d3eb

Merge "Allow usage of mediawiki.api.options on mobile"

Merge "Remove a hack, and a hack for the hack, for MediaWiki UI input fields"

Merge "Make sure template function is callable from processTemplate()"

Make sure template function is callable from processTemplate()

Change-Id: Id9df4062754b02e7217779bc6c0ba42d457e9942

Localisation updates from https://translatewiki.net.

Change-Id: I2a30a623d1e70298d50d59fc7d2261bf638fb4db

Merge "Add unicode-bidi: embed to mw-title on changeslists"

Remove table header from Special:Tags when there are no tags

The table header shows up even when there are no tags to be
listed in the table. This patch fixes the same

Bug: T92728
Change-Id: I22d2c0435bc365f65354cea604b1c87905c769f8

Merge "Add more aliases for Special:Diff for language 'de'"

Add more aliases for Special:Diff for language 'de'

The local community wants 'Diff' as canonical form, add also the long
form for what Diff stands.
Keep the old alias, was added with:
Ic2b34a948d527cee6b76abd7e24a6d8a33b65bd2

https://translatewiki.net/w/i.php?title=MediaWiki:Sp-translate-data-SpecialPageAliases/de&diff=6066852&oldid=6049012
https://de.wikipedia.org/w/index.php?title=Wikipedia:Fragen_zur_Wikipedia&oldid=139947048#Spezial:Diff_.E2.86.92_Spezial:Unterschied

Change-Id: I504cd4b0e7c7b5c22812fe9eb0cf8dec9ff9514b

Use rspec-expectations "expect" syntax instead of "should" syntax

Bug: T68369
Change-Id: I502ba4b0d10c35aaf32fd4dcbfff8cec8ea34582

mediawiki.helplink: Use a SVG+PNG icon

Icon copied from OOjs UI library.

Bug: T45591
Change-Id: I0378e26471a72ab3a756e3cbb8c16a6878b3dbde

Add unicode-bidi: embed to mw-title on changeslists

Bug: T93336
Change-Id: I49214553f7151b2abd3d48a60b476f5792ac3a89

build: Increase qunit browserNoActivityTimeout from 10s to 60s

Bug: T89075
Change-Id: I3ef601a38052e38247cebfb19475a31ac2448eac

Merge "mediawiki.util: Add mw.util.isIPAddress"

build: Add assert-mw-env task when running grunt-qunit

Previously it gave cryptic error messages like:

> Running "karma" task
> > Warning: Parameter 'url' must be a string, not number

Change-Id: I0adb0f2f501dd5db2d6738a40efbce349f0175d2

mediawiki.util: Add mw.util.isIPAddress

Test it by reusing all the IPv4 and IPv6 cases.

Also, spelling fix

Change-Id: I57e1f00d18a74147da3200be133a4ceb989b1b9d

Allow usage of mediawiki.api.options on mobile

Change-Id: Ic5b8bc2e5e5e87cd5545812b7c20d73f630ec8e7

Update OOjs UI to v0.9.3

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

Change-Id: Iab68173bddc023b874423ad87d1dd874925a227f

Add example code showing to completely disable logging

Change-Id: I965e1ff51fe31579cf24502b148d9fad971cd298

Localisation updates from https://translatewiki.net.

Change-Id: Ib9939c0c66487999e57e72e57767612460023b04

Use ksort() in getDeduplicationInfo for sanity

Change-Id: Id23cc987b3f1e23b35b0eedbfc5c9978774a0ebd

Merge "Remove EditFilterMerged detection define"

Remove FormSpecialPage::$fields

Just use the result of FormSpecialPage::getFormFields() directly

Change-Id: I2f6f8e20b83a24e744ab4c6ea5417d9627a3e0aa

Merge "Split SiteLookup interface from SiteStore"

Merge "EditPage: Implement 'editnotice-notext' message"

EditPage: Implement 'editnotice-notext' message

Bug: T91715
Change-Id: Iff6735d93da8b877063690ff67bda652a03d4906

Update OOjs to v1.1.6

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

Change-Id: I7c1299c8049983746158a05e47a13f1f4bf7032f

build: Declare grunt-cli dependency

A long time ago, npm did not support using command line programs
within 'npm test' (not unless harcoding a path to node_modules/bin/...
within packages.json/scripts/test). But this has been long supported.

Declaring a (dev)dependency will provide its binaries to the
PATH used by the subshell 'npm test' creates internally.

Add grunt-cli so that developers don't have to manually install it.

Change-Id: I6a7fafa3c6e40d2407f07c514167ab9fc7661685

Send messages logged via MWLoggerLegacyLogger::log() to the debug toolbar

Bug: T93145
Change-Id: Ia96525faf4ffcbd2b5beae9716d6be009f2b6074

Merge "mediawiki.api.options: Add module for API action=options"

Merge "Remove unused global define RLH_FOR_UPDATE"

Remove EditFilterMerged detection define

We've supported this hook since 1.12 days and it's actually
deprecated by EditFilterMergedContent. No extensions in Git
or SVN check for this define anymore.

Change-Id: If638094a2fb2f047a3b40a24e11e2c0e2fa9c1b9

Remove unused global define RLH_FOR_UPDATE

Change-Id: I8f6df920e70ae1d94e6d9fe32cbaf468d31ffdc0

Merge "Removed odd "partitionsNoPush" setting to simplify the code"

Split SiteLookup interface from SiteStore

* SiteLookup interface is added, and SiteStore extends
  it. (any SiteStore type hints can be changed to use
  SiteLookup if all they need is lookup functionality)
* Memcached based SiteStore code is split from the
  database SiteStore, and SiteSQLStore is deprecated.
  If no caching is desired when using a SiteStore, then
  use a SiteDBStore instance, instead of passing $source
  parameter in SiteStore::getSite and SiteStore::getSites.
* SiteListFileCache renamed to FileBasedSiteLookup and
  implements SiteLookup.

Bug: T77990
Change-Id: I36b599884c211580ea6806a8a190c65c4f9087cf

mediawiki.api.options: Add module for API action=options

Implemented mw.Api#saveOptions to save user preferences.

If necessary, the options will be saved using several parallel API
requests. Only one promise is returned that resolves when all requests
are complete.

If a value of `null` is provided, the given option will be to reset to
the default value.

Any warnings returned by the API, including warnings about invalid
option names or values, are currently ignored. This basically means
that all requests will succeed (barring networks problems, internal
server errors and such).

Change-Id: Ia015898ca910923e00bc53f099b4e5631d6ad45c

Merge "Force LocalFile::purgeMetadataCache use the master DB"

Fixed Lint/UnusedMethodArgument RuboCop offense

$ bundle exec rubocop --auto-correct
Inspecting 25 files
.W.......................

Offenses:

maintenance/jsduck/custom_tags.rb:14:26: W: [Corrected] Unused method
argument - position. If it's necessary, use _ or _position as an
argument name to indicate that it won't be used.
  def parse_doc(scanner, position)
                         ^^^^^^^^
...
25 files inspected, 2 offenses detected, 2 offenses corrected

Bug: T91485
Change-Id: Ide96774c284af97f8d587bd6cde302c4500496ed

Force LocalFile::purgeMetadataCache use the master DB

This used to happen automatically via markVolatile() but that was
removed in I808c3a6, which caused cache updates after move/delete
to retain outdated information from slaves.

Bug: T93009
Bug: T88506
Change-Id: Ie4a6c5fbecec9b38ad3ff45fbc22c5d74d66601c

Fixed Style/FileName RuboCop offense

$ bundle exec rubocop
Inspecting 25 files
.C.......................

Offenses:

maintenance/jsduck/CustomTags.rb:1:1: C: Use snake_case for source file
names.
^

25 files inspected, 1 offense detected

Bug: T91485
Change-Id: I91b43362f265a05284495727dab5465e1c4f0542

Merge "Add framework for file warnings"

Merge "Drop outdated "documentation reviewed" tags"

Merge "Fix and make PHPDoc tags in FileBackend more specific"

Merge "Drop not needed @dataProvider doc tags from MediaWikiTitleCodecTest"

Drop outdated "documentation reviewed" tags

We review documentation all the time. Even if this was a big, notable
review, it was 5 years ago. It's probably outdated again, e.g. because
methods changed but the corresponting documentaion did not. In my
opinion the fact that a review happened 5 years ago is not useful any
more.

Change-Id: I6f4fb88ea790520bf2443aae4144cdde394b5e78

Fix and make PHPDoc tags in FileBackend more specific

A pure documentation patch. Should almost be a no-brainer. ;-)

Change-Id: I9bf4d6056f7cde193fe5882a04a7973af9b5a7f4

Drop not needed @dataProvider doc tags from MediaWikiTitleCodecTest

No other @dataProvider in this file does have it's parameters documented.
We usually don't do this because it doesn't add useful information and
is not possible in all cases anyway, e.g. in providers for tests that
are expected to fail. The @param tags would be "mixed" there.

Change-Id: I12a44482e7934a4a18d284d57bf510cb0467cee3

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

Don't log HttpErrors in the exception log, use MWLogger

Bug: T85795
Change-Id: Ic36f657a6447dd99c78681536e5dfb066d97b409

Add framework for file warnings

See I674bf7f6c1b21ffc9870aa84382479af5f966561 for an example in the
PdfHandler extension.

Bug: T89765
Change-Id: I3c4b7af7284b5e16e458dd72de789e74db489895

Fix exit code from rebuildLocalisationCache.php

When --threads was used, the exit code was unconditionally 0, even if a
child thread threw an exception, because the exit code was "abused" for
the count of rebuilt languages. So, don't do that. We don't really need
a count of rebuilt languages that badly, just pass through the exit
status of the child.

Bug: T92900
Change-Id: I77922e686fe7da4fc25bb7f579ed70e1b107eaca

Merge "Add help link to three other "minor" special pages"

Localisation updates from https://translatewiki.net.

Change-Id: I0dcd569bbef7ee98f9341af08a2c3d19e7a02e4f

Fix "Bad value 1 for attribute autofocus" on special pages

Xml::inputLabel does not work with boolean parameters, therefore change
true to an empty string to avoid the '1' in html.

This is done for Special:BookSources (from the bug),
Special:ExpandTemplates, Special:Undelete and Special:Userrights

Bug: T68134
Change-Id: I53ff5701840750f17a2ea9a1ec56cd88c00f3cf5

Migrate import log to new log system

There is no need for an extra formatter class, because at the moment
there is nothing extra to format.

This allows use of gender on Special:Log. Old messages are kept for use
in IRC. A test was added to ensure an unchanged IRC message.

The new log message for import-interwiki is used from
I3ca8b21bce49b41cac7109efb8056ca4469b88d7 to avoid "transwiki jargon".

Bug: T57404
Change-Id: I590075b203548c071a98c1eaa4b1bc9250f60e22

Add help link to three other "minor" special pages

Bug: T45591
Change-Id: If8efa314fbd8f7469aa018135ad0f611f8806b48

Merge "Truncate null revision comment for whole multibyte characters"

Emergency remove .mw-headline-anchor

Follows-up 6c7480e5f0, d2a6a73d2.

The should stop the § character from being indexed by Google et al.

Bug: T18691
Bug: T93000
Change-Id: Ie9e334e973e3ded270f1897a2c3816d9df739fc0

Merge "Optimize order of styles and scripts"

Merge "Fixed timestamp comparison in fixUserRegistration"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 9d213e1eb420bfd9ef6fbc661556b40c8aee32f6

Change-Id: Icb9de1ae26a7148fb0fe68840c5a9cfd45640560

Merge "Tidy up tidy usage"

Merge "API unittests, pass optional params to doApiRequest()"

Merge "New testing wrapper to circumvent object access"

New testing wrapper to circumvent object access

The new TestingAccessWrapper class provides a convenient way to make
all of an object's methods and properties public.

TODO: We should organize test helpers into a source directory.  Note that the
helper and its test are in the same directory.

Change-Id: I958d55df18c74e9d2b25d98cd0316989a0fbbe6f

Fixed timestamp comparison in fixUserRegistration

* Also optimized batching logic slightly

Change-Id: I0cedb5b39f996b7baec067890f230cff51468e4b

Optimize order of styles and scripts

The current ordering of scripts and stylesheets in <head> causes all major
browsers to serialize and defer requests that could be performed in parallel.

The problem is that external stylesheets are loaded before inline scripts. As
Steven Souders explains, "all major browsers preserve the order of CSS and
JavaScript. The stylesheet has to be fully downloaded, parsed, and applied
before the inline script is executed. And the inline script must be executed
before the remaining resources can be downloaded. Therefore, resources that
follow a stylesheet and inline script are blocked from downloading."[1]

In other words: the browser could start loading body images, but it refuses to
do that until it has executed inline scripts in head. And it refuses to execute
those scripts until the external CSS is downloaded, parsed and applied. You can
see the effect of this in this image, showing the request waterfall for
[[en:Gothic Alphabet]]: [2]. Notice how no images were requested before the
browser had finished processing the three load.php requests at the top.

To fix this, we want to move the inline scripts above the external CSS. This is
a little bit tricky, because the inline scripts depend on mw.loader, which is
loaded via an external script. If we move the external script so that it too is
above the external stylesheet, we force the browser to serialize requests,
because the browser will not retrieve the external CSS until it has retrieved
and executed the external JS code. So what we want is to move the inline
scripts above the external stylesheet, but keep the external script (which the
inline scripts depend on) below the external stylesheet.

We can do this by wrapping the inline script code in a closure (which binds
'mw') and enqueuing the closure in a global array which will be processed by
the startup module at just the right time.

Net result: external CSS and JS is retrieved in parallel, retrieval of images
(and other external assets) is unblocked, but the order in which code is
evaluated remains the same.

[1]: <http://www.stevesouders.com/blog/2009/05/06/positioning-inline-scripts/>
[2]: <http://people.wikimedia.org/~ori/enwiki-waterfall.png> (excerpted from
 <http://www.webpagetest.org/result/150316_0C_7MB/1/details/>.

Change-Id: I98d383a6299ffbd10210431544a505338ca8643f

API unittests, pass optional params to doApiRequest()

ApiQueryTestBase::check() should pass all optional params
like session and user to the doApiRequest()

Change-Id: If1cb52b03faba06c97d330a91a23893fe08acf22

Merge "Regenerate autoload.php"

Regenerate autoload.php

Follows-up 87dfc20b1e.

It recently started generating dirty output because
BufferingStatsdDataFactory was manually placed – in a different
position than script would (i.e. not alphabetical).

Change-Id: I030a6ec9eb6f3a9b6cb54c9646acf2a898b93a80