Merge "resourceloader: Reduce 'implement' overhead for modules without scripts"

Merge "Move migrateActors.php to includes"

Merge "RCFilters: Convert RL modules to packageFiles"

RCFilters: Convert RL modules to packageFiles

* Use per-file require() to get classes' dependencies
* Export all DM classes to the global mw.rcfilters.dm object because the
  unit tests need that
* Don't make any of the UI classes globally available except
  MainWrapperWidget (and we can probably drop that one too)
* Rename mw.rcfilters.{dm,ui}.XYZ.js to XYZ.js

Change-Id: I8e541abf15c6987827a29eecc6a6f19f27378abb

Merge "wdio-mediawiki: Rename internal "Front page" ref to "Main page""

Merge "resources: Update jquery-client from v2.0.1 to v2.0.2"

Merge "mediawiki.user: Remove deprecated stickyRandomId()"

resources: Update jquery-client from v2.0.1 to v2.0.2

Bug: T101483
Change-Id: Ib1537cd92d733998818fe28b04ee81bec48a75d3

Merge "wdio-mediawiki: Move non-public methods to local scope"

wdio-mediawiki: Rename internal "Front page" ref to "Main page"

Also remove redundant 'new Promise' indirection, in favour
of native chaining through "then()".

This avoids a situation where wdio could hang indefinitely (until
timeout) when MWBot.request fails because we did not attach any
"catch()" handler to the promise. By always keeping a chain, the
failures are (by default) forwarded and thus propagated and thus
caught eventually.

Change-Id: If2f09479600029db2fa967c57082031744a06d8f

Merge "Follow-up I5bcae848: Remove from 1.33 RL, backported to 1.32"

wdio-mediawiki: Move non-public methods to local scope

Change-Id: I56258290d7c731b913f4df21c0bf4adc7fe4c69f

Merge "mediawiki.language: Deprecate mw.language.specialCharacters"

Merge "mediawiki.libs.jpegmeta: Remove deprecated mw.libs property"

Follow-up I5bcae848: Remove from 1.33 RL, backported to 1.32

Change-Id: I84fa5e2fae785f80833a3128c18790d6ad3d1042

resourceloader: Reduce 'implement' overhead for modules without scripts

Change-Id: I55728c526711545be3923d3e1e2f276cbfa52224

mediawiki.language: Deprecate mw.language.specialCharacters

Use require( 'mediawiki.language.specialCharacters' ) instead.

Remove mw.language.setSpecialCharacters(), because it was only used by
the specialCharacters module. There already was no getter.

Depends-On: I9cf525a1e1aadc12d72284d08b98882b0ab95f5e
Depends-On: I74e706a6603bd8fa899193de91e49f246608c4c1
Change-Id: I02e0e84b79309caa314954925cc9c0c2874b9bde

Merge "Drop SessionManager::autoCreateUser(), deprecated in 1.27 and unused"

mediawiki.user: Remove deprecated stickyRandomId()

Introduced and renamed/deprecated in the same cycle (MW 1.32), was
meant to be removed before release but forgotten.

Change-Id: I4bb219a1e38e15651115b70277e1968886dca690

mediawiki.libs.jpegmeta: Remove deprecated mw.libs property

Deprecated since 1.31.

Also fix existing uses to not use the private mw.loader access
pattern, which is unsupported and not needed here.

Depends-On: If0a125320af15b92216d98f54348f8b375cc1988
Change-Id: I403379ebf710077b77a02854c22071014d1a6bc4

Merge "Drop b/c reading of wgSessionsInObjectCache and wgSessionHandler, deprecated in 1.27"

Merge "resourceloader: Misc optimisations in startup module"

Drop SessionManager::autoCreateUser(), deprecated in 1.27 and unused

Depends-On: I261ea755c1d273ccc7fcc7c3b041623ccf4e331b
Change-Id: Ice2d755bee694b97e5ddbfc72caddf2b87ba0fc9

Drop b/c reading of wgSessionsInObjectCache and wgSessionHandler, deprecated in 1.27

Depends-On: I0d861a24b6cfd5c881112f15c682d97cef715cf0
Change-Id: I6b69d9ad3863512879f287f960828c1fdf7b214c

Drop User::getGroup{Name|Member}(), deprecated in 1.29 and unused

Except in the Configure extension.

Change-Id: Id6e2511cc2c5f86afcf0af2faa69d4a712efbe26

Drop EmailNotification::updateWatchlistTimestamp(), deprecated in 1.27 and unused

Change-Id: Idad6f5aa579e41e941d1074465dd2f0e0f4577dc

Move migrateActors.php to includes

