Add @var to Pager classes

Some are needed to make better results with phan in extensions

For example:
File CentralNoticeCampaignLogPager.php line 12:
Assigning array{0:20,1:50,2:100} to property but
\CentralNoticeCampaignLogPager->mLimitsShown is
array{0:20,1:50,2:100,3:250,4:500}
[PhanTypeMismatchProperty]

With "@var int[]" the type is correct

Change-Id: Ic68910bf17344852ad11fcc000a47891e4bf0179

Update PHPDoc types in several maintenance scripts and related

There is like a billion of these in our core codebase. I would love to
update them all. But I don't have the time, at least not to do them all
at once.

TL;DR: It's an improvement. :-)

Change-Id: I0a9c51bdcf0f261f971586bb8703f418324b0f98

Merge "Remove various references to cURL in code comments"

Merge "Report logs for each individual test failure"

Merge "Title: Minor clean up for stricter checks, casting style, and doc types"

Report logs for each individual test failure

The initial implementation of reporting logs with test failures was
incorrect, it always reported the logs of the most recent test run.
Attach logs to the Test when a failure is reported and pull them
back out in the result printer.

Bug: T217489
Change-Id: I5aa55d6fa7a7ec03a2e71636b6b0366ea40605cb

Merge "Use the .= operator to concatenate a var with another"

Localisation updates from https://translatewiki.net.

Change-Id: I337a2609e45ccd9b3bb71c7d60e3847e320b5db6

Use the .= operator to concatenate a var with another

Rather than using $var = $var . $foo, to avoid the redundancy, use
$var .= $foo which is a valid PHP syntax and popularly used.

Change-Id: Idbbdb31a7b5561ed97f9ba0f05f6ac78c9419f82

Merge "ActiveUsersPager: Fix ordering and return 0-action users"

Merge "Stash WatchedItem changes so that the jobs run from the queue"

Merge "resourceloader: Remove wgAvailableSkins from mw.config"

Merge "Normalize use of "INNER JOIN" to "JOIN" in database queries"

resourceloader: Remove wgAvailableSkins from mw.config

Added in 2011 for use in QUnit tests (r80790, r80792).

That was a terrible idea in hindsight, which I removed again in
2015 with 6b758fc982972e, and 0f9e4ca0fb7c.

If a need for it were to arise in the future, we'd export it
within the file module that needs it, or via addJsConfigVars()
on only the pages that need it – not globally.

Bug: T217772
Change-Id: I3885cacaa9b33e6947dbaf26f9b6839e19588603

ActiveUsersPager: Fix ordering and return 0-action users

Ia9d2ff00 introduced two incompatibilities with what the base classes
were trying to do:

* The $descending flag passed to buildQueryInfo() does not actually mean
  DESC if $this->mIsBackwards is set. The old code just blindly assumed
  that was the case.
* For paging to work right, we must return all $limit rows, we can't
  filter out users with 0 actions. Thus we need to LEFT JOIN
  recentchanges (and move some conditions around to cope)..

Bug: T217525
Change-Id: Iba105a31ff61fbb14931bf8903538bfe7b0ba48b

Merge "resourceloader: Remove ResourceLoaderMaxQueryLength from mw.config"

Merge "resourceloader: Use local var for getConfig() calls in StartUpModule"

Merge "Fix HeaderCallback failing on headers without a colon"

Merge "Add help text to "partial" and "sitewide" options on Special:Block"

Normalize use of "INNER JOIN" to "JOIN" in database queries

The ANSI SQL default join type is INNER and this might save
some line breaks here and there.

Change-Id: Ibd39976f46ca3f9b71190d3b60b76ca085787a00

Merge "Make LinksUpdate::getPropertyDeletions private"

Merge "Add more documentation to DeferredUpdates::doUpdates"

Add help text to "partial" and "sitewide" options on Special:Block

Describe the differences between sitewide and partial blocks under the
respective radio options.

Bug: T213101
Change-Id: Ibf098c6becaf4824f8da2cade22e23f4ce00283b

resourceloader: Remove ResourceLoaderMaxQueryLength from mw.config

This is only used internally. It doesn't need to be exported
as part of mw.config.

Change-Id: If4206d797b5d426eb999c34ad5dde3ba141c6f05

