Remove $wgMemCachedDebug

Change-Id: I98380ac1cb66e33719b4836410311aea3513115c

Merge "build: Upgrade eslint-utils dependency from 1.3.1 to 1.4.2 for security issues"

Merge "Fix FileBackendGroup test for some configurations"

Fix FileBackendGroup test for some configurations

The expected value of FileBackendGroup::config()['srvCache'] was being
obtained from MediaWikiServices::getLocalServerObjectCache(), but the
class actually used ObjectCache::getLocalServerInstance( 'hash' ). It
happens these are often the same, so it passed the gate-and-submit jobs,
but it's causing failures on Travis CI.

Follow-up to bd2a4395025, which added the failing test. Thanks to
Krinkle for pointing out the failures on CI.

Change-Id: I17651766f4ee2752dfcae9574d2538269dec4ebe

Merge "Remove unneeded overrideMwServices/resetServices"

Merge "Call resetServices() when setting globals in tests"

Merge "Add UnknownContentHandler."

Remove unneeded overrideMwServices/resetServices

Change-Id: If6cbdec05b8f310ef3a0b4649aaa16d9fb80a047

Call resetServices() when setting globals in tests

Now that resetServices() will preserve (but reset) customized services,
it should be reasonably safe to call it every time globals are changed,
and much more effective than relying on tests to call it every time
themselves.

Depends-On: Iab8ea3a61bbc6803805d855ef23c071067646f71
Depends-On: I00e35ecea6a27468674b2a6e7d9d9eb6518e3bd5
Change-Id: Ie7a89f6ed7d52a0bc01672019ff92e7ee105a1f3

Avoid fatal errors when reporting exceptions.

When reporting exceptions that occur during initialization, wgUser may
be null. Don't die when that happens.

Change-Id: I65d5a17d80f9021e28a218c7a5a17e399bc7ce98

Add UnknownContentHandler.

UnknownContentHandler can be configued to handle models that
belong to extensions that have been undeployed:

  $wgContentHandlers['xyzzy'] = 'UnknownContentHandler';

This way, no errors will be thrown when trying to access
pages with the unsupported model. Instead, an error message is
shown, and editing is prevented.

This patch also improves handling of non-editable content in
EditPage and in DifferenceEngine.

Bug: T220608
Change-Id: Ia94521b786c0a5225a674e4dc3cb6761a723d75b

Merge "Fix categories detele SPARQL clause"

Merge "profiler: Centralise output responsibility from ProfilerOutputText to Profiler"

Merge "mediawiki.Uri: Add another test for arrayParams"

mediawiki.Uri: Add another test for arrayParams

Verify that the added handling is correctly conditional.

Bug: T231382
Change-Id: I71b834c1c0a7bdaf28d990eafd4ce5d70a55e141

Merge "Special:NewFiles: Use a proper user widget instead"

Merge "resourceloader: Purge localStorage blob if last written 30+ days ago"

Merge "Pagers: Drop 'newbie' feature"

Merge "Fix call to non-existing TempFSFileFactory::getTempFSFile()"

Merge "Special:DeletedContributions: Drop 'newbies' feature"

Fix call to non-existing TempFSFileFactory::getTempFSFile()

Should be TempFSFileFactory::newTempFSFile()

Bug: T231488
Change-Id: I9fbf7d993773f55965268ac10b347110148671c9

Merge "Fix typo in comment"

Merge "Make WantedQueryPage::makeWlhLink protected"

Merge "PermissionManager should not cache anonymous rights under ID 0"

Special:NewFiles: Use a proper user widget instead

Change-Id: I276f55e9d4dd2d539dae232c7a4c1e71745a6168

Pagers: Drop 'newbie' feature

Change-Id: I14c4da7b107aa9c712bd81522021ae7ecaf2b790

Special:DeletedContributions: Drop 'newbies' feature

Change-Id: I32c67fb19f50f3949ffe981488c009c82a244999

Merge "Sync up with Parsoid parserTests.txt"

Merge "Remove unused localisation message 'wlshowlast'"

Merge "Remove unused localisation messages 'ascending_abbrev', 'descending_abbrev'"

Merge "Upstream "pulsating dot" widget from VE"

Make WantedQueryPage::makeWlhLink protected

Bug: T231481
Change-Id: I6ace1bb99879ec8256b29f86220e8fc2e785f593