This way it can be subclassed in extensions, like AbuseFilter
(Ic755526d5f989c4a66b1d37527cda235f61cb437).

Bug: T188180
Change-Id: Idf320232011c72e39267b1f3c39848aea35d37fe

Drop SearchEngine::getNearMatchResultSet(), deprecated in 1.27 and unused

Change-Id: I87987af702b9b92c4d1b3578e69f1b2e68529b52

Drop MWExceptionHandler::getLogId(), deprecated in 1.27 and unused

Change-Id: Id03aaf4f2ea5c910e13274826e44033e12adcd2e

Drop JsonContent->getJsonData(), deprecated in 1.25 and unused

Change-Id: I2cac3a1d2cda799c302ca2364fb91916c3240c11

Drop LinkCache->addLink(), deprecated in 1.27 and unused

Change-Id: Id6395cc9a122375349b7a13c051049bca3f5ab08

Merge "Drop EventRelayerGroup::singleton(), deprecated in 1.27 and unused"

Merge "Drop Linker::formatTemplates, deprecated in 1.28 and unused"

Merge "Drop Linker::formatSize, deprecated in 1.28 and unused"

Drop EventRelayerGroup::singleton(), deprecated in 1.27 and unused

Change-Id: I9e63fc255a575086a6c0934e51e593b5ee0a31d8

Merge "Persist sessions pre-send instead of post-send"

Merge "resourceloader: Require logName parameter in mw.log.deprecate()"

resourceloader: Misc optimisations in startup module

* Remove redundant trackQueue var.
* Simplify logError() code, and remove some local vars.
* Simplify Map#set() code.
* Simplify performance.now check. There are no known browsers,
  supported and unsupported, in which this property is existant
  as anything other than a function.
* Optimised baseModules iteration in sortDependencies().
  Slightly shorter and slightly faster as for-loop.
* Omit trailing dot from error message in one case.
  (None of the other error messages had trailing dots.)
* Add missing space to error message for mainScript.
* Simplify splitModuleKey() code.
* Remove redundant quotes in mw.loader.store.set() for encodedScript.

Differences in response body size on stock MediaWiki.

> uncomp 42152 - 41984 = 168 bytes saved.
> gzip-1 15437 - 15364 =  73 bytes saved.
> gzip-9 13506 - 13419 =  87 bytes saved.

Change-Id: I54c9ab30522c36ece054b915809fe20bff6867b3

resourceloader: Require logName parameter in mw.log.deprecate()

When omitted it, it previously defaulted to the key string, which
would automatically lead to the creation of confusing and
non-descript Graphite metrics. (Via the WikimediaEvents extension
via the 'mw.deprecate' tracking topic.)

If Graphite tracking is intended, require the logName parameter
to be set. For example, in wikibits, a deprecated 'write' method
is set on the 'document' object. Its Graphite property under
mw/js/deprecate/ will be document_write because logName is set.
Without logName, it would previously create 'mw/js/deprecate/write'
as metric, which easily clashes and is not very understandable
in Grafana.

An exception is kept for properties of the window object as their
fully-qualified property name would always be identical due to
them being available as global variables.

Change-Id: I9d99fe0395ec5309eac0895f2419f7cd16caeb94

Drop Linker::formatTemplates, deprecated in 1.28 and unused

Change-Id: I02e677021cdc1fc7ac6da4839a23b15afcfb2296

Drop Linker::formatSize, deprecated in 1.28 and unused

Change-Id: Id119e9f3ff0d27b59654876bad9212cc0f3c5a2d

Drop SiteSQLStore, deprecated in 1.27, unused

Change-Id: I6743417c3aafbba59ac02350e0190fbf688a5a40

Merge "Remove old Title CSS/JS methods, deprecated in 1.31, unused"

Merge "Remove Title->canTalk, deprecated in 1.30, unused"

Merge "Remove Title::escapeFragmentForURL, deprecated in 1.30, unused"

Use packageFiles feature to replace special-purpose RL modules

For config vars, add a virtual package file called 'config.js', and use
require( './config.js' ) in the module. For most data modules, add a
virtual package file called 'data.js', use require( './data.js' ) in the
module. Where needed, add wrapper files that put the data in the
relevant global object and in module.exports.

LanguageDataModule is the only special-purpose module not being removed
in this commit, because it uses languageScripts, and those are not
compatible with packageFiles (yet).

Also merge mediawiki.ForeignStructuredUpload.config into
mediawiki.ForeignStructuredUpload, since that was the only thing that
used it.

Change-Id: I203d4e3ecdeeeb16729eba2dcf40d11a41d2e582

Remove old Title CSS/JS methods, deprecated in 1.31, unused

Change-Id: I500b5efbdc2ee3b4ca09662ed0df89e9d7514bbd

