Remove excessive indenting of parameters on newlines

Change-Id: I4e214b55b9936be2336a97192f9716b3ea7d6d61

Merge "Use commit-and-wait when processing more than updateRowsPerQuery"

Merge "OOUI: Backport I407b0d6fe7b81465054b640d4b5ac4bf352a9901"

Merge "Localisation updates from https://translatewiki.net."

Merge "RCFilters: Use `display: flex` for better responsiveness on narrow canvas"

Merge "Preferences class deprecation, step 2"

Merge "Preferences::miscPreferences: Hard-deprecate this"

Merge "rdbms: Add @deprecate tag to MaintainableDBConnRef class alias"

Merge "Ignore master connections for POST-nonwrite in $wgTrxProfilerLimits"

Localisation updates from https://translatewiki.net.

Change-Id: If17e7446210675a8d91df4e3b6ae80cb35d3afa3

Use commit-and-wait when processing more than updateRowsPerQuery

This is needed to avoid triggering "does not have outer scope" errors when
addWatchBatchForUser() is invoked from WatchAction::doWatch() or
WatchAction::doUnwatch() in the context of uploading a file, moving a page, or
other instances when we don't have outer scope.

Bug: T208003
Change-Id: Ice5cb8fced64883476daea5cdac36e47dfcccb61

OOUI: Backport I407b0d6fe7b81465054b640d4b5ac4bf352a9901

Bug: T208595
Change-Id: I480067d9897dde25e89058a4da6b065de11b9b31

Merge "Sync up with Parsoid parserTests.txt"

rdbms: Add @deprecate tag to MaintainableDBConnRef class alias

Added as per when this was aliased to ease removal
in the future. It was there since REL_1.29.

Bug: T195576
Change-Id: Ia9061cb9592b39d0f6bac1c96f87c9b0346c105b

Preferences class deprecation, step 2

Remove all functions that have been hard (or superhard, with exceptions)
deprecated previously. They all had no callers with one exception:
* getTimezoneOptions() is called from 3rd party extention, already broken
  anyway.

Mark the remaining functions for removal in 1.34, where whole class
will become {{db-empty}}. They in principle have no callers already, but
let's follow the proper process and give extension developers a bit more
time.

Change-Id: I0e3e98ed1e6242b6ab7c5f21d71478c392fe89a9

Preferences::miscPreferences: Hard-deprecate this

Change-Id: Ic1bdf7fc132a1410af1ac24a4c0b8fbad3a6158e

Merge "Upstream VE's MWCancelConfirmDialog as AbandonEditDialog"

Merge "Hard deprecate OutputPage::parse() and OutputPage::parseInline()"

Merge "Support multi-content diffs on Special:Undelete"

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 68bfb079ab3c62d5cc79b05d496a4bc2966e9553

Change-Id: I54359f91c28ba5d622175dc1caf10a8ed8e91ad3

Merge "RCFilters: Fix highlighted rows overlapping legend"

RCFilters: Fix unintendly wrapped ButtonSelectWidget

With I2102f6d3397 OOUI's ButtonSelectWidgets behave differently on
limited parent element horizontal space.
We re-introduce `white-space: nowrap` here specfically.

Bug: T208705
Change-Id: I154c2d9c5469a789d365c5ccb771a9fc0cc018a4

Merge "installer: Use getUrl() for CSS link in WebInstallerOutput"

Localisation updates from https://translatewiki.net.

Change-Id: I10ac08f52ac9650f4897dc9aaffc7dc9a4dfbb68

installer: Use getUrl() for CSS link in WebInstallerOutput

This change changes the CSS URL in the stylesheet link from

  <link rel="stylesheet" href="/mediawiki/mw-config/index.php?css=1"/>

to

  <link rel="stylesheet" href="/mediawiki/mw-config/?css=1"/>

on the website /mediawiki/mw-config/.

Change-Id: Ib46c4d944df862cf928d3ebd050ff2d1e16ba6a5

Upstream VE's MWCancelConfirmDialog as AbandonEditDialog

So this dialog can be shared across editors.

Bug: T192333
Change-Id: Ib581528363a3fe5461529515835bd7fb1ce66e56

