EditPage: Extract some edit conflict code into EditConflictHelper

Extensions with custom content models (e.g. ProofreadPage) will extend
EditPage to provide a customized editing experience. However when
another extension like TwoColConflict wants to change just how edit
conflicts look, this causes problems since only one class can take the
place of EditPage.

So instead, split most of the frontend code for edit conflicts into
TextConflictHelper, and call it from edit page. Extensions can override
with the instance created by calling
EditPage::setEditConflictHelperFactory().

And to make that split possible also move EditPage::addNewLineAtEnd()
and ::buildTextboxAttribs() into a separate TextboxBuilder class that
both EditPage and TextConflictHelper can use.

Bug: T176393
Change-Id: Ie415edd84329c02d5762477f8a171fced85b01a2

Merge "Remove unused mediawiki.ui 'checked' icons"

Localisation updates from https://translatewiki.net.

Change-Id: Ic741440ded2718acbe29973af639442eb722d0ff

Merge "HistoryPager: Add missing $tagFilter property"

HistoryPager: Add missing $tagFilter property

Change-Id: Iec2d944d5583eb13fa9383c0a6a183b550f60aab

Article: Drop partial purge code and constants, deprecated in 1.29

Change-Id: I4f20d823e6c9206121212b8a73ecc28f91e68b45

Merge "Rename some cache keys with odd virtual hierarchies"

Merge "Make runJobs.php treat --json value as case insensitive"

Merge "API: Include setupSkinUserCss in prop=modules for useskin mode"

Merge "API: Add tests for useskin parameter of ApiParse"

Remove unused mediawiki.ui 'checked' icons

Removing unused mediawiki.ui 'checked' image, which was in use before
the switch to the unified checkbox with blue and white appearance.

Change-Id: I4ba88583a2ed59b44a1f7bbffe823468af8e3874

Make runJobs.php treat --json value as case insensitive

The docs mentioned "JSON", which did not actually work.

Change-Id: Ie9802551d05f7d80d1db0fd316e36f4b35bbc521

Rename some cache keys with odd virtual hierarchies

This makes automatic statsd metrics more useful.

Change-Id: I07d87dcb8ce9b42cc2f1c84ac4c06d177d463b1d

API: Include setupSkinUserCss in prop=modules for useskin mode

Follows-up 90c95fc7f290f, which included result of Skin::getDefaultModules
in the prop=modules list. All hardcoded modules in OutputPage and Parser
were also subsequently moved into Skin::getDefaultModules.

However, a number of modules cannot be moved there because fundamentally
Skin::getDefaultModules can only load modules via OutputPage::addModules().

For style modules, addModuleStyles() must be used.

Fortunately, there is already a centralised place for that, namely
Skin::setupSkinUserCss(). Include that in the ApiParse return as well.
That should resolve the last bit of inconsistency between ApiParse
and OutputPage when it comes to the module queue.

Bug: T140664
Change-Id: I35e2e3bbdccdd1aa2a259b8e624daa80c609ba8c

API: Add tests for useskin parameter of ApiParse

Change-Id: If3dff7be5ccb6791f95d37c06998fcbadf1f469f

Merge "Setup: Merge PreConfigSetup into Setup.php"

Merge "Add action/user tracking to link refresh jobs"

Setup: Merge PreConfigSetup into Setup.php

Follows-up 41ea7e2fefff65.

The following previously happened between PreConfigSetup and Setup
and must now happen either before it, after it, or moved inside it.

* WebStart: Detect missing composer.
  This must be after Autoloader/Vendor but before the first call to
  wfDebugLog (or other loggers) so that we can output a more descriptive
  error instead of a generic "Unknown class" fatal error.
  Moving it to before Setup is too early, and after is too late.
  Move it to within Setup.php and make it work in CLI mode.

* WebStart: Install header callback
  Moving it to before Setup is too early, and after is too late.
  Move it to within Setup.php (no-op in CLI mode).

* WebStart/Maintenance: Load LocalSetings.
  Must be between PreConfigSetup and Setup.
  Move to Setup.php to maintain execution order.
  Utilise MW_CONFIG_File for custom handling in Maintenance.php.

