Merge "Migrate various callers away from wfWikiId() to WikiMap"

rdbms: add ILoadBalancer::getReplicaResumePos method

This does what ChronologyProtector wants more rigorously and is better
named. Not all replica servers will have the same position, so they
should be compared to get the highest one.

Simplify the getMasterPos() method to only return master positions
as the other current callers do not need anything else. It will now
connect if needed as well. This should make the method naming better.

Reducing the use of replica derived replication postitions (instead
of those from the master) makes certain GTID issues less likely,
such as the matter of obsolete domain IDs.

Increase general test coverage of LoadBalancer.

Bug: T224422
Change-Id: I5420721ee339a24d09c26c38709500c7bbe797c2

Merge "rdbms: better normalize and document constructor $params in Database"

rdbms: better normalize and document constructor $params in Database

Change-Id: I6531cd3a34d7d6bdf277db779301d88ca1e45a95

Fix stale client caching of history pages with $wgShowUpdateMarker enabled

Change-Id: I5b9d986e4f34166f2a1602129d2e01573bfe7ac5

Merge "Add helper for HTTPFileStreamer header syntax"

Merge "Update wikimedia/object-factory to 2.1.0"

Update wikimedia/object-factory to 2.1.0

Vendor patch: I5571d3ea4b50a866b2444af648b22b1c5a1b5ebe

Change-Id: If5884a31c3f468fc55502c27e978f2eb2668441d

Merge "RedirectSpecialPage: handle interwiki redirects."

RedirectSpecialPage: handle interwiki redirects.

Previously, WikiPage::performRequest() would assume that Titles returned
by RedirectSpecialPage::getRedirect() are local pages, and would set
$wgTitle to whatever was returned. That would lead to a confused state
where the skin would try to render for an interwiki Title.

Instead, WikiPage::performRequest() should wrap the interwiki redirect
in a call to Special:GoToInterwiki/xyz, just like
Title::getFullUrlForRedirect() does, but still avoid the HTTP redirect,
to avoid leaking private information via view counters (T109724).

There are two things to test:
1) call Special:MyLanguage with an interwiki prefix,
   e.g. Special:MyLanguage/wikipedia:XYZ.
2) create a page that contains an interwiki redirect,
   e.g. #REDIRECT [[wikipedia:XYZ]], then call Special:MyLanguage
   for that page.

For these tests, the user language should be the same as the content
language. That is the critical case. If the user language differs
from the content language, the problem would be obscured by another
bug which is addressed by Ib4cbeec47a877c473.

Bug: T227700
Change-Id: I2852c5a9774f0c76e49f1e3876fcfe85a305f9ce

Merge "Some fixes to page updater docs"

Merge "resourceloader: Remove use of object registering in test suites"

Merge "mediawiki.dic: Remove Zero-related stuff"

resourceloader: Remove use of object registering in test suites

This was done as a "clever" shortcut to make sure tests a little
but shorter, but also made them less consistent with normal code.

Remove this in favour of 'class' or 'factory' options as needed.
Also remove a bunch of unneeded register() calls.

The tests cover everything affected by this change.

Side fix - isFileModule should reject modules with 'factory'
the same way it rejected raw objects and non-FileModule 'class'
cases already. This is now covered by tests as well.

Bug: T222637
Change-Id: I3996317dbcd780cc6e0f82c84e769c08a3fc42bb

Migrate various callers away from wfWikiId() to WikiMap

Change-Id: Idc2980c05873c428fb5ffa6ab89f723d33217a8e

Add helper for HTTPFileStreamer header syntax

Adds a helper function for transforming an intuitive header array
to the peculiar syntax expected by HTTPFileStreamer and the related
FileRepo/FileBackend streaming methods.

Change-Id: Idac9281b0f1b3c93f4ec1d1c3f336db110e5d260

Merge "Log errors in DeferredUpdates::handleUpdateQueue()"

Merge "Reduce contention of getScopedLockAndFlush() callers by using the DB domain in the key"

Merge "Avoid using deprecated SiteStatsUpdate constructor"

Merge "Optimize JobQueue utilization in UserGroupMembership::insert()"

Merge "Remove unused methods and bogus method call in Title"

Merge "Fix various fatals and IDEA warnings in CopyFileBackend"

Reduce contention of getScopedLockAndFlush() callers by using the DB domain in the key

