Merge "Add Wikidata to default interwikis"

Merge "API: Document a level of 'all' in action=protect"

Merge "API: Remove deprecated response values from action=login"

Merge "Add User::findUsersByGroup()"

Add/update doc blocks for MWTidy

Change-Id: I0b87e119048fd993f8bfda25a6c6b744d59804d1

Merge "Clean up array() in docs, Part I"

Merge "ParserTest: Declare $recorder property"

resourceloader: Log each deprecation only once from mw.log.deprecate

Bug: T100412
Change-Id: I0111f0ed84b74774b6665aca2625d07ea53cfc8c

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

Localisation updates from https://translatewiki.net.

Change-Id: I8bc7496b6c811cd32810d67239ce7788c4934e71

Merge "Improve how slashes are stripped from filenames"

Merge "ApiParse: Deprecate prop=headitems"

Rename OpenSearch qiprofile parameter back to profile

In I6f987db this parameter was inadvertantly renamed to qiprofile. The
qiprofile is something different, and is also a BC break. The BC break is
pretty minor, for a randomly chosen day this looks to be used 157 times
out of 28.7M requests, but since this really isn't a qiprofile rename
it back to what it was.

Change-Id: I1cc07945888c15ea4b4c9596eea447b706606fae

ApiParse: Deprecate prop=headitems

This code has been rotting for a while. It is important missing stylesheets
and critical inline scripts. Consuming these items is quite risky.
One can't safely use it to update an existing document (as some parts are
duplicative or conflicting). One also can't use it to build a new document as
several critical pieces are missing.

* Since ResourceLoader was introduced in 2011, buildCssLinksArray() doesn't have any stylesheets.

For most pages, this will produce an empty array for headitems.

Mark it as deprecated since in favour of 'prop=headhtml' or 'prop=modules'.

Change-Id: I6084e4454e245f59bf3c82d6db830977725f71e5

Use isset in WikiPage prepareContentForEdit for newContent

This will stop the following:

Undefined property: stdClass::$newContent in
includes/page/WikiPage.php on line 2056

Bug: T75474
Change-Id: I1bc840faff1b4dc1444434b4edec223950bc6eb1

Merge "Fix Undefined variable issue in ApiQueryUserContributions"

Merge "Remove missed WatchedItem::resetNotificationTimestamp in test"

Remove missed WatchedItem::resetNotificationTimestamp in test

Change-Id: I3ece74fecd55f248025d9c10aaf1960d5cd3fd7a

Merge "Use display name in category page subheadings if provided"

Fix Undefined variable issue in ApiQueryUserContributions

Some ApiQueryUserContributions::prepareQuery code paths don't define
the $index array, but try to use it anyway.

That raises the following notice:
     Undefined variable: index
     in …/includes/api/ApiQueryUserContributions.php on line 340

This commit ensures we only try to use the variable when needed.

Bug: T141497
Change-Id: Ib9c038b84219bc1886139cb079a33aba74bc5220

Merge "Improve WikiMap::getWikiReferenceFromWgConf()"

Localisation updates from https://translatewiki.net.

Change-Id: Ia932678049b7bcd615d9c1e74490f6014e92738b

Add User::findUsersByGroup()

Needed by Iee6e0074d64.

Change-Id: Idbaf2102410f82343ad38ff51ccb7ab70b15e654

Merge "Remove AutoLoader::loadClass()"

Merge "Don't run the non-Tidy "bug 2702" hack unless Tidy is really missing"

API: Remove deprecated response values from action=login

Bug: T121527
Change-Id: Ie370d6dc5a922ad9c50758ba81b98aa8492f83cb

Merge "Adding support for numeric collation when using UCA collations"

API: Document a level of 'all' in action=protect

It should be mentioned in the documentation for the 'protections'
parameter that 'all' exists to explicitly allow everyone to use the
action.

And the example using it seems to confuse some people into illogically
thinking that "action=all" means "restrict from all" rather than
"restrict to all" even though every other possible value means
"restrict to X", so reiterate it there too.

Bug: T141367
Change-Id: I2c3a2bf193c2ccb1ed28991ac785f7fee4948e6a

Merge "Push common search api parameters into SearchApi class"

Don't run the non-Tidy "bug 2702" hack unless Tidy is really missing

