Merge "Fix typo in apierror message"

Localisation updates from https://translatewiki.net.

Change-Id: I19182d49545679c7e86a71009616f23ba335efa5

Merge "RELEASE-NOTES: Note upgrade of OOUI (to be updated as we go)"

Merge "Fix Title::getFragmentForURL for bad interwiki prefix."

RELEASE-NOTES: Note upgrade of OOUI (to be updated as we go)

Follows-up Ibb166a30ad; let's start doing this regularly now.

Change-Id: I9dd092aed1da5d4d89cdb586fe27491142f20b4b

Update OOUI to v0.30.2

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

Bug: T198231
Bug: T210862
Bug: T212806
Bug: T213137
Bug: T213199
Bug: T213507
Bug: T213517
Bug: T213544
Bug: T213574
Depends-On: Ic2163347ed522199ba3a9dbd5cb45b32e138cb5d
Change-Id: Ibb166a30adfa87225fec3e256450db55b71e0f14

Merge "Add namespace restrictions to Special:Block and API"

Merge "Introduce multiselect widgets for namespaces"

Merge "Add NamespaceRestriction class so that BlockRestriction can handle namespaces."

Fix Title::getFragmentForURL for bad interwiki prefix.

Calling Title::getLinkURL or any other method that relies on
getFragmentForURL on a title with an unknown interwiki prefix
was triggering a fatal error. With this patch, that situation is
handled more gracefully.

Bug: T204800
Change-Id: I665cd5e983a80c15c68c89541d9c856082c460bb

build: Remove newer HHVM versions we'll never support

Been failing for months. The believe that, defacto, the only version
we support is HHVM 3.18. Interestingly, that build is failing too,
possibly due to configuration differences between WMF and Travis.
I'll keep it for now, in case wants to try and make it pass.

Change-Id: Id2fd32db14686cba1f65f9aa015e2b7f9275f00c

Merge "Datatable: Unify `padding` with `.wikitable`"

Merge "Datatable: Apply `padding` only on `th` & `td` elements"

Datatable: Unify `padding` with `.wikitable`

Unifying `padding` with `.wikitable`.

Bug: T214208
Change-Id: Ic04b647d2fa90ee0002a4d5b4597e271ea543194

Datatable: Apply `padding` only on `th` & `td` elements

Applying it on `table` as well results in issues when wanting
to override `border-radius` together with `border-collapse`.
It's not needed in a well-formed table as every child element would
need to reside within a `th` or `td` anyways.
Also the only other content carrying element `caption` doesn't receive
table's `padding` by default.

Change-Id: I5e8c9d47fab806b56e6d03f1611ddbf6d4ea8924

Merge "Better detection for old MS Office files"

Better detection for old MS Office files

* Introduce MSCompoundFileReader, which reads the CFB directory and
  detects the file type from well-known names in the root directory
* Do not detect a ZIP file if the EOCDR is not at the end. Other
  containers, especially CFB files, may contain ZIP files embedded
  within them in the last 64KB, but this is not a security concern
  unless the EOCDR is exactly at the end of the file.

Bug: T40432
Change-Id: Id5b1a258ccf3c3c8951e32f6b7a5b1bafe941082

Merge "Remove deprecated module alias "jquery.byteLimit""

Merge "mediawiki.api: Remove deprecated module aliases"

Fix typo in apierror message

Replace "conjuction" with "conjunction" in Apierror-redirect-appendonly/en message

Bug: T201491
Change-Id: Ia121deecf256888736a9a4de18967f85f063d153

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

Localisation updates from https://translatewiki.net.

Change-Id: Ia9ecbc8465dd445aa109916f2e8dda5161ed2ebd

Merge "Fix more jquery event shorthand usages"

Merge "rdbms: Remove references to pg_attrdef.adsrc in Postgres code"

Merge "Convert RandomImageGenerator to use the new execution framework"

rdbms: Remove references to pg_attrdef.adsrc in Postgres code

PostgreSQL v12 will remove the long-deprecated column
pg_attrdef.adsrc.  The supported way to introspect into column
default values is pg_get_expr(adbin, adrelid), which works
back through all versions of PostgreSQL supported by wikimedia.

Changing to the supported method will allow the upcoming v12 of the
database to be used while maintaining compatibility with older
versions, without needing to write version-specific code.

This patch has been tested with maintenance/update.php and
with phpunit in PostgreSQL versions 9.2, 11, and 12dev.  It does
not harm the first two, and fixes errors that would otherwise
arise in the dev version.  All unit tests which pass under version
11 now pass under 12dev as well.

