Cache redirects from Special:Redirect

People sometimes link these from high traffic places, so it is
important to cache in varnish.

File's with height can change so only cache that for 10 seconds.

Also change from 302 to 301.

Change-Id: I87a60c812cd1aa78a36359090c0cb8390be7183f

Merge "Remove Revision::fetchRevision()"

Update type hints and documentation of RevisionDeleteUser

Technically unrelated, but motivated by Ib59cf77.

Change-Id: I54cb9c396552d6982f5ca8463e00a0769e60a291

Localisation updates from https://translatewiki.net.

Change-Id: Ie2c639a0adbb91dc68109e58b6d74263e8ae0f0c

languages: Fix some soft type hints in the Language class

Change-Id: Ic9ffbc79c6932db97175d0a1da75df45ba3455e0

Merge "Correct whitespace and parentheses typos in BlockLogFormatter"

Merge "Fix documentation for RevisionRecord::$mPageId"

Correct whitespace and parentheses typos in BlockLogFormatter

Change-Id: I9e79e9d274e8ed1fa94966f5e1ac01f8fcd6d96b

Merge "Avoid making master connection from Skin::getUndeleteLink"

Merge "build: Don't install symfony/polyfill-* that we require"

Merge "docs: Remove 'code-coverage' placeholder directory"

Merge "Remove deprecated wgEnableParserCache"

Remove Revision::fetchRevision()

Deprecated in 1.28, hard deprecated since 1.31. No callers, never had
callers outside of core.

Change-Id: I39721d12ad0d221931d23f2807f0a87cf05f0e3a

Merge "RemexCompatMunger: Don't split p-wrapping on style/link tags"

Merge "Password: replace equals() with verify()"

Avoid making master connection from Skin::getUndeleteLink

Do this by calling Title:quickUserCan instead of Title::userCan
from Skin::getUndeleteLink.

Bug: T214735
Change-Id: I24dfd86275638e52012a5647ab3e5c848af840c2

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

Localisation updates from https://translatewiki.net.

Change-Id: If0c6e253be60e3b6b0b36f1281936b4719cb1eeb

Remove a few obscure "done" and "empty" comments

These don't add any knowledge to what is already obvious from the
code, I find.

Change-Id: Ia613b6a059f78dbeefdfd020899bd1a6e239a731

RemexCompatMunger: Don't split p-wrapping on style/link tags

<style> and <link> tags are metadata tags, they shouldn't split the <p>
tag when p-wrapping content.

Bug: T208901
Change-Id: I2ef5da68c9ccde4477d8295dfe4abf8497c5d26e

Remove some exemptions that are no longer needed

Manually identified, it's possible that there are more.

Change-Id: Ifda5ecdb12915f4005161f1f98eae5f1162fbee1

Require ClassMatchesFilename sniff to pass for most of tests/

phpunit-patch-coverage assumes that the filename matches the classname
as a performance optimization. And for most test cases, this is true. We
should enforce this with PHPCS, mostly to help developers not make
mistakes.

Test cases that have mock classes will need to ensure that the test case
class that matches the filename comes first, since that's the only class
the sniff will look at.

Tests in GlobalFunctions/ and maintenance/ are still exempted for now,
since they don't match yet.

Change-Id: Iede341504290f5ba2da1c81908069ba9d465600f

Rename FirejailCommandIntegrationTest to match class name

Change-Id: I581c0d95c38e6d22ab19d68e58e2f48c98b2379b

Rename UncategorizedCategoriesPageTest to match class name

Change-Id: Iedf32a15e0a8cb79a2823ccffa3b486a38fb1dc6

Rename tests/phpunit/includes/media files to match class names

The classes were renamed in 9bf39163, this updates the test cases to
match. Also take care of XCF while we're at it too.

Change-Id: Iaaeee93e496af6cdd610df5bc75302ecfe273f64

Use TestingAccessWrapper in FormOptionsInitializationTest

Avoids the need for an extra wrapper class just to make a property
public.

Change-Id: I08ae2fb24604a2bde352525abdcadf6251045f5b

installer: Suggest using 127.0.0.1 in the db not found error message

Its a really common error people have where they specify
localhost, not realizing that mysql does socket magic for that
value, and they need to specify 127.0.0.1 instead.

Also remove windows specific language in the help text.
Although more of an issue on windows, it is not unheard of
to happen on linux if the mysql socket is not configured.

Change-Id: Ie8199ab50d37dc2bc43dfafe2e5ef7812e9d832c

Localisation updates from https://translatewiki.net.

Change-Id: I55f7b5a4c28e9756e5432573a67a179aed70fa4c

Add Namespace Restrictions to Special:BlockList

The editing restrictions will be split by type and add a heading for each type.
The namespace will be linked to Special:AllPages with the namespace set so the
user can see what pages the user is blocked from.

Bug: T204990
Change-Id: Idb1de20c1a780562b072ea350e5ba7dd1518d177

Merge "Fix typo in tests/phpunit/includes/Revision/MainSlotRoleHandlerTest.php"

Fix typo in tests/phpunit/includes/Revision/MainSlotRoleHandlerTest.php

Bug: T201491
Depends-On: I47530e01d8061d6826deaf0b9111d77a0b269d0a
Change-Id: Iaf1abd06cd35444cf4c6c4edef65f973089579c1

Localisation updates from https://translatewiki.net.

Change-Id: Ia1d66a4d2b2aa5fb06b85b15c89cdc941dea363d

Add missing dependency on 'mediawiki.storage'

mediawiki.special.preferences.ooui/tabs.js uses mw.storage from module
'mediawiki.storage'.

Change-Id: Ia0d02a9445449645ce1054b4eb9c935f51e374c0

Remove deprecated wgEnableParserCache

One less thing to do in Setup.php.

