ResourceLoaderClientHtml: Simplify code

* Use nowdoc instead of heredoc syntax where variable expansion is not
  wanted.
* Combine code paths for debug mode and normal mode.

Change-Id: Ic70d82aaec6396a31783e5a60419409f08c4ef73

Merge "rcfilters mobile cleanup"

Merge "Code style cleanups to FileBackendGroup"

Merge "Make LinksUpdate no longer extend EnqueueableDataUpdate"

Make LinksUpdate no longer extend EnqueueableDataUpdate

LinksUpdate does not match RefreshLinksJob since the former is only a subset
of the later. Also, DeferredUpdates::doUpdates() only runs in "enqueue" mode
for cases in MediaWiki::restInPeace() if there is no post-send support.

In a future commit, the deferred callback in which LinksUpdate runs
currently, will be abstracted into its own deferred update, which
will then bring back EnqueueableDataUpdate for this update.

Bug: T206283
Change-Id: I0680be445e8b8e8d0dba85df135b84640f4fcb81

Merge "OutputPage: Remove 'html5shiv' module from RL registration"

Merge "Remove old comment and unused setting"

Merge "Avoid error in querypage API when there's no value field"

Merge "rdbms: update ILBFactory comments and simplify LoadBalancer "load" code a bit"

Merge "rdbms: In the query log, show the server name in the message"

Remove old comment and unused setting

Change-Id: I813c90197ba0fc76d493a47f83107ba4bb8dff2a

Merge "selenium: stop running extensions/skins specs"

rdbms: update ILBFactory comments and simplify LoadBalancer "load" code a bit

Change-Id: I391a2f873c623b3086703a9b078def40363d9520

rcfilters mobile cleanup

- ensure the advanced filters are hidden when clicking the "hide/show" button
- prevent horizontal scrolling of the "saved filters" and
  "other review tools" buttons on small screens.

Bug: T229360
Change-Id: I445df9124a6bb520dcf49b245f0a8392bf723c08

Localisation updates from https://translatewiki.net.

Change-Id: I8dfd153ca5be10f42e616ed11039f8e3fc8b7e46

Merge "changes: Avoid use of deprecated method Revision::userCanBitfield()"

changes: Avoid use of deprecated method Revision::userCanBitfield()

Revision::userCanBitfield() is deprecated and RevisionRecord should
be used instead.

Change-Id: I0d6ad6863be520d3264809be69532dfb2e3dab9a

OutputPage: Remove 'html5shiv' module from RL registration

Follows-up 66a011797d8e2fd, which changed the reference to this
JS file to be without the indirection of ResourceLoader.
It's been deployed well over the needed 7 days, so this can be
removed now.

Change-Id: I823c0b31c4478e5e34f4191d851b6a9c83a6019b

Merge "resourceloader: Remove unused 'getBaseModulesInternal' internal method"

Merge "Use `text` element for current content model on Special:ChangeContentModel"

Merge "Convert maintenance scripts to LoadBalancer::getMaintenanceConnectionRef()"

Convert maintenance scripts to LoadBalancer::getMaintenanceConnectionRef()

Change-Id: I8944a052f51a1941ad3b76a40fc9c46d1363c426

Use `text` element for current content model on Special:ChangeContentModel

Change-Id: I6881154fa5baf4d55634c691c45596f49e2cd4b0

Move WANObjectCache to wancache/ instead of WinCache

Follow up 6ca3de5fa0.

Change-Id: I7021dfe5c900f810ab3b6dc701a04e2c3e1c4c66

Merge "Show current content model on Special:ChangeContentModel"

Show current content model on Special:ChangeContentModel

Bug: T226914
Change-Id: I60c7075751cb180cb86cde0b9caea1a4bcf9b2a3

Merge "Move WAN cache classes under a new wancache/ directory"

resourceloader: Remove unused 'getBaseModulesInternal' internal method

I added this as wrapper for the private 'getBaseModules' method
in commit dec800968eb6 for "use by SpecialJavaScriptTest", but
I never did. And it doesn't need it so.. let's remove that.

Change-Id: Ia301bd1c1f3a15098b9e6cd2934dd8e80879dd50

Reorganize ResourceLoader fields

