Merge "maintenance: Sanity check that --replicadb isn't using the master in sql.php"

maintenance: Sanity check that --replicadb isn't using the master in sql.php

Change-Id: I0b7fd896c88b06cbc17b31b1ae2b9cb8e375da1b

Merge "API: Warn about POST without Content-Type"

Merge "filebackend: Remove private handleWarning in favour of local closure"

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

Merge "Skin: Add new hook, UndeletePageToolLinks, for Special:Undelete views"

Localisation updates from https://translatewiki.net.

Change-Id: Ifdfe6ec25954eb7c61a85b6494bc0d2a18bae9a7

Merge "rdbms: various field name and style cleanups to Database"

Merge "Short-circuit WebRequest::getGPCVal() for printable ASCII strings"

Skin: Add new hook, UndeletePageToolLinks, for Special:Undelete views

This allows extensions to add further links; the particular use case
in mind is for the AbuseFilter extension, but others may want this
too.

Bug: T231055
Change-Id: I671a0479e877e6c37606b688064cb9c893717709

filebackend: Remove private handleWarning in favour of local closure

This is a pattern from before PHP 5.3, which we no longer need.

Change-Id: I886d1ded25ffe1ee12b6a3f8d48c04aa7dd2ef51

rdbms: various field name and style cleanups to Database

Rename Database::hasFlags() to sound less similar to getFlag()/setFlag().
The order of class fields is roughly:
* Objects and resources
* Configuration
* Mutable options that do not have to be kept in sync with the connection
* Session level state tracking (should be in sync with the connection)
* Transaction level state tracking (should be in sync with the connection)
* Results from or timestamp of the last time a certain event occurred

Change-Id: Ibdca2fefe7ed2c792344c5602b5191a950eed933

Merge "objectcache: move lag waiting to SqlBagOStuff::doCas() instead of overriding merge()"

Merge "objectcache: add BagOStuff::fieldHasFlags() convenience method"

Merge "rdbms: remove reference to READ_LATEST in ChronologyProtector::shutdown()"

Merge "filebackend: use AtEase in FileBackend related classes"

Merge "Deprecate skin functions that are not skin responsibilities"

Merge "objectcache: make more SqlBagOStuff methods private and rename shard variables"

Short-circuit WebRequest::getGPCVal() for printable ASCII strings

This avoids accessing the content Language instance in many cases

Change-Id: I82bd66496180f947c1af16c2728a1548df03dcf9

Merge "objectcache: move MemcachedClient class to /utils subdir"

filebackend: use AtEase in FileBackend related classes

Change-Id: I819e0e923dfd3ffa7299acd1d6103e3cdf2baa93

Deprecate skin functions that are not skin responsibilities

Skin shouldn't be responsible for providing requested revisionId
nor if that revision is the current revision.
The OutputPage object has all required information (both the
currentRevisionID and the current Title object).

Change-Id: I2dbae4c6968a2b3b3cea3e09977e9579609b4cc5

Merge "Move getRestrictionLevels from NamespaceInfo to PermissionManager."

Merge "Split some Language methods to LanguageNameUtils"

Merge "Allow justifications in `Special:EditTags` to use full comment length"

Split some Language methods to LanguageNameUtils

These are static methods that have to do with processing language names
and codes. I didn't include fallback behavior, because that would mean a
circular dependency with LocalisationCache.

In the new class, I renamed AS_AUTONYMS to AUTONYMS, and added a class
constant DEFINED for 'mw' to match the existing SUPPORTED and ALL. I
also renamed fetchLanguageName(s) to getLanguageName(s).

There is 100% test coverage for the code in the new class.

Change-Id: I245ae94bfc1f62b6af75ea57525139adf2539fe6

Add ObjectFactory as a service

ObjectFactory exists to handle lazy instantiation of objects based on
declarative specifications.

One drawback to the current use of static ObjectFactory::getObjectFromSpec()
is that it doesn't have a good way to handle dependency injection. The
best you could do is refer to a static 'factory' function in your
specification, and that function would access the global service
container. But DI would prefer not to have that "global service
container" being accessed.

The solution is to have a non-static ObjectFactory that can supply
the needed services based on the object definition. For example,

 [
     'class' => MyObject::class,
     'services' => [ 'FooFactory', 'BarFactory' ],
 ]

would get the FooFactory and BarFactory from a service container to pass
into MyObject's constructor.

