Verify parameter for MapCacheLRU::has() can be passed to array_key_exists()

This prevents warnings from PHP from array_key_exists().

Also make sure Title::newFromText throws an exception before it can trigger this.

Bug: T76305
Change-Id: I2b36b7a3b96b37e29fe06f69c13a185b3ec592a7

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

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"

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

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

Fixed phpunit exception test

phpunit has removed that class in
https://github.com/sebastianbergmann/phpunit/commit/f0322b6978bc2b6fd9a17f499950a4b023ef3db4
instead adding a dependency on "sebastian/comparator": "dev-master"
https://github.com/sebastianbergmann/comparator

The class name changed from PHPUnit_Framework_ComparisonFailure
to \SebastianBergmann\Comparator\ComparisonFailure\ComparisonFailure

Bug: T90880
Change-Id: I8988d0b07c00f54c8c8dd38a5e92a5736d88b163

Merge "Ignore phpcs in autoload.php"

Merge "Add batching support to fixUserRegistration.php"

Merge "JsonContent: Pass FormatJson::UTF8_OK when beautifying JSON"

Merge "Fix log message for blocks without duration"

Add batching support to fixUserRegistration.php

Also:
* Add wfWaitForSlaves() call
* Clear User cache after updating registration time
* Don't use empty()
* Use the master for everything

Bug: T92890
Change-Id: I88b97befdbd78ef12eda9a9571f6943c7b232207

JsonContent: Pass FormatJson::UTF8_OK when beautifying JSON

Currently it is not beautiful at all.

Change-Id: I82a07238b4213212b8e4a7150e35486f6dc90bc6

Tidy up tidy usage

 * There's a branch path in the sanitizer that depends on $wgUseTidy,
   which means the test output differs from on wiki.

 * In general, we should set these variables to match the wiki behaviour
   in tests.

 * Exposes T92892, Sanitizer removes empty tags when tidy is disabled.

 * Tweaked tests for T19663 to use an extension tag to show that
   HTML5 tags with non-word characters make it through the parser
   intact (before being ultimately sanitized).

Change-Id: I09c72fd739e11a8b757f37dc4c790758d782ad73

Localisation updates from https://translatewiki.net.

Change-Id: I4ca8bda44de1a3e7efec4e7322fb9659446d3988

Fix log message for blocks without duration

Very old blocks form 2004 have not any log_params and therefor enforce a
duration of infinite to be shown on Special:Log.

Follow-Up: Ibc7fcaa5a952ff90d42a6477da4baa429f3de64b
Change-Id: I4bdfab1ff865501a780ed0c7c50e2b1bab467a4b

Fix phpcs errors and warnings in includes/specials

Change-Id: I7608459fd50963fa8478b97ccd8f5a1a9d87aae8

Ignore phpcs in autoload.php

Causes 20 warnings for line length
https://integration.wikimedia.org/ci/job/mediawiki-core-phpcs-HEAD/13849/console

As being autogenerated, migth as well ignore so we can have
mediawiki-core pass.

Change-Id: Iedbe527456144c9b09b7ba93a8ce67edf0b47015

Merge "Re-introduce AvailableRightsTest for User::getAllRights completeness"

Merge "registration: Fix merging of $wgExtensionCredits"

Merge "Clean up $wgSQLiteDataDir handling and removed standalone sqlite class"

registration: Fix merging of $wgExtensionCredits

$wgExtensionCredits has two-levels of arrays, so it needs to be merged
recursively.

Bug: T91609
Change-Id: I4e5fc50059745a89fb69bc1e05a299fd9aaee968

Merge "Lower timeout of upload stash -> image scaler requests"

ApiParse: Support previewing section=new

When live previewing a 'new' section, we want to automatically add the
edit summary/sectiontitle as an H2 element, as the Edit Page and API
do as well.

Bug: T84877
Change-Id: I40925c16284bb97a4d491e12a8e7878b9d1b4810

Merge "Enable mw-ui-radio styles on Special:Preferences"

Localisation updates from https://translatewiki.net.

Change-Id: I5547cae4cf155c31b20a231e1fd7f81848e14612

Set main RequestContext title in api.php

We set $wgTitle to a dummy title, and RequestContext::getMain()->getTitle()
falls back to that so everything *works*, but RequestContext whines
about it.

Set the same dummy title explicitly on RequestContext::getMain() to shut
things up.

Bug: T78414
Change-Id: I5b90df16cf2f51a6edd5898e9712161f335202a7

Merge "Fix phpcs errors and warnings in includes/specialpage"

Fix phpcs errors and warnings in includes/specialpage

Change-Id: Ie9276c162e20b28915d31e5a87b2f18cf1c295b6

API: Return parsedsummary on onlypst

This updates the summary preview on "Show changes" for LivePreview.

Bug: T84836
Change-Id: I169fd1e1b6ed4d4f43ca69f92bc7a38e7823ddae

Merge "Fix phpcs errors and warnings in includes/site"

resourceloader: Add ResourceLoader::isModuleRegistered()

Currently if code wants to check whether a module is registered it
has to call getModule() and see if the response !== null.

Change-Id: I4b470083ddaa5d8cd6be50d5c5b690d4b99b6c4a