Merge "Permit destructors in MediaWikiTestCaseTrait::createNoOpMock"

Merge "phpunit: Add more info to LocalSettings.php error in MediaWikiIntegrationTestCase"

Localisation updates from https://translatewiki.net.

Change-Id: I17cb40921799c2b97f8c8bfca201c6a77939ce48

Merge "Remove Oracle reference from WikiMap"

Remove Oracle reference from WikiMap

Bug: T230418
Change-Id: I06536034a9d7f176e65b087cc2d3ad65d172961b

Reset $wgContLang consistently for tests

We already reset $wgParser to a stub in a number of places where it
might have changed. Let's give $wgContLang the same treatment so we can
avoid special-case code.

This deprecates setContentLang(), which is no longer needed. The magic
of $wgContLang is now handled by setMwGlobals() and setService().

This is a follow-up to e4f69ee, which fixed one case of this problem.

Change-Id: I90925ef8b2a7478cce90d474db1b8b4539e45c15

Merge "Avoid multiply calls to MediaWikiServices::getInstance() in one function"

Merge "rdbms: clean up use of ATTACH queries in DatabaseSqlite"

Merge "objectcache: add "sessionConsistencyWindow" option to ReplicatedBagOStuff"

Merge "filebackend: improve FileBackendMultiWrite consistencyCheck()/resyncFiles()"

rdbms: clean up use of ATTACH queries in DatabaseSqlite

Defer the queries until a connection exists. Only issue issue the
them for databases that are different than the currently opened file.
Also, make handleSessionLossPreconnect() aware of attached databases.

In LoadBalancer::reallyOpenConnection(), avoid having the "catch" block
appear like it returns a half-constructed Database.

Change-Id: I9f676bb72a1ab06f0eac5820dce28231741c283d

objectcache: add "sessionConsistencyWindow" option to ReplicatedBagOStuff

Change-Id: I25af780f063879eda2de1b9a1168e37115f823ed

Merge "Allow CompositeBlock::appliesToRight to return null when unsure"

phpunit: Add more info to LocalSettings.php error in MediaWikiIntegrationTestCase

Change-Id: I3df68aab40f4b0bb1d5134330347d77ed3399f02

Allow CompositeBlock::appliesToRight to return null when unsure

CompositeBlock::appliesToRight checks $block->appliesToRight()
for each of the original blocks from which it is made.

AbstractBlock::appliesToRight returns:
* true if the block applies to the right
* false if the block does not apply to the right
* null if unsure

Before this, CompositeBlock::appliesToRight can only return true
or false. After this, it returns:
* false if false for all of the original blocks
* true if true for one or more original blocks
* null otherwise

Bug: T229417
Bug: T231145
Change-Id: Ie93b7691b57ac6a8f86b3641ad07a1d54babcd42

Merge "Unit tests: Remove duplicated code in ExtensionRegistry"

Avoid multiply calls to MediaWikiServices::getInstance() in one function

RepoGroup::singleton() results in
MediaWikiServices::getInstance()->getRepoGroup()

Also avoid getting the service in loops, doing it once before the loop
is cheaper

Change-Id: I29e0c7487e3e498559ff16a567e2fad0c0f8bb69

Add string cast when outputting exceptions in sql.php

Change-Id: Ibf21869a2525a8c55f53ec79727c818888344e99

Merge "Avoid usage of deprecated setTags() method, use addTags() instead"

Avoid usage of deprecated setTags() method, use addTags() instead

$tags can be `null` as seen in most cases and this causes failures,
so, the safer path is to make sure if $tags is a null, do nothing.

Change-Id: I5b7e39adba5d08fdcd42c437a72a391be98c8695

Unsuppress phan issues part 6

Bug: T231636
Depends-On: I50377746f01749b058c39fd8229f9d566224cc43
Change-Id: I2cd24e73726394e3200a570c45d5e86b6849bfa9

Unsuppress more phan issues (part 5)

Bug: T231636
Depends-On: I6e5fba7bd273219b1206559420b5bdb78734aa84
Change-Id: I50377746f01749b058c39fd8229f9d566224cc43

Merge "Unpack arrays on variadic arguments"

Merge "Remove unneeded redeclaration of mPage in Article subclasses"

Unpack arrays on variadic arguments

This avoids extra handling in the function to unpack an array

Change-Id: I93277599bc8125e23fca53d43a728fb7e9e39aad

Remove unneeded redeclaration of mPage in Article subclasses

It is already set in the parent class

To make phan happy, set the new type with a @property annotation

Change-Id: I47a83e3c4848b0a75284dc08c72df60b3d5dd19b

Fix method/function names case mismatch in core files

PHP doesn't care much but I think we humans do because we should
call methods by the name we give them. Method fixed are;

