Introduce ResourceLoaderLessVarFileModule

This ResourceLoader module provides a way to ship
messages to CSS variables.

We will need this going forward to deal with flash of
unstyled content in various JavaScript based UIs that
are subject to i18n such as table sorting and
collapsible elements.

To avoid overhead of hitting the database to fetch and
transform localisation messages we make use of the MessageBlobStore
making use of `messages` definition already inside
ResourceLoaderFileModule. Given this resource is only intended
for render blocking styles without JavaScript this should be okay
(although if requested in JavaScript will also ship associated
messages)

Bug: T42812
Change-Id: I2bf12cdc848478889acbe9a7a970e46f8aefa287

Merge "resourceloader: Consider having dependencies as non-empty in WikiModule::isKnownEmpty"

resourceloader: Consider having dependencies as non-empty in WikiModule::isKnownEmpty

If a module itself is empty, it must consider any dependencies
it has before bailing out as empty.

Bug: T191596
Change-Id: I2b45b948a6f78060e53513d3b4b77f48d7bf4a6b

Merge "SiteConfiguration: Add test to confirm null as valid override"

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

Localisation updates from https://translatewiki.net.

Change-Id: I08e57f36ecea92e65b4af4f03b22b8790b7e3f8f

Merge "SiteConfiguration: Clean up test cases"

Merge "Create Expiry Widget with Date Time Selector"

SiteConfiguration: Add test to confirm null as valid override

E.g. not being misinterpreted as unset or something like that.

Change-Id: I253f17a828ae47886d1c4f2ca5097a546c38d119

SiteConfiguration: Clean up test cases

Make a clear distinction between names of configuration keys,
and "magic" keys that relate to SiteConfiguration class options.

Previously words like 'tag', 'wiki', and 'params' were over-used
in the test fixture making it very hard to see what is what.

Change all the configuration keys to camelcase so that 'WithParams'
and 'params' are different strings.

Change-Id: I279e57e30fb0a858ffd1f883f0601e96354ddead

Merge "Add "developer tools" preferences section"

Merge "Remove onhashchange fallback code"

mw.special.apisandbox: Offer to fill in token when not given

Change-Id: I53c414f0a80ce76907d3293971a47e87aa67fb06

Remove onhashchange fallback code

We no longer support any browsers that don't have
the hashchange event.

Change-Id: I5e6bfd1d5c634df21b6b3d06dcf007112cdf15fa

Merge "Fix width of OOUI TextInputWidget with .mw-editfont-... classes"

Add "developer tools" preferences section

Under editing. Will be used for the ParserMigration
and PerformanceInspector extensions

Bug: T129322
Change-Id: Id98d8d5245f207bd817854e43794800e89fe0f50

Merge "Improve docs for Enotif-related settings"

Merge "rdbms: remove deprecated getLaggedSlaveMode() method alias"

Improve docs for Enotif-related settings

Also fix documentation that was claiming wgEmergencyContact as
the default enotif sender, which is wrong. EmailNotification uses
$wgPasswordSender instead.

Change-Id: I5b060acac7bdfdbca9ac31e3473286225d1caa6f

Merge "Drop archive.ar_text and ar_flags"

Localisation updates from https://translatewiki.net.

Change-Id: Iac53ea2bc5fb098aa3dddae111e7a52532cad4f3

Merge "Make populateRevisionLength fix rows with ar_len = 0."

Merge "rdbms: make select() warn when FOR UPDATE is used with aggregation"

Merge "Avoid locking aggregated SELECT in Category::refresh"

rdbms: remove deprecated getLaggedSlaveMode() method alias

Change-Id: Iecdd39270d27ee1a2f9e1d714ec02c710fa241ef

Merge "Use CONN_TRX_AUTO in WikiPage speculative revision ID callback"

Make populateRevisionLength fix rows with ar_len = 0.

Due to T192189, some rows in the archive table may have ar_len set to 0
erronously. This can be detected by checking if they have ar_sha1 set to
"phoiac9h4m842xq45sp7s6u21eteeq1", which is the hash of the empty string.

This patch makes populateRevisionLength.php detect and fix such rows.