* WebStart: Initialise output buffering
  Utilise (new) MW_SETUP_CALLBACK hook.

* WebStart: Display NoLocalSettings.php
  Utilise MW_CONFIG_CALLBACK hook.

* Maintenance: Setting $wgLocalisationCacheConf, calling Maintenance::finalSetup.
  Utilise (new) MW_SETUP_CALLBACK hook.

Change-Id: I633a6ff235b4275391c48034c0525d2fbfa3fecd

Merge "Removed Toki Pona localization files"

Removed Toki Pona localization files

Bug: T132899
Bug: T178730
Change-Id: I4c61b3ef42cdc24fee74587965240ca08242867e

Localisation updates from https://translatewiki.net.

Change-Id: Ic5c7a0f485ea4efe772c791965d5c43c98780940

Merge "Add support for SVGs to $wgLogoHD with PNG fallback"

Merge "Follow-up 2454f51b274: Upgrade qunit from 1.x to 2.x too"

Merge "Remove unused table pager icons"

Merge "registration: Only allow one extension to set a specific config setting"

Add support for SVGs to $wgLogoHD with PNG fallback

SVGs could already be used through $wgLogo. However, if a PNG fallback
is desired for older browsers, using SVGs was previously not possible.

This commit adds support for using an SVG image in $wgLogoHD and,
using $wgLogo as the fallback image.

Usage example:

> $wgLogo = '/path/to/png';
> $wgLogoHD = [
>     'svg' => 'path/to/svg',
> ];

Note: When the 'svg' key is set in $wgLogoHD, any '1.5x' and '2x' keys will
no longer be used because SVGs can render optimally on any screen sizes.

@Reedy, @Krinkle and @Brion VIBBER helped me alot with this.

Bug: T86229
Change-Id: I6197d96ce9110f4711ef2c4b198445bc5c6ae110

Remove unused table pager icons

Removing unused table pager icons, which have not been in use since
Ia750131aae52

Change-Id: Ia9a8cf12c2efcc7760f3ca028aff6c7e82253ccb

api.js: Add an else to avoid unnecessary checks

If the first if matches then the second if can never match.

Change-Id: I96d59b57cac414c9ee7008e2f284a7a4e3348f3f

Merge "RCFilters: Display specific error if query times out"

Merge "objectcache: Always use interim values on WAN cache tombstones"

objectcache: Always use interim values on WAN cache tombstones

This stores values for very short times while the main
value is a tombstone in case of particularly high traffic.

Also make mutex keys expire immediately on unlock.

Change-Id: I4ec5cf7f8b49239fdd2518e5d955534877a0f7ee

objectcache: Add more hotTTR comments to WANObjectCache

Change-Id: I123e5231206350ee51098fcb6528da126d1a86ce

Localisation updates from https://translatewiki.net.

Change-Id: I6b0ec9e393e8f8af256a7dd7f833312d6acb5504

Add action/user tracking to link refresh jobs

Change-Id: Ie7261eacddb869988b005ba2f17968df88c7003e

Merge "resources: Deprecate the 'jquery.badge' module"

Add missing @throws documentation to Language::factory

Change-Id: Id5ff9d438864e2d39a10441c43aba4188191f623

LocalFile: Fix copy-paste error

No idea how that got through.

Change-Id: Ic7eebe624434005ff962f19eaa85cdf40e131284

Merge "SpecialEmailUser: Don't increment pingLimiter() just for opening the page"

SpecialEmailUser: Don't increment pingLimiter() just for opening the page

When checking whether the user can even send am email, don't increment
the rate limits since they haven't sent an email yet. Only increment it
once we're fairly sure that the email will actually get sent.

Bug: T177575
Change-Id: I3f832accd6259c040438ebee4c94847faa108ea6

Merge "RCFilters: Truncate saved query menu item"

Merge "Fix changes list misaligned arrow"

RCFilters: Display specific error if query times out

This catches DBQueryTimeoutError, logs it, returns HTTP 500,
and displays an error message.

Live update just ignores it (effectively treating it the same as
a live update check that happens to have no updates).

Bug: T175776
Change-Id: If4d880e9e6a56989895956798fc6918a43841065

Merge "Remove @codingStandardsIgnore from long lines"