resourceloader: Use local var for getConfig() calls in StartUpModule

Change-Id: I53af241454872fd228869b45dc74175c56026d6e

Merge "Hard deprecate MWNamespace::canTalk()"

Title: Minor clean up for stricter checks, casting style, and doc types

These improvements touch areas around cleaning up deprecated functions
such as intval(), strval() into casting with (int), (string), etc.

If such improvements is welcomed into the Title class, I can make more
of such but for now, I've just made a few, like 1/20 of these kind of
changes in the file so that, if it's merged and encouraged to continue,
I'll cleanup the file. Or as proposed on this patch, phpcs rules can be
used to handle such cases.

Change-Id: Ib90e880cb124e765379ddad531d4c89289d364de

Merge "user_group, the nonexistent table that keeps on giving"

Make LinksUpdate::getPropertyDeletions private

Change-Id: Iec1ae929566e25a278ef918f9f6b22343dce6cec

user_group, the nonexistent table that keeps on giving

Follow-up to 27c61fb1e94da9114314468fd00bcf129ec064b6.

Bug: T199474

Change-Id: Ie8e054f5898209c51538669149e966bee7754f1e

Revert "Convert MultiHttpClient to use Guzzle"

That breaks on Wikimedia beta cluster (T217733):

Warning: Invalid argument: option: 6 in
vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php on line 56

Warning: Invalid argument: option: 6 in
vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php on line 56

Warning: Invalid argument: function: not string, closure, or array in
vendor/guzzlehttp/guzzle/src/Handler/CurlMultiHandler.php on line 108

Fatal error: Uncaught exception 'ConfigException'
Failed to load configuration from etcd: cURL error 23: Failed writing
header

This reverts commit 1e048a08b565ae909e85465f8b09a27ed8480ce2.

Bug: T202352
Bug: T217733
Change-Id: I2384355043896128d3f191941e8da00fdc62361e

Add more documentation to DeferredUpdates::doUpdates

Change-Id: Id2f1fb624d03943e5eb4bed49d3435cf9c99fc25

Merge "resourceloader: Minor clean up in saveFileDependencies()"

resourceloader: Minor clean up in saveFileDependencies()

* Use early return instead of all-encapsulating conditional.

* Document why the try/catch is so big.

Change-Id: Ie19e18556e7ac0a12ad6b979367f8c6b786bbe31

Remove various references to cURL in code comments

Given we now use Guzzle as well, and don't require cURL for any
features at a basic level, our code should not be as assumptious
about the backend being curl. Such assumptions may be wrong
and lead to confusion and/or bugs.

Bug: T137926
Change-Id: I6ad7f76768348e1eb8c1fb46c8125cce9285dc22

Merge "Fix a rather fatal typo in rebuildrecentchanges.php"

Merge "API: Avoid duplicate logs to api-feature-usage"

Fix a rather fatal typo in rebuildrecentchanges.php

The JOIN condition was being ignored because there is no table called "user_group" in MediaWiki core.
Thus if and when using $wgSharedDB, the query would end up listing *all* registered users from the shared user table.
And even without $wgSharedDB, running rebuildrecentchanges.php would result in everyone's edits being marked as bot edits (recentchanges.rc_bot = 1) and thus hidden from the Special:RecentChanges page.

Thanks to Lcawte for reporting this bug.

Follow-up to 27c61fb1e94da9114314468fd00bcf129ec064b6

Change-Id: I18d658b67c50f2200341f732783c2e7524dd27f1

Merge "Block Special pages only if the user is sitewide blocked"

API: Avoid duplicate logs to api-feature-usage

It can wind up logging deprecations twice if extractRequestParams() is
called with different values for 'parseLimit', for example.

Change-Id: I921777089fb8cfb4339af6fd08ee3475ed31b7f6

Localisation updates from https://translatewiki.net.

Change-Id: I82b2f9fbde2a8c7524bd2de7ee207c829941a2cf

Merge "Clear cached HTML artifacts"

Merge "Convert MultiHttpClient to use Guzzle"

Merge "resources: Provide the moment gom-Latn i18n to gom as well"

Merge "Specify licence of mediawiki ontology.owl"