- isOk() -> isOK()
- setOk() -> setOK()
- teardown() -> tearDown()

Change-Id: I6b3f0cf3902887058efa426968da380803869e0b

Merge "jobqueue: Avoid usage of deprecated `MWHttpRequest::factory()`"

Merge "phpunit: Don't bother resetting unused global in UploadFromUrlTestSuite"

Merge "maintenance: Remove cross-wiki purging from rebuildmessages.php"

Merge "Add ext-dom to composer.json"

Merge "INSTALL/RELEASE-NOTES-1.34: Add required PHP extensions"

Setup: Remove debug message for "caches"

This is currently forcing more object cache interfaces to be constructed
than might be needed in a given request. The configuration for these
interfaces is usually quite straight forward or can be trivially
verified from eval.php.

If we do want to log this, I would recommend doing it from the classes
constructor instead e.g. something like:

  LoggerFactory::getInstance('MessageCache')->debug( get_class( $cache ) );

Bug: T189966
Change-Id: I06dc6dfbdf274bb57e9295b0c757b8d52c02acbe

Merge "lockmanager: sort key by bucket in QuorumLockManager::doLockByType"

Merge "Optimize HashRing to avoid hashing for the common single-location case"

Merge "Unsuppress other phan issues (part 4)"

Merge "Replace unecessary rollbackMasterChangesAndLog() call in doPostOutputShutdown()"

Merge "Remove redundant ignore_user_abort() call in ApiStashEdit"

Unsuppress other phan issues (part 4)

Bug: T231636
Depends-On: I58e67c2b38389df874438deada4239510d21654f
Change-Id: I6e5fba7bd273219b1206559420b5bdb78734aa84

Unsuppress more phan issues (part 3)

Bug: T231636
Depends-On: I78354bf5f0c831108c8f606e50c87cf6bc00d8bd
Change-Id: I58e67c2b38389df874438deada4239510d21654f

phpunit: Don't bother resetting unused global in UploadFromUrlTestSuite

messageMemc is not used for anything important. Should be removed
soon, but at least don't reset it in tests. If something new depends
on it, it should fail.

Bug: T189966
Change-Id: Id707db028e6eba03242fc9e1541d785a607d9be9

maintenance: Remove cross-wiki purging from rebuildmessages.php

This script relied on numerous outdated or unofficial methods that
are no longer supported.

* Global variable $messageMemc holding a BagOStuff based on
  MessageCacheType. This logic is now in ServiceWiring
  for the MessageCache service.

* String "{$db}:messages", should have been using BagOStuff::makeKey()
  and may've fallen out of sync.

* Deleting keys outright instead of touching check keys as
  MessageCache does, since it's been using WANObjectCache for
  several years.

All of this should be done via MessageCache::clear(), but that
can't be easily constructed across wikis on a wiki farm. That
requires instantiating the script for other wikis separately.

Remove support for that, recommending instead that site admins
run it in a loop for each wiki as needed.

Realistically though, that should never be needed. The script has
virtually no known use case, except (as documented in maintenance/README)
"after changing a wiki's content language" - which applies to one
wiki, not all a once.

Change-Id: I9f45d65f5cef93d6e332baf26d955d3f7a7c19d2

Localisation updates from https://translatewiki.net.

Change-Id: I130e7b793e6327e0d1f3b56d0b321f7bb3448c70

Change the autonym of Banjar from "Bahasa Banjar" to "Banjar"

Change requested on the Sundanese Wikipedia village pump

Bug: T231283
Change-Id: Ib2f49f77634c497135c0b32256fca4e919866a38

Change the autonym of Sunda from "Basa Sunda" to "Sunda"

Change requested on the Sundanese Wikipedia village pump

Bug: T228832
Change-Id: Id26493395a028b72f0254c7b866eb074eccbe1f9

Merge "Use DatabaseBlock instead of deprecated Block in CompositeBlock tests"

Merge "user: Hard deprecate `User::addNewUserLogEntryAutoCreate()` method"

Merge "ContribsPage: Re-remove the getContribs() method"

Merge "filebackend: fix PhpStorm warnings in FileBackend"

Merge "MessageFormatterFactory"

user: Hard deprecate `User::addNewUserLogEntryAutoCreate()` method

This method is no longer used in core or any extension per CS below;

https://codesearch.wmflabs.org/search/?q=addNewUserLogEntryAutoCreate&i=nope&files=&repos=

Change-Id: I906db9a123f6eb4210642b68525324071422eb61

jobqueue: Avoid usage of deprecated `MWHttpRequest::factory()`

Change-Id: I58c007436d38e4d0edd1ce14034b2f3bfb536df9

ContribsPage: Re-remove the getContribs() method

Dependencies remove the use from Flow and ArticleFeedback.

This reverts commit e6a8e5268d8b70867e58b1c827d42fec56bb315f.

