Merge "Fix localization of time in Turkish"

Localisation updates from https://translatewiki.net.

Change-Id: I70882c277fb173e316e0ed90712fbe93628ab4d6

DI for CommentStore in WatchedItemQueryService

Change-Id: I93d9ed5f66297da7009d5b99aa9ed48e1b10582f

Merge "Mock CommentStore in some RevisionTest tests"

Merge "Pass $key into CommentStore methods and use MediawikiServices"

Merge "Add @covers for RemexStripTagHandler"

Mock CommentStore in some RevisionTest tests

Change-Id: I16b00a5514547d93a2308393098e3b363505374e

Merge "[MCR] RevisionStore::getTitle final logged fallback to master"

Add @covers for RemexStripTagHandler

This internal class is only used by Sanitizer::stripAllTags().

Change-Id: Ib913ee14524539216305da7e3183c07ab7d72cb5

Merge "Use HTTPS instead of HTTP for interwiki"

Merge "installer messages: Move to https: links when available"

Merge "Follow-up ed76b6736: Also remove the now-unused $mExtStyles"

installer messages: Move to https: links when available

Change links used in the installer from http:// to https://
when the linked content is available at the same hostname
using HTTPS.

Change-Id: Ib5afcf5ba4aa6da360c926794e6a021e85008d6c

Localisation updates from https://translatewiki.net.

Change-Id: Ia4e5683f7d850983a5fd76097f2cd3c82daca667

Remove superfluous spaces and semicolons in comments

Change-Id: Ib4b452f1843ec250c8c1fcc2a738d80726b6135d

Follow-up ed76b6736: Also remove the now-unused $mExtStyles

Change-Id: I7622ba15b03df97213acd82e4a3fa78267a30d55

Merge "jquery.textSelection: Fix typo"

Merge "jquery.localize: Fix rendering of generated documentation"

jquery.textSelection: Fix typo

Probably nothing is using this event.

Change-Id: I5e4a8a741b46868f8a1d55257a980aa0853fbb6c

Merge "Force CommentMigration MIGRATION_OLD in 2 RevisionTest tests"

Pass $key into CommentStore methods and use MediawikiServices

This allows CommentStore to be added to MediaWikiServices
without the need of an aditional Factory.

This change includes a compatability layer to allow the behaviour
from 1.30 to continue to be used while deprecated.

CommentStore::newKey has been deprecated.
Keys are now passed into the public methods of CommentStore
where needed.
The following CommentStore methods have had their signatures changed
to introduced a $key parameter, but when used in conjunction with
CommentStore::newKey behaviour will remain unchanged:
  * CommentStore::getFields
  * CommentStore::getJoin
  * CommentStore::getComment
  * CommentStore::getCommentLegacy
  * CommentStore::insert
  * CommentStore::insertWithTemplate

Change-Id: I3abb62a5cfb0dcd456da9f4eb35583476ae41cfb

Use HTTPS instead of HTTP for interwiki

Switch from HTTP to HTTPS where the HTTP URL redirects to the HTTPS URL
or where the HTTP URL and the HTTPS URL deliveres the same content.

Change-Id: I51947ae93cc9e6a30bb31eee7ff13561fc94ece4

jquery.localize: Fix rendering of generated documentation

JSDuck expects an empty line before and after code samples.

Change-Id: I45cf3a8e7c88476e61a681cfc44fc497e8f1c67e

Merge "Preload module 'mediawiki.notification'"

Merge "Fixing typos for ucfirst magic word translation into Ukrainian: ПЕША_ДІТЕРА → ПЕРША_ЛІТЕРА."

convertExtensionToRegistration: Set requires key in extension.json

The highest manifest version is not supported from the start of
extension.json
For extensions converted to this highest version the mininum core
version must be specified to avoid load problems in the extension.

Using 1.29.0 for manifest version 2 due to T149757 /
Id1071fc0647892438e5cd0e3ee621fbdaaa64014

Change-Id: Iea5ba589c70958db7500cf3587b5ebd738532026

Fix back-compat method Article::updateRedirectOn

No reason to set $lastRevIsRedirect to null before pass it as argument

Change-Id: I5e3c54d828a381c682725a43e322e660186b38cb

Merge "Use wikimedia/object-factory 1.0.0"

Localisation updates from https://translatewiki.net.

Change-Id: Ie15458314b698c74878d2582acfde34b8e066b68

Use wikimedia/object-factory 1.0.0

