Clean up spacing of doc comments

Align the doc stars and normalize start and end tokens

Change-Id: Ib0d92e128e7b882bb5b838bd00c74fc16ef14303

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

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

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"

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

Merge "Update gradient mixins from OOUI, add horizontal-gradient"

Merge "MessageCache: Restore 'loadedLanguages' tracking for load()"

Merge "Deprecate several public properties on the block classes"

Deprecate several public properties on the block classes

Public methods for checking and setting these properties already
exist where needed. Also update the remaining direct uses of these
properties in core.

Change-Id: Icdef025c9700e625aeb2a07975e69f1b1cc2466c

Update gradient mixins from OOUI, add horizontal-gradient

Change-Id: I31ab76ea938f29c3d39456f093268780a44c014e

Merge "Fix SlotDiffRenderer documentation"

Merge "rdbms: make DatabaseSqlite::open() also check DBO_DEFAULT"

Merge "externalstore: use CONN_TRX_AUTOCOMMIT instead of IDatabase::clearFlag()"

Merge "CopyTextLayout: Only select-all on focus"

MessageCache: Restore 'loadedLanguages' tracking for load()

This was removed in 97e86d934b3 in 2018 in favour of using
`$this->cache->has($code)`. This is a problem because there
are cases where only a narrow subset of that structure is
populated (by MessageCache->replace) without things like
$this->overridable (or anything else that MessageCache->load does)
having ocurred yet.

The assumption that keys are only added to $this->cache by
MessageCache->load (or after that method has been called) was
actually true at some point. But, this changed in 2017 when
commit c962b480568e optimised MessageCache->replace to not call
MessageCache->load.

Bug: T208897
Change-Id: Ie8bb4a4793675e5f1454e65c427f3100035c8b4d

Fix documentation of languageinfo name prop

A separate lilang parameter existed in some patch sets of change
Ic66991cd85, but was ultimately abandoned again in favor of respecting
the general uselang parameter. Apparently I forgot to update the
documentation, though.

Change-Id: I2722f7a513c963bb9fec0308c413ae02e00851c6

CopyTextLayout: Only select-all on focus

This allows users to partially select the text, which is useful
in some cases, but still makes it easy to copy either using a keyboard
shortcut or the ButtonWidget.

Bug: T110579
Change-Id: I5be1530bb2cca9f8251e2e1c87a85e1e79cfe47f

Merge "Remove TODO for unblockself"

Make CliInstaller control the processing logic of the error

Previously, if there was an error during CLI installation,
CliInstaller::showStatusMessage() exited the script directly. The exit
timing of the script should be given to the caller, not the callee.
So, I coding:
[1] Remove `exit()` from CliInstaller::showStatusMessage()
[2] Make the callee to return Status, the caller determine how to handle these Status
[3] Strictly check the key database type instead of just outputting message

Bug: T46511
Change-Id: I72ffd33fe5c592b9ea78f37bae5a9c081295c624

Better handle "no accounts to unlink" case in Special:UnlinkAccounts

When there are no accounts to unlink, say so rather than displaying a
button that results in an error when clicked.

Bug: T228717
Change-Id: I17f8aed213f114338c4b46e26ce369bc63e36a99

rdbms: make DatabaseSqlite::open() also check DBO_DEFAULT

Change-Id: Ib21078dc9d4aa8cba6c5cb6c25868af328c1cab6

Merge "objectcache: make "busyValue" stricter to avoid callback ambigiuity"