Merge "redo: don't die producing xml files if rev text export conversion fails"

Merge "Update table header color of `.mw-datatable`"

Merge "Use IMaintainableDatabase type hint for DatabaseUpdater::newForDB()"

Localisation updates from https://translatewiki.net.

Change-Id: I1dcfd8baa72e77a0c43a30270cf3c39810f29760

Update table header color of `.mw-datatable`

Bug: T214208
Change-Id: Id5f14b9a3af40a83385dcb4baf0e2d464c5e07a9

Merge "Fix phpdoc for type hint for User::makeUpdateConditions()"

Merge "resourceloader: Remove unused 'skipped' property from registry"

Merge "resourceloader: Move registration of Resources.php to ServiceWiring"

Fix phpdoc for type hint for User::makeUpdateConditions()

Change-Id: I46b3842495b3b9b8d44d16f434de4b44c65c1d80

Use IMaintainableDatabase type hint for DatabaseUpdater::newForDB()

Change-Id: I56dd9f8d195ad60cf3ec9b81af2b4f25d1ca0a3a

resourceloader: Remove unused 'skipped' property from registry

This was added for debugging purposes, but I've never needed it
so far. It also makes the shape of objects in the registry
inconsistent, which can be a de-opt.

Keep it for now, but with a more minimal impact by storing it
in the same property.

Change-Id: I297e3adb4765b83b1e7356b81cd056f99b3de93d

Merge "Replace deprecated function wfEscapeShellArg with Shell::escape()"

Replace deprecated function wfEscapeShellArg with Shell::escape()

Change-Id: I4046d593d1450cfffc489ca2abadba1084a540e4

Merge "Improve docs for Title::getInternalURL/getCanonicalURL"

Make Special:Version escape extension names that aren't links

This is to make phan-taint-check happy. Technically this is
a breaking change, but I am unaware of any extensions actually
having their names be arbitrary html, and support for this
is not documented anywhere.

Bug: T216348
Change-Id: I065d7e57f36e079e0b02180379e2df1f8535f3a8

Merge "resourceloader: Fix ensureNewline( '' ) support for PHP 7"

Merge "resourceloader: Use self::class instead of 'self'"

Merge "Fix/suppress phan errors related to arrays (#11)"

resourceloader: Fix ensureNewline( '' ) support for PHP 7

substr( '', -1 ) returns '', not false, in PHP 7.

Change-Id: Ie48fe064385e70f070e3fbfe0b62198f36cccd6d

resourceloader: Use self::class instead of 'self'

Change-Id: I66d9974875d404fb5f7669f3962e694e1d56c350