Change-Id: I874d347fd286b26773113d4f0c6c30d9a4055ad3

Merge "build: Remove unused 'deepmerge' package"

Add namespace restrictions to Special:Block and API

This adds a UI for blocking namespaces to Special:Block
and a namespacerestrictions parameter to the block API.

The number of namespace restrictions in a single block
is not limited as page restrictions are.

The checkbox allowing the blocker to specify whether
the target can edit their own user page is normally
disabled for a partial block, but is re-enabled if
the block is to the user talk namespace.

If the config $wgBlockAllowsUTEdit is set to false, the
checkbox will not appear, and the target will not be
able to edit their own user talk page if they are
sitewide-blocked, namespace-blocked from the user talk
namespace, or page-blocked from their user talk page.

Bug: T204986
Change-Id: I9e231ad109d7285486ec332b26780339592b8df7

Introduce multiselect widgets for namespaces

Bug: T204986
Change-Id: Ie3916e2322d8b1a7effe9ba4604b596b568004e6

Merge "tests: Remove use of $IP and MediaWikiTestCase in libs/composer tests"

tests: Remove use of $IP and MediaWikiTestCase in libs/composer tests

Reference the fixture files relatively instead of absolutely,
this is done for most other references to data files as well
I believe.

Use plain arrayEquals(), which seems to suffice here. In PHP 7 at
least, regular == doesn't require the declaration order of the
keys to be the same in order to evaluate to true.

Change-Id: Iddc874ec811f5c960e13d480d70bcb20334cfa1e

Hard-deprecate wfArrayFilter() and wfArrayFilterByKey()

Deprecated in 1.32, no callers.

Change-Id: Id6147cff1336cc39863c985023933ce9c6afca31

Convert RandomImageGenerator to use the new execution framework

Introduces a stringifier for Command, useful for debugging.

Change-Id: Ifcfccaef5a609e0cf30186e39a6bd0fa971c2dbd

Merge "htmlform: Move HTMLSelectLanguageField into includes/htmlform/fields"

htmlform: Move HTMLSelectLanguageField into includes/htmlform/fields

This was originally directly under includes/htmlform. This is
a specific type of field, so it should go here instead.

Follows-up d0c31ac988888764.

Change-Id: Iac196068ce2cbca063948bc78d21913482717d9a

registration: Fix typo in schemas

Change-Id: I8d39290904593c1ef962be2995a536028d0635bc

Merge "Remove unnecessary placeholders from RELEASE-NOTES"

Merge "Replace deprecated `setIconTitle` with `setTitle`"

Merge "RELEASE-NOTES: Add note for jquery.async deprecation"

Merge "Return to LESS multiple value escape mechanism to prevent invalid output"

Remove unnecessary placeholders from RELEASE-NOTES

Change-Id: Ic30da7d03b0d34774e7d24d8b0616b6c6f66ae40

RELEASE-NOTES: Add note for jquery.async deprecation

Change-Id: I245fd9141f1b3040bed031ede7cb4327d900f5af

Return to LESS multiple value escape mechanism to prevent invalid output

LESS doesn't accept JS evaluation in PHP via `'@{arguments}'.replace`,
therefore we need to go back to manual escaping multiple comma separated
values.

Change-Id: I4c46afc012d804350ccee2cd5b1eed4e6ea61d8a

build: Remove unused 'deepmerge' package

Was added for wdio-related code, but not used in the end.
The only reference to this module anywhere in Wikimedia Git is
mediawiki/extensions/CirrusSearch, which doesn't use it for
its regular or daily tests, but rather the integration test,
which already has its own copy of this module in package.json.

Change-Id: Ifdf4362077f4684a2866363e35d0aca2f89f48b5

Merge "WebInstallerOutput: Add getLanguage()"

Add NamespaceRestriction class so that BlockRestriction can handle namespaces.

This begins work on making namespaces a valid restriction type. The CRUD
operations of BlockRestriction can now handle namespaces. Since
NamespaceRestriction implements Restriction, enforcement should start working
immediately, but testing enforcement will come in a subsequent patch since it's
impossible to create them.

Bug: T204991
Change-Id: I7264b452d9ad788c146d6ea25d01d4d7cb5ac4f6

Document header case in MWHttpRequest

The lack of this documentation resulted in some confusion in T214032 and
related tasks.

Change-Id: I08fc201b625197614ef5e163ed3686638b630b13

Remove deprecated module alias "jquery.byteLimit"

Follows-up 777336288bc8 (MW 1.31).