Convert MultiHttpClient to use Guzzle

Convert MultiHttpClient to use the Guzzle library.
Guzzle includes built-in support for concurrency, and automatic
fallback to php streams if curl is unavailable.

Bug: T202352
Change-Id: I703af901f9da33d20b5e0989941f3f7fd6609298

Merge "resourceloader: Remove addModuleScripts, and deprecate getModuleScripts."

resourceloader: Remove addModuleScripts, and deprecate getModuleScripts.

The addModuleScripts() methods were deprecated in 1.31 and 1.32,
these are now removed.

The getModuleScripts() are now deprecated as well, always returning
an empty array. To be removed in 1.34.

Depends on commits for bundled/wmf-deployed extensions that
remove the last few remaining callers to the deprecated functions
in: 3D, Collection, Flow, GlobalUserPage, and Wikibase.

Bug: T188689
Depends-On: If9f0bc6aef85117587fa1929f34f8861c8d80314
Depends-On: Ia8d41b97fbf6822f5f8f7ac889408acce1ac9a3a
Depends-On: I503b919739ea474ff33726815b0da55e2f7e2724
Depends-On: I236ef637fd03b810a46eb361e25067a037e9d183
Depends-On: I62e17779753b977a452cc0c9694947941e999cc3
Change-Id: I5a19b8f164ccf666485d2971202194b747f882df

resources: Provide the moment gom-Latn i18n to gom as well

Bug: T191339
Change-Id: Icf7556a1fa71e12dd1029765f81914a36a3c1c22

Merge "objectcache: optimize WAN cache key updates during HOLDOFF_TTL"

Merge "Remove references to obsolete special page and messages"

Merge "Various small code and comment cleanups to IndexPager"

resources: Remove deprecated 'jquery.hidpi' module

Deprecated in 1.32 and has no further purpose in its current
form as a jQuery plugin for <img srcset>, which we now use
natively without fallback.

The remaining logic for bracketed window.devicePixelRatio is
simple enough to inline as needed without the cruft and overhead
that comes with a centralised approach.

Bug: T202154
Change-Id: I729dfabcbb40a0a794d6b166a584f45a64ac0338

Merge "logging: Remove 'null' return from ManualLogEntry::getTags()"

Various small code and comment cleanups to IndexPager

Change-Id: I351c12d51468bd428207017929fec1e796097206

Drop AuthPlugin and related code, deprecated in 1.27

Bug: T215843
Depends-On: Ie49709faa6f67b0a31bd546823d29dbfea0a70eb
Depends-On: If8109dbfdeeb88a6265f4cd79d1ae81bff48d934
Depends-On: I36b195fa2d6c23a76631ebaf869c787e44427d60
Depends-On: Iaa171b085fde331eab7d85c7de74523e27fc625a
Change-Id: I330b30d6582034a233fed204c0680b1ce84eec6e

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

Merge "Recentchanges: Hide instead of disable namespace selector checkboxes"

Merge "Set api-request log http.request_headers properly"

Set api-request log http.request_headers properly

$request->getHeader( 'Api-user-agent' ) was returning false,
and we can only accept string header values.

Bug: T214080
Change-Id: I3de8a455ad7dab9e3a67e24439e95f4968c93243

Localisation updates from https://translatewiki.net.

Change-Id: If706f62773b164f4c26a0eedb1e6709971c77b17

Clear cached HTML artifacts

Follow up to I6aad563e48f41c783df8b176a4f437e60a1255cc
Can be merged when that patch has been in production for a week.

Change-Id: I9f6b523f3ece055f8e6e45dfbfeb9852bebe4772

Remove outdated reference to design.txt

docs/design.txt was removed half a year ago in change I85be15a68f
(commit 6760216e10), and was long outdated even then, so let’s stop
pointing people at it.

Change-Id: I165ff1e12fb79c480930289d25833992be61b891

Merge "resourceloader: Remove warning for handled localStorage error"

Block Special pages only if the user is sitewide blocked

Update the default implementation of FormSpecialPage::checkExecutePermissions()
so that a Special page is only blocked if the user has a sitewide block.

This change allows the user to continue performing critical functions (like
resetting their password) even if they are partially blocked.