Merge "Integration tests for FileBackendGroup"

Merge "Grunt: Expand js{,on} for readability"

PermissionManager should not cache anonymous rights under ID 0

Bug: T228253
Change-Id: I8a54830842f220ff1ac4402a3380c2229a99b619

profiler: Centralise output responsibility from ProfilerOutputText to Profiler

Make it Profiler.php's responsibility to enforce this, based on the
existing signal from ProfilerOutput::logsToOutput().

The ProfilerOutputText class should not have to double-check this
a second time.

Long-term, I'd like even this check in Profiler::logDataPageOutputOnly
to be removed, because really the external caller of that should
know whether it is safe to output stuff or not rather than stashing
its own state inside Profiler::$allowOutput and then implicitly
reading it back out again later on. But, that's for another time.

Also:
* Remove use of deprecated Profiler::setTemplated while at it.
* Make 'visible' parameter explicit, as for other parameters.

Change-Id: Iaa3fc4ea25a059b90235d769db60c04b8f152f05

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 06a41a99d7811a361446b894da7c5c8224398ad1

Change-Id: I9d324b3e6e9167683c15b7fee2a53b598d79a67c

Docs: Standardise "messages used" comments

Change-Id: Ie99f8a62da734956863b1e1c41c8768f0e083fed

Grunt: Expand js{,on} for readability

Change-Id: Ib3bfb1a9d6547e1062ea3ad08de7ea721c111586

Merge "ApiFeedContributions: Drop 'newbies' feature"

ApiFeedContributions: Drop 'newbies' feature

It has been broken since 2011 (r100315), so no need for
deprecation.

Change-Id: I099f52211c8ce401d827405cb313e1d3932b92f8

Merge "skins: Skin::getSkinNameMessages() method is now deprecated"

Merge "skins: Remove use of deprecated MessageCache::singleton()"

Remove unused localisation message 'wlshowlast'

Not used since 34a3efd5f75ca25f1b8b3b3a2efd13bfb1f7c631.

Note that a similar message 'rclinks' is still used.

Bug: T224429
Change-Id: I471a6a88b838bd7f1fab34c531c556e8ccd2779a

Remove unused localisation messages 'ascending_abbrev', 'descending_abbrev'

Not used since 987a860df428e3c70fd81c404f1f7a300345dc48.

The messages 'sort-descending', 'sort-ascending' serve the same purpose.

Bug: T224429
Change-Id: Iabcf83c80aa3fc387ea908868838b73254765a82

Re-add missing messages in languages/i18n/exif/qqq.json

Added in 7c68604e4c5e3f61884d529a37a2878c3155d343, incorrectly removed
by translation updater in 7881d44f690cc1e7415bb6cbaedc02d586a64498?
Checks are failing because of this.

Change-Id: I264f0f1cf3dcd69bbf8d43ad057a81e3f39ed623

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

Merge "Move Title::isNamespaceProtected() to PermissionManager."

Merge "Fix entites typos"

Merge "LockManagerGroup tweaks"

Merge "profiler: Rename Profile::setTemplated to Profile::setAllowOutput"

Merge "Document that img_description_id is a comment id"

Integration tests for FileBackendGroup

100% coverage except for one bit of the code that I didn't understand.
Unit tests to come, together with rewrite as a service.

Change-Id: Ib01758d994a9e5587a4fcb5edc3d80010ef05615

LockManagerGroup tweaks

One small change that was supposed to be in 5a6c18a0863 but didn't make
it into the patch version that got merged, and removal of some long-dead
code.

phan objected to the use of new $class without being explicitly
reassured that $class is, in fact, a string. I don't know why.

Change-Id: Ifa00c59ab4464109414f21db37e3a6db21decdf3

Localisation updates from https://translatewiki.net.

Change-Id: I8380b27a31ae89200aa7e7ccf38424acb244e288

Fix typo in comment

Change-Id: Iebb57a49156f4efc6f45079d239eef2464c0f554

Merge "Let Html::element do the HTML encoding"

Merge "parserTests: Add parser test with filename containing single quotes"

Fix entites typos

Bug: T201491
Change-Id: I241fda87b0e53e804f6bdd48eb975a79e9f176d2

Merge "Revert actor changes to populateLogSearch.php"

Merge "MessageCache: Add STRAIGHT_JOIN to avoid planner oddness"

