Perform edit stashing when the edit preview or diff form is requested

* Users are likely to save after they inspect and review their changes.
  The buttons to do so are also located below the edit summary box.
* This obsoletes the backend stashing on preview. Most of those parses
  would have been useless due to being per-section only. Also, some
  extensions like the Graph extension disable stashing for "preview"
  output anyway. Simplify the code by removing that method.

Bug: T136678
Change-Id: Ied77bdbd191dd9267d4295b0fa7b942f65b062db

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 969556c9cdc7f03dd02bcf985fc476b22e8053f0

Change-Id: Id124cc5aa32233e7b48578c0ddc4bb88c06fa893

Merge "Remove unused and undocumented $wgUseMasterForMaintenance"

Merge "Add an audio file parser test"

Merge "Explicitly specify the width of the search input on Special:Search"

Explicitly specify the width of the search input on Special:Search

This ensures that, even with float:left on the element, Firefox tries to render
the input field with as much space as possible (up to 50em).

Bug: T137535
Change-Id: I10fed035328d82f680d268f3bb116ca07d1ee576

Don't misname login button

Particularly don't name it "wpRemember", which is the name of a
completely different field.

Bug: T137799
Change-Id: I4b5f79ec59edf33b3ec91f019bf08ef5507ed55b

Merge "FormatMetadata: Use binary integer literals instead of bindec()"

FormatMetadata: Use binary integer literals instead of bindec()

We can do this now that we have dropped support for PHP 5.3.

Change-Id: I1e78249e244105496e976bf01e35f4067410b2f0

Remove unused and undocumented $wgUseMasterForMaintenance

Change-Id: I32b58a869c796f5170fc61d6cedca9b07b7f5762

Remove PHPVersionError, deprecated and unused since 1.25

Change-Id: Id6637886dc1d5ce3a13c9ba23dd23e76b2168eba

Merge "Make it possible to disable the upload dialog functionality"

parser: Remove redundant comment about revisionsize cache vary

Follows-up 457431b.

Change-Id: Iac3e4d6c11de3737155e7f7ff35ec7a6a3873865

Avoid setting vary-revision for {{REVISIONSIZE}}

Just always use the input size for new revisions. If they are
saved, then that should be the revision size. If they are just
null edits, then the size must have matched the current revision.

This also enables edit stashing for this case.

Change-Id: I428c0cc87750eeddd1d7dcebd1a2b03817cec441

Send the edit summary/user to the edit stash API

This can be useful for pre-caching AbuseFilter check results

Bug: T137698
Change-Id: I03e3924408a2a5da09a411b66d09958a1b78cb99

Merge "Move mbstring initializaton to Setup.php, add checks"

Move mbstring initializaton to Setup.php, add checks

Bug: T137509
Change-Id: Ib3496c9465a7b229b97793c6fa5dafb878c04968

Support short array syntax in findHooks.php

Bug: T135257
Change-Id: I19d4f073ff730821230ccb28d29df303fbaccbc2

Make it possible to disable the upload dialog functionality

One can disable it by setting `$wgForeignUploadTargets = [];`.

Change-Id: I2489004271078a78a1ce698304fb591196eeb941

Merge "mediawiki.checkboxtoggle: Fix minor issues."

Localisation updates from https://translatewiki.net.

Change-Id: Ia3ca901c6a7ca042b2ab6690bf3affa7c8b3ffd4

mediawiki.checkboxtoggle: Fix minor issues.

Follows-up 606a21c, 79414f8, 870c5e2.

* Mention class name in PHP source. Since there is no obvious link
  between "mediawiki.checkboxtoggle.js" and "ListToggle.php" these
  comments make it possible to find the PHP class. Previously
  these css classes had 0 matches across all source code, thus
  risking accidental removing.

* Quote attribute selector.

* Ensure default prevented before instead of after the handler. These toggles
  have no fallback (href="#"). As such, navigation should be prevented
  regardless of whether the JS fails or succeeds.

* Remove use of is(':checked'). These is no need to reach out to the
  selector engine to determine whether 'checked' property is true or false.
  Instead, use prop('checked') for both getting and setting.

* Simplify code by using this.checked directly instead of $().prop().

* Simplify code by using one loop (from $.prop) instead of two loops
  ($.prop inside $.each). This pattern is also used by selectAll().