Fix/suppress phan errors related to arrays (#11)

Change-Id: Ie5c05fbc88c51d493bc1462005d2f8dde5f72101

Merge "Change Database type hints to use IDatabase"

Merge "Fix phan errors by improving documentation (#10)"

Merge "Fix PhanPluginDuplicateExpressionBinaryOp in DjVuImage (#9)"

Merge "Fix misc phan errors by adjusting documentation (#8)"

Merge "Follow-up 8e1342ed47: Add @since for Title::NEW_CLONE"

Change Database type hints to use IDatabase

Change-Id: Ie1e86cd961a619335138402ba22fe2a73a3e62fd

Merge "debug: Use __CLASS__ to get the name of the class"

Merge "resourceloader: Add missing (string) cast to ClientHtml tests"

Merge "phpunit: Don't include PHPUnitCommand channel in MediaWikiLoggerPHPUnitTestListener"

debug: Use __CLASS__ to get the name of the class

self::class would have been better here for consistency across
the code base but seems HHVM handles self::class diferrently from
PHP 7, when used inside a trait. The difference is significant to the
tests and thus we keep the result the same by using __CLASS__ instead.

Change-Id: I58b452341a0567f4d6e6636a7e489531a5569d20

Follow-up 8e1342ed47: Add @since for Title::NEW_CLONE

Change-Id: I6ad81dad179b2f2f009c8dc43f51eb0083c066f3

Merge "mediawiki.Title: Remove redundant closure"

resourceloader: Move registration of Resources.php to ServiceWiring

This identified something we hadn't codified before which is that
we expect the 'startup' module to always exist. The code and tests
for ResourceLoaderClientHtml also assume this, which we now
acknowledge explicitly by leaving it in the constructor.

Bug: T32956
Change-Id: Iffc545eb32078aed0d059e5902a5c9382e14c641

mediawiki.Title: Remove redundant closure

This module is loaded with packageFiles, which is always executed
in module scope (with a closure), even in debug mode.

The behaviour of non-packageFiles debug mode is the only reason
files have closures.

Bug: T50886
Change-Id: I6c0e686cb1d6971a6a9c5eeab2d2a093d2dae2d3

resourceloader: Add missing (string) cast to ClientHtml tests

One of the assertions already did this, but the others not yet.

The reason we do this is because the return value is a WrappedString
object. The assertEquals() method is able to satisfy the expected
value because of loose comparison casting it to a string for the
case where it matches.

But, when it fails during development, PHPUnit was still printing
the object as "Actual" and the string as "Expected" which makes
it difficult to understand what's going on, and also no diff.

Change-Id: Ice9fe5061fb76867e9bee2f34c88d3ddac3144f9

phpunit: Don't include PHPUnitCommand channel in MediaWikiLoggerPHPUnitTestListener

Follows-up 96657099fc (T217489) adding MediaWikiLoggerPHPUnitTestListener.
It started printing messages from log channels to CLI output when there
is one or more failed assertions. This is useful.

But, it also interacts with MediaWikiPHPUnitTestListener from 2018,
which calls wfDebugLog('PHPUnitCommand', …) for every test and every
assertion. That allows one to later open debug.log and see MediaWiki's
messages interleaved with PHPUnit activity. This provides messages from
a much larger scope (including before test setup, and data providers).

But, this means we get:

 > There were N failures:
 > 1) ExampleTest::testGetThing
 > === Logs generated by test case
 > [PHPUnitCommand] [info] FAILURE in ExampleTest::testGetThing:
 > [PHPUnit\Framework\ExpectationFailedException] Failed asserting …
 > …
 > …
 > …
 > … {"private":false}
 > ===
 > Failed asserting that …
 > --- Expected
 > +++ Actual
 > @@ @@
 > …
 > /var/www/mediawiki/tests/phpunit/includes/…:…

This needlessly repeats the test name and method, and takes up
vertical space to describe the failed assertion in a way that is
less useful than the one that PHPUnit already prints after it.

Exclude these by default from CLI.

Bug: T217489
Change-Id: I0f471f82b16ba869d15389ed470bd73dc6deea78

Merge "jobqueue: update claim array in JobQueueMemory::doAck"

Merge "rdbms: deprecate unused aggregateValue() method"

Merge "jobqueue: clean up __construct() signature for some job classes"

jobqueue: update claim array in JobQueueMemory::doAck

Fixes regression from a51ea350bed6f66b.

Change-Id: Id4d703b5537069e785a5c1454b9fc2785494247b

Merge "for exports, make sure we compare page titles as strings only"

jobqueue: clean up __construct() signature for some job classes

The $params value should always be an array

Change-Id: If7a0350b64241c7b5bbbdbc7bc90379a93484013

Merge "resourceloader: Fix variable spacings where need be"

resourceloader: Fix variable spacings where need be

This fix is to make sure resource loader code conforms with convension
https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#Spaces.

Change-Id: I34c0a0addbf7c373fc3a64b644a3098bb485a3f4

Move PackedHoverImageGallery from PackedOverlayImageGallery.php

Change-Id: I7a764fa89637dc7c3865dbbbbe0d90d5bc268ed2

for exports, make sure we compare page titles as strings only

...and not as numbers!! Also added strict compare for the namespaces
field while we're in here.

Bug: T220257
Change-Id: If68b79334188c2f3be5d254bea3c1e27d52c4a9f

Merge "rdbms: add and enforce DB_REPLICA/DB_MASTER roles in DBConnRef"

Merge "Add Special:Users as a synonym for Special:ListUsers"