We have two hacks which are used when Tidy is not available: one in
Sanitizer::removeHTMLtags(), and the second here as a late Parser pass
equivalent to Tidy itself. But the Sanitizer one was enabled only if
MWTidy::isEnabled() returned false, whereas the Parser one was enabled
also when tidy was disabled in ParserOptions. This patch makes them both
consistent, it enables the bug 2702 hack only when MWTidy::isEnabled()
returns false, and when Tidy is disabled in parser options, the output
is simply passed through.

This allows tidying to be done separately on the ParserOutput, as is
required by the proposed ParserMigration extension (I24d0776a933fa3f).

Eventually the bug 2702 hack will be removed in favour of a pure-PHP
HTML 5 parser, but it looks like it is too early for that.

Change-Id: I94be6c9dec531c23ef80cb36732243bd6858bf22

Remove web-request usages of deadlockLoop()

Add comment discouraging use of the method. There are no problems
with deadlocks/timeouts in these code paths according to WMF logs.

Change-Id: I5b21cc423df584efa881361063000e01932cc2ea

Merge "Make revision deletion acquire file locks to avoid races"

Merge "Make onTransactionIdle() safer for multi-DB commits"

Merge "Only pretty-print the parser report JS vars"

Adding support for numeric collation when using UCA collations

To use, add '-u-kn' to the end of a collation name and set it as
the value for $wgCategoryCollation.

Bug: T8948
Change-Id: Ica7908daf80624fa2648127114d01665e96234c0

Only pretty-print the parser report JS vars

Follow-up to b7c4c8717f9.

Change-Id: I3030df4ed890d9d20daca31fbd33c6d9db91b94c

Make revision deletion acquire file locks to avoid races

Also made RevisionListBase an Iterator to avoid ugly loops here

Change-Id: I40d2d8cf63df95c59d0e1275e3ec45aff238e1cb

Make onTransactionIdle() safer for multi-DB commits

* For DBs managed by LBFactory, they will trigger after the COMMIT
  step finishes for all DBs.
* Make commitAll() respect the same logic as commitMasterChanges().
* Add LoadBalancer::forEachOpenMasterConnection() convenience method.
* Rename logMultiDbTransaction() to less confusing logIfMultiDbTransaction().
* Various other small code cleanups.

Change-Id: I6c9b40ba8b9e7600cce774f26b9c401e60fa8803

Merge "Add comment to ChangesList::showCharacterDifference"

Clean up forEachOpenConnection() loop and use the method more

Change-Id: I431b68c763970101804e09ea6634480e1ba79cda

Localisation updates from https://translatewiki.net.

Change-Id: I288d171b23b6376328c90c95409801c76147acf1

Move NewPP limit report HTML comments to JS variables

* Instead of having messy code to create a hidden HTML
  comment of English strings at the bottom of the page,
  expose the structured data of the parse information
  to JS so tools can use it.
* Make makeConfigSetScript() use pretty output so these
  variables are also easy to read in "view source".
* Remove ParserLimitReportFormat hook, since the data
  is not formatted to HTML anymore.

Bug: T110763
Change-Id: I2783c46c6d80f828f9ecf5e71fc8f35910454582

Push common search api parameters into SearchApi class

We have a number of parameters that are pretty much the same between
these different search api's. Lets make them actually the same by
sharing the definitions, and then letting individual classes tweak them
as needed by removing the offset, or adjusting the max limits as
necessary.

Change-Id: I6f987db8ecb63dc943b4d2518bfe3703c677448e

Merge "Don't magically make MediaWiki\Widget classes infusable"

Mark Linker::link() as @deprecated

...and add a link to the on-wiki LinkRenderer documentation to the
class. Actual wfDeprecated and replacements in all the uses to come in
follow-up patches.

Change-Id: I2fefb6c38153028d2cc68c13bc62434d3df69cd2

Merge "Apply $wgMaxArticleSize more exactly"

Merge "API: Use rev_user when possible for list=usercontribs"

Merge "Make importTextFiles.php work with wildcards on the Windows shell"

Merge "WebRequest::getText(): Update more of the doc comment"

Merge "mediawiki.htmlform: Remove deprecated $.fn.liveAndTestAtStart"

Merge "syncFileBackend.php: ellapsed => elapsed"

Merge "SectionProfiler: Remove $profileOutCallback"

Merge "Remove modulemessages from ApiParse and Output (deprecated in 1.26)"

Merge "Add MWTidy::factory()"

Don't magically make MediaWiki\Widget classes infusable

For consistency with how normal OOjs UI widgets behave.
Only one usage outside of HTMLForm needed updating. There
are no usages in extensions in Gerrit.

Change-Id: I9b5de86307a13ca7355b7541e8683e4d6a10431f

