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()"

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

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

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

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

Localisation updates from https://translatewiki.net.

Change-Id: I333fd839423300bc3d1b9c62791a2d672a24d070

Move FauxResponse class to own file

Similar to FauxRequest

Change-Id: Ia9e4e7287ade605899d935db7c9f1f017e0bcb85

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

The documentation already claimed it returned array,
this is now actually true. Follows-up 037c06e886c63.

There are two ways it could be null:

* The setTags() method was never called, and the class member
  was implicitly null by default.

* The setTags() method was called with $tags = null.
  This happens WikiPage::doUpdateRestrictions() among other places.
  That could probably be fixed, but is out of scope for this change.

The getTags() method of this clss appears to have no outside
callers in MediaWiki core, nor in any other project indexed
by Codesearch.

Change-Id: Ib89c79ef90870506187c3c3b61464c2aa788c594

Merge "Show password policy flags on Special:PasswordPolicies"

Show password policy flags on Special:PasswordPolicies

Follow-up to I28c31fc4ea.

Also improves what policy values are considered disabled, documents
how to extend core checks/flags and adds a structure test for it.

Bug: T118774
Change-Id: I66bf396e8e8a8c310a47ba337abe9070e7e83ff6

Recentchanges: Hide instead of disable namespace selector checkboxes

* Set the initial state of the visibility of the checkboxes already in
  HTML. This avoids a flash of not-hidden checkboxes while loading.
* Also add class="mw-input-with-label" to prevent line breaks in the
  label.

Change-Id: I15781b95590faa9cf65c19fe6fc84fa94ec786dc

Merge "Simplify strings in PHP code"

Merge "maintenance: Replace custom .ext extraction with pathinfo()"

Merge "Add mediawiki.interface.helpers.styles where it is needed"

Simplify strings in PHP code

Change-Id: I481810ade68b0c5a5be21d22e2a107646d5813e6

Merge "Add UserSendConfirmationMail hook"

Merge "Hygiene: use null-coalescing operator when operating on tags"

Add mediawiki.interface.helpers.styles where it is needed

Bug: T212613
Change-Id: I7d5920c3464838f68dc1169779e2f2662a3e336e

Hygiene: use null-coalescing operator when operating on tags

Change-Id: I7d5b4e05ee6c010b4f9438bc4871a3369b1871a6

Replace lowercase {@inheritdoc} with @inheritDoc

According to the coding standards we even enforce with a custom PHPCS sniff.
It currently does not pick these mistakes up because of the curly brackets.
I'm not sure if this is worth an update of the PHPCS sniff. I wanted to
suggest this fix anyway.

Change-Id: I9041ea7a00baf7f55e0ff0e56879a89fb74bb479

maintenance: Add missing limit parameters to some explode()

This is, in theory, a loophole that can not only cause such code to
consume suprising amounts of memory and runtime. It can also create
suprising results. For example, an input like

 -param="might contain a = char"

might result in a cut-off value.

Not so much of a problem in a maintenance script. But still good
practice, I find.

Change-Id: I14fb278e6fdb61d0c486ca7e23229851ea479408

Remove references to obsolete special page and messages

Remove references to Special:BlockMe, and messages 'proxyblocker-disabled'
and 'proxyblocksuccess' from the documentation for proxy block messages.
Restore the updated documentation introduced in the patch that removed
these (Id4c7521443f049), which was reversed in I922e660fafc7a95.

Change-Id: I98fa1b48506b5592b5f59e23520c3f0872ff4b4a

API: Handle Messages in errorArrayToStatus()

Two bugs here:
* If the error array contains an entry using a Message object instead of
  a string as the key, it'll blow up trying to do
  `self::$blockMsgMap[$error[0]]`.
* If the error array contains a Message object not wrapped in an array,
  it'll blow up trying to do `...(array)$error`.

Bug: T217382
Change-Id: I2a08e02bca0fb194416b3f2e6a1d6192d5c13cb2

Merge "rdbms: document Database::doQuery() return value"

Merge "Avoid use of deprecated ObjectCache::getMainWANInstance method"

Merge "obectcache: split out some WAN cache refresh logic into scheduleAsyncRefresh()"

Avoid use of deprecated ObjectCache::getMainWANInstance method

Change-Id: I429e68c8642a00afa812f5ae900c1fb4f8308528

Merge "Remove comments literally repeating the next line of code"

rdbms: document Database::doQuery() return value

Removed random @throws tag from a subclass while at it

Change-Id: I1cad1f66b62bb4a306feb5c773ed5556891f82a7

Merge "Fix is_array check in ResourceLoaderImageModule"

RCFilters: Make SaveFiltersPopup header's label and icon wrap

Wrapping SaveFiltersPopup header's label and icon in languages like
Catalan and German.
Also re-positioning close button from absolute top and remove duplicated,
inherited properties.

Change-Id: I3fd35b237f82ab95b66e6e710cba36cf1887618f

Localisation updates from https://translatewiki.net.

Change-Id: Ibd3587a45d063cdd7f7276bd15319f1067819eab