Deprecate the unnamespaced version and move it to includes/compat.

Bug: T147167
Depends-On: I39c805bfb98b32f32f3d0dc1eee9e823afe1c21a
Change-Id: I3780c7adf51683f3f7adb35a88f9a25a0a2e2530

Fixing typos for ucfirst magic word translation into Ukrainian: ПЕША_ДІТЕРА → ПЕРША_ЛІТЕРА.

Change-Id: I921945fd6a4121465844deece2a964e7d79f8117

Merge "rdbms: turn LoadBalancer waitTimeout default into a class constant"

Localisation updates from https://translatewiki.net.

Change-Id: Ib0da9c11a473475633c26e418a3fb3a1eb901653

Merge "mediawiki.api.edit: Remove dependency on 'mediawiki.Title'"

Merge "EmailNotification: Hard deprecate updateWatchlistTimestamp()"

rdbms: clean up and improve LoadBalancer logging calls

Change-Id: I8a17ad36ccc770fbd101c4b280e8ad1b32837125

EmailNotification: Hard deprecate updateWatchlistTimestamp()

Soft-deprecated in 1.27.

Depends-On: Ie7073f2048ba2b79a8b36ad913453008ec3555ce
Change-Id: If1dd2f8211c808ab0e284d7a3fb0b24610a0b32d

rdbms: turn LoadBalancer waitTimeout default into a class constant

Change-Id: Icd1e1aba1c45cb79a8249a9e16dce16f1c8183d2

Merge "Add Siberian Tatar (sty) language"

Merge "skins: Deprecate MediaWikiI18N::translate()"

Add Siberian Tatar (sty) language

Bug: T186359
Change-Id: Iecfdda0bcfbcc491565b6de36046791a3e5c540a

Merge "JavaScriptMinifier: Fix "Uninitialized offset" in regexp char class parsing"

skins: Deprecate MediaWikiI18N::translate()

Remove all usage of $tpl->translator->translate() with wfMessage()->text(),
which does the same thing, given the deprecated $tpl->translator->set()
and $tpl->setTranslator() methods have no callers.

These new wfMessage() callers should probably be changed to context calls
via BaseTemplate::msg() or Skin::msg() at some point, but I'm reserving that
for a separate commit given wfMessage() is what MediaWikiI18N::translate()
currently does - to reduce risk of unrelated behaviour changes.

While `$tpl->setTranslator` and `MediaWikiI18N::set` ($tpl->translator->set)
have no known callers in Wikimedia Git, the `$tpl->translator->translate`
method does have a handful of calls (although less than three, and not in
bundled or WMF-deployed repos). As such, proceeding with hard-deprecation.

Bug: T186090
Change-Id: I93d503de5515298288852ec4c150959fd688786b

Merge "skins: Deprecate QuickTemplate::setTranslator and MediaWikiI18N::set"

Merge "mediawiki.api.edit: Fix errors"

Merge "Do not concat exception class name in Timestamp compat"

mediawiki.api.edit: Fix errors

* Use title = String( title ) to support title with type mw.Title also on the
  POST request.
* Use titles: [ title ] instead of titles: title to ensure that a '|' in the
  title is not interpreted as multi value separator.
* Reject on invalid === true to prevent processing invalid titles.

Change-Id: Ia6a3d9b4b658a89f8969f0d64bf06a53638eb553

Merge "Remove Skin::showIPinHeader(), deprecated since 1.27"

Merge "OutputPage: Remove methods deprecated in 1.27 or before"

Localisation updates from https://translatewiki.net.

Change-Id: Id6a360383034c78b0723dc7896640d365add4ab5

skins: Deprecate QuickTemplate::setTranslator and MediaWikiI18N::set

These methods have no callers anywhere in Wikimedia Git.
Deprecate it now, remove in MediaWiki 1.32.

This is part of removing the legacy MediaWikiI18N class.

Bug: T186090
Change-Id: Ie2ca6a4b4f4481dc5cd81e7ed5543a05ee611f42

Remove Skin::showIPinHeader(), deprecated since 1.27

Change-Id: I46e5f84512413c9c4fbdfa43abf8908d6a6e11be

OutputPage: Remove methods deprecated in 1.27 or before

Eighteen uses of readOnlyPage and three of addExtensionStyle; all
have fixes awaiting merge. Marking as dependent on the only one
in a Wikimedia-deployed extension.