Bug: T209097
Change-Id: I5190297b7b235b6ebbdfa522323ce9bbd46b6729

Allows to add subsections to special pages groups on special page "Specialpages"

Adding all pages into one section creates some inconsistencies where users
cannot clearly distinguish pages. Thus extensions and extension suites like,
e.g. semantic extensions, translation extensions or social profile extensions
can improve the situation.

Bug: T216467
Change-Id: Iddc28e040fe975f8f823c84919fa476d2a13af0a

Merge "Document the 'sitewide' option for the Block class"

Specify licence of mediawiki ontology.owl

Bug: T216842
Change-Id: Ide976016bc5d421458cfe4ad661d88bb32da61a6

Merge "Fix typo in Priority Hints static variable name"

Merge "mediawiki.cldr: Move file to its own directory"

objectcache: optimize WAN cache key updates during HOLDOFF_TTL

Avoid the ADD operation spam from all threads trying to access
a tombstoned key by checking the interim value cache timestamp.
This also avoids the GET/CAS spam from threads that manage to
get the mutex. If a single thread repeatedly accesses the same
tombstoned value in rapid succession, there will significantly
less cache operation spam.

Do the same for cache updates to keys in the holdoff state
due to "check keys" or the "touchedCallback" function.

Relatedly, fix getWithSetCallback() to disregard interim values
set prior to or at the same time as the latest delete() call.
This can slightly reduce the chance of the cache being behind
replica DBs for a second. It also avoids unit test failures
were a series of deletes and cache access happen at the same
timestamp (via time injection or regular system time calls).

In addition:
* Add PASS_BY_REF flag with backwards compatibility to avoid
  bloating the signature of get()/getMulti() with the new
  tombstone information needed for the above changes.
* Avoid confusing pass-by-reference in getInterimValue() and
  fix use of incorrect $asOf parameter.
* Move some more logic into setInterimValue().
* Update some comments regarding broadcasted operations that
  were not true for the currently assumed mcrouter setup.
* Rename $cValue => $curValue and $versioned => $needsVersion
  for better readability.

Bug: T203786
Change-Id: I0eb3f9b697193d39a70dd3c0967311ad7e194f20

Fix typo in Priority Hints static variable name

Bug: T216499
Change-Id: I444292afd1b8fcc81e7cf204f82ed512f53bcc85

Merge "Remove unused method Title::validateFileMoveOperation()"

Merge "API: Handle Messages in errorArrayToStatus()"

Remove unused method Title::validateFileMoveOperation()

This method has never been used and isn't covered by the deprecation
policy since it was protected.

Bug: T214316
Change-Id: I96404792e895f12abec048c62c4b70d86a3d8310

Merge "Avoid cached lag logging spam from changes list pages"

Merge "Various fixes to make phan-taint-check happier"

Merge "Various fixes for phan-taint-check"

Various fixes to make phan-taint-check happier

Bug: T216348
Change-Id: If4071e689f476d2138d8964598c5a02b09448677

Merge "Use htmlspecialchars() not htmlentities in xml export for validity"

Avoid cached lag logging spam from changes list pages

We do not really care about snapshot lag for these cases, so make
them use the same method that getCacheSetOptions() uses.

Bug: T160833
Change-Id: I238aa48e0cdafb2a7f06cae93642797439f94091

Various fixes for phan-taint-check

Bug: T216348
Change-Id: Id9e7a4704d62457cac9bd1ca6044dec76472e371

Merge "Status: Make various improvements to the Status class"

Status: Make various improvements to the Status class

These improvements touches areas such as;
- strict comparison as count() returns in int.
- use of single quotes for string literals.
- use of RequestContext to get the user's Language object.
- fix minor typo.

The todo for using RequestContext was added in 7c543bb6c25.
It used `$wgLang` at the time for consistency with Message.php,
which was changed in 2016 with 0beb5ca992c. Hence, it's now
fine here as well.

Change-Id: I943dfd761a8fca7e8b6658e3c9510ddc6694f042

Localisation updates from https://translatewiki.net.

Change-Id: I88973d2b1ab48171ec8d909a69540d03ed229a4e

Merge "resources: Remove 'jquery.mockjax' module"

resourceloader: Remove warning for handled localStorage error