Change-Id: Ie9fb6a9ff384c72cca559f74d8e409d108207ae3

Merge "Add "help" links for Special:BrokenRedirects, Special:DoubleRedirects, and Special:ListRedirects"

Merge "Release notes for ResultWrapper indexing change"

Log errors in DeferredUpdates::handleUpdateQueue()

Other cleanups and fixes:
* Split up handleUpdate() method into run() and jobify()
* Handle Throwable errors
* Use 'cli' in stats keys in CLI mode instead of "get"
* Tweak some code comments

Change-Id: I7749465df2d7b58e66ee5ebdd3c3d25aea52eeb3

Merge "rdbms: cleanup some Database error message wording for consistency"

Merge "rdbms: cleanup DatabaseSqlite::lock() error handling"

Merge "WatchedItemStore: Fix fatal when revision is deleted"

rdbms: cleanup some Database error message wording for consistency

Change-Id: I7b338e6e856c62ecaab2ef97f76431c2220b430d

Merge "installer: make SqliteInstaller set "synchronous" for the job queue DB like the other DBs"

Merge "Add NS_MEDIAWIKI to LinkCache::isCacheable() to help MessageBlobStore cache regeneration"

Merge "Make MapCacheLRU::get() accept a default value and improve $maxAge default"

Merge "Correctly distinguish the latest revision in action=history"

Merge "Don't show tools on history view when no revisions can be shown"

Merge "jobqueue: fix IDEA warnings in JobQueueRedis"

Merge "Add ResultWrapper/FakeResultWrapper tests"

rdbms: cleanup DatabaseSqlite::lock() error handling

Consolidate more code, including error checks, in open() like with
the other Database subclasses.

Change-Id: I55acae55a219f66c7e45b3a06d76b1d8741a4159

Merge "Add help links to core special pages"

Localisation updates from https://translatewiki.net.

Change-Id: Ie478cd3901ba6be3946f48048ccecf4c35d7f0e6

installer: make SqliteInstaller set "synchronous" for the job queue DB like the other DBs

Change-Id: Ifc51ea735020fd7557c99b02bb7426d78b5f8b47

Add ResultWrapper/FakeResultWrapper tests

Bug: T227665
Change-Id: I0709d0fb55640756b19614552b42b6b8341959f4

Add help links to core special pages

Bug: T227534
Change-Id: I832507223b5e006dfeaf041a071e9b7c280c6b64

Merge "resourceloader: Omit parameter 'lang' from image URLs when not vary"

Merge "Move LanguageCodeTest from integration to unit"

resourceloader: Omit parameter 'lang' from image URLs when not vary

The parameter 'lang' is only added to the URL when the image vary on the
language or on the direction.

Also omit the parameter 'lang' when the value is equal to
ResourceLoaderContext::DEFAULT_LANG.

This change makes the URLs shorter and reduces the size of the
stylesheets.

This change also improves caching because the URLs do not vary on the
language for the same image.

Change-Id: Id7d8ecc7d95747f3c157f9abc12e8489e5085aff

Merge "Speed up code coverage generation for local development"

Move LanguageCodeTest from integration to unit

76 more unit tests

Change-Id: I68e9ffff30a744ef6b31598e0176e67892defb95

Merge "Stop using wfDebugLog in LocalisationCache"

Merge "Use SessionManagerInterface in PHPSessionHandler"

Merge "objectcache: normalize BagOStuff method overriding pattern for *Multi() methods"

Allow PHP version check to execute on older version of PHP

Gerrit change 522014 (merged as b8b2514) mostly corrected this
issue, but one line was missed. Fixing that now.

Bug: T213893
Change-Id: I0a57f5902f1af2619fe47f7ee53695d3fd03adaf

Release notes for ResultWrapper indexing change

Bug: T227665
Change-Id: I81c0f715688a581edc193b96b96db93ff7679670

objectcache: normalize BagOStuff method overriding pattern for *Multi() methods

Change-Id: I1bebb60307b1a166461cb5f9a55a79194cc0e363

Release notes for deprecating User::getRights() and breaking User::$mRights

The actual change was in I88992403.

Change-Id: Ibecec431001a53db4502cf0377342c8aacc7bebe

jobqueue: fix IDEA warnings in JobQueueRedis

