ImageListPager: Don't fatal when unable to thumbnail a file

Change-Id: I2e2879de2b0dd4cd347bf545c46c98b891def9eb

Merge "Convert Special:NewFiles to use OOUI."

Localisation updates from https://translatewiki.net.

Change-Id: I47121cdf937af7abdfdf27fa876926149ed08c83

maintenance: Allow having a nicer error message if an extension isn't enabled

Maintenance scripts can be invoked regardless of whether an extension is
enabled on a wiki or not. On wiki farms where some wikis may have an
extension or not, this can potentially be rather confusing. Especially
when the script bails out with a fatal class missing error or a database
table missing.

This allows maintenance scripts to specify that they require an
extension by calling:
 $this->requireExtension( 'ExtensionName' );
in the script's constructor.

Bug: T141531
Change-Id: Icfbf063bb9c9ac9e55c3a5a8ed815528a2c1ce1e

Merge "Unbreak file caching"

Unbreak file caching

This was totally broken by 796d62d0

Change-Id: I34ee08e926117b9472151d9fd415c1dfc6a57de1

Automatically detect READ_ONLY mode for MySQL/MariaDB

This avoids having users think they can make edits when an
exception will just be thrown when they try to save. Likewise
for other write actions.

Bug: T24923
Change-Id: I49c4057b672875ec6f34681a5668a509cec05677

Merge "Remove return by reference in Title::makeTitle"

Merge "Use mw.notification for success message on Special:UserRights"

Use mw.notification for success message on Special:UserRights

Split out Special:Preferences handling of successbox into it's own
mediawiki.notification.convertmessagebox module and make it reuseable.

Use this module on Special:UserRights.

Bug: T115463
Change-Id: I87054b55053d209835d6fdea1f6e3e67f10e3ac8

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

Merge "registration: Support conditionally registered namespaces"

Localisation updates from https://translatewiki.net.

Change-Id: Iab10db9d190dbbf1f46fbf9726e7b78210f4ddab

Remove return by reference in Title::makeTitle

This made sense back in php4, but has no useful effect anymore
with our minimum php version being 5.5. This also has the effect
of searching for 'function makeTitle' now actually finding something.

Change-Id: I0ea02a2e7dff285341b0e5b5b7773da78bfaaff3

Merge "Special:Search replace visualClear with mw-search-visualclear"

Special:Search replace visualClear with mw-search-visualclear

Bug: T140951
Change-Id: I41b135fd14011045fc8c9584d52d0b853c2ef49e

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"

registration: Support conditionally registered namespaces

Some extensions (e.g. EventLogging) conditionally register a namespace
depending upon the wiki's configuration using the CanonicalNamespaces
hook. Since the "namespaces" key automatically registers the namespace,
we need an option to not register it and let the extension take care of
that manually using the hook.

All of the other namespace related settings are safe to unconditionally
set since they won't do anything if the namespace doesn't actually
exist.

Bug: T141604
Change-Id: Ie8c217fdc8bd783b30f98210309ea56fef39c0da

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"