Equalization translations for Serbian

Change-Id: I3c79ded77f16beee404ca3005ae7678838ab9d60

Merge "Special:Preferences: Expose `.mw-navigation-hint` on keyboard focus only"

Localisation updates from https://translatewiki.net.

Change-Id: I52eda764f5cba9b1f88bd6a2d3817f88214849ab

Merge "Use new ScopedCallback::newScopedIgnoreUserAbort helper function"

Merge "Update wikimedia/scoped-callback to 3.0.0"

RCFilters: Use `display: flex` for better responsiveness on narrow canvas

Bug: T197065
Change-Id: I0fdff660a480aeedc096bc4ec48b05569e67e595

Merge "Don't infer parsoid-only parser tests based on presence of parsoid option."

Merge "PHPVersionCheck: Remove obsolete load.php code and simplify"

RCFilters: Fix highlighted rows overlapping legend

While the new rows are being faded in, they each have an opacity not
equal to 1. Setting the opacity on something creates a new stacking
context, and because the rows are below the legend in the DOM, this made
them appear on top of the legend.

Fix this by setting a z-index on the legend so it stays on top.

Bug: T186841
Change-Id: I2eee201d1e0aed94974a85d6d46b86fd4a4db796

Don't infer parsoid-only parser tests based on presence of parsoid option.

For historical reasons, we used a complicated test to try to determine
if a test is "parsoid only" based on the presence of the string "parsoid"
in the parser test options clause.  But really test clauses should be
marked up specifically as html/php or html/parsoid, we don't need to
play hacky games any more.  Remove some unnecessary code and simplify
the test logic.

Change-Id: Ia331e7e12a75aa19aeb96601712f169f2aa75bf9

Merge "Upgrade wikimedia/xmp-reader to 0.6.1"

Merge "Sync up with Parsoid parserTests.txt"

Merge "Update deprecations after OutputPage::parse*As*() backport to 1.32"

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 24b6363b9a99435479305fa98651b07e2ddb118a

Change-Id: I9090541605411af1ff3b4252b2e0902736e1ed48

Update deprecations after OutputPage::parse*As*() backport to 1.32

Originally landed in 1.33 in a3f63785eebfa2f729d42da880b6722736ef1e7d.

Backported to 1.32 in cff9ab4b5c26dc87137aa2c04efd5d1f31226fe8.

Change-Id: I402c31b57904f6a6fd0b3a2c32a1c0a73c553254

Fix typo in tests/includes/SampleTest.php in core

Minor typo fix in comments for setUp() and provideTitles()
methods in SampleTests.php file.

Change-Id: I09e0fb041d6809509e780d22bf8419497cd50ee7

Add start, end to allowed params of Special:MyContributions

I assume they were just omitted by accident.

Bug: T207895
Change-Id: I99c14849a6f312f0014298d92de773205d05ad04

Merge "Handle <nowiki> and other markup consistently in image link/alt options"

Use new ScopedCallback::newScopedIgnoreUserAbort helper function

Introduced in Ib6e307d76f93.

This patch updates FileBackend, LoadBalancer and LBFactory to use a public
function newScopedIgnoreUserAbort in ScopedCallback instead of all using
the exact same function but duplicated.

Bug: T184044
Change-Id: I27d7dc16abfe4b9447d7f3d8bd89f0de3ddeb662

Update wikimedia/scoped-callback to 3.0.0

Depends-On: Idf3ee1f785104257dec8c6bfead821244184b126
Change-Id: I005575a341d4eda4595dc36537c4f2996e32eb45

Upgrade wikimedia/xmp-reader to 0.6.1

Depends-On: I3eba0c81aed14f1087c46fb11088412cad5a505f
Change-Id: I215ab354b39a276aefc54fabad4e3ddcd792a796

Ignore master connections for POST-nonwrite in $wgTrxProfilerLimits

Unless an actuall write happens, avoid create log spam over things
like CentralAuthUser->getSafeReadDB().

Change-Id: I1cf43f4b1cdd12eb458458448f0f0ea328925b47

Hard deprecate OutputPage::parse() and OutputPage::parseInline()