Bug: T192189
Change-Id: I9b6cd62f4b8c5bf93cd305b56af4352a527e19c9

RELEASE-NOTES: Fix typo in CSSMin release note

Follows-up c058bc4af3.

Bug: T189375
Change-Id: I21200dba6649d775d519249ae09cc21660dc864b

Use CONN_TRX_AUTO in WikiPage speculative revision ID callback

This increases the odds of seeing newer data and not guessing a
wrong revision ID due to getting raced out between stash and save.

Change-Id: Icf0f9b9c4d559f4ddf05a4ca8679e3c4dca5620a

Merge "wikibits: Remove redundant stubs for document.write"

Merge "selenium: Abstract test command behind 'npm run'"

Merge "build: Switch QUnit package from deprecated 'qunitjs' to 'qunit'"

Merge "Add release notes for CSSMin bug fix"

Merge "userrights: Add array typehints to SpecialUserrights methods"

Add release notes for CSSMin bug fix

Follows-up 5333d63cb.

Change-Id: Ib504500308630daa1051c21da8721b3058d8d793

rdbms: make sure cpPosIndex cookie is applied to LBFactory in time

Once getMain() was called in setSchemaAliases(), the ChronologyProtector
was initialized and the setRequestInfo() call in Setup.php had no effect.
Only the request values read in LBFactory::__construct() were used, which
reflect $_GET but not cookie values.

Use the $wgDBtype variable to avoid this and add an exception when that
sort of thing happens.

Further defer instantiation of ChronologyProtector so that methods like
ILBFactory::getMainLB() do not trigger construction.

Bug: T192611
Change-Id: I735d3ade5cd12a5d609f4dae19ac88fec4b18b51

build: Switch QUnit package from deprecated 'qunitjs' to 'qunit'

This should fix the deprecation warning from npm in CI logs.

Change-Id: I5d02dd8d6a86123d7935db51b8d0b5d88218869f

selenium: Abstract test command behind 'npm run'

This makes it possible to use in Quibble. Right now, Quibble
cannot use 'npm run selenium' yet because it sets up its own
chromedriver, which would conflict with selenium.sh.

But, in preparation for Id95f18927b5, we can at least abstract
the test command so that it can easily be changed without having
to update Quibble and without having to worry about it being the
same in all MediaWiki branches.

Bug: T179190
Change-Id: I622ec3bf36746502cae891cc6bec23982b21f876

Merge "jquery.tablesorter: References ignored in sortkey."

jquery.tablesorter: References ignored in sortkey.

Text in elements with class='reference' excluded for sorting.

Bug: T29745
Change-Id: If3969c00cbb7a74362fe85f4cad361da488526d5

Localisation updates from https://translatewiki.net.

Change-Id: Ic59e840c7e00449454f0d402f57e26003e169417

Fix typo: bytePointLimit -> byteLimit

Change-Id: Ifafc4bdd6663e1ccfa6be938e5fc72fd1442a9a5

Remove cssclass in the favor of 'type' => 'user'

Bug: T192694
Change-Id: If385c16c9099475d50dcb775484f710450360ed2

Merge "Remove cssclass in the favor of 'type' => 'user'"

Localisation updates from https://translatewiki.net.

Change-Id: If368d4b40a1bde11c66da9bf24be7ad60452e665

Merge "Add Gorontalo language file"

Add Gorontalo language file

Bug: T189127
Change-Id: Ib15fcc1b1b63674afdc0642e7139e11972898c95

Fix width of OOUI TextInputWidget with .mw-editfont-... classes

Bug: T192373
Change-Id: I6a6757874a3502b42cff0a3edb55115a305ad609

Merge "MessagesEs.php: add non-camelcase names to some special page names"

wikibits: Remove redundant stubs for document.write

The browser already warns for these, and our stubs were actually
making it somewhat worse by appending to a weird place in the page
instead of what the browser does (which is to either allow it
properly, or ignore).

Bug: T192623
Change-Id: I1924b252a8b986c4f1ad65f96149b8c06c1fad9b

