Merge "Fix the last Generic.Files.LineLength phpcs failures"

Localisation updates from https://translatewiki.net.

Change-Id: I2c6d6bb15910feb1b8da5565bc27bfe7878bc0e4

Converted DatabaseBase::mTrxAtomicLevels to an array

* This simplifies the code and cleans up __construct()
  a bit while also making it less likely to cause
  trouble with unit testing mocks.
* Also add a sanity mTrxLevel check around usage
  of mTrxAtomicLevels in the one place it was missing

Change-Id: Ia0a7f22f5c27b3d4d8b51e04629f42a1ed9c3993

resourceloader: cache minified user and site modules

* Add support for a '/* @nomin */' annotation in ResourceLoader. If present in
  JavaScript or CSS, the code will not be minified or cached. This allows
  modules like the ResourceLoaderUserTokensModule to declare themselves unfit
  for minification / caching without requiring a complicated refactor.
* Make ResourceLoader::filter() static, at the cost of not having minifier
  errors in the ResourceLoader log bucket. (They will continue to be logged as
  exceptions, however).

Change-Id: Ic1d802ee20565e61046bfbd8fd209bc56a4cbd6c

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

Localisation updates from https://translatewiki.net.

Change-Id: Ie3f19a2b5cea902108126c528fb4c8f489a4d551

Merge "Rename ObjectFactoryTest_Fixture to camel case name"

Rename ObjectFactoryTest_Fixture to camel case name

Change-Id: Ib909c4f694b9774a0983abbb23bfd38ed8c05d5b

Enable MediaWiki.NamingConventions.PrefixedGlobalFunctions.wfPrefix

Should work after Ie094f78781b2d6f7af1a0f1eea1f7ea4d0b9c69c

Bug: T104501
Change-Id: I1e3e8a1480b822272c762f2771d8554980c32c1b

Enable Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed

All remaining lines should be fixed now

Bug: T113854
Change-Id: I85269d4c84ae3a676a773e24b0a96a4e7a89ac38

Merge "Fix ObjectCache::newAccelerator with string fallback in PHP 5.3"

Fix the last Generic.Files.LineLength phpcs failures

In both cases the long lines are inside method documentation,
and because @codingStandardsIgnoreStart comments only
work with // and not /*, they have to be outside the method comment.

Bug: T102614
Bug: T114213
Change-Id: I7343e1cf662c5bf12a5e4385646c5c2c4a798260

Fix remaining Generic.Files.LineLength failures

Except the ones for which there is no clear way to fix
because of T114213.

Bug: T102614
Change-Id: I94a1fc7c890c82a36f0b51e42d311585c275775c

Fix Generic.Files.LineLength phpcs check under phpunit/includes/api

Bug: T102614
Change-Id: Ic0b3d9b58fee8dc20a976ae65dcb7072a3afc3b1

Fix ObjectCache::newAccelerator with string fallback in PHP 5.3

Because in PHP5.3:

php > $a = "hash";
php > echo isset( $a['fallback'] );
1
php > echo $a['fallback'];
h

This will fix using MediaWiki with MySQL on PHP5.3
if neither APC, xcache nor wincache are available.

Change-Id: Iebf034be75b282e2654cd298713455caf062eda4

Fix Generic.Files.LineLength phpcs check in files under phpunit/includes

Bug: T102614
Change-Id: Iee3df5f064f595ecebe8210cc936bc3d20a122c9

Fix Generic.Files.LineLength phpcs failure in 11 files under includes/

Bug: T102614
Change-Id: I0d759be6ef568c2c6f28606d3002484ad77a1830

Merge "Fix spelling of "fragment" in test descriptions"

Merge "Make lines short to pass phpcs in WikiReferenceTest.php"

Merge "Make lines short to pass phpcs in WikitextContentTest.php"

Merge "Add @codingStandardsIgnoreStart to 5 files under tests/phpunit/includes"

mediawiki.widgets: Add title attribute to TitleOptionWidget description

As the description is likely truncated, this will provide
the full description on hover.

Change-Id: I0abce42a2582e2ac8d0db5a7fb98adeb3eadf62c

mediawiki.widgets: Fix TitleOptionWidget styles

* Make the description display:block so ellipsis works
* Fix assumption that option is a child of titleWidget,
  instead make a child of titleWidget-menu.

Bug: T114565
Change-Id: I033c83b249c710b6a2a54889501174ad56d971ce

Fix spelling of "fragment" in test descriptions

