Add missing "@return StatusValue" to all abstract do… methods

All implementations do this, and all callers expect this. I wonder how
the documentation for this detail got lost?

Change-Id: If4977c9f4032ca5d93ea91f49cdc70f7ef1175ba

Add missing empty lines between methods

This might hint at an edge-case in the PHP CodeSniffer sniff that should
detect if methods are separated by a single empty line. Feel free to
investigate. I, personally, can't invest more time in this than
suggesting this quick fix.

Change-Id: Ib3c60eac76f255b4fe929f7933de256222716576

Merge "Fix non-editing partial block creation on Special:Block"

Fix non-editing partial block creation on Special:Block

Bug: T213707
Change-Id: Iaf8b3a6a0b2715808c9e6f8de94d315404368b12

Merge "registration: Allow overriding attributes in tests"

Merge "OOUI: Backport Ie43a0a346fea2dd2fda540d62f24410de1bc0d76"

registration: Allow overriding attributes in tests

ExtensionRegistry is a rather special singleton that can't use the normal
MediaWikiServices reset due to its early initialization, so introduce
a ->setAttributeForTest method to override attributes that are typically
loaded from extension.json.

Bug: T200013
Change-Id: I9e62a02ed2044c847e9ab2dcdfab094001f88986

OOUI: Backport Ie43a0a346fea2dd2fda540d62f24410de1bc0d76

Bug: T213544
Change-Id: I0bec76fd6d394329331c23ef43c28cb502cb5605

Remove leading space from 'selfmove' message

It would cause the error message to render as preformatted text.
https://phabricator.wikimedia.org/F27889207

Change-Id: I83136e4ac29eb091b4a86ab3bc22c907bb8ca7bc

Fix @param values on private functions

Change-Id: Id7442d5fc557227b3973f5008a3bc1390c528737

Merge "Rescue some more HISTORY"

Rescue some more HISTORY

Change-Id: Ieb7a95b91aa3d83574e8553acfb89e2b9eee027a

Localisation updates from https://translatewiki.net.

Change-Id: I1fe81ace8ae5a401ac3f7e3dbf29b82fa3f274f3

Merge "Clarify warnings about missing software diff3 and git."

Use the ?: shortcut from PHP 5.3 where it makes sense

Change-Id: Ieff70f23b19f0be3670c4ed3e2a5c30ef3792d7f

Localisation updates from https://translatewiki.net.

Change-Id: I0d13f0ef78e9ee1936af9f642c441853457e5409

Merge "Add special page class for disabling special pages"

Add special page class for disabling special pages

Similar to the ApiDisabled class, but for special pages.

Change-Id: Ib56860bc3f8a14fd332b775b1e7f50953641ee82

Merge "ImageListPager: Kill that annoying GROUP BY"

Merge "Move RELEASE-NOTES-1.32 (from REL1_32) into HISTORY"

Move RELEASE-NOTES-1.32 (from REL1_32) into HISTORY

REL1_32 has gone out of the door!

Change-Id: Id93f97d73b5213935379c8ab62724897504adaad

export: Mark DumpFilter::mark() as being protected

I used
https://codesearch.wmflabs.org/search/?q=-%3Epass%5C(
to make sure there really is no other call to this. This function really
is meant to be protected.

I also used
https://codesearch.wmflabs.org/search/?q=function%20pass%5C(&files=php
to make sure I got all subclasses.

Required for I7da632c43681438aa886bdb709379f10cd9cc658.

Change-Id: I9aaf95c66a6efa22131de627ce015587a109858b

Merge "Update OOUI to v0.30.1"

Merge "clientpool: Fix non-canonical order of elements (PHPDocs)"

Merge "block: Avoid use of is_null() PHP function where necessary"

Merge "collation: Fix non-canonical order of elements"

Merge "actions: Avoid use of is_null() PHP function where necessary"

clientpool: Fix non-canonical order of elements (PHPDocs)

Change-Id: Ia45326a11ae6da2b3196804d70f823c841570e7c

Update OOUI to v0.30.1

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

Bug: T76638
Bug: T210847
Bug: T212977
Depends-On: Ib466527110d5068e297c42ebb3dd381c0c83754a
Change-Id: Ib466527110d5068e297c42ebb3dd381c0c83754b

block: Avoid use of is_null() PHP function where necessary

