Merge "Fix phpdoc description of RevisionStore::insertRevisionOn"

Merge "DI for CommentStore in RevisionStore"

Merge "Remove unused code from LegacyHookPreAuthenticationProvider"

Fix some broken @var comments and add missing uses

Change-Id: I2091f4107b6c23d9c3b698999147ce3a2ec38b66

Fix phpdoc description of RevisionStore::insertRevisionOn

Change-Id: Iae0ac4cf98b1cc93eb5a9f8bb97e64b5ea1d9f0f

DI for CommentStore in RevisionStore

Change-Id: I527388514489e79c53b6016a8bd3119ee1750c83

Remove unused code from LegacyHookPreAuthenticationProvider

Change-Id: I699947fe02f649e24c957eb9790fcff91cb2c14a

Merge "rdbms: apply minimum sanity timeout for all cases of LoadBalancer::doWait()"

Merge "objectcache: remove unused constant and expand on a comment"

Merge "Revert legend when results are grouped"

objectcache: remove unused constant and expand on a comment

Change-Id: I3342497d657291d20325a3d87cc634f53d7776c4

Merge "Drop XCache support"

Merge "Fix for warning in ForeignAPIFile when no thumbnails"

Drop XCache support

It was never super popular anyway, APC was always the best option.
The project has no plans to move to PHP7, so it's quickly reaching
its end of life. Oh, and Fedora dropped it from their repos 2 years
ago.

Change-Id: Ia3257e86a6323d8943f04a5c05c72c0bd4c4b0a9

Localisation updates from https://translatewiki.net.

Change-Id: I7fa8feca5bdc1394bc24f3d93aaee1f7be2c2f9e

jquery.textSelection: Remove incorrect browser hack for old Opera

This code appears to be incorrect, at least on Opera 12.18 on Windows 10,
where I just tested it. It causes the selection to be misaligned, rather
than correct it.

For example, the following code:

    $textbox.textSelection( 'encapsulateSelection', { peri: 'foo\nfoo' } );

Causes the selection to extend one character past the inserted text with
the hack, and to be shown correctly without the hack.

It is theoretically possible that this hack was necessary for some other
operating system, but then it should have also been applied to 'insertText'.
Also, Opera 12 only has a usage share (barely) large enough to show up on
<https://analytics.wikimedia.org/dashboards/browsers/> on Windows (XP).

Change-Id: Ifee877c6202bcfc0fecb65220e34b4b39659dbeb

jquery.textSelection: Remove unnecessary checks

Some of the commands were wrapped in unnecessary checks (which were
also inconsistent between the different commands):

* Checking if this element is hidden. I'm guessing this was a
  workaround for some IE 6 bugs where doing things to the selection
  of hidden elements would break horribly. In any case, this should
  not be this code's responsibility.

* Checking for undefined this.selectionStart. I probably should have
  removed these in 1430ca85dcf9cb3f77a92bee6a915473e465f649. The only
  remaining case for them would be if .textSelection() is called on
  something that is not a text input element, which should not happen.

Change-Id: Ib09e64399921fe3861efd4877c6ecb2f900aef09

Merge "rdbms: remove obsolete getReplicaPos() code"

Fix for warning in ForeignAPIFile when no thumbnails

Remote audio files have no thumbnails, so this check in
ForeignAPIFile for thumbnails by iterating over the result
from FileBackend::getFileList() could throw a warning about
invalid input to foreach, as it may return null.

Change-Id: I07f37ed5a299e1a3c9a1e6bddcbc5582f99e5274

Merge "Add @covers for BlockLevelPass"

Merge "jquery.textSelection: Rewrite 'scrollToCaretPosition'"

Add support for Kumyk (kum)

Bug: T186647
Change-Id: I704e493d743d54071fc6f5341ea04e8a48ebaad7

Merge "Add support from Guianan Creole (gcr)"

Use new TagMultiselectWidget in ApiSandbox

It seems to be generally a 1:1 replacement.

Bug: T183299
Change-Id: I191a99bd373b0fb320770e9089278a16964d80d2

rdbms: remove obsolete getReplicaPos() code

Also fixed incorrect comment for getGtidCoordinates()

Change-Id: Ie2ce8a8bba230ceb6d068957e5ffeb360c3fe473

Update OOUI to v0.25.2

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.25.2

Depends-on: Ib00f16ef5e74732068822dc10d43314098b8fa59
Change-Id: Ie327a10622b64b7f969283497822c494c513585b

rdbms: apply minimum sanity timeout for all cases of LoadBalancer::doWait()

Change-Id: I61b4da6429b390e5aae65d1a30adc6df59687d8c

Merge "MWHttpRequest: Restore ability to pass null for $options"

jquery.textSelection: Rewrite 'scrollToCaretPosition'

The previous version tried to assume some platform-dependent
things about text rendering and guess the position based on
that. It did not work very well.

Change-Id: I9f4c465692cf128b7758de4ca75f977e8f8be83e

Merge "Hard deprecate UtfNormalUtil"

Hard deprecate UtfNormalUtil

Deprecated in 1.25, last vestiges have been removed.

Inline one last function that's not supposed to be public anyway.

Change-Id: If44c6a9cce7d2d95dc63e6f135b73b8c969f1c07

Merge "jquery.textSelection: Document methods and parameters"

Revert legend when results are grouped

Since Ifecaaff572, legend was using position: absolute when results are
grouped by page. Most users found such behavior annoying and unexpected,
so positioning is being reverted to previous state.

Only change besides reverting is preventing collapsed legend to push
away the surrounding results. Without that, we would have scenarios
where legend is expanded and collapsed, but results are pushed away as
if legend is still expanded.

Bug: T182156
Bug: T184165
Change-Id: I5adb7e1cb793ec30fba9d5d9674a7fa294d9a3a5

Merge "jquery.textSelection: Move copyright comments to the top of the file"

Merge "Fix localization of time in Turkish"

Add support from Guianan Creole (gcr)

Bug: T186635
Change-Id: I13334bb83562fc943fd69ef1f0fc3cd85aceffe1

Localisation updates from https://translatewiki.net.

Change-Id: I70882c277fb173e316e0ed90712fbe93628ab4d6

MWHttpRequest: Restore ability to pass null for $options

Prior to 3de744597e34, it was possible to pass `null` for $options.
Restore that by setting the default type to be null, and explicitly
document that null is a supported value in the doc block.

Also update the signature of MWHttpRequestTester to match.

Change-Id: I74d586afa5527e0a30ea99f3989f4dd12fa9fea1

jquery.textSelection: Document methods and parameters

Change-Id: I9e4112a75ae7abc8c479efc39a438c3b59933418

jquery.textSelection: Move copyright comments to the top of the file

Functions' doc comments should only document the function.

Change-Id: I56f21b49a35bf191a930d39af2f2020cdeb6ca30

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 BlockLevelPass

This class was split out of Parser, so it now needs separate covers tags.

Change-Id: I06c4a6a4fac3d6ff13924e3ca45ee134f7eeab20

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"