Change-Id: I97ca75b9c587443e195b1c162dcfee7880a5e226

Make lines short to pass phpcs in WikiReferenceTest.php

Bug: T102614
Change-Id: I557744a48a6a6889fde0b050d8e646ba081d5504

Make lines short to pass phpcs in WikitextContentTest.php

Bug: T102614
Change-Id: Id5f12d4cca1d952580d2bb68fdbcb3cfb2467085

Add @codingStandardsIgnoreStart to 5 files under tests/phpunit/includes

To pass phpcs.

Bug: T102614
Change-Id: I4c7d60d26112c7ba67b66f1923ce945e96175d15

Converted InfoAction::pageInfo() to using getWithSetCallback()

Change-Id: I5b780d6e21d9fc7f38d365db1389153b07c6c461

Apply $wgReadOnly to all file backends

* Also added a wfConfiguredReadOnly() method
  to avoid DB_SLAVE connections

Change-Id: I9e7ec95c4b2f763505166d2345d27abaef6257a3

Merge "Add page purge limiter"

Localisation updates from https://translatewiki.net.

Change-Id: Iac862f8797a3fb3be6b7be0e2a0cab1bac88b245

Various getHTMLDebugLog() cleanups

* Removed used Entering/Exiting stuff
* Removed bogus empty entry on the first line

Change-Id: I0ade132e04d745a21f4ec9fdd52a43b10d85e4d3

Add page purge limiter

Change-Id: I737dc77ce803432b0792f7e49323a29661bc056f

Merge "Clean up getLagTimes() docs"

Merge "mw.Upload.BookletLayout: Show errors and try to recover from warnings"

Clean up getLagTimes() docs

Change-Id: Icc11619136b6ed10f507e25cd0798da975af4ad1

Merge "Remove redundant ApiQuery::shouldCheckMaxlag"

Remove redundant ApiQuery::shouldCheckMaxlag

* Parent version already returns true

Change-Id: I118d59492b8dadd3c7128d2c74f6d37216206d0a

objectcache: Implement 'fallback' param for newAccelerator()

This way it can be used in $wgObjectCaches using 'factory', whilst
also specifying a fallback.

Change-Id: I8d4de1e2a394e7e915513fae91808fe32a21f75d

Added pt-heartbeat support to DatabaseMysqlBase

Bug: T111266
Bug: T19179
Change-Id: I04c93d5c7389fbb8fba633d955591b30a9504f31

mw.Upload.BookletLayout: Show errors and try to recover from warnings

Show useful errors and try and recover from warning by changing the
state of the upload.

Bug: T114130
Change-Id: I08704108704ffaa70245c5facfca4b063a61f01b

Merge "mw.Upload: Add details of error when uploading to stash"

Don't clobber existing styles when adding more in IE9

Follow-up to c2e8f505c12ace6af42161bf9930d483b1721f2e. This was really
stupid and I feel stupid.

Bug: T114323
Change-Id: I80292083d8fd50c2e2727446e4468b6a13ad40e5

mw.Upload: Add details of error when uploading to stash

Add a new method 'getStateDetails' to mw.Upload. The Upload.Booklet
layout uses this in the error message.

Bug: T114130
Change-Id: I932af41a7ae561774097c91f857d1daa39d8c49f

Merge "Fix-up for I21bb3f08e7f: make ResourceLoaderModule::expandRelativePaths public"

Merge "build: Enable jscs jsDoc rule 'checkTypes' and make pass"

Fix-up for I21bb3f08e7f: make ResourceLoaderModule::expandRelativePaths public

Change-Id: Iba76d7e788c3ca5457fe462b7fd9ecfaedba50f8

Fix-up for I21bb3f08e7f: ResourceLoaderModule::expandRelativePaths

ResourceLoaderModule::expandRelativePaths(), not ResourceLoader::expandRelativePaths().

Change-Id: Ibb2a8398641d360240889988189d2ef51f114c0b

Merge "resourceloader: Fix-up missing expandRelativePaths() in preloadModuleInfo"

Modernized slave wait checks in recompressTracked.php

Change-Id: I5771af8135530677dc8223389f572896918c6b72

resourceloader: Fix-up missing expandRelativePaths() in preloadModuleInfo

Follows-up 280d292.

Change-Id: I21bb3f08e7fc9c6b364df2934640c2d5ce1e86b0

Added replication=async option to FileBackendMultiWrite