Change-Id: I9c9aece69b869165b8d9037336c8a2e7d7189c1e

resourceloader: reorganize ResourceLoaderModule fields

Also use AtEase for error suppression to avoid phpstorm warnings

Change-Id: Iaa8ca107f4e7c923e894416b40b4fadd2b3d162e

Move WAN cache classes under a new wancache/ directory

Change-Id: I6837761ebca7557e029e1f65beca738266e48efb

Avoid error in querypage API when there's no value field

QueryPage does not require a value field, and post I8b67268e omitting
it might even be useful, so the API should not error out when it's
not present.

Change-Id: I83b49d4451ba71bb2161459bddf17b9ac00088d2

Merge "Make Special:MovePage note about redirects dependent on content model"

Merge "Deprecate SearchEngine:getTextFromContent() and SearchEngine::textAlreadyUpdatedForIndex()"

Merge "Simplify SearchUpdate constructor and hard deprecate some param types"

Merge "resourceloader: Merge $fileCache conditional blocks"

Merge "Document SearchDatabase::doSearchTextInDB to return null"

Merge "Deprecate SearchEngine::userHighlightPrefs()"

Merge "Fix bogus field use in RevisionRenderer::getSpeculativePageId"

Code style cleanups to FileBackendGroup

Set domainId in register() instead of on every config() call.
Use array_merge() to make the override precedence clearer.

Change-Id: I8792281cd9f1a4018255f9a9b87126c909095e68

Fix bogus field use in RevisionRenderer::getSpeculativePageId

This field was renamed, causing a functional merge conflict

Follow-up 5099ee9f7273

Bug: T229589
Change-Id: I7a6bb68ff1fe320313276dc5a67c70de6715ccb6

Clean up $wgFileBackends/$wgLockManagers comments

Change-Id: Ib9692176f0e100f3b0c39544109ef2d8a8c8cd92

Merge "Add missing title info in RenderedRevision::outputVariesOnRevisionMetaData()"

Merge "Cleanup UserGroupMembership::insert() and make it more atomic"

Cleanup UserGroupMembership::insert() and make it more atomic

Bug: T229456
Change-Id: Ibf7900dd9273d3befb5c7f0c4ed46b65bd3b0ae4

Merge "API: Create a module bundle for 'apipretty' and 'apihelp'"

Merge "filebackend: avoid use of IDatabase::clearFlags() in DBLockManager"

Merge "resourceloader: Remove getters for private constants in mw.loader.store"

Merge "Suppress notice from ParserOutput::__sleep()"

Merge "resourceloader: Remove slow structure test for checking getVersionHash"

Deprecate SearchEngine:getTextFromContent() and SearchEngine::textAlreadyUpdatedForIndex()

It was just a wrapper to Content::getTextForSearchIndex(), simply use
this method rather than depending on (and sometimes constructing) a
SearchEngine.

Change-Id: I8541248ffdca303f0af3b959cf2f051dcb497925

resourceloader: Remove slow structure test for checking getVersionHash

This isn't needed because the startup module validates this already.

The vast majority of modules are FileModule instances which can't be invalid,
because a separate test asserts that class already. This test existed for
validating the format of version hashes returned by a theoretical Module
sub class in an extension that (badly) overrides the getVersionHash method.

As of writing, no extension overrides that method. And more importantly,
the startup module already validates this at run-time, and logs a warning.
This commit turns that into an exception, which would get logged in a way
that Jenkins will fail the build if encountered.

This structure test, which computed the response for all registered modules,
previously took 3-5 seconds in CI.

Bug: T225730
Change-Id: Id2e37434b0ccd95dd2279f04e2230e9c06b09ccb

Simplify SearchUpdate constructor and hard deprecate some param types

Change-Id: I5677041169402014f1afc1a9012460c760ca24b6

Deprecate SearchEngine::userHighlightPrefs()

was only returning two constants and forced a cyclic dep bewteen
SearchEngine and its SearchResult instance.
Simple make these values a constant in SearchHighlighter and use
them as default values for the highlight methods.

Change-Id: Ia78408d5266d0a305006027fe6265a2a1d68b0b9

Merge "resourceloader: Avoid escaping of UTF-8 characters in messages"

Add missing title info in RenderedRevision::outputVariesOnRevisionMetaData()