Bug: T189966
Change-Id: Ie9b631897fda51f6db9fc4d634bcbd6bf865dfc9

docs: Remove 'code-coverage' placeholder directory

In 8 years, this is the first time I see this directory exists
and that there is a Makefile that would write to it.

I don't know if anyone uses that, but PHPUnit auto-creates this
directory as needed. Hence, it need not exist ahead of time.

It seems most contributors generate code coverage by invoking
PHPUnit directory, which requires a path to be specified.

Per https://mediawiki.org/wiki/Manual:PHP_unit_testing/Code_coverage,
this tends to be docs/coverage, not docs/code-coverage.

Change the Makefile to write there as well, as better example,
and also add it to gitignore.

Change-Id: I0a4cf716ea9b7fae89c282945b160b0dc7b2d02f

Merge "Add namespace restrictions to block's log messages"

Localisation updates from https://translatewiki.net.

Change-Id: I371f47ad51f9cce54686e9e7dad8e186274c8962

Add namespace restrictions to block's log messages

Currently, there are 3 block messages: sitewide, partial with restrictions, and
non-editing partial blocks. This will add namespace restrictions to the
partial editing blocks message type.

Bug: T204985
Change-Id: Ic17d5459e67c267fdee1fb2513d67428148ac85d

Merge "wdio-mediawiki: Release 0.3.0"

wdio-mediawiki: Release 0.3.0

Bug: T211038
Change-Id: I5036034be75a39d21234987e78edb927d6a9e7a4

Merge "Add block and unblock commands to WDIO"

Merge "Don't check anon permissions for maint scripts in autoCreateUser()"

Don't check anon permissions for maint scripts in autoCreateUser()

AuthManager::autoCreateUser() causes createAndPromote.php to give error
"Automatic account creation is not allowed." when
$wgGroupPermissions['*']['createaccount']=false is set. Anonymous user
checks should be skipped for maintenance scripts.

Change-Id: Ib61889a758e542abe991707d8b7853a25cfed8e9

http: Support callback functions in GuzzleHttpRequest

Provide backward compatibility for callback functions in
GuzzleHttpRequest, which was missing in T202110, and restore
GuzzleHttpRequest as the default provided by HttpRequestFactory.

Bug: T212175
Depends-On: I4b45e79d35252d13f714f3271b87301ca515121a
Change-Id: I60d1a034b44874f6d24a04058db264eeb565f5e1

Password: replace equals() with verify()

So far, our key derivation code assumed that it has control over
the salt used by the derivation routines, however I want to add Argon2
support and it doesn't work this way: password_hash() generates the
salt itself, and the only way to verify a password is by using
password_verify(). Current way the things are done doesn't support it
because it relies on the result of password hashing with parameters we
provide to be deterministic.

Therefore, I'm deprecating Password::equals(), as well as whole concept
of comparing Password objects - it's used only in tests anyway. It's
getting replaced with verify() that only accepts password strings.
Uses of old function are fixed with exception of a few calls in tests
that will be addressed in my Argon2 patch.

Change-Id: I2b2be9a422ee0f773490eac316ad81505c3f8571

Localisation updates from https://translatewiki.net.

Change-Id: Ia3e8a3a29198b7032e95618445aa53ab72f65cf0

Merge "Add missing `@return` PHPDoc to __sleep() function"

Merge "Clone the Title object to prevent mutations."

Add missing `@return` PHPDoc to __sleep() function

Change-Id: I97547938041264440af621694f08a067f1d0791f

Merge "HTMLForm OOUI: Reduce distance between flatlist label and widgets"

Merge "Document Uri.js constructor and methods throwing Errors"

Merge "MIME: Re-classify DjVu files as OFFICE, like PDFs, and not as BITMAP"

Merge "Make sure database update succeeds from older database versions too."

Document Uri.js constructor and methods throwing Errors

I, personally, think the fact an empty `new Uri()` call can fail with
an Error being thrown is a bug. But it seems others think this is a
feature (see Ib4dc568). Can we at least document it then? And then
somehow (not sure how) tell everybody who ever used this class to fix
their code?

Change-Id: I86ef1ccf90f9be9c61bc6a8312369ae09d622b0c

Special:AllMessages: Visually hide repeating text

Leaving it in for better screen reader navigation though.
Also, in order for this to use 'mediawiki.mixins' renaming special
stylesheet from CSS to LESS.

Bug: T209253
Change-Id: I40ac43d0a4a379a71ce78f2bd1f62b828170f4d8

HTMLForm OOUI: Reduce distance between flatlist label and widgets

With `.oo-ui-fieldLayout-header` set to `padding-bottom: 0;` label
is visually appearing equally to other “normal” label and widget
combinations.

Change-Id: I1efa11301b631304ef1d9cc7613fc139e84fa9c0

Merge "Add tests to ensure that NamespaceRestrictions are returned from Block APIs."

Clone the Title object to prevent mutations.

The Title object that is loaded from master gets reloaded from the replicas
and mutates the original object. When pages are moved, the Title no longer
exists on master, but still exists on the replicas. Cloning the object allows
the item to be loaded from the replicas, without mutating the original Title.

Bug: T210739
Change-Id: I9ad973e9a609124749909605f37bc1e1fc549585

Merge "Fix typo in apierror message"

Add tests to ensure that NamespaceRestrictions are returned from Block APIs.

Bug: T204989
Change-Id: Ibe44da41e9efa6688615dc9b0712f9713d4e8158

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 block and unblock commands to WDIO

Defaults to expiry time of never to mirror API default

This leaves it up to individual test suite to assert that the user
they use is in the right blocked/unblocked state

Bug: T211038
Change-Id: Icf94458eb9c9a1fa25e5ef9291d9fc2c54a5567c

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"