Merge "rdbms: add SHOW to isTransactableQuery() and add QUERY_IGNORE_DBO_TRX uses"

Make UserEditCountUpdate::doUpdate avoid comparing IDatabase instances

Also make User::initEditCountInternal take the specific DB handle that
was waited on for replication. This shouldn't make a difference but makes
things more explicit.

Change-Id: Ibb8e083406eb4f4453afce94a2b33450239fce94

Merge "rdbms: fix comment for Database::runOnAtomicSectionCancelCallbacks()"

Merge "tests: Use a fancy generator provider in ReleaseNotesTest"

Localisation updates from https://translatewiki.net.

Change-Id: I2bd83f45ca37191f255513b0def3a65dbdf08bc3

Merge "LanguageConverter performance: Reuse the same string object for regexp"

rcfilters: Simplify FormWrapperWidget submit code by reducing jQuery use

None of this really benefits from the indirection of jQuery objects,
CSS selector matching, property accessors.

Somewhat follows fa261ed50f3a8c6e.

Change-Id: I22ea5216324c6bc9221ce77006430290c79d454f

Merge "Improve LanguageConverter performance on pages with many HTML tags"

Merge "objectcache: Hard deprecate ObjectCache::getMainStashInstance()"

Merge "when getting file (img) properties, suppress whines that it's not xml"

tests: Use a fancy generator provider in ReleaseNotesTest

Follows-up ffc23ee333fffb.

Change-Id: I99bf0d7115f6d2a84eb499d104d9cd227331c20f

Merge "Move trivial unit tests, round II"

LanguageConverter performance: Reuse the same string object for regexp

The regular expression used by LanguageConverter::autoConvert() is a
constant, but it is being created on-the-fly by every invocation.
This causes an expensive full-string comparison when the compiled
regular expression is fetched from the cache -- since the regex is 332
bytes long, the time taken for this comparison can add up quickly: on
page with a lot of tags, the regexp cache may spend more time looking
up the regexp than it takes to execute it.

Bug: T223969
Change-Id: I53c3e631e47a791cf3f0844dd79d4357605c59e3

Improve LanguageConverter performance on pages with many HTML tags

We were concatenating a single character to the end of the wikitext
source (which copies the entire string) every time through an inner
loop; when the page was large and the loop count was large this took
an excessive amount of time.

Bug: T223969
Change-Id: Ib80306b0bc6c73b750d492764f0e2dfd3a7a5450

Merge "test: refactor/speed up release note file test"

Move trivial unit tests, round II

Change-Id: I18e5a1514d7372b34f7fb460adf506a1ac65001f

objectcache: Hard deprecate ObjectCache::getMainStashInstance()

This method was soft deprecated in 1.28 and usage is no longer in core
except for FlaggedRevs ext (here in removed in patch I3ada0b53e8b2a6f118f).

Usage
=====

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

Depends-On: I3ada0b53e8b2a6f118fae37e7fc644b187bf95f8
Change-Id: I3f06b3732f67b703cdc12ec5849b67067475cdda

Merge "Move MultiHttpClient.php to libs/http"

Merge "s/Import Images/Importing Files/ in importImages.php"

Merge "Add 1.33.0 to HISTORY"

Merge "Add 1.31.3/1.32.3 to HISTORY"

Merge "API: Removed unused local variable `$p` from ApiQueryDeletedrevs"

Assert that LocalSettings file exists

This time, use $IP to get correct path to LocalSettings so that tests can be run
from outside MediaWiki root.

Change-Id: I0dd3dd281cff67b0d8acdc09b97328c44eca2bb6
Follows-Up: Ia36d22217f94e019bb5945705243bd0daace72e0
Bug: T226977

API: Removed unused local variable `$p` from ApiQueryDeletedrevs

This local var was introduced in fdddf94 and usage of it removed in
4e6810e. This is no longer used so cleaned up.

Change-Id: I2acf28b3a027284adbda70f50bee3c4a6b4a68b2

Add 1.33.0 to HISTORY

Change-Id: I11cfe407745ef7a3f2b48dc78f62cd1657290522

Add 1.31.3/1.32.3 to HISTORY

Change-Id: Ie7ae1bda5ffc910954852eb412ed30cb56810cb5

specials: Combine small miscellaneous 'mediawiki.special.*' modules

The format for this would be cleaner if T225842 were solved.

Bug: T193826
Bug: T219342
Change-Id: I9aaa52b567db254e1e79649b3b99260214da74b3

test: refactor/speed up release note file test

The test ReleaseNotesTest:testReleaseNotesFilesExistAndAreNotMalformed
takes 4 seconds on my machine. That is due to assertLessThanOrEqual
being invoked for each lines of various files at the root of the
repository. HISTORY has more than 20000 lines and assert functions are
rather slow.

