Merge "installer: Add a defined check to overrideConfig method"

Merge "Increase test coverage for BlockManager class"

Merge "resourceloader: Complete test coverage for FilePath"

resourceloader: Complete test coverage for FilePath

The 'getLocalBasePath' and 'getRemoteBasePath' methods were not
yet covered.

Change-Id: If2eacca3a908048ec62b357e14e4e2322363e296

Merge "API: Fix 'user_id' field of ApiCSPReport"

Merge "API: Add test case for ApiCSPReportTest"

API: Fix 'user_id' field of ApiCSPReport

Was accidentally turned into an always-true boolean in 5f343617,
due to confusion with JavaScript's default '||' operator.

Change-Id: I24071e22f8bb7a296ae138303f63acdb8ea4950a

API: Add test case for ApiCSPReportTest

Covers the basic functioning of the class, as well as the specific
behaviours introduced or changed by 5f34361759 and 0ca1b8a0e621c9.

Also includes a (bad) expectation for 'user_id' (bool instead of string),
which is a bug caused by 5f34361759 that the next commit will fix.

Change-Id: I2c57c813b8a31e51a61778951227ccbd1217a547

Merge "DevelopmentSettings: Disable legacy javascript globals in CI and for devs"

DevelopmentSettings: Disable legacy javascript globals in CI and for devs

Bug: T72470
Change-Id: Iedd3a9c8e541bc4cc657753e6ae3389f0393ed64

Drop UploadBase:: and UploadFromChunks::stashFileGetKey() and stashSession(), deprecated in 1.28

Change-Id: I2411753a24092a5c37434b095e594831ecff04e1

Drop the UploadVerification hook, deprecated in 1.28

Change-Id: Ic1917fc95223cbf97845c2702ddf98accf8ba9c2

Merge "objectcache: add MediumSpecificBagOStuff base class for non-proxy subclasses"

Merge "objectcache: refactor WANObjectCache::fetchOrRegenerate() locking code stylistically"

objectcache: refactor WANObjectCache::fetchOrRegenerate() locking code stylistically

Change-Id: I5e289989ef91923b650a9c325febd7410d1b2caf

objectcache: add MediumSpecificBagOStuff base class for non-proxy subclasses

This make it much clearer what needs to be overridden

Change-Id: I3073f8a0605f557c6a3a93d0d8401cddd0fb8dbe

Merge "MediaWikiUnitTestCase: Use DIRECTORY_SEPARATOR instead of `/`"

MediaWikiUnitTestCase: Use DIRECTORY_SEPARATOR instead of `/`

`strpos( $reflection->getFilename(), '/unit/' )` will always
return FALSE under Windows

Change-Id: Ib13d32858e238b9a18094f829939f6bba08366ab

Merge "wdio-mediawiki: Release 0.4.0"

Merge "Improve links in several specialmute-*"

Merge "Allow empty input in namespaces multiselect on Special:Block"

Merge "Clarify WikiRevision::downloadSource() and related code"

Merge "resourceloader: Add Config parameter to packageFiles callbacks"

Merge "Clean up DeferredUpdates transaction handling"

Improve links in several specialmute-*

* Remove namespace prefix from user page links
* Add "user" as translatable text instead

Bug: T218265
Change-Id: If20b9023379f00e0b82992322e340362613dce09

resourceloader: Add Config parameter to packageFiles callbacks

Callbacks used to generate the contents of virtual files in packageFiles
modules only received a ResourceLoaderContext object. They could access
the Config object through ResourceLoaderContext::getConfig(), but that
method is deprecated. Pass the Config object as a second parameter to
these callbacks, so that they don't have to use a deprecated method.

Change-Id: Ia4666914e9b07f298ee5ae30ae8c70a3e83b0910

Increase test coverage for BlockManager class

Change-Id: If2c50248448f3f633d9531039094f12c7d712c41

Merge "SiteConfigurationTest: Fix recurring typo ("an non-existing")"

Clean up DeferredUpdates transaction handling

Bail out in attemptUpdate() if the transaction state is dirty rather
that failing at some later point. Also, flush implicit transaction
rounds before calling DeferrableUpdate::doUpdate() for fresher data.

Bug: T225103
Change-Id: I4f5d2f9814a562069619f05e003663fcedbd3f64

Merge "Separate right for foreign user js redirects"

Merge "Revert "objectcache: fix race conditions in RedisBagOStuff::incr()""

Revert "objectcache: fix race conditions in RedisBagOStuff::incr()"

This commit reverts most of 7e647d2f0f5b, but keeps unrelated code
clean ups from it, as well as the conflicting changes from d8b952ae47.

From WMF production nutcracker:

> nc_redis.c parsed unsupported command 'WATCH'