Merge "Remove Title->getSquidURLs, deprecated in 1.27, unused"

Merge "Fix mw-ui-quiet+progressive/destructive selectors"

MWHttpRequest: Include the request ID in outgoing HTTP requests

In order to be able to trace requests across the production environment,
we attach the X-Request-Id to outgoing requests.

Bug: T201409
Change-Id: Iadd6cbd51a78a1876a1a318783b51635717d054f

NoLocalSettings: Remove style for the html element

There is no need to style the html element. A style of the body element
is enough.

Change-Id: I4ab90824de393de3569ff55cf0138a7aede5770c

Fix mw-ui-quiet+progressive/destructive selectors

Bonus: Fix color of disabled button text

Bug: T215610
Change-Id: I6e38355b46d87920605604beb0141cf097efab08

Remove double check of $oldRevision

The

 if ( $oldRevision )

above already do the same check.

Change-Id: Ibbd5b6209d67f195b6b7431bccc31c4401f0a799

Merge "Collapse some else statements and nested if statements"

Merge "Expose external link additions and deletions"

Collapse some else statements and nested if statements

Change-Id: Idd4d751e149e2ee11b8749ba847323f189b6ae51

Merge "Move upload stash related exceptions to own files"

Remove Title->canTalk, deprecated in 1.30, unused

Change-Id: I0c397f8d33a2841343c7907b233530bf3c88e212

Change HttpRequestFactory default to guzzle

Activate the changes from T212175 by changing the default
MWHttpRequest-derived type to guzzle.  This is being done
as a separate change so that it is conveniently revertable
if issues arise.

Bug: T214703
Change-Id: I0868d2e7f3eed6d7c6b15334eb3187f3edcd7bec

Remove Title::escapeFragmentForURL, deprecated in 1.30, unused

Change-Id: I649f3f1346c366e722fe4fd8690773bedce7ec54

Remove Title->getSquidURLs, deprecated in 1.27, unused

Change-Id: I9e9b8ba33ba5d5cb219d00879ecb45b6b005b7e3

resourceloader: Fix localStorage for multi-file modules

I had forgotten to update mw.loader.store.set() for the new multi-file
module format.

Follows-up fbbd65d2df43f522f4deae719.

Change-Id: I27edeb43b218c93761ea7c394e18fac7a5cc914d

Expose external link additions and deletions

Needed for links event stream.

Bug: T214706
Change-Id: Ie93c8d90c2f69d7a08b8a6e9a0a5bb8266d4dd51

Merge "API: Don't return a deprecation warning for default values"

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

Localisation updates from https://translatewiki.net.

Change-Id: I36b74b2277e53dceeb60a3662eb14da9fa4f0589

API: Don't return a deprecation warning for default values

If a deprecated parameter has a default value, or a deprecated value is
part of the default value for a parameter, don't give the client a
deprecation warning about it.

Bug: T215548
Change-Id: I980763e3d44fb1b7459c64b175fcaddf5fd44a13

Merge "build: Add 'svgmin' Grunt task and crush SVGs"

Merge "Ensure action object matches passed in right in Title::checkUserBlock()"

Ensure action object matches passed in right in Title::checkUserBlock()

There is a possibility that the retrieved action (by name) will not match the
passed in right, we should ensure that the retrieved action matches.

Also adds 'purge' to the whitelist of actions that are not prevented by a
block.

Bug: T213220
Change-Id: I5671dcd6f004b1663c8a5c5aec200abbf742ec1d

build: Updating mediawiki/mediawiki-codesniffer to 24.0.0

Change-Id: I66b1775b7c1d36076d9ca78cbeb42787a743f2aa

resources: Allow jquery.ui.widgets on mobile

Change-Id: If0d7b687e5996e5dcdce0651c85a4dc4461b482b

Merge "Stop overwriting $cache in User::getCacheKey()"

Prevent fatal PHP errors when PageRestriction::getTitle() returns null.

Update usages of PageRestriction::getTitle() to handle a null response.

Bug: T214763
Change-Id: Ied33e2c3c9442c47ae8084a97bb0921869fb9d49

build: Add 'svgmin' Grunt task and crush SVGs

Adding 'grunt-svgmin' package and corresponding 'svgmin' task and
also crush SVGs. Several of them are featuring bad UX and are not optimal
in many different angles (a.o. clarity, universality, recognizablity etc.).
Specifically the 'mediawiki.skinning' ones.
But while they are here, let's make them suck less.

Change-Id: I8191ceac14565ae31d941982fab1586fa9b6bbdd

Mostly drop old comment schemas