Remove unused deprecated WatchedItem methods

Not used in core or in any extensions on Gerrit

Change-Id: I10f0e98f52189f60480f5adb824a5cca9820d5f5

Renamed MWCallableUpdate class file to match the class

Change-Id: Ia595cee94a696b83b00fe9f2a91b49bce51c08dd

ParserTest: Declare $recorder property

Change-Id: I68df25e44235de02c1c8adc9b8ed13505aadf45a

Add MWTidy::factory()

A convenient factory function to eliminate code duplication in
ParserMigration's MigrationEditPage::tidyParserOutput().

Change-Id: I058912885025e7a9402912236c65c44e32ef036e

Merge "Convert onTransactionIdle() callers to DeferredUpdate subclasses"

MediaHandler: Fix @return for getHandler()

Change-Id: I9373501e0055d0cddcb6c5bf72f5496ed22133c3

Convert onTransactionIdle() callers to DeferredUpdate subclasses

* This puts the complex logic here after the commit step for
  all DBs, making the main multi-DB transaction more likely
  to be atomic. Previously, the idle callbacks could be hit
  bewteen DB commits.
* Enforce transactionality via AtomicSectionUpdate.
* Use $this instead of $that hacks for old PHP versions.

Change-Id: Idf7d54fdac6487f86907099680f5c1c4f5530b4e

Merge "Update justinrainbow/json-schema in require-dev to ~1.6"

Remove modulemessages from ApiParse and Output (deprecated in 1.26)

No uses of 'modulemessages', getModuleMessages() or addModuleMessages()
anywhere in Wikimedia Git.

Change-Id: I59420880f3545d1aabf9bcbea1e34b1475697d26

Merge "Avoid use of DB rollback() in LocalFileMoveBatch"

registration: Fix "config" schema validation

The "properties" field is only interpreted if the value is an object,
which was not being required. Add that missing type in, so
manifest_version 1 type config entries should fail.

Bug: T141289
Change-Id: Iffb084eb7a81a398a74370a1f9561198a88f5779

Avoid use of DB rollback() in LocalFileMoveBatch

* Verify the DB updates and bail before doing anything instead
  of relying on rollback() if something does not match up.
* Do the file copying before updating the DB so that there is
  nothing to rollback if they fail.
* Improved failCount in case the current version is missing.
  It should also reflect all the missing old versions.

Change-Id: Ie2d316548d8e5584cc69bb9f1425db108b05be5c

Count edit conflicts for each namespace separately

Bug: T140368
Change-Id: I5591f53063db32b2c74a2f74751d4af0ef5424d7

Add addition of 'maxarticlesize' in ApiQuerySiteinfo to RELEASE-NOTES

Follows-up I25a4c1b9013f98e5b0a263a637c8b006dccfd5ad.

Change-Id: If3ffa6635e18e53a642ebaa0b93c4c265612299f

Merge "Make onTransactionPreCommitOrIdle() atomic when immediate"

Localisation updates from https://translatewiki.net.

Change-Id: Iea286077578232c907a4bddf4516c3fe9994397f

Merge "objectcache: Disable RedisBagOStuff constructor in unit test"

Make onTransactionPreCommitOrIdle() atomic when immediate

One of the main uses of this method is to tack on some DB
updates to the end of atomic transactions to reduce lock
contention problems. In this case, multiple updates in the
callback are atomic. However, if no transaction is active,
then such updates were previously not atomic. Wrap them in
transactions now.

Change-Id: I18c56e65063a61000c4dfd1979bf972800b173ac

Merge "objectcache: Add callable doc comments for WANObjectCache"

objectcache: Add callable doc comments for WANObjectCache

Change-Id: I28a705ae12e065dc8a3921e33c873f9a7fbb7490

MW_DIFF_VERSION is globally defined, drop MW_ prefix when used as
the class constant

Bug: T141272
Change-Id: I0a9e9037b2259515df6366184cc781ecb78f424c

objectcache: Disable RedisBagOStuff constructor in unit test

Follows-up c4e698dc2. The tests are failing on Travis CI because
php5-redis isn't installed by default. These tests shouldn't need
that to be installed (and even then, we should skip gracefully as
it is an optional dependency).

> RedisBagOStuffTest::testUnserialize with data set #0
> Exception: RedisConnectionPool requires a Redis client library.
> ./mediawiki/includes/clientpool/RedisConnectionPool.php:86

Fixed by using a mock and disabling the original constructor.