* This will defer writes to non-master backends till the
  end up the web request. This is useful for multi-DC setups.

Bug: T112708
Change-Id: I118c07764dd4a4f4f2590d4548238df12860e750

resourceloader: Minify per-module instead of per-response

* Decline to cache minified private modules, because they exist in as many
  variants as there are users, and are unlikely to be cache hits now that we
  use APC.
* Other modules are minified individually, to improve cache hit rate.

Bug: T107377
Change-Id: Id6f5142062d73b5701126724e0fe8264105f7813

Merge "mediawiki.Uri: Support names of Object prototypes as keys in query"

mediawiki.Uri: Support names of Object prototypes as keys in query

JSHint W001 has to be disabled to avoid warnings about a key named 'hasOwnProperty'.

Bug: T114344
Change-Id: I7a58204c98d7a490906c6a23620e9f24ebb3db0a

build: Enable jscs jsDoc rule 'checkTypes' and make pass

Change-Id: I64c3b5bd7afc4686355c80ef68009d978f7b3097

Upgrade composer/semver to 1.0.0

And update the class names that changed.

Bug: T113431
Change-Id: Icf73630e60fe7d586d7ed249abc1d3014b67cbe6

Merge "objectcache: Add some newlines to WANObjectCache docs"

objectcache: Add some newlines to WANObjectCache docs

* Doxygen needs this or the paragraph ends up in the
  last bullet point
* Also removed some redundant comments

Change-Id: Ie4bf2b67fff27ecad5f24f55688161fe484c300d

Merge "Removed wfDataCenter()/wfDataCenter() roles"

Removed wfDataCenter()/wfDataCenter() roles

* Follows-up 0a1c04bea
* The DC name ended up not being used in the cookie,
  making these unused

Change-Id: I5f0d822a334102ee9596b1ec2a8d911010294e7c

Merge "Make lines short to pass phpcs in media tests PHP files"

Merge "Make lines short to pass phpcs in parser tests PHP files"

Merge "Use correct comment format for codingStandardsIgnoreStart"

Make lines short to pass phpcs in media tests PHP files

Some lines that weren't too long are also broken up
for consistency and readability.

Bug: T102614
Change-Id: I36e3225e135b58bc22705afbb888b06f93b5a9a8

Make lines short to pass phpcs in parser tests PHP files

Bug: T102614
Change-Id: I14a28815af83efd38f04def8eb7d42ef124ab918

Use correct comment format for codingStandardsIgnoreStart

Apparently, codingStandardsIgnoreStart doesn't work with /*,
and does work with //.

Bug: T113852
Change-Id: I2e7f200617091acc8a79be09763a91c28f5045f2

Localisation updates from https://translatewiki.net.

Change-Id: I4096b4a9ea51f480a79d6b07df27c0bce55aa534

Make lines short to pass phpcs in 3 files under tests/phpunit/includes/

Bug: T102614
Change-Id: I8f3d570fb6e9866d4376f42d4efa05f9c5e7f14d

Merge "Made WikiPage::isRedirect() actually use page_is_redirect"

Add @codingStandardsIgnoreStart to pass phpcs in 2 files in tests/phpunit/includes

Change-Id: I7f551dafcf437cc23aa9aed24f9e4d385816a0a0

Remove unused minor and major in WikiReference

Given that the class is not constructed anywhere outside
of core the constructor can be altered without having to
worry about b/c.

Change-Id: Ie49c43a1724f05cb95a296dc0fad0f1c587d80ef

Merge "API: mention how to get page property names"

UploadBase: Remove UPLOAD_VERIFICATION_ERROR

It was replaced by HOOK_ABORTED five years ago and isn't being used
anywhere now.

Change-Id: I20feb33c108ae56f25a0cd01da1a326b290106c2

API: mention how to get page property names

Mention action=query&list=pagepropnames to get a list of properties in
use.
Also rephrase apihelp-main-param-uselang to use active voice.

To test, visit
  api.php?action=help
  api.php?action=help&modules=query%2Bpagepropnames
  api.php?action=help&modules=query%2Bpageswithprop

Change-Id: Ic13f6c00396aed1a1c016bfcf6a39d621a4ebc06

Merge "Added $wgDataCenterId/$wgDataCenterRoles"

Made WikiPage::isRedirect() actually use page_is_redirect

* Previously it would always fetch the whole content,
  which is silly since followRedirect() uses the redirect
  table to avoid loading the text. The initializeArticle()
  uses isRedirect() and then possibly followRedirect(). It
  makes no sense for the former to fetch all the text anway.