Follows 3a044ca51ce9b.

Bug: T226785
Change-Id: I03cf35ee2773cfeb887edec8bc447c834bab5a2a

resourceloader: Avoid escaping of UTF-8 characters in messages

It's not needed to escape UTF-8 characters in messages.

The characters '&', '<' and '>' are still escaped. For the output in
load.php this escaping is not necessary but messages can also embedded
into <script> tags and here are the characters '<' and '>' problematic.

Bug: T229301
Change-Id: If424a21df4a813ba6cb79b939f0857d96f162a17

mediawiki.Upload.BookletLayout: Add missing message

Bug: T203058
Change-Id: If1274da9b9dca5595201c298ffd35576ef7b7e52

resourceloader: Remove getters for private constants in mw.loader.store

Turn the private getter methods into private properties.
Shorter, simpler, faster.

Change-Id: I8c7a7c566983ba6ac78cd2feb5d0ec6debb3773c

Suppress notice from ParserOutput::__sleep()

Bug: T229366
Change-Id: I8f0a537f0b6b76aac0c52e691ec4653c51c49940

Merge "Make CliInstaller control the processing logic of the error"

Merge "context: Use getRawVal instead of getVal for 'uselang' and 'useskin'"

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

Merge "Fix documentation of languageinfo name prop"

Localisation updates from https://translatewiki.net.

Change-Id: Ie2a14a24bf788729b75ce23b7fec1b0bf5d9c02e

Merge "resourceloader: Remove deprecated ResourceLoaderContext::expandModuleNames"

Merge "resourceloader: Only register 'jquery.qunit' in test context"

Merge "resourceloader: Don't explicitly enqueue test libs on SpecialJavaScriptTest"

Merge "resourceloader: Hard-deprecate ResourceLoaderContext::getConfig"

Document SearchDatabase::doSearchTextInDB to return null

The function calling this abstract function allows to return null,
so it should be okay to return null here

The null for empty result in SearchOracle

Change-Id: I66a8fb3a4190bf5506f358a47f6f4833b1715c7f

Merge "preferences: Fix skinname msg check in generateSkinOptions"

Merge "Add QueryPage::delete method"

resourceloader: Hard-deprecate ResourceLoaderContext::getConfig

Already in 1.34 release notes from when it was soft-deprecated, earlier
this release cycle.

Depends-On: I949b91daabb0cf16519ef02e718ad97c827c4eec
Depends-On: Icf1ead8829268505709c71f46dede5cdfc4471a8
Change-Id: I471088b322848a779d0dd58651dc1bd0d49067f1

Merge "Allow SelectWithInput to be marked as required and handle that dynamically"

Merge "Fix SpecialBlock validation for ipb_allow_usertalk"

Revert rename of mSpeculativeRevId to speculativeRevIdUsed

And add a test which is confirmed to fail on HHVM prior to this change
with the error message "serialize(): "" returned as member variable from
__sleep() but does not exist".

Bug: T229366
Change-Id: I236bb4d64bc2e9f7756885e8c418399804eac5e1

preferences: Fix skinname msg check in generateSkinOptions

Fixes a bug introduced in b2b204eeafc.

$useSkin is not the appropriate variable to check here - we want
to check the skin set in preferences.

Bug: T223824
Change-Id: I666e30dd9ff396fd39c47b362d66670dfaa92ca9

Merge "context: Make the getSkin() fallback logic more explicit"

Merge "Fix recent changes and watchlist separator whitespace"

Allow SelectWithInput to be marked as required and handle that dynamically

Alters the SelectWithInput to allow a required config to be passed from a
parent widget. Also handles the required state dynamically. If the widget is
an OR widget, then only the select dropdown is required. The text input will
be required when the other option is selected. If the widget is an AND widget
then both the select dropdown and the text input will be required.

Bug: T220533
Change-Id: I8479743126756f2b1bd7bcd53b100a0134f34d07

Merge "Fix return type hints in media related classes"

Fix SpecialBlock validation for ipb_allow_usertalk

ipb_allow_usertalk should be saved as false only when:
    A block is sitewide
    A block is partial and there is a restriction on the User_talk namespace

Bug: T224468
Change-Id: Ic85368991b5905af1bab99b0cb5fcba0e993405e