Merge "resources: Also add jQuery Migrate to foreign-resources.yaml"

profiler: Rename Profile::setTemplated to Profile::setAllowOutput

This is set from three places:
1) SkinTemplate.php, 2) Maintenance.php, and 3) load.php.

These last two are very much *not* HTML-templated output.
Rename these this method pair to getAllowOutput/setAllowOutput instead,
which is less confusing going forward. No known callers outside of
core (updated in the next commit), but I'll keep compat for one
release cycle just in case.

Change-Id: I828f95332dca3c6766b2b485ffb71762542b42d7

Merge "Follow-up 225b462: Remove 'apisandbox-api-disabled' i18n too"

Fix categories detele SPARQL clause

Bug: T228348
Change-Id: I87b3b5831e31dc619b5eefb7854d3223acd605fc

MessageCache: Increase APC 'messages-big' expiry from 1min to 1h

Bug: T218207
Change-Id: Ic5d2a556912e2a16ee899eec3a0670f00dec9a8c

Merge "mw.Uri: Add support for array parameters with explicit indexes"

Follow-up 225b462: Remove 'apisandbox-api-disabled' i18n too

Initially removed, lost in rebase.

Bug: T224429
Change-Id: Ibb9c0ab1dcd2b3119cda570552d54ff866677d95

skins: Skin::getSkinNameMessages() method is now deprecated

No longer used per CS below;
https://codesearch.wmflabs.org/search/?q=getSkinNameMessages&i=nope&files=&repos=

Change-Id: I2be9f417444024c3612db3e771a385ce5ed12ec4

skins: Remove use of deprecated MessageCache::singleton()

Change-Id: I1c6d3874dd78023d743a418039c23adb55578f98

Merge "Recognize exif values for Apple iOS photo modes"

mw.Uri: Add support for array parameters with explicit indexes

When the new 'arrayParams' option is set, query strings like
`&foo[0]=a&foo[1]=b` will be parsed as a single parameter `foo`
containing an array, rather than two separate parameters.

The new option also affects the behavior of array parameters like
`&foo[]=a&foo[]=b`, which will be parsed as a parameter named `foo`
rather than `foo[]`, and disables array handling for parameters that
don't contain an array index at the end.

Unlike in PHP, this does not handle associative or multi-dimensional
arrays, but that may be improved in the future.

Bug: T231382
Change-Id: I48d4bb3fdf0ea7f5eb133c59bf63651ba356fc42

Revert "Output more MW version info in update.php"

Left changes to SpecialVersion.php

This reverts commit e73328601d51674e8fef89c3db12b30ceafd702f.

Bug: T227489
Change-Id: I673b6a6abb73e93d2253fc554f94ef9c833dae0c

Recognize exif values for Apple iOS photo modes

CustomRendered value 2-8 are used by Apple to indicate the processing
modes used like HDR, Portrait and Panorama.

Bug: T231385
Change-Id: I767a81a8bebdf25c230b104d35236a4b38cbe4ed

resourceloader: Purge localStorage blob if last written 30+ days ago

Our version hashes are 6-7 chars of base36 from a fnv132 digest.
Using the formula of <https://en.wikipedia.org/wiki/Birthday_attack>
that provides enough range to publish 2087 different versions of
a given module before there is a 0.1% probability to clash with
another version, 660 versions for a 0.01% probability, and
209 versions of a 0.001% probability.

I think 200 versions of a single module is a good enough space
for most use cases we have of the version hash (such as the E-Tag
header for browser caches and HTTP proxies, whic have have a 30-day
TTL).

However, for mw.loader.store it's a bit tricky. It's generally
more than enough given that (unlike HTTP caches) we only store
1 version of any given module so we don't need it to be different
from N different versions, just the last one.

But, also unlike HTTP caches, localStorage has no expiry. This means
that while for a single user it only has to be different from their
last-seen version, but from the server perspective, it needs to be
different from all possible versions a given user may have last seen.
This is problematic and effectively unbounded.

Plug this hole by discarding the localStorage value and starting
fresh, if the user last visited the site more than 30 days ago.

This is also in preparation for T229245, which will reduce the
hash from 6-7 chars to 5 chars. With that size, we can support only
348 different versions at a 0.1% probability (instead of 2087).
Which is fine for the bounded use cases with a TTL, but would make
the unbounded nature of localStorage even more problematic.