Bug: T202154
Depends-On: I91b88601334f72e9d7e897853d0e075c86928c60
Change-Id: Ief22737b9f64cec4f0cc7b25e79e385f2e9eb2b1

mediawiki.api: Remove deprecated module aliases

Bug: T202154
Bug: T196802
Depends-On: If1bd7521563b8574d12d45988737e0e838337f8a
Change-Id: Ic7020e43747774bfdad2e63263ee8caadc5d2f45

Localisation updates from https://translatewiki.net.

Change-Id: I74147fe057b8b4c35deba3146787d4e7082d3656

Localisation updates from https://translatewiki.net.

Change-Id: I9718a0adfc58858c00092a962f7d592cdc6f55db

WebInstallerOutput: Add getLanguage()

Combine the functions getLanguageCode() and getDir() to the single
function getLanguage().

Also mark the class WebInstallerOutput as @private.

Change-Id: Ic7deea5f3040fe8f1211b96622bdae222ca9e072

Merge "TablePager: Unify `padding` with `.wikitable`"

Merge "Reduce selector specificity of `.wikitable`"

TablePager: Unify `padding` with `.wikitable`

This should be at best temporarily until `.mw-datatable` gets same
`padding`.

Bug: T214208
Change-Id: Ib86808803fd6fa41704e7a69687a1ab41ef3c43d

Merge "registration: Add @codeCoverageIgnore for statement that can't be covered"

Reduce selector specificity of `.wikitable`

Reducing selector specificity by removing `table` from `.wikitable`.
All use cases seem to be covered by direct child `>` selectors anyways.

Bug: T214218
Change-Id: Iae532d6389348972a85404926aaa0c45de7e02e1

Merge "filerepo: avoid dynamic calls to FileRepo::isVirtualUrl"

registration: Add @codeCoverageIgnore for statement that can't be covered

MW_PHPUNIT_TEST will always be defined during tests, so we can't have a
test that requires it to be undefined.

Change-Id: I67b011af31026bc9c568725f8f294359b63f2fd5

filerepo: avoid dynamic calls to FileRepo::isVirtualUrl

Change-Id: Ie851ec33ba5d7965393e41595d0b4b9df070bd27

Localisation updates from https://translatewiki.net.

Change-Id: I1fc5d6f7500437df862f9f3f7b8e42af4702a9d8

Merge "Make sure page restrictions are handled as a string"

Merge "TablePager: Reduce pagination button in visual strength"

Fix more jquery event shorthand usages

Not yet caught by the linter.

Change-Id: I9601bff47c099c4901487e7faaf871f23e2e2d1c

Make sure page restrictions are handled as a string

Bug: T214173
Change-Id: I3ec9b0a4f797b0e89fd30a9ceafcaaf8a6dce5d9

TablePager: Reduce pagination button in visual strength

Reducing buttons to `frameless` buttons and reposition icons of
next and last page button to be symmetrically harmonious.

Bug: T214128
Change-Id: I1c0a621d7699b5fadc2559abdf6003a0db0aabae

Fix exception language in MovePage

Follows up I8f6c9647ee.

Change-Id: Iff2941bfc9e14f52ddd33f0a3d522c5a0d594c3e

Merge "dao: Add a type hint to getConnection() and some extra newlines"

Merge "DateTimeInputWidget: Set `max-width` equivalent to calendar popup"

Merge "WikiTextStructure: Add an exclusion from WikibaseMediaInfo"

Merge "WikiTextStructure: Explain the source of two non-Core exclusions"

Merge "Try to avoid master queries for GET requests"

WikiTextStructure: Add an exclusion from WikibaseMediaInfo

This is not lovely, and probably should be a hook or similar.

Bug: T213638
Change-Id: I042ac81b630dede55887e644692ea1a3b1fd6fe1

WikiTextStructure: Explain the source of two non-Core exclusions

Change-Id: I2673afb25c6f21789a4c89f390ca13dae2cc2fa9

Merge "Linker: Simplify return paths"

Merge "ApiSetNotificationTimestamp: Make entirewatchlist more efficient"

Adding $digitTransformTable for Santali (sat)

Bug: T212221
Change-Id: I30b9b55f16186e7fd404b6e3c9460a3fd81187d0

Linker: Simplify return paths

* Avoid else after return.
* Use early-return pattern.

Change-Id: I059e233d019dbc2233756776dfa09f8f2fdb2df7

Localisation updates from https://translatewiki.net.

Change-Id: I86f2b814991862ac1d750445543e6d9a8117c23a

