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

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

[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"

foreign-resources.yaml: Add TODOs for the four remaining unverified libraries

Change-Id: I493415f89745eddf4a64ae23aced586028cc16aa

Avoid hitting DB in Category getters when they have the required info

Bug: T63045
Change-Id: Ib881ce263fa7a4a8256f627a56b774aaa815d3df

Merge "Collapse some nested if statements"

Collapse some nested if statements

Change-Id: I9a97325d738d09370d29d35d5254bc0dadc57ff4

Merge "resourceloader: Fix typo in comment to make more sense"

Merge "resourceloader: Fix return type doc for getSelectors() method"

Merge "jobqueue: dependency inject more objects into JobQueue"

resourceloader: Fix typo in comment to make more sense

Change-Id: I5d519fcded529d022461a040efe0fe17e7c1c589

Merge "Add &$tags argument to MarkPatrolled hook"

Merge "Fix wpCreateAccount bug when set on Special:Block url params"

resourceloader: Fix return type doc for getSelectors() method

So I've checked and this method should return an array and this array
specifically is an array of strings (string[]). In addition, remove an
irrelevant semicolon after foreach().

Change-Id: Id7dffe882583a7555d87a5b9c409584478c30a8c

Use HTTP method as hint for the locking mode of sqlite

Bug: T93097
Change-Id: I4aa44345e2f9e252c1a351f252bdcb9e15f180a5

jobqueue: dependency inject more objects into JobQueue

Also moved some WikiMap/$wgJobClasses checks to JobQueueGroup::pop
which is the method callers are supposed to use.

Change-Id: I2ab82d8adc4ae1f54697d2935afa2053539cf2db

Merge "tests: enable MariaDB/MySQL strict mode"

Localisation updates from https://translatewiki.net.

Change-Id: I41a457d73c94bab3dd7a1104d513aa9598aba4e3

Merge "phpunit: Call 'teardownTestDB' from shutdown instead of destruct."

Merge "Define a default Monolog logger if inexistant"

Define a default Monolog logger if inexistant

In the array 'loggers' the key '@default' is assumed to be present,
hence be sure it is defined: this is the system default for MediaWiki,
it can be overwritten if desired.

This default php://stderr with level ERROR is inspired from Monolog’s
own default, which is php://stderr with level DEBUG: this is similar
but less violent for the average MediaWiki sysadmin.

Bug: T196906
Change-Id: Id74083fc20ddf906c40e6d149095e8ade4c68572

Remove `background-size()` LESS mixin.

It is not useful any more, as it only applies to Android 2 and
got deprecated in 1.27.

Bug: T219956
Change-Id: If5174704c2b50ce45b6a02e1b53691df380c6f3c

Merge "Check line length of HISTORY"

Merge "ChangesList: Comment out newlines in EnhancedChangesListGroup.mustache"

Merge "protect.js: Use event 'input' instead of 'keyup'"

Merge "protect.js: Simplify by using jQuery"

Merge "protect.js: Reorder function declararion to avoid forward reference"

Merge "protect.js: Simplify by removing object ProtectionForm"

Merge "protect.js: Remove JavaScript global variable window.ProtectionForm"

Merge "Language: Use type hint instead of manual type check"

protect.js: Use event 'input' instead of 'keyup'

The event 'input' is already triggered when a character gets inserted.
The event 'keyup' gets only triggered when the key gets released.

* https://developer.mozilla.org/en-US/docs/Web/Events/input
* https://developer.mozilla.org/en-US/docs/Web/Events/keyup

Change-Id: If1990af8304e4941565367ab558e1f870480cea3

Merge "LogFormatter: ignore unrecoverable data"

Merge "Remove `.background-size()` mixin usage"

Fix wpCreateAccount bug when set on Special:Block url params

wpCreateAccount default is dynamically changed if the user
has not posted the form or interacted with the field in any way

Bug: T218554
Change-Id: Ied68dc60d57ed35ed3285c5cf89b772c8c4e41d1

Set synchronous = NORMAL for cache tables in Sqlite installer

In local testing, this makes SqlBagOStuff::set 17x faster.

Change-Id: I54d0d4f009ca1289485b89a1dcab46ffb31d8f11

Merge "ServiceWiring: Use RequestContext instead of $wgUser global"

Merge "installer: make sqlite installer move the job queue to another DB"

Merge "Add Job::getMetadata() and Job::setMetadata() accessors"

Merge "jobqueue: remove entire unused JobQueueAggregator class hierarchy"

Language: Use type hint instead of manual type check

Change-Id: I9c68d0838d5865c29bff85b9a6bf50e02e83f556

Add Job::getMetadata() and Job::setMetadata() accessors

Change-Id: I3a97008d324f600a1c9f6005673073277ee564fa

Fix comment typo in fresnel config file and remove an unnecessary comma

Change-Id: I43123d9c001ea59f38f904cd232cb5f8a11e15d8

LogFormatter: ignore unrecoverable data

It is possible for the log_params column of the logging
table to contain serialized data that cannot be deserialized
anymore because the types it references are missing.

It is currently the case with old Flow log entries on enwiki.
The extension is uninstalled and the UUID class is not found.

This patch proposes to simply skip the params and log a
warning in that case.

Bug: T212742
Change-Id: I3226b8fb338dd2b81e087af5d798d8f35368282d

Merge "MigrateActors: Improve query for log_search rows"

ChangesList: Comment out newlines in EnhancedChangesListGroup.mustache

This is a follow-up to 9c5a4773952f2e914eda668b71dedb7c78a9a258.

This change removes all spaces and newlines inside of the <td> elements
by a Mustache comment. This prevents that the newline get converted to a
<p> element when the recent changes get included by

  {{Special:RecentChanges|enhanced=1}}

Bug: T215737
Change-Id: I179d0b36b122efc3e1b7c193354ad03c3c9cefcf

protect.js: Simplify by using jQuery

Also remove useless boolean return.

Change-Id: I5a1ee977c2e7548bab98f8a7b6731262d175db51

Remove `.background-size()` mixin usage

Removing deprecated LESS `.background-size()` mixin calls.

Bug: T219956
Change-Id: I48a939eae125802778d0a8822961bef826970f46

Merge "RecentChanges updated to use pseudo elements for presentation"

Merge "Fully utilize LinkTarget passed to getRevisionByTitle"

RecentChanges updated to use pseudo elements for presentation

Bug: T219348
Change-Id: I6eeeaa3b58d37adb7fefb4cc6915022229b3b324

Merge "jobqueue: change MWException throw statements to using JobQueueError"

Fully utilize LinkTarget passed to getRevisionByTitle

Failure of getRevisionByTitle to pass its LinkTarget argument
to newRevisionFromRow resulted in a needless second instantiation
of the Title (an extra query).

Because newRevisionFromRow needs a Title, not just a LinkTarget,
it is unfortunately necessary to call Title::newFromLinkTarget
on it for now -- however this does not involve a DB lookup and
is on track to be fixed with revisions to the Title class.

Bug: T206498
Change-Id: Ic6f98d8fbf66d85121668571c17e148efc5ec2be

Fix PhanUndeclaredType* errors (#3)

Change-Id: I09420688dcd7dd7ba9dd17b189805a6f3f15f6fd

Merge "Style change tags correctly on log extracts"

Merge "Disable expensive {{REVISIONID}} magic word in miser mode"

Check line length of HISTORY

Slight modifications, almost all for URLs.

Change-Id: I612852e9a677a6a14fd48e2ccd7fcb998da9152a

Style change tags correctly on log extracts

You can see them when you view a previously deleted page
and some of those events have been tagged.

Bug: T212613
Change-Id: I88e637a0ee688dd23c0ec90790e8189ecf1a8789

Merge "Add test for Special:page::buildPrevNextNavigation"