The use of WATCH, in addition to failing the commands that use it,
also appears to also have caused a chain reaction making nutcracker
intermittently unavailable to other web requests.

Bug: T228303
Change-Id: Ic37efc2963b147e461837571ae0b65acf3f60cb4

Merge "GlobalFunctions: Hard deprecate `wfGlobalCacheKey()` function"

Merge "Fix param doc of TitleArrayFromResult::setCurrent"

Pass message params as array to RawMessage

To match documentation of RawMessage::__construct

Change-Id: I4d9fce6b63af89a0012c9eaac72c0b46f75b43ac

Fix param doc of TitleArrayFromResult::setCurrent

$row is a stdClass, not the ResultWrapper, which is a list of stdClass

Change-Id: Ic96354a6283fcf42ac1c75f524f1da45117100ce

Allow empty input in namespaces multiselect on Special:Block

Follow-up to 83ebbb519f

Bug: T219882
Change-Id: I5f3d907a8f8be02f9e2c23d2a66c42752daa38b5

Merge "Add support for extentions to change Special:Mute form"

Merge "objectcache: optimize MemcachedPeclBagOStuff::*Multi() write methods"

wdio-mediawiki: Release 0.4.0

a1e51bf94325130fa3d54d65c071d3ab57a4ab79 unpublished 3 RunJobs methods
They do not appear to be used according to wmf codesearch
and, given their limited usefulness, are unlikely to be used elsewhere.

Change-Id: Iaefe6f71320c2c3ab86468bf827b6e8d2e80095d

Add support for extentions to change Special:Mute form

The hook (SpecialMuteModifyFormFields) is used to append
the option to mute/unmute notifications from a specified user.

Special:Mute handles posting and saving the fields, the only
requirement is that the field name is the same as the property
that wants to be modified.

Currently there are only two notifications "blacklists":
* `email-blacklist` is directly handled in this page 'cause it is part of core.
* `echo-notifications-blacklist` is part of Echo, so this change is required
  to support it. See I77b3ccfdce9b501e

Bug: T220163
Change-Id: I2b3eee0802cb086091f35ecce13ae77a8e7d518d

GlobalFunctions: Hard deprecate `wfGlobalCacheKey()` function

The only extension that used this was ConfirmEdit and it has been
cleaned up with Ia7b276ee65fdf58c, so is it time we hard deprecate
it?

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5CbwfGlobalCacheKey%5Cb&i=nope&files=&repos=

Depends-On: Ia7b276ee65fdf58c4fc0859563930528d44a03ca
Change-Id: I3fda797c64ec62df4454625ad1a3bf89f6a88246

Delete very incomplete translation

Change-Id: Ic8df4cc9894a366148632dbb2a97803cd6feccef

Localisation updates from https://translatewiki.net.

Change-Id: I06dcc9ee33533abbfa09d8ae7bafd6f6547f0e22

Merge "objectcache: fix WinCacheBagOStuf::doGet() is_string() check"

objectcache: fix WinCacheBagOStuf::doGet() is_string() check

Allow integers to so that incr() and other cases work. This class
now passes all the unit tests again.

Change-Id: Ie7aae36180472a0e913fb8275bccee3fbc6fe7d9

Merge "Tests: Remove old TODO file and update README with integration notes"

Merge "Remove deprecated mw.Api.errors and mw.Api.warnings"

Merge "installer: Remove <doclink/> parser function and last use of it"

Merge "objectcache: clean up RedisBagOStuff and optimize changeTTLMulti()"

objectcache: clean up RedisBagOStuff and optimize changeTTLMulti()

Also fix some IDEA warnings in redis classes and make it easy for IDEs
to recognize the Redis (phpredis) class calls to RedisConnRef proxies.

Bug: T113916
Change-Id: If45a37da412ac37e8c07dc3d1053826aa0a62077

installer: Remove <doclink/> parser function and last use of it

Pass the url to the Copying page as a message parameter instead.
This results in identical text and styling. Not even "external link"
vs plain link styling difference, because that doesn't exist in the
installer stylesheet.

The message is renamed in this commit because it is a breaking
change (due to <doclink> being removed, and due to $2 being
required).

Bug: T227297
Change-Id: I73efd9ecb193523a7e384a7a40c768d32b2361b8

Merge "Remove pointless PermissionManager::revokeTemporaryUserRights method"

Merge "doMaintenance: Try to print errors to stderr"

Merge "parser: add vary-revision-sha1 and related ParserOutput methods"

Separate right for foreign user js redirects

Require a new editmyuserjsredirect permission for users to edit
Javascript redirects in their userspace when the redirect target
is not in their userspace (unless they have edituserjs and can
edit any user JS anyway). This is to prevent attacks where a
popular userscript has been moved into the system namespace or
another safe location but many users still load it through the
original userspace redirect, and the attacker manages to take
over the userspace by compromising the account or getting it
renamed.