But in order for that to be possible, you need to have an instance of
ObjectFactory that has reference to the service container from which to
fetch the services. And the most DI-correct way to get that is to have
it dependency-injected into your constructor, at which point the
ObjectFactory instance should itself be a service in the DI service
container.

Bug: T222409
Change-Id: I2a56059a9209abce0b22fb93c9f9c6a09a825c56

rdbms: remove reference to READ_LATEST in ChronologyProtector::shutdown()

Change-Id: I34b2fa940475d5e5e81130221f61d782818a6d17

objectcache: make more SqlBagOStuff methods private and rename shard variables

Also optimize silenceTransactionProfiler() when LoadBalancer is not in use

Change-Id: I83fd5c17058ba082a13e957e4b4590e2d1d5b581

PermissionManager::userHas{All,Any}Right: don't specify a variadic param.

Change-Id: Ife9d01be57a4926f4a5efa99661163a391564a6e

objectcache: move lag waiting to SqlBagOStuff::doCas() instead of overriding merge()

Change-Id: I61b49bc7b00ffa8308d93e76f4bb94cf09ba9936

objectcache: add BagOStuff::fieldHasFlags() convenience method

Change-Id: Id2bf4ec0dd0999f988b70a895003c4b1aaae51a2

Move getRestrictionLevels from NamespaceInfo to PermissionManager.

Bug: T11977
Change-Id: I051be9148c98086fdf53a66a74bf7c28699016db

Allow justifications in `Special:EditTags` to use full comment length

Bug: T188460
Change-Id: Ib3d5207432b741b9382db0d963a42ed4c8a59a79

Merge "Deprecate and replace usages of User:isAllowed{All,Any}"

Merge "Make LocalisationCache a service"

Merge "Some integration tests for LockManagerGroup"

Merge "Update .mailmap for Aryeh Gregor"

Some integration tests for LockManagerGroup

Change-Id: Ide94a260e5f96c4cfcf76c1ee9b8dcf486976263

Make LocalisationCache a service

This removes Language::$dataCache without deprecation, because 1) I
don't know of a way to properly simulate it in the new paradigm, and 2)
I found no direct access to the member outside of the Language and
LanguageTest classes.

Change-Id: Iaa86c48e7434f081a53f5bae8723c37c5a34f503

Merge "Remove deprecated User groups/permission static calls."

Update .mailmap for Aryeh Gregor

