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"

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

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

Merge "Replace wfGetDB in ParserTestRunner"

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"

Update documentation for $wgAutopromote

Follow-up to 1ad840da68

Change-Id: I7c2dedf3824154c0c844f73a98e6824fe8ff141a

Merge "Remove deprecated handling of array keys for $wgProxyList"

Replace wfGetDB in ParserTestRunner

After I0d7dacee3, wfGetDB returns a connection reference instead of
a real connection, which means it can't be passed to CloneDatabase
anymore, and testcase / test runner classes should avoid it.

Bug: T228928
Change-Id: I42be4214e33a4d4f8d53fafc925c8d708b4c0106

Remove deprecated handling of array keys for $wgProxyList

Change-Id: Ic9cc2a5585180ab57fd361342cbac8210b094a5c

Localisation updates from https://translatewiki.net.

Change-Id: I70c061d14d3768c28234257004f35299c80a3201

Merge "Revert "Make wfGetDB() return a MaintainableDBConnRef instance""

Revert "Make wfGetDB() return a MaintainableDBConnRef instance"

This reverts commit 335066505a3ea6121f3c5b5eae43b0c37ca6e2eb.

The standalone parser test runner, as used by Parsoid Jenkins builds,
is (indirectly) using wfGetDB in a way that isn't supported (changing
the selected database). This needs to be fixed, but we can give it
a few days.

Change-Id: I07ff422dd56d5700e570100747aa49b7764ec80f

Merge "Add null to Parser::getTitle return doc"

Make wfGetDB() return a MaintainableDBConnRef instance

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

Depends-on: I9328e709fe5d81099338a31deef24d34db22d784
Change-Id: I0d7dacee3ec4ef67dc0b0f6551ad046c74dc47dc

Merge "installer: Stop offering MyISAM option for new installs and fix bogus method calls"

.travis.yml: Don't allow failures for PHP 7.3 any more

Bug: T204884
Change-Id: I797855405cced7471ab40a7cf80980cf68c840ce

Merge "Users with partial access to user rights should not be blocked"

Merge "make XmlDumpwriter more resilient to blob store corruption"

Merge "Drop newlines in LocalisationCache debug logs"

Drop newlines in LocalisationCache debug logs

316d205115c migrated LocalisationCache from wfDebugLog() to the Logger
interface which no more requires the messages to be terminated with a
new line.

Change-Id: Ibe2f750e7ab16cf91801988cb726adb67f7cd9d2

Merge "Validate sort order in Special:Search"

make XmlDumpwriter more resilient to blob store corruption

Loading content can also throw InvalidArgumentException when
the cluster address is an unknown cluster.

Bug: T228720
Change-Id: I313f9a5a27b21a33e90639abae3f505640c30e23

Add null to Parser::getTitle return doc

Change-Id: I5086f636114cec83f0188d2309ea70aef27df13d

Fix and re-apply "RedirectSpecialPage: handle interwiki redirects"

This re-applies commit 41106688abbe6dfff61c5642924ced42af3f0d33
(thereby reverting commit 6c57748aeee6e4f2a197d64785102306fbd4a297)
and fixes it for local interwiki redirects by adding and using a
forcing parameter in Special:GoToInterwiki to treat local redirects
like external ones.

Bug: T227700
Change-Id: I4bc2ed998430fc2bac71baf850b8988fdb24c1ac

Merge "Drop mediawiki.dic, useless and poorly updated"

Merge "Don't double wrap rollback links"

Merge "Add Hebrew aliases for some special pages"

Merge "Drop wgPasswordSalt, deprecated since 1.24"

Don't double wrap rollback links

Bug: T228766
Change-Id: Iaca54e1ceefc73815fe3d9898ca490b74d11dac6

Merge "objectcache: Use variadic signature for makeKey()"

objectcache: Use variadic signature for makeKey()

This should help fix the following issues that various repos
are getting from Phan as of late:

> Call with 5 arg(s) to \BagOStuff::makeKey() which only takes 2 arg(s)
> defined at ../../includes/libs/objectcache/BagOStuff.php:456
> <source="PhanParamTooMany"/>

Bug: T228563
Depends-On: I5cfba063821101325a5a7359e6b8ad71a0fb1b2f
Depends-On: Ifa5b96735376f2fbe3680799f960616ba8d357ff
Change-Id: Ic9df7f3ad7f356c7cbdfe1edfbe35821b931dda6

Validate sort order in Special:Search

Providing an invalid sort order to Special:Search could trigger an
exception from the search engine when trying to apply it. Validate the
sort order, much like API classes do, and let the user know that the
sort they requested could not be applied.

We also have a unreported error for invalid profile requested, so
added that warning to the display while here.

Bug: T228171
Change-Id: I79079eea8c03a90b5b65f1dad11c99e514de00e1

Merge "Make XmlDumpwriter resilient to blob store corruption."

Merge "Avoid the use of silence operator (@) and use AtEase methods"

Merge "filerepo: make ForeignApiRepo::httpGetCached() use "pcGroup""

Merge "filebackend: avoid use of LoadBalancer::newMainLB() in LockManagerGroup"

Merge "Replace wfWiki() usage with WikiMap in some additional places"

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

Also simplify the code by removing the is_callable() check and relying on
regular PHP errors instead of an exception for bad callbacks.

Change-Id: I084b0132c5fb05f1941a6d6839cfa74e2cf677f0

API: Only take HTTP code from ApiUsageException

Codes set on other Exception types are unlikely to be intended as HTTP
codes.

Bug: T228758
Change-Id: Ia6a53cb621f87ff97d5f16215a1b09ae11ca8f53

Add release note for custom OOUI themes using OOUIThemePaths (T100896)

Change-Id: I98f370310d93fc178e0baf55798808fa4ece2f92

Make XmlDumpwriter resilient to blob store corruption.

In the WMF databases, we have several revisions for which we cannot
load the content. They typically (but not necessarily) have
content_address = "tt:0" and content_sha1 = "" and rev_sha1 = ""
and content_size = 0 and rev_len = 0.

This patch makes sure we can still generate dumps in the presence of
such revisions.

Bug: T228720
Change-Id: Iaadad44eb5b5fe5a4f2e60da406ffc11f39c735b