Since this is only a concern on large community wikis, by
default all users have the editmyuserjsredirect permission.

Bug: T207750
Change-Id: I36a879d5da04cb6f49ed1bc40dbe144f6862c6a1
Depends-On: I072cf857c1fff4578402904aa9cb5a0c8833f16f

Remove pointless PermissionManager::revokeTemporaryUserRights method

Also add missing @since and make phpdoc formatting match (most) other
methods.

Change-Id: I087205ad23836f283e5ba291488a9bc1f4c3ce99

Merge "Remember checkbox state on Special:Block if checkbox disabled"

Merge "rdbms: better handle a non-existing  "defaultGroup" in LoadBalancer"

Merge "objectcache: relax WANObjectCache "pcTTL" nesting rule to allow set()"

objectcache: relax WANObjectCache "pcTTL" nesting rule to allow set()

As long as get()s are disallowed from the process cache, the sets() should
at least still be up-to-date, so there is little reason to prevent them.

Change-Id: Ic62c8380801130de7f8412cddcf85b246e33b3cd

Merge "Exclude extensions/skins test directories from coverage analysis"

Tests: Remove old TODO file and update README with integration notes

Change-Id: I006a238f5241a70841f0d25ef7196f13654fb7ac

Merge "externalstore: unbreak writes to non-default storage clusters due to isReadOnly()"

Exclude extensions/skins test directories from coverage analysis

Follows-Up: I3d19627fa36f6cc6666c29fdb638272fdaa30630

Bug: T192078
Change-Id: I29dec891ee2327dde96dc293f3d9cda0e6a891fc

Merge "Remove unused variable"

Merge "Report more information about composite blocks in block error messages"

Report more information about composite blocks in block error messages

For any messages that use CompositeBlock::getPermissionsError, include
details of the original blocks from which the composite block is made.

If there are any database blocks, give their IDs and also explain that
there may also be blocks due to IP blacklisting.

If there are no database blocks, then explain that the IP must be
blacklisted in multiple places.

Bug: T212326
Change-Id: Id6ad0019f8add4d5e000da5e872338e87cca485e

Remove unused variable

Change-Id: I70c76e05e46f3432e580eb6e93d7a49ecfc5cfb4

Remember checkbox state on Special:Block if checkbox disabled

Many of the blocking options on Special:Block are only valid for
certain inputs. These options are represented with checkboxes. If
an option is not valid with the current form inputs, its checkbox
is disabled (and not submitted) and it is set to false by default.
To make this clear to the user, we uncheck the checkbox when it
is disabled.

This stores the value of the checkbox before it was disabled and
restores it if the checkbox is enabled again.

Bug: T221117
Change-Id: I3e52d7c3eb70f2798de3fb5ac3824d56e73306a6

MessagesKo.php: periodic updates

Add Mute and sort BlockList alphabetically.

Change-Id: I8125979b61e489b2c99af258254c4bb83675b9c4
Signed-off-by: Yongmin Hong <revi@pobox.com>

parser: add vary-revision-sha1 and related ParserOutput methods

This can be used to avoid double parsed on save if the prior output
can be reused in-spite of involving a self content reference.

Change-Id: Idcd30a3fa3f7012dac76ce8bbf46625453ae331f

externalstore: unbreak writes to non-default storage clusters due to isReadOnly()

Instead of telling ExternalStoreMedium classes the default write stores and using
that to make them read-only, let them be configured via other means. For example,
ExternalStoreMwstore already respects FileBackend::isReadOnly() for each location
(e.g. file backends) and ExternalStoreDB checks LoadBalancer::getReadOnlyMode()
for each location (e.g. DB cluster).

Make ExternalStoreAccess::isReadOnly() take a list of base URLs, default to the
default write stores if not specified.

Bug: T227156
Change-Id: I3161890fb2ccb46d6206628f0cd88f8af9f1688c
Follows-Up: I40c3b5534fc8a31116c4c5eb64ee6e4903a6197a

Merge "Add mechanism for temporary user rights"

Add mechanism for temporary user rights

Add a mechanism for adding temporary user rights that only exist
for the current request. This is occasionally needed to let normal
users act with a bot flag; traditionally the fact that User::$mRights
was public has been abused to do it, but I88992403 broke that.

Bug: T227772
Change-Id: Ife8f9d8affa750701e4e5d646ed8cd153c1d867b

Hard-deprecate wgProfileOnly, deprecated in 1.23

Change-Id: I1b66931ca0d3ae3e044b14f5b97dd98861ec5e23

Hard-deprecate wgLocalInterwiki, deprecated in 1.23

Change-Id: Ib18c3122c4d9b8a50d314f67004cc73b8c14d40e

