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

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

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"

Re-introduce "Templatize Special:Contributions lines"

The original change was 1bce6db10e99840cfbd10fa074c93d8d96d88412
It got reverted in I8f63e002e34a9d13e2597a27e3dd918a687d1e7c
Fixed issues raised

Bug: T122537
Change-Id: I047038d6164bd6e32909d25bb37002b43bfd3a92

Merge "resourceloader: Load ResourceModuleSkinStyles config once per register() call"

resourceloader: Load ResourceModuleSkinStyles config once per register() call

Change-Id: I052503027f656fe15fca1069017a05ab589efaac

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

Follow up: I698d9728c70cfee5693594f2f1e0513c942aa2cd

Bug: T137285
Change-Id: I4a41d549ae40965d0698234ea308d14f9b14edcb

Localisation updates from https://translatewiki.net.

Change-Id: Ia7ecf877adc3a35e9c02e27115095435f9484d94

Avoid getWithSetCallback() warnings on unversioned key migration

If an unversioned key is present, this generated a warning
previously.

Bug: T137244
Change-Id: I92df9b9fe0436348b5b27e684d239238d57c9b5f

Allow configuring distinct upload dialog comments for local/foreign uploads

Change-Id: I192d6989af9613730a1e0b8ce077e651bc450d37

mw.ForeignStructuredUpload: Fix confusing typo in a comment

Change-Id: I47694fa97bf82ca1532542d65ecae5826e1640b2

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