Refactor the test:
* extract the logic to verify the file length of various files to a
  standalone test and with a dataprovider.
* flip the boolean logic ensuring whether the file exists. When it does
  not, PHP emit a warning which breaks the test anyway.
* Check the line lenght and collect errors in an array, then for each
  file run a single assertion to ensure the array of errors is empty.
  That effectively get rid of assertLessThanOrEqual overhead.
* Use assertSame() to show the full faulty line. assertEqual() shorten
  the arrays content.

Running tests went from 4 seconds to 700 ms on my machine.

Bug: T227067
Change-Id: I9bbbc6647ba9732b462e331e4b6d4acffe35e7cd

Type hint using interface

Fixes phan errors in FileImporter

Change-Id: I4ea2d081062f5f509040c73ff5f59ee5ffd700ad

rdbms: fix comment for Database::runOnAtomicSectionCancelCallbacks()

Change-Id: I9ab830be69fdaa46629edb4d96c29aa99005f92f

when getting file (img) properties, suppress whines that it's not xml

Imports eventually succeed but the log fills up with stack traces
needlessly.

Bug: T206013
Change-Id: Icb004954272ea8fc6fbc4fd5090cd1310d66946c

Merge "Type hint against interface for LoadBalancer in MediaWikiServices"

Merge "Unit tests: Support unit tests in extensions and skins"

Merge "Remove group validator from UnitTestCase"

Merge "GlobalFunctions: Tighten version number type for wfDeprecated()"

Merge "ApiBase: Drop get(Examples|(Param)?Description(Message)?)\(\), deprecated since 1.25 and 1.30"

GlobalFunctions: Tighten version number type for wfDeprecated()

To avoid cases like: facddc4 and Ifaf6ab0d36bc02bd170, make sure the
value of the mediawiki version  must be a string (e.g. '1.33') or a
boolean (e.g. `false`).

For some reason, typos can slip through for this value to be a float.
Let's safe guard for future cases like this.

Change-Id: I52bdf94c957bda67548a937d51649e925195f926

ApiBase: Drop get(Examples|(Param)?Description(Message)?)\(\), deprecated since 1.25 and 1.30

Change-Id: Ie62436692f39825237d36002aa01bedf1caa1fa3

Merge "Title: ensure getBaseTitle and getRootTitle return valid Titles"

Move MultiHttpClient.php to libs/http

Change-Id: I199e615fad363b1439604c9d01a9a475856d6128

Merge "Pass column as string to IDatabase::selectField"

Title: ensure getBaseTitle and getRootTitle return valid Titles

Since getBaseText() and getRootText() may return text with trailing
whitespace, getBaseTitle and getRootTitle must use makeTitleSafe instead
of makeTitle.

Bug: T225585
Change-Id: Id92df552f05e6a9ed7c9259a8779fa94c3587a3e

Unit tests: Support unit tests in extensions and skins

For now only extensions with skin/extension.json which follow PSR-4 are
supported.

DefaultSettings is loaded in bootstrap to work around errors when unit tests are
run followed by integration tests (e.g. `vendor/bin/phpunit`)

Usage:

 - composer phpunit:unit will run all unit tests in core and any tests in
   `{extensions/skins}/tests/phpunit/unit` which also extend
   MediaWikiUnitTestCase, and which have an extension.json file
- Pass a specific directory to only run unit tests for a particular extension,
   e.g. `composer phpunit:unit -- extensions/GrowthExperiments`

Bug: T226911
Change-Id: I237a9f82e4d1b05cf2f08b3e4bb7ffcd8d47111c

Localisation updates from https://translatewiki.net.

Change-Id: I4c609d902775f74131d2dc16ac0aef3bc418c140

Merge "RCFilters: Reduce startup overhead from 'config.json' computation"

Drop APIEditBeforeSave hook

Removed the source code and the test code.

Bug: T220656
Change-Id: I3674da6d9c33c08ec2f1774cb793b3a952480571

Merge "htmlform: Reveal method getOOUI() is called from when spitting deprecation notice"

Remove group validator from UnitTestCase

Whether a unit test has `@group Database` is irrelevant, since it will simply
not work if it relies on database access.

Change-Id: Ibe01d4912160ae44ca86ef6f7c3946501a8c067e

Drop the check for existences of LocalSettings.php in MediaWikiIntegrationTestCase

This is not needed and we should let testers run integration tests without
the need to define LocalSettings.php

This reverts small part of Ie717b0ecf4fcfd089d46248f14853c80b7ef4a76

Bug: T226977
Change-Id: Ia36d22217f94e019bb5945705243bd0daace72e0

Merge "Mention relevant tasks in comment (follow-up to c8733333)"