WRT performance, is_null() is a few nanoseconds slower than === null
due to function call overhead. Also, I personally think using the
identical check on null is slightly more readable than using is_null().

Change-Id: I83cd7a6ca49acc548a7f46b2c37dfa189889bd19

actions: Avoid use of is_null() PHP function where necessary

WRT performance, is_null() is a few nanoseconds slower than === null
due to function call overhead. Also, I personally think using the
identical check on null is slightly more readable than using is_null().

Change-Id: Ie5b1ac17a18907e92eb3042decb7fc75e903123d

Merge "Fix SVG metadata handler by changing offset from -1 to 0"

collation: Fix non-canonical order of elements

Change-Id: Ie3f19e9e1f91faf6cee32c3c63d86215ba52c8e2

Fix Revison typo

Bug: T201491
Change-Id: I4a91d64dafdafd022462730c221cdf2a113d2ef2

Fix SVG metadata handler by changing offset from -1 to 0

Starting in php 7.1, negative offsets are supported in
file_get_contents. Previously they were not (and in fact, the
php docs said -1 was the default value for the offset if unspecified).

The result of this, is starting in php 7.1, MediaWiki could not determine
the aspect ratio of SVG files that were larger than 256kb.

Follows up e6de99be

Bug: T213501
Change-Id: I565e51cd8131542b9a70da49e9cc36c5594ecda3

Merge "rollbackEdits.php: Cosmetic changes of outputs"

Merge "build: Remove 'stylelint' package as it gets pulled-in by 'stylelint-config-wikimedia'"

Localisation updates from https://translatewiki.net.

Change-Id: I707519be5cbb133b724b02d0659fe35ce545df82

build: Remove 'stylelint' package as it gets pulled-in by 'stylelint-config-wikimedia'

Change-Id: I734ebf6c18a3266bec2d9c9e38e827c3ec412586

Merge "maintenance: Fix detection of bad hooks (wfRunHooks -> Hooks::run)"

Merge "filerepo: Remove begin/done comments from LocalFile"

maintenance: Fix detection of bad hooks (wfRunHooks -> Hooks::run)

It was still looking for wfRunHooks, which no longer exists
as of MediaWiki 1.32.

After this, it is now able to find one bad hook:

> Unclear hook calls:
> - Hooks::run( $action . 'ArticleComplete', [ .. ] ); # SpecialEditWatchlist

Also, remove the matching of wfRunHooks generally, given it no
longer exists. And also remove the historic notes from hooks.txt.

Change-Id: I4ac52ed75fb99d7775d4b4755e3f0871003d70a8

Add missing message for OOUI v0.30.0

Follow-up to 221b03c2436dd99f395e22e09fd356e4d380e04d.

Change-Id: I18193b191c516c14e32f0959dee2ac4c78564ce8

Merge "createAndPromote: use AuthManager::autoCreateUser"

Merge "Fix slow queries in migrateActors.php"

rollbackEdits.php: Cosmetic changes of outputs

Change-Id: I18313edf4fe9577fb488159b019689a450454b2f

Merge "Improve readability of LogFormatter::makePageLink"

Merge "Don't check for LIBXML_PARSEHUGE presence"

Merge "Remove a CURLOPT_CONNECTTIMEOUT_MS check"

Localisation updates from https://translatewiki.net.

Change-Id: I9ee05968b2a310c032969eaf5f52f703b796ba16

Fix malformed output of block logs

Avoid adding <span> to expiry parameter whenever block log is
stored as plaintext.

Bug: T208523
Change-Id: Id3fcdfda7311484911ae1f57ec262a2c6e6ab00b

filerepo: Remove begin/done comments from LocalFile

I argue that these comments don't provide any additional information.
I think the code explains itself already.

Change-Id: I7b495dfde5e8b27a4cdd4bfcf84d388ffc2db5e3

Improve readability of LogFormatter::makePageLink

Change-Id: I912d9626b99697d5a0394b72a56ffc8a5e7aa1bb

Fix order of arguments in ChangeTags::getPrevTags

Bug: T212703
Change-Id: I9e38c999718762daeb5c155ea8eaaa5d16dd85c1

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

Merge "Fix missing ATOMIC_CANCELABLE in MovePage::move()"

Localisation updates from https://translatewiki.net.

Change-Id: Ibe90f37243cab5598d9a1d319e0a88ae9fcf4fdb