Leads to more confusion that actual utility in practice.
The storage being disabled is an entirely supported and expected
scenario, which we handle gracefully.

When wanting to ad-hoc check whether localStorage is enabled,
browsers tend to show that in their interface already (e.g. to
the side of the address bar in Chrome). And it can also be
trivially checked from the console by looking at localStorage
or mw.loader.store.

Bug: T195647
Change-Id: I1edf89e52ec56b4919816878924f096d5b7f7751

Move specialpage redirects related classes to own files

Change-Id: Ibddbb0c4e40fff310b1720ec9c72f2683cc93f54

mediawiki.cldr: Move file to its own directory

Bug: T193826
Change-Id: I763ce0e8b500a9f499a4fc3f76da036b745a0054

maintenance: Remove unused code from several maintenance scripts

The most notable removal is done in the orphans script. This code was
really never used. Brion introduced it in 2005, already disabled.

I have all the respect for what Brion did. I just think it does not make
much sense to keep code around for so long if it does not work anyway,
and must be rewritten from scratch anyway now that we have multi-content
revisions and such.

Change-Id: I4e8050929f90e44a6e6051bf938993a8b0cdf649

Merge "Fix usage of MediaWikiServices in comments and documentation"

Fix usage of MediaWikiServices in comments and documentation

MediaWikiServices::getFoo()

is wrong. Right is:

MediaWikiServices::getInstance()->getFoo()

Change-Id: Ib6d844ddfe5bd6ccd72b887a63d9ad476c8d196f

installer: Replace the use of global $wgParser with services

global $wgParser was deprecated in 1.32 and services are used
instead to get the parser object via MediaWikiServices. Making
this replacement as we're avoiding globals.

Change-Id: I967a52c481bfcc83552053a03e88acb038810828

resources: Remove 'jquery.mockjax' module

Added in r93908 (2011, aa1bc6f742c0) for use by UploadWizard,
but this never happened. Instead, the UploadWizard extension
had its own copy for a awhile, but that isn't used anymore
either.

It's never been updated or seen other maintenance, and is
redundant nowadays with Sinon.

Bug: T202154
Change-Id: Ief4c87ac62efa32dbbfadea3e52bfce5db8bf184

resources: Remove 'jquery.xmldom' module

Added in r93908 (2011, aa1bc6f742c0), for use by UploadWizard,
but as far as I can tell this never happened, and it's not
in use currently anywhere in Wikimedia Git or elsewhere indexed
by Codesearch.

It's never been updated or otherwise seen maintenance, and seems
obsolete with native browser features nowadays.

Bug: T202154
Change-Id: I6eccf4f488b6e8d7afa5eb81505895d0729cbfea

build: Add names to Fresnel scenarios

Per T216854, it's now supported to give the scenarios a name.
This way, the Jenkins build will have headings like "scenario View a page"
instead of "scenario 0", which is difficult to look up or
understand without the config file side-by-side.

Change-Id: If9884c38eef8b05637a8045c4beb3f2bbf8f5de5

Various fixes to make phan-taint-check happier

Bug: T216348
Change-Id: Ice672eed3b7e4a199e1307a6477ffe31502b97b5

Merge "Add separate log action for file reverts"

validateRegistrationFile: Accept glob patterns

I've had this locally for a while now - it makes it easier to validate all
extension.json files that you might have checked out. The only catch is that
you have to escape the glob pattern from your shell.

Change-Id: Ic220034574129fab9e850f91c05dbd5e241f556c

Add separate log action for file reverts

This change s adds 'revert' as a separate log action for file reverts, as it
allows special formatting of log entries and revisions in UI and filtering
for file reverts specifically.

Even though there are no log entries with this log action right now, it does
seem that this was intended as some point, as there are corresponding
test cases in UploadLogFormatterTest, and is listed in
$wgLogActionsHandlers and https://www.mediawiki.org/wiki/API:Logevents
as well. Furthermore, the i18n message 'logentry-upload-revert' already
existed before this change.

Because this functionality can not be provided by tags, the 'mw-undo' tag
is not suited for this use case. However, it could be added additionally to
all log entries with this log action..

Bug: T60209
Change-Id: Ie1ccd8053dc5de58b2297a8460219f0233aab968