Depends-On: I2cc3a4631bcb45b7f8cd913e9b6dba14349e5e9e
Depends-On: Ieaac7a198cacec406a8240ed07b2d9f32ef9e56a
Depends-On: Ia4b63715380d97ccb3133bf39a260834c20b4f5a
Depends-On: I88fb74c3007360e2301c8bca7db6a940e966a735
Change-Id: If5c619cdd3e7f786687cfc2ca166074d9197ca11

Replace deprecated call to OutputPage::parse() in HTMLForm

Use OutputPage::parseAsInterface() to tidy the output and make the
selection of user interface language explicit.

Follow-up to Ifeb1ca6eb8b5c743421b8f9e329f1e3658050e47.

Bug: T198214
Change-Id: Ia4b63715380d97ccb3133bf39a260834c20b4f5a

Localisation updates from https://translatewiki.net.

Change-Id: Ibbaeb400c9e5a1c744d8690cccc56d8d2fb9d0cf

BlockRestriction::update() unnecessarily does a SELECT on the page table.

This unnecessary SELECT causes a performence issue where it takes an
exceedingly long time to perform an update.

Bug: T208526
Change-Id: I0238234146fc838067019351ea97046a7d37d6fc

Merge "Only expand `{{...}}` in messages once (part 2)"

Merge "Replace deprecated calls to OutputPage::parse()"

Merge "Use Html:rawElement() where possible in special pages."

Merge "RollbackAction/SpecialUndelete: Use OutputPage::addWikiMsg() where possible"

Fix undefined NS with @covers tags in /includes/auth/

Should MW Codesniffer also handle these as well?

Change-Id: Ieb493bddd5f3674fb889b4a780d51b50d89c4f65

Merge "Add docstrings for text search index field types"

Merge "Add langauge 'kjp' Eastern Pwo"

Merge "Recognize CSS/JS redirects with non-ASCII targets"

Merge "Improve some queries ordering by rev_timestamp with actor migration READ_NEW"

Recognize CSS/JS redirects with non-ASCII targets

The regex in JavaScriptContent and CssContent was trying to pass
urlencoded stuff to Title::newFromText(), which would fail. Make sure we
urldecode it first.

Bug: T208264
Change-Id: I189c4c308da2875839ad8c1061332500f0e6d244

Merge "Diff and history link separated via CSS"

Merge "Fix IP handling in DerivativeRequest"

Merge "Pass $context to 'GetPreferences' hook"

Merge "Update OOUI to v0.29.3"

Fix IP handling in DerivativeRequest

$r->getIP() should return what was set by $r->setIP(); currently
that breaks for DerivativeRequest as get reads from the base but
set writes on the derivative.

Change-Id: If30c233931ceb1ca05da8866da3c3ab78de1a339

Update OOUI to v0.29.3

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

Bug: T94749
Bug: T112982
Bug: T179755
Bug: T206925
Bug: T206211
Bug: T206768
Bug: T207728
Bug: T207905
Bug: T207917
Depends-on: I4c090cff7b1ec1dbaeefd9d1ed7f9d9ebd8092f5
Change-Id: I9106e927f1b4fd07006df1e3c9d8ef9832cf919d

Merge "Fix for missing end tag </page> on some exports"

Add docstrings for text search index field types

Change-Id: Icd34ec79db4d8fd024a82045aaa9bedb6556caa8

Fix for missing end tag </page> on some exports

T203424 replaced streaming mode with batched queries.
However, it did not properly handle some values of
the $wgExportMaxHistory config variable, and emitted
broken XML. This change fixes that issue.

Bug: T207974
Change-Id: Iade3fc603e513da51b7a970c16275516c02ede49

Merge "User: Don't fail mysteriously when passing a User object to idFromName()"

User: Don't fail mysteriously when passing a User object to idFromName()

If $name is a User object, some code magically works because the object
gets converted to a string, but other code blows up because objects
aren't valid array keys. Prevent this from happening by explicitly
forcing $name to be a string.

Bug: T208469
Change-Id: Icc9ebec93d18609605e2633ccd23b90478e05e51

Doc: add missed return type to Title::getEarliestRevTime()