Depends-On: Ib3ea649bc68dd0edc025133721b46996d1e8901f
Change-Id: I636ebc7dead55f1ae654a13d3b6ebb023ecfe8e1

Merge "SpecialPage: Hard deprecate getTitle, deprecated since 1.23"

Merge "Add tablesUsed to RevisionStoreDbTest"

Add tablesUsed to RevisionStoreDbTest

Bug: T183777
Change-Id: Ie8bc2a9aca341a44c22828c9e9c3a7f89c7f87cd

Merge "SiteStats row initialization cleanups"

Merge "Remove wrapclass from parser cache key"

Do not concat exception class name in Timestamp compat

It seems this is not necessary and confused my IDE in thinking
'Exception' is an alias for TimestampException.

Given that we use a namespaced reference to the class,
ClassCollector still won't see it. Updated test to verify that.

Change-Id: I7c9258c5739a64959442af1296e2b93395260894

Preload module 'mediawiki.notification'

Preload the module 'mediawiki.notification' in parallel to the API
request because mw.notify is called on API response.

Change-Id: Ief8de75bc9392315a867fe1c4cf11ce4d0e9bfb8

Merge "mediawiki.api.category: Avoid exceptions"

Merge "Throw exception if php-curl is missing"

Merge "Print: pre-wrap <pre> elements"

Throw exception if php-curl is missing

Currently MediaWiki just dies silently, if the php
curl extension is missing.

Throw an exception instead that redirects the admin to the appropriate help page.

Change-Id: I25edfcb34ee5451a742cac6ae099019a16f6c417

mediawiki.api.edit: Remove dependency on 'mediawiki.Title'

The module 'mediawiki.api.edit' supports parameter with type mw.Title
but it does not need an explicit dependency on the module
'mediawiki.Title'.

Change-Id: I75b08ac0c906c1cdea7275f6bb209dd7655da5ef

mediawiki.api.category: Avoid exceptions

This change avoids some exceptions in processing the API response
* Ensure that a "|" in the title is not interpreted as multi value
  separator.
* The key query is missing on empty title.
* The key query.pages is missing on title="#" or title="mw:".

Change-Id: Idc82412337d0733cc58a78b1e202f5fdfd8f31cd

Print: pre-wrap <pre> elements

This seems to have been overlooked when T103780 was fixed during 1.26

Bug: T185129
Change-Id: I6f0068d66a5641c4f5092d5a2d0356c061fd2763

SiteStats row initialization cleanups

* Remove unreachable code (field cannot be both null and -1)
* Avoid PHP warnings during test runs in miser mode due to
  the $row field being "false".
* Init the site_stats row in unit tests to unbreak miser mode.

Change-Id: Ieb45cb3068b71e01aa12c674e0bfcf13a7f2493c

Merge "rdbms: set the schema in the LBFactory local domain"

Merge "Shell: Set pipes to non-blocking"

Add mobile and desktop thresholds

This will allow us to centralise these values similar to tablet
device width and allow us to remove a Minerva/MobileFrontend
dependency

Going forward we will encourage explicitly tracked imports
deviceWidthTablet is retained for backwards compatibility
but on long term it should be imported directly from mediawiki.ui's
variables

Bug: T171365
Change-Id: Ib9f843147db4473ce5590741e0fb490384b0007e

Merge "Make interim WAN cache key deactivation logic broader"

Merge "rdbms: Bump TransactionProfiler log entries to WARNING"

rdbms: Bump TransactionProfiler log entries to WARNING

Change-Id: I4816cfb8c8cd3b8e3dd2fe10c028c4654cc82854

Merge "deferred: Make MWCallableUpdate support a list of DB handles"

Merge "objectcache: use region prefixes for mcrouter-backed WAN cache"

Merge "Warn if stateful ParserOutput transforms are used"

Merge "benchmarks: Fix divide by zero in Benchmarker"

benchmarks: Fix divide by zero in Benchmarker

Follows-up 593442683dc471c0.

Bug: T186269
Change-Id: I011a8614e0acf489daf5bd9dcb932e920b995092

Merge "qunit: Test `before` and `after` hooks in QUnit testrunner"

Merge "qunit: Refactor and simplify testrunner to fix nested modules"

Merge "Added letter Kra (ĸ) to special characters toolbar"

Added letter Kra (ĸ) to special characters toolbar

Added 'Kra' to latin extended collection in the file /resources/src/mediawiki.language/specialcharacters.json