Previously we just ignored the problem and used the config from the local
wiki. This could result in incorrectly formatted uploads (if the error was
transient and the upload succeeded), or more confusing error messages
appearing later (if the error wasn't transient).

Change-Id: Ica6e65abbb678a9942c2bba6370e3ba156085bb8

LinksDeletionUpdate: Fix typos in column names

Bug: T137280
Change-Id: I78ee489f10e796d2b3fa39dcb4f65f0763860bf3

Flag internal page retrieve/save cycles with EDIT_INTERNAL

This is used to avoid edit stash checks and stats noise.

Bug: T136678
Change-Id: I6158b8b218240ef3875f2d1d2e8ca3bc2f69f980

Merge "Use callable type-hint in a few places"

Merge "Use LinkCache::getSelectFields() in OutputPage::addCategoryLinks()"

Use callable type-hint in a few places

Mostly places which immediately had a:
 if ( !is_callable( $callback ) ) {
  throw new Exception(...);
 }
check at the beginning of the function.

Change-Id: Ia78663b2231629010816bd1cda8814b996968d1d

Merge "Avoid contention in updateLinksTimestamp()"

Use LinkCache::getSelectFields() in OutputPage::addCategoryLinks()

Change-Id: I71d95cbb6c22ae92586c883e7de5353d26c4838f

Merge "Fix Message::newFromSpecifier for nested RawMessage"

Fix Message::newFromSpecifier for nested RawMessage

This can happen e.g. when something processes Status contents
and expects [$key, $param1, ...] and instead gets [$messageObject]

Change-Id: I346b35e08bd38ce231e16d0616438ea408b55bff

Merge "Change invalid password reset form language"

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

It's possible that an authentication extension implements its own HTMLForm classes
for a submit button or info field. To omit the default submit button of HTMLForm
in these cases, too, it's necessary to check if one of the provided HTMLForm field
classes inherit from HTMLSubmitField or HTMLInfoField (assuming, that subclasses
in extensions inherit from one of them).

Bug: T137236
Change-Id: Id0ec914088b527a9c7224d545b04986068138f93

Fix Status::getMessage for the multiple errors, $shortContext only case

Remove $msgCount++ which does not seem to have any non-harmful purpose.

Also try to explain what the hell the method does.

Change-Id: Ibdf40ff13a4a9d294a6bc6f778a14fb2633c2f70

Localisation updates from https://translatewiki.net.

Change-Id: I4174fc3bca40d45f06dd93af10d009d23acae375

Merge "Reject authentication data change when there are warnings"

Reject authentication data change when there are warnings

AuthManager uses a status which is OK but not good for authentication
data which is discouraged but still accepted for authentication.
Users should not be allowed to change credentials into such invalid state.
This change brings the web logic and the AuthPlugin fallback in sync with
the API which already used the stricter check.

Change-Id: I4ff54fcc901f6fe11f15ed60fc1a3d8753de9f6c

i18n: Use https: scheme for HSTS external links

When the local wiki is accessed in HTTP, clicking an external link
starting with // will (if this is the first visit) do an unencrypted
HTTP call to the target server before being redirected to the HTTPS
version, potentially allowing a man in the middle attack.

This commit changes the URI scheme to https: for every link with HSTS
enabled in the codebase, namely www.mediawiki.org, *.wikimedia.org,
*.wikipedia.org, translatewiki.net and bugs.php.net.

Change-Id: I002b502952f32fb49823e18b281252977ecca8cf

Merge "Map dummy language codes in sites"

Merge "resourceloader: Add support for variables in WikiModule"

Merge "Call ChangeAuthenticationDataAudit from the API"

Call ChangeAuthenticationDataAudit from the API

Bug: T137193
Change-Id: I2f7a7da2eb9973ed8f946c52e9821ed962dcd7ba

Map dummy language codes in sites

The script that populates the sites table assumes that the
subdomain is the content language code. This is not true for
all wikis. This patch introduces a mapping to fix this issue
on the fly, based on $wgDummyLanguageCodes.

This is driven by the need to avoid "bad" language codes when
creating wikidata items from the client, when "linking" two
pages on different wikis. When we do this, we use the language
code from the sites table for the labels of the new item.
We would like to forbid "dummy" languages in labels and
descriptions, for consistency.

Change-Id: I6452761e14d9902bb069e32d0f499bc39e680453

Avoid contention in updateLinksTimestamp()

Since LinksUpdate::doUpdate() already flushes the transaction,
go ahead and flush before other DataUpdates might run (e.g.
from RefreshLinksJob). Also release the lock before running
the LinksUpdateComplete handlers, as the lock is just to keep
LinksUpdate instances from racing with each other.

Change-Id: Ied97fa36fbca0203123e9fc966d2e23bfd621c0e

Change invalid password reset form language

The old workflow was login -> go to a password change form (at which
point the user is fully logged on and can just abandon the form).
The AuthManager workflow is go through a string of login providers
(including the password reset), and get logged in at the very end
- so the user has to submit the form. The submit button text for not
updating the password was changed from Cancel to Skip to account for
that; this patch updates the messages which reference that text.

Bug: T136894
Change-Id: I38ac7883d2acfc73c5662dc2c382a395ef1b91c1

Merge "jsduck: Update paths to OOjs UI files in eg-iframe.html"

Merge "Chinese Conversion Table Update 2016-3"

Remove getOptions() checks from getStashKey() entirely

It only avoided a tiny edge case of options lag, which is
not worth the complexity.

Change-Id: I94bc35fc89eb46436a23b10aee1b7d339859c7ea

Merge "Make edit stash keys less preference sensitive"

Merge "InfoAction: Don't pass non-dbkeys to LinkBatch"

Make edit stash keys less preference sensitive

Only factor in the user ID/name/sig instead of anything that
might happen to touch the user row.

Bug: T136678
Change-Id: I004790a017c68cfcff6729f6fbfd0f79b6904c20

InfoAction: Don't pass non-dbkeys to LinkBatch

Usernames are stored with spaces instead of underscores, so this was
causing exceptions later on when dbkeys were expected to be provided to
TitleValue.

Let Title take care of normalization from spaces format to underscores
in Title::makeTitle(), and pass that object onto LinkBatch instead.

Bug: T137147
Change-Id: Ia3606d30de61173e4d5526a9d7a77bd4b1710a05

Localisation updates from https://translatewiki.net.

Change-Id: Ia8baf1ed8a5cd70ebf65955f95d47f3b73745e43

Merge "Avoid rollback and bot flagged edits in edit stash stats"

jsduck: Update paths to OOjs UI files in eg-iframe.html

Follow-up to 1f49b66c6f89d56dee457196296a7869f0f7475d.

Bug: T136954
Change-Id: Idb29a69992fff96c51d18d34b36ad751f3b0469f

Merge "Follow-up c81ab8ae5f: add forgotten dependency on mediawiki.language"

Avoid rollback and bot flagged edits in edit stash stats

This just skews the numbers with cases that could not use
the cache anyway.

Bug: T136678
Change-Id: Iaa90d4f6ee80e886f59bc43a1392e59fe2a8f900

resourceloader: Add support for variables in WikiModule

To be used for the 'site' module. This will make it easier to split up
'site' into 'site' and 'site.styles' (per T92459).

Change-Id: Iaac3e458d5107e4c10c2826bd64608d5c47e8b87

Merge "Avoid exceptions for empty titles in addResultToCache() for sanity"

Follow-up c81ab8ae5f: add forgotten dependency on mediawiki.language

Bug: T137002
Change-Id: If35416f4bd08f72f53dad26e80f8417ca2a6d7d9

Avoid exceptions for empty titles in addResultToCache() for sanity

Bug: T137083
Change-Id: Ie8aabba30bc0d7a6910a58104ff08ea036ba962d

Merge "Password change request should be optional on soft reset"

Password change request should be optional on soft reset

Non-optional fields are required by HTMLForm now so the skip button
would not work properly.

Change-Id: Iad937767f0f7cf6dce43f7882720d1ed730ee28d

HTMLForm: Properly support 'flatlist' in OOUI mode

Previously it worked mostly by chance for multiselect fields (until
2f33194497bd94d1cecb21dcb88e9bf3abf2ddaa broke it), and did not work
at all for radio fields.

Bug: T123935
Change-Id: I94396303bcf16110a505883bec2ce352d833fe2b

Merge "Fix float of search input on Special:Search"

Merge "Don't override action in UI and REDIRECT responses"

Merge "Add column alias support in BatchRowIterator"

Localisation updates from https://translatewiki.net.

Change-Id: I961120816132dc1a46f62e25b7223006a727aa32

Localisation updates from https://translatewiki.net.

Change-Id: I66385ba8e6a50980cfc957fd13e58cd5a43ed438

revisiondelete: Fix $oldBits/$newBits usage in setVisibility()

This addresses the FIXME comment around updateLog()

Change-Id: Iedb9df0b4cf65308a6b58bd9d9295d1850bc2c28

Merge "Avoid DBPerformance warnings in Category::initialize()"

Avoid DBPerformance warnings in Category::initialize()

Use DeferredUpdates and zero-out negative values in the meantime.

Bug: T92357
Change-Id: Ia1339395b70b042edba6dc1d570acb92c90a287b

logging: Simplify DeleteLogFormatter b/c check

Change-Id: Iab16663579d2eece4a94f80f7120b231aed0e572

Clean up setVisibility() log type logic

The log type used should be based on whether any item
was (un)suppressed, not just the last log entry. Otherwise,
unsuppression could end up in the wrong log if the last
item in the list was not unsuppressed but others were.

Change-Id: I7b6af524cc45a1d83b2b719bfa00138531455e35

Localisation updates from https://translatewiki.net.

Change-Id: Ib2318bbccf83f08e9f32c97b6fea0c6baa6d1cd7

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 8fa3996c2656b66789e658c50150f45b19c69ae1

Change-Id: I7f839aaddddf6ac96b7d155989f20a2b386d5622

Don't override action in UI and REDIRECT responses

In Ic8caf57eb, we changed things so the requests returned in a UI or
REDIRECT response would have the action forced to that appropriate for
the action being peformed. But ResetPasswordSecondaryAuthenticationProvider
has a use case where a mismatch is necessary: it's run during the login
action, but it needs a PasswordAuthenticationResponse for a change
action.

Bug: T136894
Change-Id: I9d109a22c5b2d2064f664f584100ecaab43199c5

Fix float of search input on Special:Search

Follow up: Icc68a8e83fc7037b96a5efec2766ffdf453c5413

Bug: T136945
Change-Id: I698d9728c70cfee5693594f2f1e0513c942aa2cd

Add column alias support in BatchRowIterator

Used by Echo/maintenance/backfillReadBundles

Bug: T136368
Change-Id: Ia8f8aa718713798a81392b5e9751c5c18ab81720

Merge "Show ParserOutput warning instead of on the actual page output for ignored display titles"

HTMLForm: Don't add a type=reset to links

Follow up: Ieb80e2ff36751abc6f00e2a02926fe9800666a8b

Change-Id: If88c80a46cba9729a9503b82532584443d6d5ba4