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: 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

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

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

Merge "rdbms: lower value of MAX_LAG_DEFAULT to 6"

Merge "rdbms: move "maxLag" parameter up to LBFactory and add comments"

Merge "Add test for User::isBlockedFrom"

rdbms: lower value of MAX_LAG_DEFAULT to 6

This matches the "max lag" value used in the server configuration
array for WMF sites for a long time.

Change-Id: Ic2e82a8cc0932e7cf28ababc2eb0849406da06ef

Merge "Follow-up dbfe1c9a935: Fix RELEASE-NOTES typo"

Merge "resourceloader: Support TestModules registration via extension.json"

rdbms: move "maxLag" parameter up to LBFactory and add comments

Also make the default value constant be define in only one place

Change-Id: I1abd2b4569910031853b25f92d7cb69a1614d05b

Follow-up dbfe1c9a935: Fix RELEASE-NOTES typo

Change-Id: Iff1acb43b32fb8c0d64f4b59d8651d9f7decff50

Improve error handling for wrong autoloader permissions

Running Composer manually with a user different from the webserver
user and ending up with a vendor/autoload.php file that's unreadable
to MediaWiki seems like an easy mistake to make. Make the error
message when that happens less cryptic.

Change-Id: I52600adceb38a7fc5384d00b5298e46a782c684f

Merge "Disallow user suppression in a partial block"

resourceloader: Support TestModules registration via extension.json

Bug: T126091
Change-Id: I27ecebe27d7aaebe6d1317bc5eaea9cca368b45d

Disallow user suppression in a partial block

Hide the "hide user" checkbox on Special:Block if the block
parameters specify a partial block. Return an error message
if a partial block with user suppression is specified via
the API. Also, make sure that an existing user suppression
is lifted if the user's block is reset to a partial block.

Bug: T210002
Change-Id: If42edfc85580d6bad6f4b397517e77e63deecc87

ResourceLoader: Add support for packageFiles

Package files are files that are part of a module, but are not
immediately executed when the module executes. Instead, they are
lazy-excecuted when require() is called on them. Package files can be
scripts (JS) or data (JSON), and can be real files on the file system,
or virtual files generated by a callback.

Using virtual data files, server-side data and config variables can be
bundled with a module. Support for file-based require() allows us to
import npm modules into ResourceLoader more easily.

The require function passed to each script execution context, which was
previously a reference to the global mw.loader.require() function, is
changed to one that is scoped to the module and the file being executed.
This is needed to support relative paths: require( '../foo.js' ) can
mean a different file depending on the path of the calling file.

The results of require()ing each file (i.e. the value of module.exports
after executing it) are stored, and calling require() on the same file a
second time won't execute it again, but will return the stored value.

Miscellaneous changes:
- Add XmlJsCode::encodeObject(), which combines an associative array of
  XmlJsCode objects into one larger XmlJsCode object. This is needed for
  encoding the packageFiles parameter in mw.loader.implement() calls.

Bug: T133462
Change-Id: I78cc86e626de0720397718cd2bed8ed279579112

Merge "objectcache: avoid occasional test flakiness due to microtime()"

Localisation updates from https://translatewiki.net.

Change-Id: I8431d4b4bd6c05fb77390e8b850edd88509fa41d

Fix message documentation to refer to Special:Block page

Change-Id: Ib17e91c123f9f8998d2739fcb6b01c313389ea24

Remove deprecated class SquidUpdate

Change-Id: Iac53864296990befe7c573a29dd2f35aca52a79e

Move interface ICacheHelper to own class

Change-Id: I6f2b6507b0038f0988e4db5566ebddbe0d734a70

Add test for User::isBlockedFrom

In the complicated decision tree for checking if a blocked
user is blocked from their talk page, one situation was not
being tested: there's a partial block to the user talk
namespace, the block is flagged as allowing a user to edit
their talk page, BUT $wgBlockAllowsUTEdit is false. In this
circumstance, the user should be blocked from editing their
talk page, as outlined in T210475.

Also, fix whitespace and make messages clearer, since this
is now quite complicated.

Change-Id: I234f3019d55a6da0da091a2eaae6c791be01b436

Merge "ApiQuerySiteinfoTest: Conversion of $wgContLang to service"

Merge "Fix 'ipb_sitewide' not matching parent block after update"

Merge "Add option for confirmation prompts on rollbacks"

Merge "Fix grammar of "config-db-host-help""

Merge "Add BeforeResetNotificationTimestamp hook"

Add BeforeResetNotificationTimestamp hook

Allows prevention of clearing of notification timestamp when a user
views a page in their watchlist.

Change-Id: Id0af5c89c648ac6dce80012a1566426d5552942f

Merge "Move interface IJobSpecification to own file"

objectcache: avoid occasional test flakiness due to microtime()

* Fix the timestamps to static hard-coded values
* Force the timestamps before various get/set tests so they do not
  use the microtime() value either.
* Remove the direct and duplicated (causing further risk) microtime()
  calls from testGetWithSeveralCheckKeys().

Bug: T207247
Change-Id: Id30a8127f11501dbe54e075b6e9d18490353f4a5

ApiQuerySiteinfoTest: Conversion of $wgContLang to service

Bug: T200246
Change-Id: Ibdf07adc7a4cf28f0f9f48f18985651a379bc10c

Merge "Display translatable images in wiki language on image pages"

Merge "specialpage: Fix login crash caused by unknown language via ?uselang"

Display translatable images in wiki language on image pages

We currently show SVGs in default languages unless overridden with
lang=... URL parameter (and we have UI for setting it). This change
makes it display thumbnails in wiki language, if translation is available.

Bug: T210814
Change-Id: Ieb0b5e9e27f45b71ef119bb3c1d3f2cd4d7100e5