Bug: T229245
Change-Id: Iba8cdbebf1bb5c7c628832708fd656fcef61c095

Merge "Special:NewFiles: Drop 'newbies' feature"

Merge "Special:Contributions: Drop 'newbies' feature"

Special:NewFiles: Drop 'newbies' feature

Change-Id: I946d7027db0fec90d8af68f9370fdd7fe5fd61e0

Special:Contributions: Drop 'newbies' feature

This old, low-value feature has been replaced by the much more featureful
functionality on RecentChanges, which has the added advantage of using an
efficient query.

Bug: T220447
Change-Id: I2e6e12b2ad627ecc346883878811ec0164cd03da

Merge "WebStart: Avoid DB or uselang for nonwrite-api-promise-error message"

Merge "MessageCache: Remove $wgMsgCacheExpiry configuration var"

Merge "MessageCache: Minor wgMsgCacheExpiry doc fix, and clear constant access"

Merge "BadFileLookup::isBadFile() expects null, not false"

build: Upgrade eslint-utils dependency from 1.3.1 to 1.4.2 for security issues

Change-Id: I70f6798eb6e42cf685d03622d4420509fc1bc236

BadFileLookup::isBadFile() expects null, not false

This deviation in behavior from wfIsBadImage() is accounted for in that
function, but I didn't account for it when changing callers to use the
service.

Bug: T231340
Bug: T231353
Change-Id: Iddf177770fb1763ed295d694ed6bab441ea9ab73

MessageCache: Remove $wgMsgCacheExpiry configuration var

This variable has never been set to anything other than the default value of
24 hours as introduced in 2003 (r2203, r2204; or 036ff960ce, edf6b38626).

The variable has never changed in core, it's not overridden at WMF,
and MessageCache is not constructed anywhere other than ServiceWiring.php
anywhere in repos on Wikimedia Gerrit, indexed by MediaWiki Codesearch,
or any GitHub-hosted repository (incl Wikia repos and WikiHow mirrors).

I've also checked all GitHub-hosted repos for boilerplates and/or public
settings files from devs or prod, and couldn't find any example of
this being overridden (after filtering out copies of the core files
themselves). Rather than having to support potentially hard-to-predict
interactions betweeen caching layers by checking its state, make it
a constant so we can code reason about it more easily.

Change-Id: Ie2e139001aae3ac54b509d94a3d917bb408eaca0

Merge "Remove hard deprecation of PasswordPolicyChecks::checkPopularPasswordBlacklist"

Merge "Restore some previous constructor calls"

Remove hard deprecation of PasswordPolicyChecks::checkPopularPasswordBlacklist

It's causing quite a bit of WMF log spam because it's still used in wmf-config

Partial revert of I4c90e361b7538efeb0c5efa7ca61aed2e468591c

Change-Id: I7d1282e4260e7a42bee35b81d77470eb40dea818

Merge "Avoid PHP warning due to bogus second tideways_xhprof_enable() argument"

MessageCache: Minor wgMsgCacheExpiry doc fix, and clear constant access

The class used is typed against BagOStuff so access the constant
from there instead.

Bug: T218207
Change-Id: Ie22d6aa5877fb5e8e2ae0b3be87f4b28f45ad763

Merge "rdbms: remove unused $fulltextEnabled field from DatabaseSqlite"

Merge "Hard deprecate new SearchResult() and introduce RevisionSearchResult"

Merge "LockManagerGroupFactory to replace singletons"

Merge "Use correct metadata when adding exact query match"

Hard deprecate new SearchResult() and introduce RevisionSearchResult

Transitional step for the transformation of SearchResult into an
abstract base class:
- RevisionSearchResult is introduced to behave like SearchResult
- methods are currently shared between  RevisionSearchResult and
  SearchResult in the RevisionSearchResultTrait

Bug: T228626
Change-Id: I13d132de50f6c66086b7f9055d036f2e76667b27

Merge "mail: Avoid use of global state and use MediaWiki services DI"

Merge "Use UserIdentity::isRegistered instead of User::isAnon in BlockManager"

Merge "Use ->getTitle() instead of ->mTitle to get the title from Parser"

mail: Avoid use of global state and use MediaWiki services DI

Change-Id: If335cb6a220ebc5eb7487e51ee097f9d2afa0d2d