Change-Id: I7258191cbae22028d76a52c005f44b7347bd86aa

Merge "Various fixes and simplifications to RefreshLinksJob::runTitle()"

mediawiki.dic: Remove Zero-related stuff

Change-Id: I5f0d8314f754d0331fe06af551b3f65df192752a

Various fixes and simplifications to RefreshLinksJob::runTitle()

* Remove logic for saving slow-to-render parser output. This has
  not worked ever since DerivedPageDataUpdater was introduced.
* Make the logic to use cached output actually work. This was
  also broken since DerivedPageDataUpdater was added. In order
  to pass the output, add a known-revision-output parameter
  to both WikiPage::doSecondaryUpdates() and
  DerivedPageDataUpdater::prepareUpdate().
* Also factored out some helper methods from runForTitle() in
  RefreshLinksJob to make it more readable and avoid the need
  for multiple transaction round commit calls. This makes the
  case of multiple-title jobs less likely to break again.
* Make use of RefreshLinksJob::runForTitle() return value.
* Add unit tests for multiple-title job case.

Change-Id: I0cd13c424a87653b5a7253c42cd48fe43befd692

Some fixes to page updater docs

A couple of typos, and also remove a thing that says isDeleted() should
be skipped for the current revision -- isDeleted() does not do a DB
query so there's no performance rationale for spreading assumptions and
increasing complexity in that way.

Change-Id: I661a6889618f188a9aa72f2f9a2c2ebf69ba09c4

Merge "Allow PHP version check to execute on older versions of PHP"

Merge "Html: Make linkButton() documentation less confusing"

Allow PHP version check to execute on older versions of PHP

A change to update.php used the null coalesce (??) operator.
While this is normally fine, the code in question executes
before the PHP version check, and causes an unfriendly PHP
error rather than the intended helpful error. Use the older-style
isset() call instead, so that the PHP version check will get
a chance to execute.

Bug: T213893
Change-Id: I22e4a24bed9e0b29e08afc7b9468e7bfd81d7d57

Merge "Normalize the code style of some ResultWrapper iteration loops"

Merge "Update $wgMainStash comments and merge in those from a deprecated ObjectCache method"

rdbms: avoid dbSchema() in Database::replaceLostConnection() and Database::__clone()

Since dbSchema() always casts the result to a string, using this method
with a call to open() is broken if the RDBMs does not support DB schemas
and thus requires null

Follows-up 7911da9c6f (last week), which added the check in
DatabaseMysqlBase::open() check. Also follows fe0af6cad (last year),
which made dbSchema() consistently return string. Before that, an
implicit null was passed in from Database::factory for mysql, which hid
the class default of empty string.

Bug: T227708
Change-Id: I67207fbaa39c5cc3fe062077cc654f048090e009

Merge "objectcache: remove inheritence from CachedBagOStuff"

Merge "objectcache: use OPT_TCP_NODELAY for MemcachedPeclBagOStuff"

Html: Make linkButton() documentation less confusing

The documentation for the first parameter said it was both "raw HTML"
and "not escaped", but those phrases have opposite meanings. Clarify that
this parameter takes plain text that will be escaped (so not raw HTML),
and rename the parameter from $contents (typically used to mean HTML) to
$text.

Change-Id: I44d6de08a68c493935b2ceff58fac6c51bc45d12

Make MapCacheLRU::get() accept a default value and improve $maxAge default

Change-Id: If562e1745ffed37b0f1ed5382a5a397fc68aac8e

Stop using wfDebugLog in LocalisationCache

Using logger interface, this would fix some unit tests, plus it
avoids using global functions and ease the way for proper dependency
injection

Change-Id: I9a545c70ace08e2fd0b832d75d6a2362a83af21e

Normalize the code style of some ResultWrapper iteration loops

Change-Id: Iced52ae65d31d1ca0cd2c64ebc9ab56b2aef9617

Add NS_MEDIAWIKI to LinkCache::isCacheable() to help MessageBlobStore cache regeneration

Change-Id: I55550a0ea5b9db0c87a67bb8bc06e846db58b9fe

objectcache: use OPT_TCP_NODELAY for MemcachedPeclBagOStuff

Clean up the option setting code in __construct() as well
and make it more strict in terms of checking bogus options

Make mctest.php recognize when memcached is used more easily