Test by using "All", "None" and "Invert" on Special:Log.

Bug: T131318
Change-Id: Idfc43f094c6147d69104416b3f8622eabb20b824

Merge "FormatMetadata: Fix comment indentation and whitespace"

FormatMetadata: Fix comment indentation and whitespace

Change-Id: I55d2e6fd160c84208fb323b4c1d31f3291ed12f2

Don't try to use the context language when auto-creating

Most of the time the context language won't be ready yet, because it
needs the user we're auto-creating.

Bug: T124367
Change-Id: I0376647be33e81593101378217b37363125cfddf

Merge "Fix docs/hooks.txt syntax for RequestHasSameOriginSecurity"

Merge "Don't look for hooks in .php5 files that don't exist"

Fix docs/hooks.txt syntax for RequestHasSameOriginSecurity

There is no place for a type before the variable name

Follows 9ec1ef7308acc0366e92f8e6af10ce3cb22b5065 (security patch not in gerrit)

Change-Id: I7c2718f8026c7163553b9135362e5de61a26c9f8

Merge "Add new param of hook UserIsBlockedGlobally to docs/hooks.txt"

Add new param of hook UserIsBlockedGlobally to docs/hooks.txt

Follows I23d29c1a0e016de4e82d5b51afa94ae9afd70ee4

Change-Id: Ib747d66d7387455c23246cc29ab88f1383e52500

Merge "Use strpos() instead of preg_match() when checking for "/" in Skin::subPageSubtitle"

Merge "Fix docs/hooks.txt syntax for SpecialSearchGoResult"

Use strpos() instead of preg_match() when checking for "/" in Skin::subPageSubtitle

Per http://php.net/manual/en/function.preg-match.php#refsect1-function.preg-match-notes

Change-Id: I0b2a0e89d77422ed50e5178fcdbbb4e31199fdc8

Merge "Put interwiki search results after local results"

Don't look for hooks in .php5 files that don't exist

Change-Id: I4c9cdcdebfe4e617361f723df624c1d3e3096e6e

Merge "findHooks.php: Work also on .inc and .php5"

Merge "Fix name of LinkRenderer hooks in docs/hooks.txt"

Localisation updates from https://translatewiki.net.

Change-Id: Ia2e64c04c2a05e0b2eb041c138e6d0fb66674595

Fix docs/hooks.txt syntax for SpecialSearchGoResult

Use colons after variable name

Follows Iec7171fcf301f1659d852afa87ce271f468177c1

Change-Id: Icf8a2b53b7746a0761fc27477fac19c53a772820

Fix name of LinkRenderer hooks in docs/hooks.txt

Follows I057cc86ae6404a080aa3c8e0e956ecbb10a897d5

Change-Id: Id4126409e179cb1f6d67a8e97281c96fbcfb76ee

findHooks.php: Work also on .inc and .php5

Also fix the use of RecursiveDirectoryIterator::SKIP_DOTS, which was
passed to the wrong constructor.

Follows I29280dfe629126c71adfb65ac1a59394bc459dc4
Follows I6c01b8768f9f70152e320d254f8025a0ffde8c62

Change-Id: Ib35ca10b809afd5f6da80cd5edd1a71479c917a7

Localisation updates from https://translatewiki.net.

Change-Id: I0d96f471b50edd2d6cd15729377ea7b40e1bda21

Make LocalFileLockError an error page exception

This presents a better message than a cryptic red exception hash box

Bug: T132921
Change-Id: Ie3f358378af54d0348f18cfb1df763a182259906

Perform edit stashing for ?undo and ?oldid reverts

Previously, these cases would not send stash requests.

Bug: T136678
Change-Id: I02d760f3c59fb09aeafc0aca429cee835f3e0b6e

Revert "Make APIEditBeforeSave give the whole revision"

This may negatively affect performance and the whole purpose of the
hook (making it possible to reject an edit from an extension while
providing detailed error information in the API result) has been
invalidated by 09a5febb7b024c0b6585141bb05cba13a642f3eb, which lets
EditFilterMergedContent do this too.

I think it was intentional that the hook was called with just the text
passed to action=edit API. Making it actually be called with the text
that's going to be saved would require more work (e.g. for
automatically resolved edit conflicts, T73947).