Merge "Remove @codingStandardsIgnore after upstream fix"

Localisation updates from https://translatewiki.net.

Change-Id: I639918d677a9ca7cfe9115fa0223c1abbf1884c8

Remove @codingStandardsIgnore from long lines

Breaks some line where the ignore is not needed.

The sniff was changed upstream to be okay
with long unbreakable lines in comments

Change-Id: I2bbe2be7cedd4d3c0ce8dc3e62d0e268bc171876

Remove @codingStandardsIgnore after upstream fix

Issue #1604 was fixed -
https://github.com/squizlabs/PHP_CodeSniffer/issues/1604

Change-Id: Id81cb820e42123007ae8696422bebf588e274437

Localisation updates from https://translatewiki.net.

Change-Id: I8e1401909269c4d0375ad61c77cc29993f02a2ce

Merge "Use LogFormatter::getLinkRenderer in LogFormatter extensions"

build: Updating mediawiki/mediawiki-codesniffer to 14.1.0

And moved phpcs.xml to .phpcs.xml (T177256).

Change-Id: Idaa1fe006a528c25ce34abd0a857c657c26e8b0f

Merge "Switch DifferenceEngine to the new shell framework"

Merge "Check minimum database server version when running update.php"

resources: Deprecate the 'jquery.badge' module

Only used in the PageTriage extension, which can provide it itself:

https://github.com/search?utf8=%E2%9C%93&q=%22jquery.badge%22+%40wikimedia+-repo%3Awikimedia%2Fjquery.badge+-repo%3Awikimedia%2Fmediawiki-debian&type=Code

Bug: T178450
Change-Id: Ibe4ff6a1f39fc554dcf4fc20cc214521d854f1ec

Switch DifferenceEngine to the new shell framework

Change-Id: Iad8718e5f54ed829efded8efbb0f3496c119e6ad

Merge "Hard deprecate Title::escapeFragmentForURL()"

Localisation updates from https://translatewiki.net.

Change-Id: I40093d586318b7bceddbe982bb0c71dfeb76dc79

Merge "objectcache: Split off some code in WANObjectCache::getWithSetCallback"

objectcache: Split off some code in WANObjectCache::getWithSetCallback

This makes it a bit easier to follow

Change-Id: I67968814ab046473eb8eca4086a8600c77417b82

Merge "Set getDeduplicationInfo() for HTMLCacheUpdateJob"

Set getDeduplicationInfo() for HTMLCacheUpdateJob

This allows de-duplication of single page jobs for the
same page due to edits to different templates. This is
the same logic that RefreshLinksJob already has.

Also fix a bug in that method in RefreshLinksJob.

Change-Id: I2f79031c945eb3d195f9dbda949077bbc3e67918

Merge "objectcache: Mention colons in BagOStuff key generation"

objectcache: Mention colons in BagOStuff key generation

Change-Id: I76967245c32aac4eea8d6672e4fc1fac5e5f3d07

Merge "Shell\Command: Move code that builds final shell command into separate method"

Shell\Command: Move code that builds final shell command into separate method

Change-Id: I6aae209fd0b20057b5f7f7129db92c184ec945f8

Merge "RCFilters: Actually toggle the collapsed class for toplinks"

RCFilters: Actually toggle the collapsed class for toplinks

Bug: T178396
Change-Id: I2206816540342012296669f8dd24ab44762f4788

Fix changes list misaligned arrow

- Revert td alignment from middle to top.
- Add top padding to fix misalignment.

Bug: T178595
Change-Id: I1211b7afe1c29b0a33c051c07921eed7eb9704e3

RCFilters: Truncate saved query menu item

Bug: T178610
Change-Id: I787237beb2f08a01bd9c6885c75401bacfcc6f9b

Localisation updates from https://translatewiki.net.

Change-Id: If6a368d96b689c022f5a0f4b835628ddef0a2b41

Merge "RCFilters: refactor highlight state"

RCFilters: refactor highlight state

* Consider highlight to be enabled when applying
  parameters that contain highlight colors.

* Don't store 'highlight=0|1' in the URL or
  saved query.

Bug: T177009
Change-Id: I8f3a1c609cef89bc08077776d453ced6f2d0f5e2