Merge "HTMLForm: Remove usage of HTMLForm::setSubmitProgressive"

HTMLForm: Remove usage of HTMLForm::setSubmitProgressive

`[ 'primary', 'progressive' ]` is already set in `$mSubmitFlags`.

Bug: T191586
Change-Id: Id20a3bc81a47c6c2979a4d2881cecc760956394f

Merge "Hard-deprecate wfUsePHP()"

Remove cssclass in the favor of 'type' => 'user'

Bug: T192694
Change-Id: I3be9fbcd990346df7c918d5ddf71f9e0a73a52df

Merge "Hard-deprecate wfCountDown()"

Merge "resourceloader: Improve titleInfo docs and simplify title key"

MessagesEs.php: add non-camelcase names to some special page names

Change-Id: Ia36b48ff40a390434864f327dcb4bfa49cd44cd8

Localisation updates from https://translatewiki.net.

Change-Id: I1b5f321f6908a18a961762bbbd7d6e515ad4067e

Merge "OutputPage: Factor out CdnCacheEpoch logic and cover with tests"

Merge "CSSMin: Fix breaking of quoted urls with outer spacing"

Merge "ParserTestRunner: Fix some documentation types"

ParserTestRunner: Fix some documentation types

Change-Id: I7d0375815dc6ac91cc3f39ea7910cdf1dff49666

userrights: Add array typehints to SpecialUserrights methods

Follows-up e3ce482e1b4da37.

* Add them to parameters of doSaveUserGroups(), which were the
  entrypoint to the error fixed by e3ce482e1b4da37.

* Add them to parameters of addLogEntry(), which is where the
  actual count() error happened.

Bug: T182377
Change-Id: I647a1ebf6dc0e80952efe2a3c1192fd8ac5dc1b7

Merge "Special:ProtectedTitles: Use HTMLForm"

Remove EDIT_TOKEN_SUFFIX constant

It isn't used anymore anywhere, according to a code search on github.

Depends-On: I56291c6a5b6c2bb359c6633748a68f4964d71923
Change-Id: I4b3f7b7bdf9bd0dbc8225bbcfb2da23016ae6bb3

OutputPage: Factor out CdnCacheEpoch logic and cover with tests

Bug: T178629
Change-Id: Ife7dd79677c2b5353317e06ac7ed521edd6193cc

Merge "Special:ProtectedPages: Convert to OOUI"

Special:ProtectedTitles: Use HTMLForm

Bug: T117723
Change-Id: I874a8614920166d68671f757497fb27554a397ac

rdbms: make select() warn when FOR UPDATE is used with aggregation

Using FOR UPDATE or LOCK IN SHARE MODE with aggregation leads to
query errors with PostgreSQL.

Bug: T160910
Change-Id: Iaed964e7e59468365cbc62cb4bfd3ad44b898452

Create Expiry Widget with Date Time Selector

Special:Block needs a date time selector for easier selection of expiry. To
accommodate this cleanly, a new Expiry Widget is created that handles this
logic.

Bug: T132220
Change-Id: I2853a2ca0ae6ccead3978f4bb50a77c2baa3a150

resourceloader: Disable ChronologyProtector on load.php (again)

This was already disabled in 2016 with 98c6743413118f by calling
setRequestInfo() to override the default 'ChronologyProtection' value
from Setup.php.

This stopped working since 4ccb228bde9294d96d which made
getDBLoadBalancerFactory (in ServiceWiring) obtains a LoadBalancer
from MWLBFactory::setSchemaAliases, and requestInfo is not
re-evaluated after that.

Bug: T192611
Change-Id: I68748e8f15bcd7bfa8d7921964788e2798682e05

Merge "Don't pass USE INDEX to a $dbType parameter"

Merge "Avoid triggering Category::refreshCounts() on HTTP GET requests"

Merge "Make getScopedLockAndFlush() in Category::refreshCounts non-blocking"

Merge "rdbms: correct atomic section examples in comments"

Merge "rdbms: make cancelAtomic() handle callbacks and work with DBO_TRX"

Merge "Add $wgJpegQuality to control jpg thumb quality"