Bug: T185918
Change-Id: Ibc739297e5a1b8d109202477e1314cc3a0d6d5b4

Remove wrapclass from parser cache key

This will result in an exception from WikiPage::getParserOutput() if
anything was missed.

This also hard-deprecates ParserOptions::setWrapOutputClass( false )

Bug: T181846
Change-Id: Ica541e1f6b52f5eec6d28cff60ba64bf525258c7
Depends-On: Ie5d6c5ce34c05b8fe2353d3bb36b2a3a4166ec4b
Depends-On: Ibfaefde2f3811151ec712554cbc9cf2415ed017f
Depends-On: I55048bbae5d4d2d0c79c241c1784448b82db3bb4
Depends-On: I23a26ba0dfbe83007cd40e97d71a2139a5ecddc7
Depends-On: Ibc013a41f4a463f4014fbbce7ce27f8690161728
Depends-On: Ie936dff918dc0869503a924298b4580402038b52

Add 'unwrap' ParserOutput post-cache transform

And deprecate passing false for ParserOptions::setWrapOutputClass().

There are three cases for the Parser wrapper: the default
mw-parser-output, a custom wrapper, or no wrapper. As things currently
stand, we have to fragment the parser cache on each of these options,
which uses a nontrival amount of storage space (T167784).

Ideally we'd do all the wrapping as a post-cache transform, but
TemplateStyles needs to know the wrapper in use in order to properly
prefix its CSS rules (that's why we added the wrapper in the first
place). So, second best option is to make *un*wrapping be a post-cache
transform and make "custom wrapper" be uncacheable.

This patch does the first bit (unwrapping as a post-cache transform),
and a followup will do the second part once the deprecation process is
satisfied.

Bug: T181846
Change-Id: Iba16e78c41be992467101e7d83e9c3134765b101

Shell: Set pipes to non-blocking

The select(2) system call only guarantees a "sufficiently small write"
can be made without blocking. It doesn't define what that means.

And on Linux the read might block too in certain cases, although I don't
know if any of them can occur here.

Regardless, set all the pipes to non-blocking, which avoids the blocking
that's behind T184171.

And then, since a non-blocking read might validly return empty-string or
a non-blocking write might validly return 0, use feof() to check for EOF
and actually close the write pipe when it runs out of data.

Bug: T184171
Change-Id: I403235a328630112b6920905730f933777e2d453

Localisation updates from https://translatewiki.net.

Change-Id: I3eb13a00f4f315cc23b29602a8ef1206f26065dc

objectcache: use region prefixes for mcrouter-backed WAN cache

This allows for mcrouter to have proper cross-DC and intra-DC timeouts.

Change-Id: If48f740f435d266a2050839f34611c0c8f36b3a7

Merge "Add ability to proxy thumbnail requests to a service"

Merge "Hygiene: Use “OOUI” as unified name in build and code documentation"

Merge "Bump composer/spdx-licenses to 1.3.0"

Merge "Fix jump-to-nav element overlap on RCFilters"

Force CommentMigration MIGRATION_OLD in 2 RevisionTest tests

These tests fail when the comment store migration is not
set to old, this probably indicates that we need some more
tests for other cases, or that we don't want to test the comment
store functionality here and should mock comment store.

Change-Id: I06d2171498da6e07997569d4a74a0c5cfc913f33

Add missing PHPDoc block to DifferenceEngine::getParserOutput

Bug: T186163
Change-Id: Ifde6f8e458d90b1ec250dc4d587cd428717fe509

qunit: Test `before` and `after` hooks in QUnit testrunner

Change-Id: Ib5b5d29e860d37dbec675e5d24c31b57c5e64102

Add ability to proxy thumbnail requests to a service

Bug: T169144
Change-Id: I4af09a8b75e7158d6ff15f97e8f067b66ac33d5c

Fix invocation of MWHttpRequest::factory with one argument.

Change-Id: Ib96c2888a6fd91ca80c162a80d283f650d6fa4b9

Bump composer/spdx-licenses to 1.3.0

https://github.com/composer/spdx-licenses/compare/1.2.0...1.3.0

Change-Id: I41be6cecc1b322796ed448f664eb58642d7511ff

Hygiene: Use “OOUI” as unified name in build and code documentation

Bug: T182360
Change-Id: I981c574003fa505fe133be6da405e73330c4e9a1

deferred: Make MWCallableUpdate support a list of DB handles

Change-Id: I0371ecd20101c5e622497e5912676300b040865e