createAndPromote: use AuthManager::autoCreateUser

Use AuthManager::autoCreateUser when creating a new user via
createAndPromote.php so that configured AuthManager providers have
a chance to perform validations and extra actions beyond local account
creation.

Bug: T212689
Change-Id: I4972507bbaf65bb542934c281c37bd8a5c7a26b7

Fix missing ATOMIC_CANCELABLE in MovePage::move()

Follow-up to I4aaa8af50d684de.

Bug: T213168
Change-Id: I0566b37117b6c69d4043e77e6368bf79fa84e325

Merge "Update the design of Special:Block"

Merge "build: Update qunitjs to 2.9.1"

Merge "Remove comments documenting constructors as "constructors""

Merge "Fix tag for partial blocks config"

Merge "Add @since tags to new public methods related to partial blocks"

Fix tag for partial blocks config

Change-Id: I421298f216bc018d1606c297b2f89ee53c8da7ec

build: Update eslint-config-wikimedia to 0.10.0

Change-Id: I2930bcabeeb7b7b2eb36063e77b26e664a691b43

Relax filter attribute filtering to allow self-referential urls

The filter attribute will often have things like filter="url( #foo )"
These local to the file filters in svgs should be fine (We already
disallow non-local xlink:href attributes on <filter> elements). In
fact, users can already do the exact same thing by doing:
style="filter: url( #foo )"

Bug: 67044
Change-Id: Ib25328c160c0d5ea7e01dc84616b76e1b9dcd0eb

Clarify warnings about missing software diff3 and git.

Sometimes we get people on irc freaking out about the warnings.
If diff3 is missing, you just get a lot of edit conflicts.

If git is missing (A somewhat more common case) it just means that
Special:Version won't show the commit hash for things. Which
first of all doesn't matter and secondly is irrelavent to 90% of
users who are not installing from git.

Change-Id: I9c9ea797a80bd4efb6826f94373deee0311dc83a

Add @since tags to new public methods related to partial blocks

Bug: T210369
Change-Id: I20197ceee8e5e43fac20addc494b841519b915c8

Remove comments documenting constructors as "constructors"

Such comments are cruft and do not add anything that's not already
obvious from the code.

Change-Id: I42068b0c6c904bc4ee38f1700e0ea14c3d659a25

Merge "HTMLSelectLanguageField: Add explicit sort"

Merge "Revert "Revert "Title::checkUserBlock should call User::isBlockedFrom for every action"""

Add me to .mailmap and remove duplicate name

Change-Id: I39de232fc6d3986ce36e205d89523fe3c8b52500

build: Update qunitjs to 2.9.1

Code and integrity from <https://code.jquery.com/qunit/>.
Release notes at <https://github.com/qunitjs/qunit/releases>.

Highlights for the browser version:

- [2.7.0] Reporter: Remove cost of DOM size when
  using "hidepassed=true".
- [2.7.0] Reporter: Use perf.now() for improved accurracy
  of unit test durations.
- [2.7.0] Reporter: Make debugging tests in a browser
  easier by adding timeline markers for DevTools.

Highlights for the npm package / CLI version:

- [2.9.0] CLI: Reduce dependency tree size
               from 143 packages to 9 packages.

Change-Id: I38408d90765cd18d5dd6952b8b6b30cbfa0c7ed5

Don't check for LIBXML_PARSEHUGE presence

It's been present since PHP 5.3.2.

Change-Id: I23a3c50c10e984abe6ff214fbf504ab6f6be763c

Remove a CURLOPT_CONNECTTIMEOUT_MS check

It was introduced in PHP 5.2.3.

Change-Id: I0c7e1684038c98677ceefbcfbb78641e90713002

Merge "objectcache: add more "touchedCallback" comments to WANObjectCache"

HTMLSelectLanguageField: Add explicit sort

Follow-up to I46dc6cc8f7d.

Change-Id: Ie5f8999cb6a24d394ceb1cf9b92facbd86431960

Localisation updates from https://translatewiki.net.

Change-Id: I4e4b7c8958b5c5626a9f5dc32f115c13f21eb8be

WikiPage::doUpdateRestrictions checks DB_MASTER

This would prevent double entries in logs when the same protection
is applied twice within a few minutes. Trying out usage of the
parent patch's new functionality.

