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

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

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

Merge "Adapt Recent Changes advanced filters for mobile usage"

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"

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"

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

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"

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"

Merge "import: Fix wrong/unspecific type hints in UploadSourceAdapter"

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

Change-Id: I8984623d3ff67d305597f05c596c2777c2c117e0

objectcache: make "busyValue" stricter to avoid callback ambigiuity

Change-Id: I01a1503ff5b37d65ef148fef79270505d8eb3146

Localisation updates from https://translatewiki.net.

Change-Id: I3289a8380de64e080ec1f19e3a443e27795f9d51

Remove TODO for unblockself

It is impossible to reach the unblockself code if someone is anonymous, so
there is no need to check that the block applies to that permission.

Bug: T228538
Change-Id: Ie7b1e0f37363cc9e75ee2c871c06cefdfb2eb1d1

Merge "Follow-up f7cddcf7c1: Move RELEASE-NOTES-1.34 entry to right place"

Merge "QueryPage: allow arbitrary sorting"

Follow-up f7cddcf7c1: Move RELEASE-NOTES-1.34 entry to right place

This isn't a deprecation, it's a breaking config change, which we
put in either the "changed configuration" or the "breaking changes"
sections. In this case I'm dumping it into the former as that's
more frequently read by sysadmins.

Change-Id: I9d63be2ec6392b66c50fb96ada510584da9ec16a

Add QueryPage::delete method

Add a method for delete the cached result for a page. This is
useful for backlog-type query pages which provide some interface
for fixing the problem in-place.

Change-Id: I36b750d71b6551bb5c662e82c24f029db01acd3b

Merge "resourceloader: Move 'mw.config' assignment from startup.js to mediawiki.js"

Merge "resourceloader: Remove internal 'wgResourceLoaderStorageEnabled' from mw.config"

Deprecate wrapper SVGMetadataExtractor::getMetadata

Use SVGReader->getMetadata() directly. Also rename the test,
because it covers the implementation and not the wrapper.

Change-Id: I61565c6aadc6d1c1e942b9bc4555ef4aeb09e5d8

QueryPage: allow arbitrary sorting

When the value field is not provided in QueryPage::getQueryInfo,
it is set to zero, essentially randomizing the result. Use a
sequential index instead.

Change-Id: I8b67268e3c59a51ed00bd832f80df184dd771c3b

Inject getLocalServerObjectCache() into the CachingSiteStore instance

This means SiteStore now uses APCu on PHP7 as well. Previously it only
used APC on HHVM, and used the main cache (e.g. Memcached) on PHP7.

Change-Id: I83ff1d2dd61c611c9976c6f9ca8026a0b6dd6662

Merge "ApiSandbox: Fix upload widget"

Merge "Don't try to store File objects to the upload session"

Merge "rdbms: remove various deprecated methods"

Merge "Fix exception when viewing special pages with relative related titles"

ApiSandbox: Fix upload widget

Attempting to send the request with an active upload widget was
producing a JavaScript error "TypeError: this.setIcon is not a
function". This was probably broken by Iaaa226ea.

Further, validation was not working correctly since getValue() was
returning undefined rather than null as documented. This was probably
broken by I8d736fbb.

Bug: T229123
Change-Id: I78a6f9f0cffcd229ec0ab80c2309a0ee2089c927

rdbms: remove various deprecated methods

Change-Id: I5ae923065a08078225b7df080cb92edca799ebaf

Merge "Code cleanup related to initSpeculativePageId()"

Merge "rdbms: make LoadBalancer::waitForAll() include servers with load in any group"

rdbms: normalize Database open() code and error handling

Mainly:
* Use oci_new_connect() for Oracle to avoid broken connection reuse
  similar to the PGSQL_CONNECT_FORCE_NEW flag in DatabasePostgres
* Set 'client_min_messages' unconditionally for PostgreSQL
* Factor out Database::getConnectExceptionAndLog() helper method
* Use the same style of query() calls in DatabaseOracle::open() as
  the other subclasses
* Make sure the Database driver handle field is null on failure
  instead of false for sanity