Very few extensions use this hook. I'm fixing AbuseFilter to use
EditFilterMergedContent in I30c1e3d0a6c10888e6ac53745313434474663cce,
we should also review ConfirmEdit, ProofreadPage and SpamBlacklist to
see what behavior they really expect.

This reverts commit be97167ab61e7d59d86f6a9dca53b7016d615735.

Change-Id: I62713419496bcf57364a8fa9de93c0c8ddc3e91c

findHooks.php: Only look at PHP files, not silly things

Change-Id: I29280dfe629126c71adfb65ac1a59394bc459dc4

Remove useless sort parameter to printArray()

All callers use the default

Change-Id: If2a5cd64242f3b3d9ad9a99f5af7fb5b93438e8e

Localisation updates from https://translatewiki.net.

Change-Id: Ia93b8aad378646799ed1a6fc4a22e8fc2d3647b8

Merge "Remove these two rights autoreview and torunblocked from mediawiki"

Rename log authevents channel back to authmanager

Fixes authentication metrics. We want to go in the opposite
direction eventually, but having auth metrics broken on the day
after the AuthManager deployment to group2 is not fun, and this
is less risky as an emergency deploy than messing with log config.

Change-Id: I5e7a6bf1ce40aa132626207f6d7c45dd0c832c10

Merge "Add support for Balinese"

Add more logging info to parseAndStash() for uncacheable content

Change-Id: I2409f0db518d366a747aa01ba5593dccdb34e68c

Merge "Fix "Undefined index: type" warning on LoginSignupSpecialPage"

Merge "Fix auth event log channel name"

Fix auth event log channel name

Change-Id: I05daab1166ffee31453692c53533efb09b7ad2ed

Fix "Undefined index: type" warning on LoginSignupSpecialPage

When discarding form descriptors that do not have an equivalent
AuthenticationRequest field, do not assume the descriptor always
has a type.

Th warning was probably caused by I2a8cd4850.

Change-Id: Id1daee1e4954199fcb2077178e8f08d1ad3ae916

Merge "MediaWiki UI: Set `max-width` on `.mw-ui-input-inline` elements"

MediaWiki UI: Set `max-width` on `.mw-ui-input-inline` elements

Make sure that MediaWiki UI `.mw-ui-input-inline` elements are not
overflowing `width` of parent element. Useful for
input textfields on narrow screens.

Bug: T137108
Change-Id: I69f50ea5d57bd24ff6999b50297a276ca4f99ec2

Fix file permissions on SearchInputWidget.php

Change-Id: I0c7b149a0c62f02b655e53c255ba01a92fc6c4a8

Merge "Add "PostSend" limits to $wgTrxProfilerLimits"

Merge "Revert "Map dummy language codes in sites""

Revert "Map dummy language codes in sites"

This is causing duplicate English Wikipedia interwiki links.
The second link is supposed to be for Simple English, but
instead is linked to English Wikipedia.

This reverts commit 1c1c74adef8a2f5597472e8c049010ff4d42ff64.

Bug: T137404
Change-Id: Ibc1c53ec1b8e643b47ad52f38a8496acfb399d65

Merge "Simplify ApiStashEdit::checkCache()"

Merge "Remove "masterPos" stuff from RefreshLinksJob"

Simplify ApiStashEdit::checkCache()

Per https://grafana.wikimedia.org/dashboard/db/edit-stash, the
proven_* cases are almost never hit. Simplify the code to avoid
complexity and slow link table queries.

Change-Id: I3f8bd79a80ddb26c6188b1a301c08766e38f6ad4

Merge "Allow minor edits to be filtered out of Special:Contributions"

Merge "Fix generateLocalAutload.php"

Fix generateLocalAutload.php

Installer overrides rework in I2661ba2036b2887d31ab356751d731cc8b499f26
made this exception uneeded, remove.

Change-Id: I952a1237ebe3d2ac86182488f1c0e297c7c6e304

Localisation updates from https://translatewiki.net.

Change-Id: I816baca545df33e40659900cefe92add4c034ae9

Merge "Revert "resourceloader: Add support for variables in WikiModule""

Merge "Use READ_NORMAL for User cache regenerations"

Revert "resourceloader: Add support for variables in WikiModule"

No longer needed per doing Ic137cb494ba23 in a different way.
This may be useful to revisit, but for now preferring to keep
simplicity and removing this unused option.

This reverts commit 9e217bf42d73ef2dfcdecfe3a753cf8d702a18fd.