Bug: T210983
Change-Id: Ibd6e481db2faa5955f52f36d5d7368adc8747c9c

Update the design of Special:Block

Update the design now we have the extra options for
partial blocks.

Bug: T202773
Bug: T212964
Change-Id: I569eb2a8b15b9ad7f9f0e99bce2cd9b818a82fb4

Merge "Do not pass $archiveName two times to OldLocalFile"

objectcache: add more "touchedCallback" comments to WANObjectCache

Change-Id: I55775216154aba0042f7c4cf1cb07336aa4a222e

Merge "Pass READ_LATEST in $flags to Title::loadRestrictions"

Pass READ_LATEST in $flags to Title::loadRestrictions

To be more consistent with other calls that do the same thing
Also bypass cache when READ_LATEST is set.
Removes duplicated $this->getArticleID call.
Removes $readLatest parameter from loadRestrictionsFromRows.

Bug: T210983
Change-Id: I2340e81fda8244b31f6a3c511ca29162f21d91c9

Merge "Make user rights conflict error a bit more obvious"

Merge "Add force option to password policy"

Make user rights conflict error a bit more obvious

Bug: T113048
Change-Id: Ia3992dcb473ec428188505e22b2def75fd23f98e

Revert "Revert "Title::checkUserBlock should call User::isBlockedFrom for every action""

This reverts commit 91fc7480301c8b2404cc280511c2fc83a6c3aa5d.

Hence it reapplies I6312a36911e5b73d773452fefef7ff25b9af08a4.

The changes made by this commit are (excluding the cases
checked for earlier):
* An action can only be blocked if prevents() is true/null and
isBlockedFrom() is truthy. Previously, any action other than
'edit' or 'create' would be blocked if prevents() was true/null,
regardless of isBlockedFrom().
* If an Action can be constructed, and requiresUnblock() is
false, the action won't be blocked. Previously, requiresUnblock()
wasn't checked.

This commit was previously reverted because it exposed that
EditEntityAction::requiresUnblock in Wikibase wrongly returned
false (fixed in I99061230023da2bbd0f98190a2907ca2e9717a4c).
Other potentially similar cases were audited in T211048.

Bug: T208862
Change-Id: If74a1d422290b8c62b7a7a8922621c73c9598269

Merge "Special:UserRights: Fix handling of empty conflict check value"

Merge "Improve OutputPage::wrapWikiMsg documentation"

Special:UserRights: Fix handling of empty conflict check value

Bug: T164211
Change-Id: I6b710a25e9be29497846fafe39c916d480989a96

Improve OutputPage::wrapWikiMsg documentation

Four spaces before the code sample makes it actually intepreted as
code sample in many phpdoc parsers (e.g. PhpStorm) instead of trying
to intepret HTML tags as HTML and making a mess of things.

Change-Id: I17d09a64116fa9b1372a7fd39f2a8049dcf805b1

Localisation updates from https://translatewiki.net.

Change-Id: I6fcdf609a7ffdacc51d7e8d1cf5974c69eba7dc6

Localisation updates from https://translatewiki.net.

Change-Id: Ie3a32793aa3d108fbef8534bbfe3d25419f365d2

Localisation updates from https://translatewiki.net.

Change-Id: I5f2f62795f2efc8b375d1ea2a0d413908792ecc9

Merge "Make ContribsPager properties private again"

ImageListPager: Kill that annoying GROUP BY

The GROUP BY is broken in later comment migration stages, we just didn't
notice because it's not used when $wgMiserMode is set. Instead of trying
to fix it, yet again, let's just do the count as a subselect.

While this code is being touched, let's also use Database's aliasing
functionality properly instead of passing "foo AS bar" as integer-keyed
values in $fields (with array_search() being used to change them!), and
let's start using Database->addQuotes() for string literals instead of
hard-coding single quotes.

Bug: T212980
Change-Id: Ide7f67893f625fe03127d4a775642cf0a9cca195

Make ContribsPager properties private again

Bug: T212771
Follows-Up: I2295a3fefe874a59458e6ea7edd2cfdd9e912096
Depends-On: I847cb6863be2b6b196b2a992c3dde13e0b21462a
Change-Id: I6b7d0c502ab0219de1b04c8e78e4aaffe43ea0d0

Localisation updates from https://translatewiki.net.

Change-Id: Ie96ca9a369fc56dfee770db77cd81378a9f9beda