Depends-On: If77a646344b3ee89505bb17be7571f63cff16a5a
Depends-On: I3b2fa1c65cfc32e8ebc21166d32f174557694d88
Bug: T220447
Bug: T231540
Change-Id: I3f87c0310f2f2de674d8c2fa017642bcc69fd834

Merge "rdbms: add more ScopedCallback::newScopedIgnoreUserAbort() calls"

Use DatabaseBlock instead of deprecated Block in CompositeBlock tests

Change-Id: I377da7e070ccd4f1ebb4a3e9619833e912180491

filebackend: improve FileBackendMultiWrite consistencyCheck()/resyncFiles()

Report file stat errors and sha1/stat mismatches in consistencyCheck().
This will trigger resyncFiles() which will make second attempt to check
the consistency while also fixing any problems if possible. Make sure
that it also bails out if such errors occur again.

Improve consistencyCheck()/resyncFiles() variable naming and add more
comments to FileBackend::UNKNOWN.

Also replace wfDebugLog() calls with PSR logger calls and wfTimestamp()
calls with ConvertibleTimestamp::convert() calls.

Bug: T231086
Change-Id: I69bcee636c6d99970e9a6448bb8296c0790c7254

Merge "Make Special:ChangeContentModel field labels consistently use colons"

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

Merge "Return null from EditPage::importContentFormData"

filebackend: fix PhpStorm warnings in FileBackend

Change-Id: Ia02bc8a94a28e7582c078bb89bd92d1ecf9e2841

Merge "Actually assign suppression-related rights to 'suppress' group"

Permit destructors in MediaWikiTestCaseTrait::createNoOpMock

709773ab5747 introduced this method, but its implementation
does not permit calls to __destruct. PHP documentation on destructors
says that they'll be called unconditionally, which makes using the
mock created with this method unusable for classes that implement
__destruct.

Bug: T231656
Change-Id: Icb4f978c78e726401d75627128c9c76f6f9afc87

Merge "Unsuppress other phan issues with low count"

Merge "Globally unsuppress phan issues with low count"

Also set $wgContLang in MWITC::setContentLang()

This assignment got lost in change Ie7a89f6ed7 (commit 47464abb4f) – the
resetServices() now implicit in setMwGlobals() resets the
ContentLanguage service, but does not update the $wgContLang global.
setService( 'ContentLanguage' ) does update it, but that method is
deliberately not used in this branch of setContentLang(), for reasons
that I assume are still relevant even though the comment explaining them
was removed – see change I8c60e37c17 (commit c70c2e4714).

Bug: T231630
Change-Id: I0170cc6bf40a5524523c45e8ffff8f72adc2fc47

Merge "Add some LocalRepo integration tests"

Unsuppress other phan issues with low count

And also update approximated counts, which for the most part are lower
than reported (hooray!)

Bug: T231636
Depends-On: Ica50297ec7c71a81ba2204f9763499da925067bd
Change-Id: I78354bf5f0c831108c8f606e50c87cf6bc00d8bd

Introduce BlobStore::getBlobBatch method

Bug: T230834
Change-Id: I56306c50a6617dc91e4eb362ef010703ac25d951

Globally unsuppress phan issues with low count

All of these suppression prevent the detection of many common mistakes,
and could easily prevent things like T231488. Especially if there are
few issues of a given type, it's way better to suppress them inline,
instead of disabling them for the whole core.
This patch only touches the one with a lower count (although those
counts may be out of date).

Bug: T231636
Change-Id: Ica50297ec7c71a81ba2204f9763499da925067bd

Add some LocalRepo integration tests

Coverage is still far from 100%, but better than nothing.

Change-Id: Ifb22491414653a5730a83434c4ea31a3540f7d26

Localisation updates from https://translatewiki.net.

Change-Id: I386c2b204c10a0d8d79d95f78b502bc3585dbb7d

Replace unecessary rollbackMasterChangesAndLog() call in doPostOutputShutdown()

The called method should not touch the databases

Change-Id: Ic1e242240a8676c5f9b6d5bce2b901412629e238

Remove redundant ignore_user_abort() call in ApiStashEdit

This API module has mustBePosted() return true and such POST
request are already covered by both WebStart.php and
ApiBase::useTransactionalTimeLimit().

Change-Id: Ie6f6af1459dde82572be8784049a307a3a769a1c

rdbms: add more ScopedCallback::newScopedIgnoreUserAbort() calls

Place more calls in LBFactory and un-owned LoadBalancer instances

Bug: T231443
Change-Id: Id93e70ada114801efe3aa81e865f0a930489bd29

Optimize HashRing to avoid hashing for the common single-location case

Change-Id: I3423ac8bd5c56db60630257ac04706838a832d36

Merge "resourceloader: Remove Grade C sniffs for WebOS <=1.4 and SymbianOS"