Also:
* Disallow changing of Database handle DBO_* flags after construction
  where it does not make sense to change them
* Do not mention DBO_* flags meant for non-config use in $wgDBservers
* Ignore DBO_PERSISTENT for SQLite if DBO_TRX is also set for sanity
* Remove $wgDBOracleDRCP variable to discourage careless automatic
  setting of DBO_PERSISTENT that breaks LoadBalancer assumptions

Change-Id: Iea948f7f872294ea8fc5d897fc10c9d29b7141d5

Merge "Convert LoadBalancer::getConnection() callers to LoadBalancer::getConnectionRef()"

Code cleanup related to initSpeculativePageId()

Change-Id: I5b97c6292a28df6633c573a05c89210b096db5a8

parser: add speculative page IDs to use with {{PAGEID}}

This works similarly to speculative rev IDs with {{REVISIONID}}.
Re-parses can be avoided if the page ID is correctly guessed.

Also make the {{PAGEID:X}} parser function set vary-page-id.

Bug: T226785
Change-Id: I0b19be45e6ddd6cde330bfcd09d243e4e5beda01

Merge "MediumSpecificBagOStuff: Make convertToRelative(0) return 0, not 1"

Don't try to store File objects to the upload session

File objects can contain closures which can't be serialized.

Instead, add makeWarningsSerializable(), which converts the warnings
to a serializable array. Make ApiUpload::transformWarnings() act on this
serializable array instead. For consistency, ApiUpload::getApiWarnings()
also needs to convert the result of checkWarnings() before transforming
it.

Bug: T228749
Change-Id: I8236aaf3683f93a03a5505803f4638e022cf6d85

Merge "API: Only take HTTP code from ApiUsageException"

Fix exception when viewing special pages with relative related titles

Bug: T227817
Change-Id: I17e4acae81792c6d13c706741ec2a953300ac004

MediumSpecificBagOStuff: Make convertToRelative(0) return 0, not 1

This matches the (explicitly documented) behavior of convertToExpiry(),
and unbreaks storing things in Redis with an indefinite TTL. Without
this change, any attempt to store something in Redis indefinitely
resulted in it being stored with a TTL of 1 second instead.

Follow-up to I1bebb60307.

Bug: T228744
Change-Id: Ia3f71fb787196b956e974beb9f20d72e58ef3eb4

Merge "resourceloader: Improve ResourceLoaderWikiModule test coverage"

Merge "objectcache: let BagOStuff::getWithSetCallback() callbacks modify the TTL"

Merge "resourceloader: Remove incomprehensible `@todo` in getTestModuleNames()"

Merge "phpunit: Avoid get_class() in MediaWikiCoversValidator"

resourceloader: Remove incomprehensible `@todo` in getTestModuleNames()

I added this 7.5 years ago with r111378 (5bf04171dc3), but I can't
recall what this would have been for. Perhaps we wanted to expose
the list of modules (or test frameworks) via ApiQuerySiteInfo, but
I don't know why, or what purpose it would serve.

Change-Id: I4005979252533a752178e6f1ac9900f32132c27e

phpunit: Avoid get_class() in MediaWikiCoversValidator

I notice that in CI, the inherited "testValidCovers" often shows
multiple times for test classes as being slow (50ms+).

E.g. from a quibble/php72 job:

> …
> 85ms to run ResourceLoaderWikiModuleTest:testValidCovers
> …

Bug: T225730
Change-Id: I524df134c52173c49ebb2b263a90bb5732255865

resourceloader: Improve ResourceLoaderWikiModule test coverage

* Remove redundant getContent() cases that were all testing the same.
  The redirect logic should indeed be tested, but exists in getContentObj(),
  not getContent(). This test was also mocking getContentObj() thus not
  actually testing what the case claims to test.
  Fortunately, the right test already exists (testGetContentForRedirects),
  so this is redundant.

* Add actual coverage of successful outcomes for getContent (previously
  they were all error/null cases), with JS content, and with CSS content.