Whoever made the mailmap for old SVN contributions mapped my account to
the e-mail address I think I used on Bugzilla etc. at the time,
simetrical+mw@gmail.com. (I was inactive then, so I wasn't consulted.)
When I started committing under the git regime, I used ayg@aryeh.name,
so that should be the address used for my old commits as well so that
they link up with my new ones.

Change-Id: If03abd0e8fc11c20e7b37948f7d10f770cd43343

Merge "BadFileLookup to replace wfIsBadImage"

Merge "Rewrite integration test for wfIsBadImage()"

Merge "Increase maximum length of automatic deletion summaries"

Localisation updates from https://translatewiki.net.

Change-Id: I2858a9ae9b37a430ea70dea57b9fbc56291b64ff

Remove deprecated User groups/permission static calls.

Bug: T220191
Change-Id: Ifa8afa90c432723b0bba0033a46b6a499c77e6fc

Merge "Add support for native image lazy loading"

Deprecate and replace usages of User:isAllowed{All,Any}

Bug: T220191
Change-Id: I197b8fadaa93e7b320fc19c10e3e09387fe09ad2

Increase maximum length of automatic deletion summaries

Currently, the maximum length is hardcoded to 255 characters.
Instead, the maximum length should be based on the actually enforce
comment store limit.

Bug: T230962
Change-Id: I4379057f8f49fd8e5efa620164eea2867fde1bfc

objectcache: move MemcachedClient class to /utils subdir

Change-Id: I2774d3e7a3c96ef691106b21908a42bfbbf25956

Merge "rdbms: add some more comments to DatabaseDomain"

Merge "rdbms: remove IDatabase::clearFlag() calls made redundant by QUERY_IGNORE_DBO_TRX"

Merge "parser: document what 0/null rev IDs do when given to Parser::parse"

Merge "deferred: make DeferredUpdates::attemptUpdate() use callback owners for $fnameTrxOwner"

rdbms: add some more comments to DatabaseDomain

Change-Id: Iee407028a2f3c7eb6497a73d57c7baa093246217

rdbms: remove IDatabase::clearFlag() calls made redundant by QUERY_IGNORE_DBO_TRX

Change-Id: I493c63f94c813ad71dc2eb8eaf9119a1f11d62cb

Merge "objectcache: Remove lock()/unlock() stubs from MemcachedClient"

Merge "Convert PermissionManager constructor to use ServiceOptions."

parser: document what 0/null rev IDs do when given to Parser::parse

This was extremely obscure and tricky to figure out before

Change-Id: Ibdbb669431819eecd34de508735864e977d51021

BadFileLookup to replace wfIsBadImage

I think this probably shouldn't be directly in the MediaWiki namespace,
but I don't know where is a better place to put it.

In order to avoid gratuitous use of TitleFormatter, I changed the cache
format -- the old implementation used getPrefixedDBkey() and I switched
to an ns/dbkey pair. I also changed the cache keys to use SHA1 instead
of MD5, by Daniel's request.

The previous implementation cached the parsed blacklist for one minute
without invalidation, so it could return slightly stale results, but it
didn't retrieve the bad image list message on a cache hit. The new
implementation unconditionally retrieves the bad image list message, but
uses a hash of it in the cache key and caches for one day. The new
behavior happens to be more cleanly implementable in a service.

Bug: T200882
Bug: T139216
Change-Id: I69fed1b1f3cfc1aa149e0739780e67f6de01609d

Rewrite integration test for wfIsBadImage()

In preparation for rewriting as a service. I didn't figure out how to
get file redirects to be tested.

Change-Id: Ic6669a19a13025744036f9f8adc4d1a25490fb42

Convert PermissionManager constructor to use ServiceOptions.

Change-Id: I36a3a2f338506ef14cc5d65b8bee2961a92d60da

Ensure block hooks keep user state consistent with realistic blocks

Several block-related hooks allow the user to be put into in a state
that is inconsistent with blocks that can actually be made:
* With UserIsHidden, User::mHideName can be set to true without there
  being a block
* With UserIsBlockedFrom, a user can be blocked from editing a page
  without there being a block
* With GetBlockedStatus, public block properties can be arbitrarily
  set on a user

These problems are mostly theoretical, but mean that it is impossible to
make some basic assumptions, e.g. that a user who is blocked from a page
must have a block. The hooks are not widely used, and with a few changes
we can make them more robust so such assumptions can be made.

This patch:
* Ensures UserIsBlockedFrom is only called if there is a block. This
  would be a breaking change if any extensions were using this to block
  an unblocked user; the intended use case is clearly for extensions to
  allow user talk page access to blocked users.
* Adds a new hook, GetUserBlockComplete, which passes the block for
  modification. This should be used instead GetBlockedStatus and
  UserIsHidden, which will be deprecated in the future.
* Allows the 'hideName' option to be passed into the AbstractBlock
  constructor so that suppressing system blocks can be made.

Bug: T228948
Bug: T229035
Change-Id: I6f145335abeb16775b08e8c7c751a01f113281e3

Merge "TempFSFileFactory service"

Merge "REST: Use ob_clean to wipe display_errors output"

TempFSFileFactory service

This replaces TempFSFile::factory(), which is now deprecated.

Change-Id: I9e65c3867e26c16687560dccc7d9f3e195a8bdd6

Merge "RedirectSpecialArticle: Fix PHP notice about undefined index"

Merge "mw.widgets.CalendarWidget: Tweak hover styling after OOUI changes"

replace wfGetDB (discouraged for use) with Maintenance::getDB in dumps scripts

Bug: T228772
Change-Id: I0d9448b2565481f61d927b2d58f44bddbf358ed8

Add support for native image lazy loading

Bug: T230897
Depends-On: I6c0b01fafe2c4498852e9c9b81151aba759dc5eb
Change-Id: I0307563b8f433dab062effe4dbfd7d4a9bbb8764

Merge "Remove usages of deprecated User::getRights."

Merge "Replace User::isAllowed with PermissionManager."

Localisation updates from https://translatewiki.net.

Change-Id: I67fba817871853916304194753f95e330841167d

Merge "Add $digitTransformTable for the Manipuri language"

Merge "Change the autonym of Manipuri to Meetei script"

Remove usages of deprecated User::getRights.

Bug: T220191
Change-Id: Ia7472cf61765fe5fee9ae72cfa9b7060565dbe87

REST: Use ob_clean to wipe display_errors output

As in the Action API

Change-Id: I48c3477f0ebb58c000fa832352a1631cfda6c909

Merge "DefaultSettings.php: Move $wgDBname/$wgDBprefix/$wgDBmwschema closer to each other"

mw.widgets.CalendarWidget: Tweak hover styling after OOUI changes

Bug: T230393
Change-Id: I9e1e2e3d7dded82e6bc60b00afedef4e56d893b6

Merge "Break the cyclic dependency between SearchEngine and SearchResult"

Merge "resourceloader: Avoid isKnownEmpty call for regular (non-embed) modules"

Replace User::isAllowed with PermissionManager.

Covers root includes, actions, api, block, changes,
changetags, diff and PermissionManager itself.

Bug: T220191
Change-Id: Ic027d32f5dd8f4c74865df0c8a9fcf91123c889c

resourceloader: Avoid isKnownEmpty call for regular (non-embed) modules

Modules that are never embedded, and not user-specific, should not be
excluded from the loader, even if they are empty.

Doing so has two problems:

1. Modules are expected to have their changes propagated within 5 minutes
   through the startup module. This depends on the fact that in page view
   HTML we queue the module by name, regardless of its current version.

   If the module is known to be needed by a page, then we need to queue it,
   even if the current version is empty. Otherwise, cached pages will be
   missing the module, despite the older ParserOutput perfectly knowing
   already that it was needed, which can cause bugs due to HTML not
   matching the module queue it was generated with.

2. The isKnownEmpty method can sometimes require a database lookup.
   The performance team tuned this with a preloader for the subset of
   modules we need the information for (user modules and style modules).
   In 0b1a7d4c59c8395, I accidentally made the conditions nested the wrong
   way, which made it call this much more frequently.

Bug: T230260
Bug: T176159
Change-Id: I4e6af2c833c92e1277713bdd0c68953d49c4dd9d

Merge "Clear block cookie if the value is invalid"

Merge "Add instrumentation to Special:Mute"

Merge "Put block cookie expiry into UTC for comparison with current time"

Merge "rdbms: simplify LoadBalancer::getLaggedReplicaMode()"

Merge "Tests: Set dbname for DatabaseSqliteTest"

Break the cyclic dependency between SearchEngine and SearchResult

Change-Id: Icda469eb675ccac6048d7a9d978d51e3e673edab

Merge "ApiSandbox: Load mediawiki.hlist RL module"

Merge "objectcache: fix WRITE_ALLOW_SEGMENTS in BagOStuff cas() and add() methods"

Put block cookie expiry into UTC for comparison with current time

Change-Id: Ia01a364a01ebacec8783ecdcfe4a129cfb746eb8

Merge "TitleOptionWidget: Documentation type fix"

Merge "RCFilters: add namespace filters for all contents and all discussions"

Merge "hooks.txt: Convert docs to modern extension registration style"

objectcache: fix WRITE_ALLOW_SEGMENTS in BagOStuff cas() and add() methods

Add MediumSpecificBagOStuff::getValueOrSegmentList() helper method.

Also:
* Use $keysMissing variable correctly in CachedBagOStuff::getMulti()
  to avoid extra overhead.
* Optimize mergeViaCas() when the current value matches the new one.

Change-Id: I5c4bd74379bc459216ac0278150ce3aecff3b851

TitleOptionWidget: Documentation type fix

Change-Id: I6ea4a1749a373c64e12d5c0ee1e5d55953523205

ApiSandbox: Load mediawiki.hlist RL module

The default l10n for the main module's description uses the hlist class,
so load it.

It might make more sense to have it as a module dependency of an
appropriate RL module like mediawiki.apipretty instead of requiring
every caller to load it manually, but it seems that's not really done
because RL is weird about handling CSS-only modules as dependencies.

Bug: T230782
Change-Id: I6493d7d789f3cebee737645528cddfb1d3acb1d7

hooks.txt: Convert docs to modern extension registration style

This changes the examples in hooks.txt from using the old format of
manually entering additions into `wgHooks` to instead use the new
`"Hooks“: {}` object format.

Bug: T230397
Change-Id: I48a9986e4243eb933088d36b4bb095b345ab62fd