Change-Id: I98faa04fb9a4710daffa3c55ba3f61df0e12d71b

Localisation updates from https://translatewiki.net.

Change-Id: I12e9cbefff0b52f7086402f42cbcc5389d692cf9

Merge "DatabaseSqlite::insert: Fix affected row count"

DatabaseSqlite::insert: Fix affected row count

Follow up to 633eb437a3b808518469c6eaf4e86a436941d837

Bug: T208331
Change-Id: I142bb8c8abd43242d098932da212aa58323a0863

Merge "Follow-up d67121f6d: Blocks can apply to non-User objects too"

Follow-up d67121f6d: Blocks can apply to non-User objects too

Bug: T208398
Change-Id: I1d39f4ff709f37e7047f49964101e83c97cda0e9

Use identical check instead of equality check

$user->getId() returns an int and it's strict on this,
so the check here should be an identical check as "=="
does unnecessary coercion. So "===" will be faster and
improves on speed/performance.

Change-Id: I28a4e86ab339f10251df8846b8d951b673fe4f9e

Display SVGs in target language

Previously, they were always displayed in defult language unless
forced explicitly in wikitext, e.g. [[File:Foo.svg|lang=ru]].
This change adds a feature flag that would enable always trying to
display in page language.

* If enabled, Parser will pass a new parameter - 'pagelang' - to
  the media handler.
* SvgHandler uses page language when determining what language to
  render the image in.
* 'pagelang' can always be overridden by 'lang'.
* If no translation in page language is available, the default
  language (English) will be used for thumbnail URLs, to prevent
  cluttering media storage and HTTP caches with useless copies.

Performance: this requires accessing image's metadata during parsing.
My testing indicates there were no code path where this wasn't the
case already, so no performance hit is expected, however we should
still keep an eye on page save performance.

Bug: T205040
Change-Id: I348840ef405e1370cc0c17d69051bce30153c9c0

Localisation updates from https://translatewiki.net.

Change-Id: I7f4659b873f1b238b90bcd59ce9e97ee48990b44

Follow-up dcf3eb79: Drop tabs.legacy.js file, unused

Change-Id: Ifbed79f8be978729fdedef7fce7c72ee20d63087

mediawiki.special.preferences/tabs.legacy.js: Drop old `jsprefs` class, unused

Change-Id: I91c3069c7588fe9a77af169bab07ae296bccd2e2

Merge "Drop the LanguageGetSpecialPageAliases and LanguageGetMagic hooks"

Merge "resources: Drop jquery.localize, deprecated in 1.32"

Merge "Drop six authentication-related hooks, deprecated in 1.27"

Merge "Remove use of old API doc methods"

Merge "Use WikiMap methods for wiki ID logic in more places"

Merge "Set the SaveParser logger in RevisionRenderer"

Merge "rdbms: clean up return values of IDatabase write methods"

Merge "Use new WikiMap::getCurrentWikiDomain() for objectcache and profiler keyspace"

Use new WikiMap::getCurrentWikiDomain() for objectcache and profiler keyspace

Change-Id: I0e77c58a55b7ce908ec1d6399ec6629d00a48953

Merge "debug: Allow the DBQuery channel to be used"

Set the SaveParser logger in RevisionRenderer

Change-Id: Iacc28ce583fed8cf2a7f7b931754c64619cb4917

rdbms: clean up return values of IDatabase write methods

Also improved the atomicity and affected row count logic for
insert/replace with sqlite.

Also remove unused "fileHandle" code from insert().

Change-Id: If7b9148fd44f3a958899885753c7c86ba66bf193

Merge "Use WikiMap methods in ClassicInterwikiLookup instead of wfWikiId()"

Merge "Add isCurrentWikiId()/isCurrentWikiDomain()/getCurrentWikiDomain() to WikiMap"

Drop the LanguageGetSpecialPageAliases and LanguageGetMagic hooks

Deprecated since 1.16 and unused in known git.

Bug: T199291
Change-Id: Idcaf1b2c24c64653f700be4fe810ed124ba4ab7c

resources: Drop jquery.localize, deprecated in 1.32

Change-Id: Iaab8637cd336ab47c789ebcf1d0610695555095f