Fix recent changes and watchlist separator whitespace

Added spaces around 'mw-changeslist-separator' spans in EnhancedChangesListGroup.mustache

Bug: T222903
Change-Id: Ie7e09a39a8f51943fa6c2c244d80435150296a5c

resourceloader: Only register 'jquery.qunit' in test context

Change-Id: I0ea4bb73d5b886ea903a19a12bdd464778af0002

resourceloader: Don't explicitly enqueue test libs on SpecialJavaScriptTest

The test-only modules registered by QUnitTestResources.php are currently
were previously caught by the array_keys() catch-all in registerTestModules()
which meant that modules like 'test.sinonjs' would be requested on
SpecialJavaScriptTest despite not doing anything by itself, nor executing
at the "right" time per se through this means.

In order for it to execute at the right time, the testrunner has to depend
on it (which it does, already). But, that also means it doesn't need to
be requested separately. Doing so could be confusing.

This is neccecary in order to move 'jquery.qunit' from Resources.php
to QUnitTestResources.php as otherwise, listing in QUnitTestResources.php,
would implicitly mean SpecialJavaScriptTest.php thinks it's a test suite
and load it. That is a problem, because when we run the tests headless from
the command-line with Karma, the environment already has a QUnit interface
defined, and should not be loaded a second time by MW.

Change-Id: I08b31cd1dee516cf0d26bafdb8cc7c1223633bad

filebackend: avoid use of IDatabase::clearFlags() in DBLockManager

Change-Id: I143533790363d8bb9b9038cf320b251131602d4a

Merge "Adapt Recent Changes advanced filters for mobile usage"

resourceloader: Remove deprecated ResourceLoaderContext::expandModuleNames

Change-Id: If9aa389cc64cb41e60c865d586d644d9f3bca9b5

Merge "Fix punctuation in HTMLForm documentation"

Fix punctuation in HTMLForm documentation

Change-Id: I22c4ed248fddc93afd486b9bee5a80bdbba1c685

pager: Add visibility methods declared in the Pager interface

In addition, also add visibility to the class(es) that implement
the method.

Change-Id: If650cfcafbe4fe8cd4d2e0629df3f170c599ea41

Adapt Recent Changes advanced filters for mobile usage

Changes the behaviour of the rcfilter search input by essentially
turning it into a button for mobile devices. Depending on the value
of `OO.ui.isMobile()` the input is set to readonly mode and given
a shorter message and different icon.

Setting the search input to readonly prevents onscreen keyboards from
being actived, but still opens the filter menu, so that mobile users
can still add/remove filters, just without the ability to search through
them.

Styles are also modified to make the search input appear as a button by
overriding the default readonly and placeholder styles.

Bug: T225499, T223230
Change-Id: Iaa67369542e658d3571d957a204daa7a53d1e520

Localisation updates from https://translatewiki.net.

Change-Id: I1b7b2936762e14eb43e9d777c3c3d198b66e4e6d

Merge "Add SearchResultSetTrait"

Merge "Add BaseSearchResultSet"

API: Create a module bundle for 'apipretty' and 'apihelp'

These are both fairly small modules used in related low-traffic
scenarios. The cost of registering both globally on all page views
as a top-level entry point does not seem worth the marginal gain
of loading only one of these two in the contexts where they are
needed.

Change-Id: I42458ebec3f8d6e14e52e44d62fb914fcbeda5db

context: Make the getSkin() fallback logic more explicit

Previously the first two post-hook checks where mutually exclusive
and the fallback was generic (not conditional) which means we
would also allow the first two conditions to succeed in a way that
leaves $this->skin as null.

Aside from that not being possible right now, it also duplicates a
null check in two places, which isn't ideal.

Codify that the first two blocks always result in $this->skin
being assigned non-null by using an 'else' branch instead of another
generic check.

Change-Id: I7986684bf1bb2daad3aaddfbcd2ca02e652f78c2

Merge "Add explicit visibility to SearchResult methods"

Merge "Add explicit visibility to ISearchResultSet methods"

context: Use getRawVal instead of getVal for 'uselang' and 'useskin'

Bug: T189966
Change-Id: I9db7b3f36f5457e80aa9b673bcb56deb83e47a18