Merge "resourceloader: Introduce metric for backend response timing"

Merge "jquery.badge: Align with Echo notifications style and WikimediaUI palette"

Merge "Make $wgExperimentalHtmlIds fall back on modern HTML5 mode"

mediawiki.ui: Remove remainder `text-shadow`

The `text-shadow` property on primary buttons was probably introduced
for a fake 3D effect and also because the colors were lighter in
order to set the white text apart. None of this is true with our
WikimediaUI color palette any more so let's remove this remainder.

Change-Id: I25e5ce7e6fafc19d8b925b6918c98dd61b423d64

Merge "RCFilters: rephrase newcomers description"

Merge "RCFilters: define consistent interface in ChangesListFilterGroup"

Localisation updates from https://translatewiki.net.

Change-Id: Ic1284de8147baa9a8436621bc2e3d1de4037a920

RCFilters: define consistent interface in ChangesListFilterGroup

Get rid of isPerGroupRequestParameter and define a consistent
interface that all filter groups can implement.

Change-Id: Ib904bcdc697c65722a0041ac611d1e00c577389f

Merge "jQuery: Remove jQuery 1 legacy option"

Merge "Revision tests, make test for constructFromRowArray a unit test"

Merge "RevisionIntegrationTest more newFromArchiveRow tests"

Merge "RevisionUnitTest::testConstructFromArray @covers tags"

Merge "Update the translation for MessagesKo.php"

RCFilters: fix stylelint error

Not sure how this got merged but it's reported as an error
locally.

Change-Id: I7cd1a9442ea07cf6e4595ff4a3611a0b682a4e6e

Merge "Fix missingcommenttext message to be more useful in different page layouts"

Fix missingcommenttext message to be more useful in different page layouts

This missingcommenttext message makes reference to a specific page layout,
which it shouldn't given the UI can be skinned in a variety of different
ways. This patch removes the specific reference to "below", which wasn't
particularly helpful anyway.

Bug: T158262
Change-Id: Ieb81db162bee6010e85b17355059e18ad447751e

RCFilters: rephrase newcomers description

Bug: T178334
Change-Id: I76a805acae4b7c6efa3707fa6c203e72c2e29ea2

Replace deprecated edit review params

- Replace old options 'hideanons' or 'hideliu' with structured UI equivalent.
- Update tests to take this new behavior into account.

Bug: T176172
Change-Id: I6ad050f7864bf51db05c3db957ac3533358cd3ac

Merge "Database: Support parenthesized JOINs"

Merge "Replace more uses of "SELECT *""

Hard deprecate Title::escapeFragmentForURL()

No usages in core and only one outside of core, being
fixed in If1191817ee2db84d8fd3b91aa13e6257faf10a41.

Change-Id: I91ca289e989a4dc32de083b9cc002c9c9d75a3e1

jquery.badge: Align with Echo notifications style and WikimediaUI palette

Aligning badge style to Echo notifications, ensuring WCAG level AA conformance
and also to WikimediaUI color palette.

Bug: T152025
Change-Id: I279a607f3c1b13ea6e21e59df0c6d21e525f587e

Introduce Shell\CommandFactory

Bug: T177038
Change-Id: Id875e68ea1fa72b44a463f977ab52270fe1e7088

Don't trigger PHP errors for unused Shell\Command

Change-Id: Id29da4f21a44ccb18d8a2ae11348d69ca3233aa5

Command: Avoid using wfDebug()

Change-Id: I92ea55d7a5b6a71a6f6b944f377215c08ea3b096

Update OOjs UI to v0.24.0

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

Depends-on: Iad57bf4f6306086cb775629f2d9f2fcf2dbcf5fd
Change-Id: Idf2bed0a1719f8a30bcc66211cbeedd2bb31ec4b

Merge "Add a few schemas to SQLite tests"

Revision tests, make test for constructFromRowArray a unit test

Change-Id: I52aa62a442011b612966bb294015730a05702cd3

RevisionIntegrationTest more newFromArchiveRow tests

There were a couple of code paths that were not tested before
for different combinations of selected fields, these are now
covered.

Change-Id: Ifd438e5c97a5f32a1744c58bd4cf0303ddb32cfc