Change-Id: I9c0c316a3b58a3d0a3d3282dd74c7fa4eef8e378

Merge "AuthManager: Fix ConfirmLink validation"

Merge "mediawiki.checkboxtoggle: Run after document ready"

Merge "mediawiki.checkboxtoggle: Remove position=top"

Merge "Add default properties for the password reset form skip button"

mediawiki.checkboxtoggle: Run after document ready

Bug: T131318
Change-Id: I74d7de03e7338affbf7f0013937ec38a30cded27

mediawiki.checkboxtoggle: Remove position=top

It's not needed. Also remove it from the .styles module too since that's
automatically implied.

Bug: T131318
Change-Id: If7cca518a318345daa43fbbfded32dceadfc6044

Add default properties for the password reset form skip button

Make "skip" the last option (so it's not the default action when
the user presses enter in a text field) and do not mark it as
primary/constructive as there shouldn't be multiple such buttons.

Bug: T136894
Change-Id: I2a8cd4850f740240520732b1b54d7876f25a00b8

Change the way installer overrides work

Instead of "don't edit this file, edit that file", do it settings.d
style where packagers can drop their stuff in mw-config/overrides.

I propose to backport it to 1.27 because LTS.

Bug: T135695
Change-Id: I2661ba2036b2887d31ab356751d731cc8b499f26

AuthManager: Fix ConfirmLink validation

I726d79de added validation of the incoming requests to
ConfirmLinkSecondaryAuthenticationProvider, but forgot to fill in the
username and action fields before doing so meaning the confirmation
would probably fail.

Change-Id: I75cb65c24538a1e60e3b47a3b9ecfbd3a79ce8bf

Add support for Balinese

Language code: ban

Bug: T137411
Change-Id: Ief8ef036cc0abae2c1c060784784e3b5453edce6
Signed-off-by: Raimond Spekking <raimond.spekking@gmail.com>

Merge "AuthManagerSpecialPage: Don't use fixed submit/info class for submit button check"

Remove "masterPos" stuff from RefreshLinksJob

Just do a single slave lag wait check when branching the base job.
Any remnant/leaf jobs after than do not have to do anything special.

This should also improve de-duplication and reduce commonswiki
errors like "Could not acquire lock on page #42482792" due to
insane pages.

Change-Id: I40f9c6e0e905bd8149bb364c33a0642628cb1423

Localisation updates from https://translatewiki.net.

Change-Id: Ic5724eb907b438b49782985441579ffcd8ebee8c

Include title/user in some edit stash log messages

Change-Id: I49d61d9cfc5edacf415f6822cb50cc9007735a27

Use READ_NORMAL for User cache regenerations

This fixes a regression from 3bbccc8da64b6. A large uptick in
master queries showed in DBPerformance logs.

Change-Id: I5cf07aba2c3f928005d3aba55554c406e87026e2

Merge "ApiQuerySiteInfo: Avoid extra getConfig() calls"

Merge "ApiQuerySiteInfo: Add $wgInterwikiMagic value"

Merge "Flag internal page retrieve/save cycles with EDIT_INTERNAL"

ApiQuerySiteInfo: Avoid extra getConfig() calls

Change-Id: Idfd11691cf579107162a730ded17c5101cd72463

Merge "Allow configuring distinct upload dialog comments for local/foreign uploads"

ApiQuerySiteInfo: Add $wgInterwikiMagic value

Bug: T131564
Change-Id: Ia7c1e944c0f65d8b1b940388aceb1f0f89005dc6

Merge "mw.ForeignStructuredUpload: Fix confusing typo in a comment"

Merge "mw.Upload.BookletLayout: Fail when unable to load config from foreign wiki"

Merge "Avoid getWithSetCallback() warnings on unversioned key migration"

Merge "Add new convenience User method for authentication data change"

Merge "Re-introduce "Templatize Special:Contributions lines""

Add new convenience User method for authentication data change

Also update createAndPromote.php to use it so it can display errors.

Note that there are two possible approaches. The other is to
do the update for all requests for which the test passed,
even if some tests fail. The approach of the patch seems
more manageable from the caller's point of view
(either the operation was a success, or it failed and
nothing happened).

Change-Id: I86abed4b80472cd888337444fac0cbcb870b1246

Merge "Add a visual clear to Special:Search input box and profile-tabs"