Change-Id: I77be506e80478ddbb4fcfa44d82aba8162fa984f

Merge "Allow false as return type of FileBackendStore::doGetFileXAttributes"

objectcache: remove inheritence from CachedBagOStuff

Change-Id: I24e5b6a95d74730c1679bdc7b7b394adcb6bd794

Merge "build: Updating mediawiki/mediawiki-phan-config to 0.6.1"

Update $wgMainStash comments and merge in those from a deprecated ObjectCache method

Change-Id: I807f36d9c51476a969d7046d57bfc32cf37c2745

Merge "PHPUnit bootstrap: less aggressive unsetting of globals"

Merge "User: support setting custom fields + array autocreation in non-existent field"

User: support setting custom fields + array autocreation in non-existent field

I889924037 added a __set method which did not actually handle fields being set.
For better or worse, setting custom fields on ubiquitous objects like User is a
common form of in-process caching, so this is a B/C break; restore for now.

PHP allows creating an array in a previously non-existent object property
with $o->foo['bar'] = $val, but doesn't properly handle that on objects
which have magic getter/setter. Add an ugly hack to make it work (but warn).

Depends on I15090ae9e4b66ac25f631f6179c4394ce8c445a9.

Bug: T227688
Change-Id: I62b80ab4fa10de984cf2c879ab12d91b0fd9bc1c

WatchedItemStore: Fix fatal when revision is deleted

People following links from their watchlist emails to since-deleted
revisions were getting fatals from
WatchedItemStore::getNotificationTimestamp().

It's hard to tell whether returning null or false is the right thing to
do here, because this function can return both and the difference is not
documented, and I wasn't able to find any callers that care about the
distinction.

Bug: T226741
Change-Id: Ib2a836099343f4c161227266dbeeafbc76dccc2b

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

Localisation updates from https://translatewiki.net.

Change-Id: I1a847d840c4263f055261c78fc9b487fc0537ffe

objectcache: rename "slaveOnly" parameter to "replicaOnly" in SQLBagOStuff

Also removed redundant check

Change-Id: I9f454eb15f223f5a9816dca4510c1a9add540b49

Merge "Tests: Assert that integration tests are not in unit test directory"

Merge "Pass in ServiceOptions to BlockManager"

Merge "Tidy up conditions for applying a block from a cookie"

PHPUnit bootstrap: less aggressive unsetting of globals

Previous approach caused breakage with "Notice: Undefined index: _SERVER"
messages, see e.g.
https://gerrit.wikimedia.org/r/c/mediawiki/core/+/521268#message-4502e6c209f53f6dc1c7cdf4f60e5045bbfb6ee4

Change-Id: Ief20e4e21fd99d219ebef865c603e336c2609ce2
Follows-Up: I16691fc8ac063705ba0c2bc63b96c4534ca8660b
Bug: T87781

Tests: Assert that integration tests are not in unit test directory

Bug: T87781
Change-Id: If82efb1829095a6cf76435921f1328801508a70a

Linker: Fix incorrect test added in Ib9816d8b

The test was intended to check for whether we have a user ID *or* a user
name. Instead, it's checking if we have a user ID *and* a user name. And
it also failed to consider User:0.

Bug: T227656
Change-Id: Ia1b5c4a6ae028513b73a65cd2c885459327d29c3

Merge "Make RouterTest extend MediawikiUnitTestCase and not integration"

build: Updating mediawiki/mediawiki-phan-config to 0.6.1

Change-Id: I370fed5248c8b243527cafb77d80ddb6f4bc7064

Merge "Replace old URLs in SampleTest"

Make RouterTest extend MediawikiUnitTestCase and not integration

it's in unit directory, it causes things to break

Change-Id: Ibeafafde5a3b9f3f8e6fda0ceef8575aa544e20b

Merge "bagostuff: optimize SqlBagOStuff and fix failing segmentation tests"

Merge "rdbms: make IDatabase::isReadOnly() return true for replica DBs managed by LoadBalancer"

Pass in ServiceOptions to BlockManager

Change-Id: Ic63d7ff35a71e36c4e6157e9d472e2870f95f00d

Tidy up conditions for applying a block from a cookie

Change-Id: Id9dd6ae395f5bb811db4c741be9db8aa2eb6fb70

Merge "Update OOUI to v0.33.2"