Merge "Hard-deprecate wgSkipSkin, deprecated in 1.23"

Merge "Update OOUI to v0.33.3"

Merge "Fix OO.ui.infuse error on misc-authed-ooui module"

Fix OO.ui.infuse error on misc-authed-ooui module

Follows-up 419849211e3a63.

Change-Id: I1e22c70fa8c467212a02b508c68a17abf79a77e8

Merge "Add mute preferences link to the tools menu"

Merge "Check namespace exists in HTMLNamespacesMultiselectField validation"

Update OOUI to v0.33.3

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

Bug: T212159
Bug: T225021
Bug: T226045
Bug: T226780
Bug: T226822
Bug: T227666
Bug: T227917
Depends-On: I7dde9d02f94c4e78ef480035d3887b68d0128233
Change-Id: I7800da87a6a10399f705b43e05c8592c766bae6f

Add mute preferences link to the tools menu

A link to Special:Mute is added to the tools menu on the sidebar
if the user is logged in and a registered relevant-user* is found
Mute preferences link will not be shown if $wgEnableSpecialMute is false

* From Skin::getRelevantUser: Special pages like Special:Contributions mark
  the user which  they are relevant to so that things like the toolbox can
  display the information they usually are only able to display on a
  user's userpage and talkpage.

Bug: T218270
Change-Id: I89eb8373963d28d52a026b60aa42d1c64b31b7c6

rdbms: better handle a non-existing  "defaultGroup" in LoadBalancer

If the specified default group does not have a corresponding server load map
defined, then ignore it and use GROUP_GENERIC.

Also, consolidate the group load and group reader index code for simplicity

Change-Id: Ic8bf9a3ebcbffb81fb14d7b1787a2adb97ac525d

Check namespace exists in HTMLNamespacesMultiselectField validation

Bug: T219882
Change-Id: I710d0abed6bd40dcea5bdd1c59c8d936565961eb

Remove deprecated mw.Api.errors and mw.Api.warnings

This static properties are deprecated since 1.29.

Change-Id: Iaf3de57105d04497026305c27cd1d745820657a4

Merge "Revert "RedirectSpecialPage: handle interwiki redirects.""

Revert "RedirectSpecialPage: handle interwiki redirects."

This reverts commit 41106688abbe6dfff61c5642924ced42af3f0d33.

The original case is changed by this commit from a MediaWiki fatal
exception with HTTP 500, to a blank 200 response due to silent
failure. Use of GoToInterwiki appears to be invalid at this point in
the code. Reverting to keep prod the same as last week, so as
to unblock the train.

Bug: T227700
Change-Id: Ieece956d2e2e4c21b5ed7a75890b9f11eaf07e66

doMaintenance: Try to print errors to stderr

Don't mix them in with stdout (which might be redirected,
like dump output, making the errors hard to discover).

Change-Id: Ibed8c0e8dde3e44de60bf32abd3fc5ce5d29e1ba

Merge "resourceloader: Replace some Xml::encodeJs calls with RL's own encodeJson"

Linker: Force type Language for $lang in tocList and generateTOC

The call of wfGetLangObj( $lang ) is not necessary anymore.

$lang still defaults to user interface language on unset parameter.

This change is a follow-up to I15b65fec987641885374dfef9e1229ea405f7c30.

Change-Id: I6f12097a0e6cf7d6035d1164092c8b87c58e2bee

objectcache: optimize MemcachedPeclBagOStuff::*Multi() write methods

Make use of Memcached::OPT_NO_BLOCK and Memcached::OPT_BUFFER_WRITES
at selective points to avoid slow internal blocking for loops in the
PECL extension methods. As this mode makes the return values less
meaningful, this is only triggered only if WRITE_BACKGROUND is
provided (it only catches some glaring format or size errors that the
client runs into).

Leave changeTTLMulti() to the default as the above approach would not
make it faster, since the diver does not buffer for that command.

Expand mctest.php to include *Multi() methods and also break down the
single key method timings by method.

Change-Id: I11fc58e1a2296778b2526a0b2f3103fef1baba0c

Merge "resourceloader: Remove support for 'object' in wgResourceModules"

resourceloader: Replace some Xml::encodeJs calls with RL's own encodeJson

Bug: T32956
Change-Id: I614fe0e80ff308b857639a27d7772f969899b468

Merge "Remove $purgeBlobs parameter from LocalisationCacheRecache hook"

Merge "Add ImgAuthModifyHeaders hook to img_auth.php to modify headers"

resourceloader: Remove support for 'object' in wgResourceModules

Obsolete since MediaWiki 1.17alpha, no known usage since.

Bug: T222637
Change-Id: Ie820b16022ced6767c32aee7f2497a99260b1641

Merge "resourceloader: Add test coverage for WikiModule::getType"