DateTimeInputWidget: Set `max-width` equivalent to calendar popup

Input fields should provide as much width as user input is needed or
in this case a little more to be aligned with inherently connected
calendar widget.

Bug: T213785
Change-Id: Id316af701cd054ed29603ffa38af9238a39cb50e

Replace deprecated `setIconTitle` with `setTitle`

Bug: T76638
Change-Id: Ia51cbfaff7325f5eef385cc00e801ede69e78954

Merge "Fix error reporting in MovePage"

Merge "Separate content parts of mw-usertoollinks from presentation"

Fix error reporting in MovePage

Bug: T210739
Change-Id: I8f6c9647ee949b33fd4daeae6aed6b94bb1988aa

Try to avoid master queries for GET requests

Skin::getUndeleteLink performs several checks, including a
Title::userCan, which in turn checks the blocked status from master in
order to decide whether to show the link. This happens during GET
requests, and a basic calculation shows that it is responsible for
roughly 15% of the DBPerformance alerts.
This patch rearranges the IF conditions so that the permission check is
performed last, and thus will be avoided if the previous condition is
false.

Change-Id: I45a18a244a26df09beb12e198e0e91a465bd1907

ApiSetNotificationTimestamp: Make entirewatchlist more efficient

Use WatchedItemStore's built-in feature for clearing the entire
watchlist when no timestamp is specified. When a timestamp is specified,
this will still use the inefficient page-by-page method, which I'll
improve in a follow-up commit.

Bug: T207941
Change-Id: I2008ff89c95fe6f66a3fd789d2cef0e8fe52bd93

Merge "AllMessagesTablePager: Simplify language handling"

Merge "Deprecate Content::getNativeData, add TextContent::getText"

Merge "Simplify boolean logic for Special:Block widgets"

Deprecate Content::getNativeData, add TextContent::getText

getNativeData() is under-specified - callers can do nothing with the
value returned by getNativeData without knowing the concrete Content
class. And if they know the concrete class, they can and should use
a specialized getter instead, anyway.

Basically, getNativeData is overly generic, an example of polymorphism
done poorly. Let's fix it now.

Bug: T155582
Change-Id: Id2c61dcd38ab30416a25746e3680edb8791ae8e8

Merge "Fix BlockList params for non-editing partial blocks"

Merge "Correct namespace 'Template' for gom-deva"

Fix BlockList params for non-editing partial blocks

When multiple blocks are listed, if there is a partial block
with restrictions, any other partial block in the list will be
shown as an editing block even if it isn't.

Bug: T213952
Change-Id: Ieafb5f71c484d9f0551fa293f7c43e1a3a2e1f2a

Correct namespace 'Template' for gom-deva

The existing name for the 'Template' namespace was incorrect. Hence,
replaced it with the correct word for 'Template' and 'Template_talk',
and added the existing ones as aliases.

Noticed that the existing file was missing a comma at the end of the
last item in the alias list.

Bug: T213717
Change-Id: Ibb8385cae3368e2c36006cf9993dec4ebbbc4491

Fully extract services framework as a library

This code doesn’t use any MediaWiki-specific code, so rename
MediaWiki\Services to Wikimedia\Services and move it below libs/. (Of
course, this does not apply to the MediaWikiServices subclass.)

Class aliases are added to retain backwards compatibity for now.

Bug: T211608
Change-Id: Ic14ea28ef21c359695b309d4293dbaaf5deedc09

Simplify boolean logic for Special:Block widgets

The boolean logic controlling the showing/hiding of widgets on
Special:Block could be expressed more simply. The logic is
becoming more complex due to partial blocks, so now is a good
time to rephrase it.

This removes safeguards against passing undefined to toggle,
so make sure none of the checks ever return undefined.

Change-Id: I1e0eb288db708be951fe5cf11f658725a38a7c4d

Avoid session double-start in Setup.php

In PHP before 7.3, the double start doesn't really matter: session_id()
changes the ID even if it was already started, and the warning from
session_start() can just be ignored. Which is what we did.

In PHP 7.3, now session_id() also warns and no longer changes the ID. To
preserve the previous behavior, we'll need to explicitly close the old
session and open the new one.

Bug: T213489
Change-Id: I02a5be1c3adb326927c156fdd00663bccee37477

dao: Add a type hint to getConnection() and some extra newlines

Change-Id: I57a060394403a6b7335fda68edaeabc02888e14d

Merge "Chinese Conversion Table Update 2018-4"

Merge "resourceloader: Avoid unnecessary xmlns attributes in ResourceLoaderImage"