Merge "Check for valid user info before calling Linker::userLink() in Linker::revUserTools()"

Mention relevant tasks in comment (follow-up to c8733333)

Change-Id: I25a8c018c879dd13967cd67fd51eae9768e3aa4b

Merge "Defer cookie block checks to resolve a circular dependency"

Merge "Rest API: urldecode path parameters"

Merge "Fix SQLite patch-(page|template)links-fix-pk.sql column order"

Merge "Mail: Add visibility for some methods without method visibility"

Localisation updates from https://translatewiki.net.

Change-Id: I4ac7bf4bbcd320d208b9cc576cd699fcbf2cc8a4

Fix SQLite patch-(page|template)links-fix-pk.sql column order

Bug: T202211
Change-Id: Ife673b88c23acdc1bfc04630715d18243471035f

Merge "Avoid :checkbox Sizzle selector"

Mail: Add visibility for some methods without method visibility

Carefully, I checked usage of these methods in our code bases and
added the best possible visibilities to them. Not sure if I missed
something but let me know if I did.

Used private & public where suitable for the various methods. As for
__toString(), this is a magic method, so should be public per PHP docs.

Change-Id: Ie0987f4a984cac2f5eb1d9e21a305ad9467a8eb2

Move trivially compatible tests to the unit tests suite

This changeset resumes work on T89432 and related tickets
by porting an initial set of tests to the new unit test suite
separated out in I69b92db3e70093570e05cc0a64c7780a278b321a.
The tests were only ported if they worked immediately without
requiring any changes other than changing the test case class
to MediaWikiUnitTestCase and moving the test to the new suite.
If a test failed for any reason (even trivial misconfiguration),
it was NOT ported.

With this change, the unit tests suite now consits of a total
of 455 tests. As before, you can run these tests via the following
command:
$ composer phpunit:unit

Bug: T84948
Bug: T89432
Bug: T87781
Change-Id: Ibb8175981092d7f41864e641cc3c118af70a5c76

Add @var annotations to XmlTypeCheck class

Also fix type of XmlTypeCheck::elementOpen argument

Change-Id: Ia54464e98440992b435eac89f14b7e60fe812f62

Pass column as string to IDatabase::selectField

To match documentation

Change-Id: I7ee69fcd90ba1cba947ca2ef205f350dc456a2c7

Merge "Hard deprecrate Language::viewPrevNext()"

Omit argument when have no attributs on Xml::label

Change-Id: Ia5090985b619e872afd05eff09256c27ce3a01c9

Fix param documentation of ProtectionForm::show

It also allows array

Change-Id: I513d6fd356933a2c2f45a1773dd2f59b967a1bf2

Hard deprecrate Language::viewPrevNext()

Bug: T207977
Change-Id: I800689b04868d144f086913df9cd6b27dd74aa81

RCFilters: Reduce startup overhead from 'config.json' computation

Follows-up 9ba1d8f526e1. Use the new 'versionCallback' to only perform
the minimum amount of work needed to detect changes, without fully
transforming the data for delivery.

Factor out message parsing into a separate step in the tag list code,
and use the messages' raw contents for the versionCallback.

Stop using WANCache for the tag list, since it's now cached by
ResourceLoader, and only regenerated when needed.

Also refactor the ChangeTags functions around tag description messages a
bit, so that we can more easily get the message keys that are going to
be used.

Bug: T201574
Bug: T223260
Change-Id: I02082aeb289ce4156170b14b8840f6d92cbadb57

rdbms: add SHOW to isTransactableQuery() and add QUERY_IGNORE_DBO_TRX uses

This assures that less pointless transactions are started

Change-Id: I61a52e989e1c9c20c58b1da773f5209ba283a83e

filerepo: remove pointless "foreign" flag in ForeignDBRepo::getMasterDB()

This connection handle is not managed by any LoadBalancer, so nothing checks that

Change-Id: Ie60859aacbcccd8439bc58d816c6b94769a4331f

filebackend: remove wfTimestamp() dependency from FileJournal

Change-Id: Ia9b9e1d2d40d240566a37012fa01e5db0c276ec0

lockmanager: disable internal QuorumLockManager methods that should never be reached

Change-Id: If19f48f8c2d3fe8a38068b09d8691bc8f4eb5d46

htmlform: Reveal method getOOUI() is called from when spitting deprecation notice

Bug: T225683
Change-Id: I654b2d165feeb77394c214ae29bfaa7a9694b46d

Merge "Re-apply: Factors out permissions check from User into PermissionManager service"

Merge "Make ExternalStore wrap ExternalStoreFactory and create access class"

Merge "revision: rename various $wikiId fields/parameters to $dbDomain"

