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"

Merge "Integration tests for FileBackendGroup"

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

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

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"

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

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"

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

Merge "wdio: Switch to dots reporter for brevity"

Use correct metadata when adding exact query match

Bug: T230819
Change-Id: If85fa1cbad1f3ef9ab2cb0da966a98aafffad05e

Document that img_description_id is a comment id

It took me a bit of time to figure this out, and it seems non-obvious,
so I thought a comment would be appropriate.

Change-Id: Ibe7220ed8f9355e3025c257057754eb4d72384b9

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

This allows to make mTitle a private property.

Change-Id: If953546cf4bb0324e4ae41d2195254c37c0ad3b4

LockManagerGroupFactory to replace singletons

100% test coverage of code that appears to be working and used, in both
LockManagerGroupFactory and also LockManagerGroup. Where possible I
wrote it as unit tests. One preexisting code path seems to be broken and
I marked the test as skipped. Two methods look unused and perhaps not
especially helpful, so I didn't write tests for them yet in case we want
to just get rid of them instead.

Change-Id: Iaa7354f31c451b87773468609c674a3bf1d4382f

Localisation updates from https://translatewiki.net.

Change-Id: I52497a0a7f1d15ca6643556e3967d9302572a40f

Restore some previous constructor calls

Follow-up to If07f10075a51fbbe9de24464cb6844faaad94780
and I082152b64141f1a9a4085bba23fe81a99ec8d886.

It will make getRequest use the provided context again.

Change-Id: I60b2598edcb0daed076876482e1e9d6cbf815eb2

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

BlockManager::getBlockFromCookieValue accepts a UserIdentity, so should
only call methods defined in that interface.

Change-Id: If30df15b800cceee0ad052a2e0524df8b28d9901

Merge "TablePager: put parent construct call back at end"

TablePager: put parent construct call back at end

Restores position of parent constructor call, changed in commit
I082152b64141f1a.

The parent constructor calls getIndexField(), which depends on the
mSort value already being set.

Bug: T231261
Change-Id: If07f10075a51fbbe9de24464cb6844faaad94780

Merge "Add more information to exception thrown"

Add more information to exception thrown

It's not just about the preference name, because current user's properties
are used for validation too.

Bug: T231029
Change-Id: I268b959017bb0dce2b4295d5302a544bfa3513eb

Merge "Add help link to Special:NewSection"

Merge "Add SearchResultTrait"

Add help link to Special:NewSection

Bug: T231260
Change-Id: I856f5256b45c3961186f27c3b0b190998e297c1a

Merge "Setup: Remove scopedProfileIn() calls"

Merge "REST: call MediaWiki::preOutputCommit and doPostOutputShutdown"

Merge "SpecialNewSection: Fix handling of 'preloadparams'"

Merge "Clean up LinkRenderer usage in IndexPager and extensions"

Merge "Revert "Make LocalisationCache a service""

Merge "objectcache: SqlBagOStuff cleanups for sqlite and postgres"

Merge "rdbms: simplify comment in LoadBalancer::reuseConnection()"

Merge "rdbms: rename $db => $conn in remaining places in LoadBalancer"

Merge "Setup: Move wgRCLinkDays logic from Setup to ChangesListSpecialPage"

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

Change-Id: I218235fa3da0c64f111c7fe2afffeba6a30b6e21

Revert actor changes to populateLogSearch.php

In all the past patches, we missed the fact that this maintenance script
is run as an upgrade from MW pre-1.16 to MW 1.16. It's apparently not
intended to be run outside of the upgrade process.

Thus, let's revert the changes that were made to the script in I8d825eb0
and I4764c1c7. Instead, the script need only detect that the pre-1.16
schema it's looking for doesn't exist anymore and bail out.

Bug: T188327
Change-Id: I7e03b45ef6d43b9069f0399089ab06e136a89fbc

rdbms: remove unused $fulltextEnabled field from DatabaseSqlite

Change-Id: Idad680485f6cf6b521a0ea33c02d30e78d1d6b50

MessageCache: Add STRAIGHT_JOIN to avoid planner oddness

For some unknown reason, when the `actor` table has few enough NS8 rows
compared to `page` MariaDB 10.1.37 decides it makes more sense to fetch
everything from `actor` then join `revision` then `page` rather than
fetching the rows from `page` in the first place.

We can work around it by telling it to not reorder the query, but then
we also have to reorder it ourselves to put `page` first instead of
`revision`.

Bug: T231196
Change-Id: I2b2fb209e648d1e407c5c2d32d3ac9e574e361d5

Merge "Use local variabe $services instead of MediaWikiServices::getInstance()"

Merge "Disambiguate confusing $fromReplica variable in BlockManager"

SpecialNewSection: Fix handling of 'preloadparams'

The parent class RedirectSpecialPage handles array parameters
automatically, and we need to specify the name without the '[]',
same as in RedirectSpecialArticle.

Bug: T231088
Change-Id: I52838a39f3311fe612c466903d7eda95fecedfa7

Revert "Make LocalisationCache a service"

This reverts commits:
 - 76a940350d36c323ebedb4ab45cc81ed1c6b6c92
 - b78b8804d076618e967c7b31ec15a1bd9e35d1d0
 - 2e52f48c2ed8dcf480843e2186f685a86810e2ac
 - e4468a1d6b6b9fdc5b64800febdc8748d21f213d

Bug: T231200
Bug: T231198
Change-Id: I1a7e46a979ae5c9c8130dd3927f6663a216ba753