* The time fetching the text showed up on xenon flamegraphs.

Change-Id: I2dc216f36d3a0ea2285e64122b4d07bd9c8ae703

Reword "passwordreset-emailsent"

Remove potential confusion.

Bug: T99533
Change-Id: Ifb28f4e1fb6fb692d6f052d5a3c1a67f881948bc

Added $wgDataCenterId/$wgDataCenterRoles

* This is used to set sticky DC cookies to avoid
  session replication lag (which also makes sure
  ChronologyProtector works)

Bug: T91816
Change-Id: I7bc2f8185a3c05cb3ca5ccc42d300eccffae48e1

Merge "Fixed remaning IDEA errors in DatabaseBase"

Made LBFactorySimple set "master" flags like LBFactoryMulti

* wfGetDB( DB_MASTER )->getLBInfo() now shows the master flag
* A corresponding slave flag was also added
* Added a few badly needed LBFactory/LoadBalancer tests

Change-Id: I9254b12cff63af7d754a3a14c5db44276f58d280

Merge "Do not split parser cache if limitation is reached."

Fixed remaning IDEA errors in DatabaseBase

Change-Id: Ife8d7ecd4f224f1e2ef6f729765c303ec69a3f5f

Refactor hashing utility functions from MWCryptRand and make public

MWCryptRand already has some useful utility functions wrapping PHP's
hash() and hash_hmac(). Let's make them public so we can use them from
other code.

But since "MWCryptRand" isn't really a good place for hashing functions,
let's move them to "MWCryptHash" instead.

Change-Id: I7542c719ac72beba7b0f6aa170bdb4c69fa6beab

Merge "build: Don't run Travis CI for wmf/* branches"

Merge "mediawiki.page.watch.ajax: Add dependency on mediawiki.page.startup"

build: Don't run Travis CI for wmf/* branches

These test runs don't seem very valuable and take up a lot of
slots that could be used for other wikimedia/* repos.

Change-Id: I47ff3a31883d02ec6b535e411d92ada92a84aa2f

Merge "Retry getting CatChange RC from master"

Retry getting CatChange RC from master

This adds optional flags to Revision::getRecentChange
And uses them in CategoryMembershipChange

Bug: T109700
Change-Id: I197ebccf1f62cdcb03ce4daa2527b973e495236c

resourceloader: Deprecate two pointless minification config vars

When minifying JavaScript, never put each statement on a separate line, and
always set a target maximum line length of 1000. These behaviors were
previously configurable via $wgResourceLoaderMinifierStatementsOnOwnLine and
$wgResourceLoaderMinifierMaxLineLength, respectively.

Change-Id: I0b0eb632875b5e16f728fd0aa62f7f5ecd79ef62

Made User::loadFromId() skip cache with READ_LATEST

* That flag is used for anti-dependencies and thus
  should never rely on cache

Bug: T95839
Change-Id: I4ffc8325e55588ef649b96e7b90bc95282f765a9

Merge "API: Remove warning about continuation change"

Merge "build: Enable jscs jsDoc rule 'checkParamNames' and make pass"

Merge "mediawiki.messagePoster: Update comments after mw.Api file renames"

mediawiki.messagePoster: Update comments after mw.Api file renames

* Use class name instead of file name
* Expand abbreviation

Follow-up 0bfdd927

Change-Id: I33ad2b10ffb4b4e5e3a38a5c6f644eb183134a36

Merge "Fix LESS file dependency tracking in ResourceLoader"

Merge "mw.Upload.BookletLayout: Go back to upload page on upload error"

Fix LESS file dependency tracking in ResourceLoader

As Timo noted in a review of I1ff6115, using '+=' to combine file dependencies
is incorrect here. I fixed it in one place but not the other, causing the file
dependencies to vary based on whether a .lesscache file was used.

Change-Id: Ie3f48048febc32c5c2d45acd87e10f946f6dfc94

mw.Upload.BookletLayout: Go back to upload page on upload error

Bug: T114130
Change-Id: Ibe50ce845a42bb5e66f842c68a35541747c9a570

mw.Upload.BookletLayout: Make all errors unrecoverable

Any error apart from 503's can't be recovered just by hitting 'Try
Again'. The user will _have_ to make some changes before the process
can continue.

Bug: T114130
Change-Id: I8e2d619be5b7e47bc903155c353bc1da36a62bb6