Merge "filebackend: avoid bogus SwiftFileBackend log warnings due to HTTP 202 responses"

Merge "rdbms: clean up LBFactory domain splitting code and remove redundant comments"

Merge "rdbms: implement IDatabase::serverIsReadOnly() for sqlite/mssql"

Merge "rdbms: update return type comments for ConnectionManager to IDatabase"

Merge "rdbms: combine trxLevel and trxShortId fields in Database"

Merge "rdbms: make FakeResultWrapper use get_object_vars() again"

Merge "DateTimeInputWidget: Avoid :input Sizzle selector"

Merge "rdbms: deprecate LoadBalancer::haveIndex() and LoadBalancer::isNonZeroLoad()"

rdbms: combine trxLevel and trxShortId fields in Database

This avoids having to keep multiple fields in sync

Change-Id: If96267afe56a9b9cd660bab333e7667e4d8dc3d4

Make ExternalStore wrap ExternalStoreFactory and create access class

* Inject settings and global instances as dependencies to the
  ExternalStoreMedium instances. This includes the local wiki
  domain, so that wfWikiId() calls are not scattered around.
* Create ExternalStoreAccess service for read/write logic.
* Deprecate the ExternalStore wrapper methods.
* Add some exception cases for bogus store URLs are used instead
  of just giving PHP warnings and failing later.
* Make moveToExternal.php require the type/protocol to decide
  which ExternalStoreMedium to use instead of assuming "DB".
* Convert logging calls to use LoggerInterface.

Change-Id: I40c3b5534fc8a31116c4c5eb64ee6e4903a6197a

revision: rename various $wikiId fields/parameters to $dbDomain

These fields are passed to methods like LoadBalancer::getConnection() and are
already expected to be DB domains. Update various comments as well.

Fix a few minor IDEA warnings.

Change-Id: I7cf76700690aec449872a80d30b5ba540d2bf315

rdbms: clean up LBFactory domain splitting code and remove redundant comments

Change-Id: I43812537e52b7a815e918512e2585e4858c06888

Avoid :checkbox Sizzle selector

Just use [type="checkbox"] for better performance.

Change-Id: I41e57873c4598ff3af6e70f913297c6b2d368f69

DateTimeInputWidget: Avoid :input Sizzle selector

This widget only creates <input> tags so just
search by name.

Change-Id: I170f67127453f979a0bbdd45f0487a5d350a4d5a

filebackend: avoid bogus SwiftFileBackend log warnings due to HTTP 202 responses

Change-Id: I12897bd2e29f18bbdce9a54a18415e83acc31365

rdbms: make FakeResultWrapper use get_object_vars() again

get_object_vars() avoids a bug in older PHP versions where
string digit fields end up as string digit array keys instead
of proper integers, which can cause errors on access.

Follow up to d9f688698ce0

Change-Id: Ic93231e4a7244f49149ce9ed35fbcb1e04ac4065

rdbms: update return type comments for ConnectionManager to IDatabase

Change-Id: I0ed1fc7b2c22192d80b99df54444a904c1778d05

Re-apply: Factors out permissions check from User into PermissionManager service

Was reverted by I549810a4cd2e424cc4a438887d2f24614a24cc00 due to
T224607.

Original change by  Vedmaka Wakalaka was
Ia0d840b772ea5f20c9594ce151cc57adc270e48b.

Original commit message:

The following methods should are factored out of the User class into PermissionManager,
leaving only deprecated stubs:

- User::isAllowed -> PermissionManager::userHasRight
- User::getRights -> PermissionManager::getUserPermissions
- User::groupHasPermission -> PermissionManager::groupHasPermission
- User::getGroupPermissions -> PermissionManager::getGroupPermissions
 -User::getGroupsWithPermission -> PermissionManager::getGroupsWithPermission
- User::groupHasPermission -> PermissionManager::groupHasPermission
- User::isEveryoneAllowed -> PermissionManager::isEveryoneAllowed
- User::getAllRights -> PermissionManager::getAllPermissions

Depends-On: I7909e9bd6bbfbd708c0a00b861a9b22a38c6665d

Bug: T218558
Bug: T223294
Change-Id: I8899240378f636ea70f447616710516c0a3c5c31

s/Import Images/Importing Files/ in importImages.php

Change-Id: Ie6cc415b8440951e3a04be4f28a30aeb47b0954e

Merge "Define unit and integration test suites"

Localisation updates from https://translatewiki.net.

Change-Id: I7711c0df21663fad4d5e5cc1c9fb7f0bb5c81a64

Merge "Avoid Sizzle's :first/:last selectors. Use $.first()/$.last()"

Merge "makeCollapsible: Avoid Sizzle selectors"

Merge "TOC: Avoid Sizzle selectors"