Localisation updates from https://translatewiki.net.

Change-Id: I28320e040be835414599ae6cc55d29312d27a88a

rdbms: make cancelAtomic() handle callbacks and work with DBO_TRX

Make transaction callbacks aware of cancelled sections. If the
statements of a section are reverted via cancelAtomic(), then the
dependant callbacks are now cancelled as well. Any callbacks for
onTransactionResolution(), which does not depend on COMMIT, will
see the triggering event as a ROLLBACK, since the unit of work it
was part of was rolled back.

Also fix the handling of topmost atomic sections with DBO_TRX.
These still need their own savepoint to make cancelAtomic() work.

Follow-up to 52aeaa7a5.

Change-Id: If4d455c98155283797678cfb9df31d5317dd91a2

Don't pass USE INDEX to a $dbType parameter

Bug: T192584
Change-Id: Ia2b761bad570532696419b4f23f84bc85ff2c0e8

Merge "resourceloader: Support loading group=user modules with addModules()"

Merge "Deprecate $wgUseAjax in 1.31"

Make getScopedLockAndFlush() in Category::refreshCounts non-blocking

Change-Id: I675ec38e064f57a4e960b3934c472833a945b695

Merge "Fix handling of ar_length and ar_sha1 in RevisionArchiveRecord."

Merge "Fix and improve PHPDoc type hints in FileBackend and FileRepo"

Deprecate $wgUseAjax in 1.31

This setting was intended to control the legacy AJAX framework, but some
extensions mistakenly used it to control any AJAX functionality at all.

But keeping this around when we're deprecating things like $wgEnableAPI
makes no sense.

[Retro-back-ported after bf61efd87 had landed in 1.32.]

Change-Id: I7b767478f9fb44236e6514b0e83228d9fb8e856f

Merge "Remove language as string for DateFormatter::getInstance"

Merge "API: Fix PHP Warning for "count(): Parameter must be an array""

Remove language as string for DateFormatter::getInstance

Change-Id: I677ab561d67b63f0c86f65fadf9319e41444a22a

API: Fix PHP Warning for "count(): Parameter must be an array"

Found by the new ApiUserrights test introduced in 5ab1bee6bcf:
> includes/specials/SpecialUserrights.php:462
> includes/specials/SpecialUserrights.php:405
> includes/api/ApiUserrights.php:116

The $tags parameter of the UserrightsPage::doSaveUserGroups()
method must be an array and is not nullable. Fix the caller
to not pass null, but an empty array.

Bug: T182377
Change-Id: Ic9be6a0bddfac023de765c810c07a64d651c33b4

Merge "Introduce SqlBlobStore::makeAddressFromTextId"

Introduce SqlBlobStore::makeAddressFromTextId

This introduces a convenience method for constructing a blob address
from a text ID. It's the inverse of SqlBlobStore::getTextIdFromAddress

Change-Id: I31b3ee5e40185c754fb2c119eb5edc50b903f5dc

Add $wgJpegQuality to control jpg thumb quality

This new configuration variable defaults to 80, which is the current
quality in use when scaling jpeg thumbnails with ImageMagick. GD used a
quality of 95, which was reduced to 80 as a result of this change.

Bug: T192248
Change-Id: I94cad01cc876ef9083250f5d1989a73cd19617fb

Merge "Avoid warnings in ObjectCache::newWANCacheFromParams if "channels" is not set""

Fix and improve PHPDoc type hints in FileBackend and FileRepo

Change-Id: I311b4a6777946b451d70c6fdb2133dbbd73f159f

Avoid warnings in ObjectCache::newWANCacheFromParams if "channels" is not set"

Bug: T192473
Change-Id: I150eef62285e99fc1269a58fee3827e5fbb0537b

Merge "Special:PrefixIndex: Convert to HTMLForm"

Avoid triggering Category::refreshCounts() on HTTP GET requests

Trigger count refreshes more often during updates instead.

Change-Id: I0f4575e648109befb86a2c8f08b491a132fe56c3

Merge "Remove deprecated function wfShellExecDisabled()"

Merge "Remove deprecated function wfInitShellLocale()"