Change-Id: Icced7c30a75516c2118489ad29eac2aa5cff80ad

syncFileBackend.php: ellapsed => elapsed

As in c099155a17d26f03 for copyFileBackend.php.

Change-Id: Ifbc0673ee092f782de654eeca636d3b1ec061ae7

Merge "Revert "mediawiki.special.userlogin.signup: Remove unnecessary field hiding code""

Revert "mediawiki.special.userlogin.signup: Remove unnecessary field hiding code"

Whoops, we actually still have the option to bring back the old
custom HTML login forms ($wgDisableAuthManager). Too early.

This reverts commit 23e5bcb767c57d3a3f90b9fc597b87c8706a4027.

Change-Id: Ifb3ab1bf27ba2190177b65f5442dde74d20140e9

SectionProfiler: Remove $profileOutCallback

Unused since 626aede99bf83733.

Change-Id: I12c0d7900cd81d8b33b1a5a8dbe866ddd9664083

Clean up array() in docs, Part I

Change-Id: Ia6bb3944c05b056677979035cb38385554ee8a4f

Merge "mediawiki.special.userlogin.signup: Remove unnecessary field hiding code"

mediawiki.htmlform: Remove deprecated $.fn.liveAndTestAtStart

I don't think anybody ever used it anywhere outside of this file.

Change-Id: I19ad0e81c0cb262a03fa10d45eaa951cdcc9848a

mediawiki.special.userlogin.signup: Remove unnecessary field hiding code

We now use HTMLForm, which implements a generic method to do this
using 'hide-if' parameters. We already use it for the signup form too.
This is just dead code.

Change-Id: I35cfc31aae714f3be8aa0d044de1119c279e3d34

Merge "Allow providing 'notices' for OOUI HTMLForm fields"

Add Wikidata to default interwikis

Change-Id: I558e9a0b6216ab91ef5fd5976b8718db51d1ffe0

Improve how slashes are stripped from filenames

* Add slash and backslash ('/' and '\') to $wgIllegalFileChars.
* Replace illegal chars before removing paths in wfStripIllegalFilenameChars().

This way users trying to upload a file with slashes in the name will
get a better filename suggestion (e.g. for 'Foo part 1/3.jpg', you
previously got '3.jpg', now you'll get 'Foo part 1-3.jpg'). Uploading
tools that don't special-case slashes will also behave better.

Change-Id: Ib78f48a5f8c92e8ab2dc773ea6789b96b3662177

Localisation updates from https://translatewiki.net.

Change-Id: Ie72f1651a43be65682e451a1ee1c44fc14f31f67

Minor transaction callback cleanups

* Give the flush commit() in close() a chance to handle callbacks.
  Also detect onTransactionResolution() callbacks when an error is
  thrown.
* Avoid resetting callbacks redundantly, as they already get
  cleared when they actually run.
* Fix a few doc comments.

Change-Id: I5c62262661b97a6c28ce0ab0949307dc119953ca

Update justinrainbow/json-schema in require-dev to ~1.6

This matches the version that Kartographer is tracking, and hence
the committed vendor repo

Change-Id: I92bc1d03e62482bf9de4903ac14f79a4340ed1f2

registration: Support config setings that are relative paths

Add support for configuration settings that are relative paths to the
extension directory. The most common use-case for this is where the
extension ships a generic default file, but sysadmins or packagers may
want to override it. Setting `"path": true` on the config definition
will make the value be interpreted as a relative path.

And add tests!

Bug: T100956
Change-Id: Ia87ced25b35be7a314ee2937e0ccc63f9acc5bb5

Merge "Session: Improvements to encryption functionality"

registration: Fix merge_strategy in manifest_version 2

merge_strategy is set in $data, which is the definition of the config
setting, not the default value of it ($value).

Change-Id: I12ee95892b2c157bb0beb97cf2cd5ef920b7d52e

registration: Update conversion script for manifest_version 2

Config settings are now an object, with the value set as the 'value'
key.

Change-Id: Iba3999fdfc8994d50331825c9a416334db602e46

Localisation updates from https://translatewiki.net.

Change-Id: I7912884168f4cae479c1e6f674961feedbb261e6

Merge "Use current content model for blank page content (not title default)"

Merge "filebackend: Fix DBLockManager IDEA warnings"

filebackend: Fix DBLockManager IDEA warnings

Change-Id: I5e671bcee0915f65082c139be8f42aafad6413f1

Merge "Make statsd counts for MWCallableUpdate actually useful"