* Fix broken test case for "Bad content model". This was not working because
  it mocked out getContentObj, thus it wasn't actually testing "bad content model",
  but rather pointlessly duplicated the previous test case.
  Fix it by actually making it use a WikitextContent object, which makes it
  test the branch that handles incompatible content models.

Change-Id: I59af5318e536c730755352e9be8f995df1f56a86

Merge "Enable RecentChanges modules on mobile"

Enable RecentChanges modules on mobile

Change-Id: I9ec4b70390700cd697faeffd8f70c67e9c542e82

Merge "Improve type hints in export related classes"

Merge "Add permission check for user is permitted to view the log type"

Merge "Type hint against LinkTarget in WatchedItemStore"

Localisation updates from https://translatewiki.net.

Change-Id: I32ed9eb31204d18421f9d986d049feca65fbb62e

Add permission check for user is permitted to view the log type

Note: formatter patch only

Note: cherry-picked I064f563cb here as well

Bug: T222038
Change-Id: I1c4e57a513e3a0e616b862a5b9d684f463ad9981

Merge "resourceloader: Also clear store in debug mode to simplify mw.loader.store.init()"

resourceloader: Move 'mw.config' assignment from startup.js to mediawiki.js

Before this change, the startup module responds as follows:

   ```lang=js
   ... contents of startup/mediawiki.js:
   ...... mw.config = null;
   ...
   ... contents of startup/startup.js:
   ...... mw.config = new mw.Map( bool );
   ```

After this change:

   ```lang=js
   ... contents of startup/mediawiki.js:
   ...... mw.config = new mw.Map( bool );
   ...
   ... contents of startup/startup.js:
   ......
   ```

Change-Id: I97fea20f17c4865aa4740482f3054532038531f0

Merge "Update documentation for $wgAutopromote"

resourceloader: Remove internal 'wgResourceLoaderStorageEnabled' from mw.config

Only needed internally by the startup module, which can export it
through private means instead.

Follows 0a8e37f042c2c and 30ddfc8a7721.

Change-Id: I5e54bd51d9b47146e1ef9a64b14c1a00697bd6a9

resourceloader: Also clear store in debug mode to simplify mw.loader.store.init()

This makes it consistent with the other two circumstances in which
mw.loader.store is disabled.

Also, it makes debug=true a more easily accessible means for users
to clear module storage, which can be useful at times (without requiring
console access).

Change-Id: I9c66815a2f6c60feef3e070cfe57c382a69cc752

Make wfGetDB() return a MaintainableDBConnRef instance (take 2)

This enforces the DB_* role checks of DBConnRef in more places.

This is a re-submission of 335066505a3ea, which was reverted due to T228928.

Change-Id: I556627dc6486e6f6539d1d2dd5aa6f009eff341e

Add SearchResultSetTrait

trait meant to hold methods that rarely overridden or that are
non trivial to implement.

Bug: T228626
Change-Id: I4fba27b22860109d648e830a435d4a036d26ad06

Merge "Replace wfGetDB in ParserTestRunner"

Add BaseSearchResultSet

Meant to hold the B/C behaviors and be the base class implemented
by extensions.

Bug: T228626
Change-Id: I072a9bd0f94bd785d1ff51f0e6ee13790e37b417

Merge "MimeAnalyzer: fix ZIP parsing failure"

Merge "parser: inject a Logger into Parser instead of using wfDebug()"

MimeAnalyzer: fix ZIP parsing failure

unpack() actually returns an array with indexes starting from 1, not
zero, so unpack(...)[0] gives a notice and always returns null. It is
lucky that ZIPs normally have zero-length comments, so this would have
had little impact on file type detection aside from log spam.

Also, add a check to make sure the unpack() will not read beyond
the end of the file. Without this, unpack() could generate a warning.

The bug was introduced by me in f12db3804882272794b.

Add tests. The test files were generated by appending an EOCDR signature
and some extra bytes to 1bit-png.png.

Bug: T223728
Change-Id: I6fab63102d1d8eea92cdcce5ab6d1eb747a0a890

Merge "Do not block autopromote on partial blocks"