lockmanager: sort key by bucket in QuorumLockManager::doLockByType

Change-Id: Ida9b3979a7d6ad36e669761f2e23eeef6c67efae

resourceloader: Remove Grade C sniffs for WebOS <=1.4 and SymbianOS

Added originally in c46fc745a9de7201e based on the support document for
jQuery Mobile 1.3, via <https://jquerymobile.com/browser-support/1.3/>.

These have been obsolete since T102318 because startup.js is now a feature
test, with UA-snifs only for browsers that both *do* have all Grade A
capabilities for which we still want to disable the JS pipeline.

* Palm WebOS: Upstream jQuery 1.x allegedly worked fine on WebOS 1.4+.
  Not sure why we excluded WebOS <= 1.4, looks like that should have
  been <= 1.3 instead.
  If it doesn't have the neccecary HTML5/ES5/CSS3 features,
  it will remain in Grade C without a sniff. Otherwise, it's probably
  fine to enable. Either way, not officially supported (Grade X).

  Usage on WMF wikis is presumed less than 0.1% as neiter Palm nor
  WebOS show up on https://analytics.wikimedia.org/.

  The example UAs mention WebKit/525 and WebKit/532 which
  correlate with Safari 3.x and Safari 4.x, which we know
  doesn't pass the feature test.

* Symbian:
  The example UAs mention WebKit/413 which corelates with Safari 2,
  which we know doesn't pass the feature test.

Change-Id: I62a83e3b4967f309739095d9c73650f9859ee856

Merge "logging: Deprecate LogEventsList::$mDefaultQuery member"

Merge "Improve formatting of constructor documentation for block classes"

Merge "Preserve grapheme clusters in upper corner completion suggester highlighting"

Preserve grapheme clusters in upper corner completion suggester highlighting

Grapheme clusters in Indic scripts can be split by highlighting,
resulting in really ugly text in the suggestions. For example,
म +  े should equal मे, but doesn't if highlighting markup comes between
the two characters. Affected scripts include Devanagari, Tamil, Telegu,
Thai, Myanmar, Khmer, and Javanese, and probably others. Latin and
Cyrillic script also have the problem, but the presentation is not as
bad (at least on some browser/OS combinations).

- Define a comboMarks regex (equivalent to \p{Mark} in other programming
languages) and use it in a new highlighting function,
prefixPlusComboHighlight, to include trailing combining characters in
highlighted text.

- Change the calling function to use prefixPlusComboHighlight instead of
prefixHighlight.

Note: This only fixes problems with **grapheme clusters** in
suggestions in the **search box in the upper corner** (upper right in
LTR languages, upper left in RTL languages). The main search box on the
Special:Search page uses different code. Ligatures, such as
Arabic ي + ا becoming يا, are not covered by this simple regex-based
solution.

Bug: T35242
Change-Id: I915c50180bc2196e0302f27835241624b7837f16

logging: Deprecate LogEventsList::$mDefaultQuery member

This is a follow up on follow-up-I91003a9a3f2a56114de5b

Change-Id: If670ed959bcbc3149b0979541c641ada63377b7f

Merge "Move User::getAllRights to PermissionManager."

Merge "Remove usages of Title::quickUserCan"

Merge "ContribsPage: bring back getContribs() method"

ContribsPage: bring back getContribs() method

Follows-up 73664393f8da2ec.

Apparently the method still has callers. Hot fix (UBN).

Bug: T231540
Change-Id: I09ba81fc7ac4afe4c5cc54c3a589a54e31e9c419

Remove usages of Title::quickUserCan

Change-Id: Ifa53e0ec800e23dc4184d133a100fb9378dfee9e

Merge "mime,logging: Remove unused private methods"

Merge "Integration tests for Language fallback methods"

Merge "createNoOpMock() method for PHPUnit tests"

Merge "filerepo: rename resolveToStoragePath() and tweak file operation arrays"

Merge "HashRing optimizations and PhpStorm warning work-arounds"

Merge "Remove $wgMemCachedDebug"

Merge "filebackend: fix @throws comment in FileBackend::makeContentDisposition()"

Merge "filebackend: add idiom constant to FileBackend for null results"

mime,logging: Remove unused private methods

These methods are defined but not used.

Change-Id: I91003a9a3f2a56114de5baede6a1034ee4ed8d80

filerepo: rename resolveToStoragePath() and tweak file operation arrays

This makes it clearer what the method actually does

Change-Id: Ifd69484560eb2949242927bea820c1c7786c43d2

HashRing optimizations and PhpStorm warning work-arounds

Change-Id: I640636575f06a419e0f78504d8e7ba61edfad8e8

Remove $wgMemCachedDebug

Change-Id: I98380ac1cb66e33719b4836410311aea3513115c