This removes most of the pre-CommentStore text columns, and the
$wgCommentTableSchemaMigrationStage setting that used to determine
whether the columns were used.

rev_comment remains in the code, as on Wikimedia wikis the revision
table is too large to alter at this time. A future change will combine
that with the removal of rev_user_text, rev_content_model, and
rev_content_format (and the addition of rev_comment_id and rev_actor).

CommentStore's constructor continues to take a $stage parameter, and
continues to have the logic for handling it, for the benefit of
extensions that might need their own migration process.

Bug: T166733
Change-Id: I1479c73774e01ead1490adf6128f820c09bce9d4

Improve performance of ActiveUsersPager query

The query can be very slow, as it has to scan all the recentchanges rows
for all the users in querycachetwo (for activeusers). We can speed that
up at the cost of not filtering out users who were active when
querycachetwo was last updated but aren't anymore.

Also in testing this I found that the query is extremely slow when the
actor table migration stage is in one of the transitional states. This
too can be sped up with some custom logic.

Bug: T199044
Change-Id: Ia9d2ff00cfcdcc6191d854eb4365ecbf67f60b1c

Stop overwriting $cache in User::getCacheKey()

Replace a few deprecated function calls

Bug: T215483
Change-Id: I000b58d8bf67d9c1ea0123a7547b986e8ca69f25

Merge "Make invalidateModuleCache() take a database domain ID"

Merge "Rename WikiMap DB domain ID methods to reduce confusion with web domains"

Merge "Ensure calls to Title::getArticleID use the same case as the definition"

Mark jquery.throttle-debounce as deprecated

Bug: T213426
Change-Id: If1c165d479bc7b47f806fa20e103059bffc761f7

Persist sessions pre-send instead of post-send

This avoids race conditions with certain web request patterns

Bug: T214471
Change-Id: I4dfee10326485e98b028585c7da2e6b30787bb91

Merge "resources: Update OOjs-Router from v0.1.0 to v0.2.0"

Ensure calls to Title::getArticleID use the same case as the definition

Change-Id: Ic92f43c9e9df2782ba20aa28ad9e6d2ef7976d2e

Merge "resources: Update OOjs-Router from 48302a572 to v0.1.0"

resources: Update OOjs-Router from v0.1.0 to v0.2.0

Change-Id: Ib6847f7328f28f31141f3ae59f0f9a0f49dbf508

Move upload stash related exceptions to own files

Change-Id: I23e81af2f1bf73f5cd78a01bcc2fcebb4c4ea038

Make invalidateModuleCache() take a database domain ID

Change-Id: I5d9880a5aee4b055365549671258eb9f242a37d6

Merge "Move exceptions JobQueueError to own file"

Rename WikiMap DB domain ID methods to reduce confusion with web domains

Those added in this same release do not need alias methods.

Change-Id: I05feeb9b0b13afe43aea1f95551965489cdbe094

resources: Update OOjs-Router from 48302a572 to v0.1.0

Add to foreign resource validation tool.

Change-Id: I06f5b2c258d1e72f2eb6268a6d24824bbe5d19eb

.phpcs.xml: Remove ignores for removed files

* includes/compat/normal/UtfNormalUtil.php
  Removed in 88ea69f2f7fa33dfc2ab787ed069380f76d4b7ef.

* maintenance/benchmarks/bench_strtr_str_replace.php
  Rewritten in 1031825f3b7732fb03fa745bff12e671c2f73d1a.

Change-Id: Ie88fb420e688f11c8a94ecf91cb8b839944d991d

Move exceptions JobQueueError to own file

Change-Id: Ibd67358cb7a14497dce8a3982ceba3b4300bdd65

Use PRC_AUTOPATROLLED for rollbacks and revdel

These are automatic, and happen regardless of whether the user
performing the action has patrol-related rights.

Bug: T190408
Bug: T184791
Change-Id: Icd1f3826c378ae759f501d87ffb9988766775b27

Fix condition if...else in getDB() &  PHPDoc comment for getUserDB()

So the conditional check should by default return $this->mDb if it's not
null, so, the else seems not to be needed(?). If we have a database handle
to process the current batch, $this->getDB() will return IMaintainableDatabase
but if it's not available (null), a call to $this->getDB() will return an
instance of \Wikimedia\Rdbms\Database is returned instead.

In accordance with the documentation (phpdoc), update the method getUserDB()
to be compliant with callers return type.

Change-Id: I95f3407dd2ffe8e4a1ad7a70be86b6cf3b65ff50

Merge "Remove deprecated class SquidUpdate"

Merge "Improve error handling for wrong autoloader permissions"

Merge "Fix message documentation to refer to Special:Block page"

Merge "objectcache: Fixes WinCache increment losing TTL."