Fix phan errors by improving documentation (#10)

PhanTypeExpectedObjectPropAccess was flagged by phan in the
DatabaseMysqli and Preprocessor_Hash classes.

In Database, the $conn property might be a standard object, such as
`\mysqli`, which is not a resource.

In Preprocessor, phan was getting confused and thinking
PPDStack::getCurrentPart() was returning bool, and not a PPDPart object.
Adding explicit documentation about the return value fixed that.

Change-Id: I0a3aa219693da5cb46ff9c0936841ed740c6968a

Fix PhanPluginDuplicateExpressionBinaryOp in DjVuImage (#9)

== has a higher precedence operator than &.

So the line `$chunkLength & 1 == 1` is interepreted as:
`$chunkLength & true`. Probably not what was intended, but it actually
works out because true is cast to 1, and the expression will either
result with 0 (falsey) or 1 (truthy), and we were looking for it to
equal 1 anyways. Incredible.

I assume that it was supposed to be `( $chunkLength & 1 ) == 1`, so I've
modified it to use that.

Change-Id: If03823b9286e0d8282519949bf46e2ba4cca0843

Fix misc phan errors by adjusting documentation (#8)

PhanTypeInvalidRightOperand was triggering on
ApiQueryBase::showHiddenUsersAddBlockInfo() on the line
`$actorQuery['tables'] + $commentQuery['tables']` (and the next one).
For whatever reason, phan was unable to automatically understand that
the right side was an array (though it was able to understand
$actorQuery, which has nearly identical code...), so I added more
specific hinting to CommentStore::getJoin() using phan's union types,
which resolved the issue.

And incorrect documentation on LogEntry::getDeleted() was triggering
PhanTypeMismatchBitwiseBinaryOperands since you can't do bitwise
operations on a string and an integer.

Change-Id: I13b791e4b754ffbc340b55cfb752e2d9226f1949

Fix PhanUnextractableAnnotation* errors (#7)

Change-Id: I8194a7b9c9abce951002590030913706e9bd743e

Fix phan errors in ChangesListSpecialPage (#6)

Specifically in registerFiltersFromDefinitions(), phan was unable to
tell that $groupDefinition['class'] was a string. I think it assumed it
was an integer because the earlier $groupDefinition['priority'] was an
integer.

Use phan's more advanced type documentation system to instruct it that
$className will be a string.

Change-Id: I09f41366b713d6159b57e798cfd1720f42ba30ef

Resync phan error counts

Change-Id: I9d893d5d05ff83c8c8e8c807e35fc2474584a24a

rdbms: deprecate unused aggregateValue() method

Change-Id: I708a5786657e8d22e4b42230bfadb11ce0d7b9ca

Merge "Make WrapOldPasswords pass "refresh" to User::clearSharedCache()"

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

Merge "rdbms: Clean up MssqlBlob constructor"

Merge "Fix/suppress misc phan errors (#5)"

Merge "Fix/suppress misc phan errors (#2)"

Merge "rdbms: rename DB server index parameter to LoadBalancer::getMaintenanceConnectionRef()"

Make WrapOldPasswords pass "refresh" to User::clearSharedCache()

Change-Id: I944488abce4ea15c2f4954ffb82b0354424335db

Merge "build: Updating mediawiki/mediawiki-codesniffer to 25.0.0"

Fix/suppress misc phan errors (#2)

* Title: phan false positive
* McrUndoAction: fixed improper use of @param
* UploadSourceAdapter: fixed wrong type
* XmlTypeCheck: Use null so phan doesn't think we're trying to call the
function ''
* Database: phan false positive
* SpecialBlock: Use phan's advanced type documentation so phan knows
specifically what's being returned
* ChangesListSpecialPage: phan false positive
* BatchRowUpdate: Have default callback take a parameter so phan doesn't
think too many arguments are being passed
* MimeAnalyzer: left FIXME for relying on PHP 7.1 unpack() signature
* LanguageConverter: Specify types for $mTables since phan couldn't
determine it automatically
* preprocessorFuzzTest: Implement User::load() method signature

Change-Id: I08080ab636c5fe67ea6a4e14b2212d7523606e21

Merge "Fixes LocalFile isMissing always returning true"

rdbms: Clean up MssqlBlob constructor

Returning inside PHP constructors doesn't work, so whatever this code
was previously trying to do didn't work. Now we directly set $this->data
to the existing $data->data. Note that we can't fall through to the
instanceof Blob due to MssqlBlob->fetch() not being idempotent.

Also expand the @param documentation for $data.

h/t to phan for highlighting this and Anomie for explaining that the
behavior was wrong and how to properly fix it.

Change-Id: Ied5ba2f63cbaf7c8e0f951000dbc4aa3089edd80

Fix/suppress misc phan errors (#5)

Add lots of missing return statements, or remove incorrect doc blocks.

Change-Id: I0881e98fbb9d0d4cf79ecc824064d24538055d3f

build: Updating mediawiki/mediawiki-codesniffer to 25.0.0

Change-Id: I1132bb024abe9a5f8a5374eb062d1b8efbfc3007

Merge "Fix/suppress misc phan errors (#4)"

Merge "Fix PhanUndeclaredType* errors (#3)"

Fix/suppress misc phan errors (#4)

* RevisionStore: phan is confused by the array not being the callable
* ApiContinuationManger: phan false positive
* ChronologyProtector: fix doc block
* MssqlBlob: phan doesn't like __construct returning things
* WikiPage: Remove Revision typehint that doesn't match doc block

Change-Id: I0661d97424d0ad6f7a3357542e79aceb6a4f6c64

Fixes LocalFile isMissing always returning true

Bug: T220244
Change-Id: I6f4e1371759559a14cb7b2125792fcfbba7e4cf3

Merge "jobqueue: Change internal $params logic to teach Phan (3)"

Localisation updates from https://translatewiki.net.

Change-Id: I6bf15beac4259c667062669f27171fc4f750f28f

Make teardownTestDB() close any dangling connections before changing the prefix

Bug: T219673
Change-Id: I028fc16b874cac744f7215601eec754a76f31a3f

Merge "media: Convert JpegHandler to shell execution framework"

jobqueue: Change internal $params logic to teach Phan (3)

Follows-up 9b4938c40d02c, 4bff6f1558, and 766549999c.

It seems despite being optional and documented as 'array|Title',
Phan still thinks it is wrong to pass Title.

Try removing the explicit default of empty array,
the lower code already does this anyway.

This should fix build failures that are preventing merges
in repos where Phan is finding Job::__construct(string, Title)
where it currently fails as follows:

 CompileArticleMetadataJob.php:11
 PhanTypeMismatchArgument Argument 2 (params) is …\Title|string but
 \Job::__construct() takes array

Change-Id: I1c3aee273e0917e30b9c18b49b24fc7f966ff57c

Merge "Remove broken explode() from MediaWikiMediaTestCase::dataFile()"

Merge "Remove warning for unnused 4th argument on wikidiff2"

Merge "Introduce PermissionManager service"

Merge "Introduce 'clone' flag for newFromLinkTarget."

Remove warning for unnused 4th argument on wikidiff2

Since we changed the signature back to 3 arguments in wikidiff2 verion
1.8.0, the warning is invalid.

Bug: T220217
Bug: T203069
Bug: T194272
Change-Id: Ia326c67de28a4e9b024466c62097b4e1e1096007

jobqueue: Document Title as valid param type for compat (2)

Follows-up 9b4938c40d02c, and 4bff6f1558. The latter of which
updated ::factory() instead of __construct(). Oops.

This should fix build failures that are preventing merges
in repos where Phan is finding Job::__construct(string, Title)
where it currently fails as follows:

 CompileArticleMetadataJob.php:11
 PhanTypeMismatchArgument Argument 2 (params) is …\Title|string but
 \Job::__construct() takes array

As Phan is reading the php doc I guess.

Change-Id: I128d57ead6bcb9dbae99d41a1f23192c3f6fbdba

Merge "Clean up LoadBalancerTest and move more code to private methods"

Introduce PermissionManager service

First iteration of adding a PermissionManager service as a replacement
for Title::userCan and User::isBlockedFrom methods.

- Created PermissionManager service
- Migrated Title::userCan to PermissionManager::userCan and deprecated the first
- Migrated Title::quickUserCan to PermissionManager::quickUserCan and deprecated the first
- Migrated User::isBlockedFrom to PermissionManager::isBlockedFrom and deprecated the first

Same for User::isBlockedFrom and PermissionManager::isBlockedFrom - the
$user parameter is now required so the declaration is changed from
isBlockedFrom( $title, ... ) to isBlockedFrom( $user, $title, .. ) which
means before User::isBlockedFrom removal all calls to it need to be updated.

Added PermissionManagerTest, it copies TitlePermissionTest but uses
PermissionManager instance instead of Title methods, this way keeping both tests
in place, we can ensure that nothing was broken and both are in working state
during the deprecation phase.

Bug: T208768
Change-Id: I94479b44afb3068695f8e327b46bda38e44e691f

Introduce 'clone' flag for newFromLinkTarget.

The "new" part of some of the newXXX pseudo-constructors
in the Title class is a lie: these methods do not always
return a new instance. This patch ensures that this fact is
documented, and we have a safe way to get a fresh Title
from a LinkTarget.

Needed by I94479b44afb30

Change-Id: I4d561ef7d7447d3d6e35079cf656bd564882d25e

Add some more missing limit parameters to explode() calls

I benchmarked this again. The runtime of an unlimited explode() can be
quite high. This is not really a DoS attack vector as it would require to
post megabytes worth of input to the code, which will hit many other
limits before. I still consider it good practice to use unlimited explode()
only when it is actually allowed to return an unlimited amount of elements.

Change-Id: I30f8ca5dba7b317bb4a046b9740fd736b4eea291

rdbms: add and enforce DB_REPLICA/DB_MASTER roles in DBConnRef

Even if there is only one database, if a DB_REPLICA connection
reference is obtained and an attempt is made to write it then
it will still fail. This can make it easier to spot incorrect
database usage even in a simple vanilla developer environment.

Note that methods like ILoadBalancer::getConnectionRef() can
be used for local connections as well as foreign ones.

Change-Id: I5523daad1bdd64624d3e0dd41bfd8d078b1078b0

[i18n] Update Czech translations of special pages

Change-Id: I0c17dd0972623ba75b4f7e4b2fced59add402bbb

rdbms: rename DB server index parameter to LoadBalancer::getMaintenanceConnectionRef()

Change-Id: I4a7a72f8cf61a0cc21d88cbb62614221f42e0540

Clean up LoadBalancerTest and move more code to private methods

Change-Id: I4c541537f9b95918f8d1a057ccbc29e8525ddc15

Merge "jobqueue: Document Title as valid param type for compat"

Merge "Add missing transaction round commit calls to RefreshLinksJob"

media: Convert JpegHandler to shell execution framework

Change-Id: I99dbdf9108d3b3dc82acd0ca7ba5da9645f41790

jobqueue: Document Title as valid param type for compat

Follows-up 9b4938c40d02c.

This should fix build failures that are preventing merges
in repos where Phan is finding Job::__construct(string, Title)
where it currently fails as follows:

 CompileArticleMetadataJob.php:11
 PhanTypeMismatchArgument Argument 2 (params) is …\Title|string but
 \Job::__construct() takes array

As Phan is reading the php doc I guess.

Change-Id: I94b442f06b6858d136546bf22f2465cf8f071ab0

Add missing transaction round commit calls to RefreshLinksJob

Follow-up to 83933a436e1ee9d

Bug: T220037
Change-Id: Ib1ac31365f9c325c56bae11aefe825ad2b2be881

Merge "Add meta=userinfo&uiprop=latestcontrib"

Update OOUI to v0.31.3

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.31.3

Bug: T193731
Bug: T194523
Bug: T217381
Bug: T218408
Bug: T219033
Bug: T219826
Depends-On: I94d1fd4398ec92c2ec63ebf1e57b3b4fc72bd76a
Change-Id: Ibb276860815fc29beaea131401bb5dfeca5159d6

Merge "Use HTTP method as hint for the locking mode of sqlite"