Move RELEASE-NOTES-1.32 (from REL1_32) into HISTORY

REL1_32 has gone out of the door!

Change-Id: Id93f97d73b5213935379c8ab62724897504adaad

Merge "build: Remove 'stylelint' package as it gets pulled-in by 'stylelint-config-wikimedia'"

Localisation updates from https://translatewiki.net.

Change-Id: I707519be5cbb133b724b02d0659fe35ce545df82

build: Remove 'stylelint' package as it gets pulled-in by 'stylelint-config-wikimedia'

Change-Id: I734ebf6c18a3266bec2d9c9e38e827c3ec412586

Merge "maintenance: Fix detection of bad hooks (wfRunHooks -> Hooks::run)"

Merge "filerepo: Remove begin/done comments from LocalFile"

maintenance: Fix detection of bad hooks (wfRunHooks -> Hooks::run)

It was still looking for wfRunHooks, which no longer exists
as of MediaWiki 1.32.

After this, it is now able to find one bad hook:

> Unclear hook calls:
> - Hooks::run( $action . 'ArticleComplete', [ .. ] ); # SpecialEditWatchlist

Also, remove the matching of wfRunHooks generally, given it no
longer exists. And also remove the historic notes from hooks.txt.

Change-Id: I4ac52ed75fb99d7775d4b4755e3f0871003d70a8

Add missing message for OOUI v0.30.0

Follow-up to 221b03c2436dd99f395e22e09fd356e4d380e04d.

Change-Id: I18193b191c516c14e32f0959dee2ac4c78564ce8

Merge "createAndPromote: use AuthManager::autoCreateUser"

Merge "Fix slow queries in migrateActors.php"

Merge "Improve readability of LogFormatter::makePageLink"

Merge "Don't check for LIBXML_PARSEHUGE presence"

Merge "Remove a CURLOPT_CONNECTTIMEOUT_MS check"

Localisation updates from https://translatewiki.net.

Change-Id: I9ee05968b2a310c032969eaf5f52f703b796ba16

Fix malformed output of block logs

Avoid adding <span> to expiry parameter whenever block log is
stored as plaintext.

Bug: T208523
Change-Id: Id3fcdfda7311484911ae1f57ec262a2c6e6ab00b

filerepo: Remove begin/done comments from LocalFile

I argue that these comments don't provide any additional information.
I think the code explains itself already.

Change-Id: I7b495dfde5e8b27a4cdd4bfcf84d388ffc2db5e3

Improve readability of LogFormatter::makePageLink

Change-Id: I912d9626b99697d5a0394b72a56ffc8a5e7aa1bb

Fix order of arguments in ChangeTags::getPrevTags

Bug: T212703
Change-Id: I9e38c999718762daeb5c155ea8eaaa5d16dd85c1

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

Merge "Fix missing ATOMIC_CANCELABLE in MovePage::move()"

Localisation updates from https://translatewiki.net.

Change-Id: Ibe90f37243cab5598d9a1d319e0a88ae9fcf4fdb

createAndPromote: use AuthManager::autoCreateUser

Use AuthManager::autoCreateUser when creating a new user via
createAndPromote.php so that configured AuthManager providers have
a chance to perform validations and extra actions beyond local account
creation.

Bug: T212689
Change-Id: I4972507bbaf65bb542934c281c37bd8a5c7a26b7

Fix missing ATOMIC_CANCELABLE in MovePage::move()

Follow-up to I4aaa8af50d684de.

Bug: T213168
Change-Id: I0566b37117b6c69d4043e77e6368bf79fa84e325

Merge "Update the design of Special:Block"

Merge "build: Update qunitjs to 2.9.1"

Merge "Remove comments documenting constructors as "constructors""

Merge "Fix tag for partial blocks config"

Merge "Add @since tags to new public methods related to partial blocks"

Fix tag for partial blocks config

Change-Id: I421298f216bc018d1606c297b2f89ee53c8da7ec

build: Update eslint-config-wikimedia to 0.10.0

Change-Id: I2930bcabeeb7b7b2eb36063e77b26e664a691b43

Relax filter attribute filtering to allow self-referential urls

The filter attribute will often have things like filter="url( #foo )"
These local to the file filters in svgs should be fine (We already
disallow non-local xlink:href attributes on <filter> elements). In
fact, users can already do the exact same thing by doing:
style="filter: url( #foo )"

Bug: 67044
Change-Id: Ib25328c160c0d5ea7e01dc84616b76e1b9dcd0eb

Add @since tags to new public methods related to partial blocks

Bug: T210369
Change-Id: I20197ceee8e5e43fac20addc494b841519b915c8

Remove comments documenting constructors as "constructors"

Such comments are cruft and do not add anything that's not already
obvious from the code.

Change-Id: I42068b0c6c904bc4ee38f1700e0ea14c3d659a25

Merge "HTMLSelectLanguageField: Add explicit sort"

Merge "Revert "Revert "Title::checkUserBlock should call User::isBlockedFrom for every action"""

Add me to .mailmap and remove duplicate name

Change-Id: I39de232fc6d3986ce36e205d89523fe3c8b52500

build: Update qunitjs to 2.9.1

Code and integrity from <https://code.jquery.com/qunit/>.
Release notes at <https://github.com/qunitjs/qunit/releases>.

Highlights for the browser version:

- [2.7.0] Reporter: Remove cost of DOM size when
  using "hidepassed=true".
- [2.7.0] Reporter: Use perf.now() for improved accurracy
  of unit test durations.
- [2.7.0] Reporter: Make debugging tests in a browser
  easier by adding timeline markers for DevTools.

Highlights for the npm package / CLI version:

- [2.9.0] CLI: Reduce dependency tree size
               from 143 packages to 9 packages.

Change-Id: I38408d90765cd18d5dd6952b8b6b30cbfa0c7ed5

Don't check for LIBXML_PARSEHUGE presence

It's been present since PHP 5.3.2.

Change-Id: I23a3c50c10e984abe6ff214fbf504ab6f6be763c

Remove a CURLOPT_CONNECTTIMEOUT_MS check

It was introduced in PHP 5.2.3.

Change-Id: I0c7e1684038c98677ceefbcfbb78641e90713002

Merge "objectcache: add more "touchedCallback" comments to WANObjectCache"

HTMLSelectLanguageField: Add explicit sort

Follow-up to I46dc6cc8f7d.

Change-Id: Ie5f8999cb6a24d394ceb1cf9b92facbd86431960

Localisation updates from https://translatewiki.net.

Change-Id: I4e4b7c8958b5c5626a9f5dc32f115c13f21eb8be

WikiPage::doUpdateRestrictions checks DB_MASTER

This would prevent double entries in logs when the same protection
is applied twice within a few minutes. Trying out usage of the
parent patch's new functionality.

Bug: T210983
Change-Id: Ibd6e481db2faa5955f52f36d5d7368adc8747c9c

Update the design of Special:Block

Update the design now we have the extra options for
partial blocks.

Bug: T202773
Bug: T212964
Change-Id: I569eb2a8b15b9ad7f9f0e99bce2cd9b818a82fb4

Merge "Do not pass $archiveName two times to OldLocalFile"

objectcache: add more "touchedCallback" comments to WANObjectCache

Change-Id: I55775216154aba0042f7c4cf1cb07336aa4a222e

Merge "Pass READ_LATEST in $flags to Title::loadRestrictions"

Pass READ_LATEST in $flags to Title::loadRestrictions

To be more consistent with other calls that do the same thing
Also bypass cache when READ_LATEST is set.
Removes duplicated $this->getArticleID call.
Removes $readLatest parameter from loadRestrictionsFromRows.

Bug: T210983
Change-Id: I2340e81fda8244b31f6a3c511ca29162f21d91c9

Merge "Make user rights conflict error a bit more obvious"

Merge "Add force option to password policy"

Make user rights conflict error a bit more obvious

Bug: T113048
Change-Id: Ia3992dcb473ec428188505e22b2def75fd23f98e

Revert "Revert "Title::checkUserBlock should call User::isBlockedFrom for every action""

This reverts commit 91fc7480301c8b2404cc280511c2fc83a6c3aa5d.

Hence it reapplies I6312a36911e5b73d773452fefef7ff25b9af08a4.

The changes made by this commit are (excluding the cases
checked for earlier):
* An action can only be blocked if prevents() is true/null and
isBlockedFrom() is truthy. Previously, any action other than
'edit' or 'create' would be blocked if prevents() was true/null,
regardless of isBlockedFrom().
* If an Action can be constructed, and requiresUnblock() is
false, the action won't be blocked. Previously, requiresUnblock()
wasn't checked.

This commit was previously reverted because it exposed that
EditEntityAction::requiresUnblock in Wikibase wrongly returned
false (fixed in I99061230023da2bbd0f98190a2907ca2e9717a4c).
Other potentially similar cases were audited in T211048.

Bug: T208862
Change-Id: If74a1d422290b8c62b7a7a8922621c73c9598269

Merge "Special:UserRights: Fix handling of empty conflict check value"

Merge "Improve OutputPage::wrapWikiMsg documentation"

Special:UserRights: Fix handling of empty conflict check value

Bug: T164211
Change-Id: I6b710a25e9be29497846fafe39c916d480989a96

Improve OutputPage::wrapWikiMsg documentation

Four spaces before the code sample makes it actually intepreted as
code sample in many phpdoc parsers (e.g. PhpStorm) instead of trying
to intepret HTML tags as HTML and making a mess of things.

Change-Id: I17d09a64116fa9b1372a7fd39f2a8049dcf805b1

Localisation updates from https://translatewiki.net.

Change-Id: I6fcdf609a7ffdacc51d7e8d1cf5974c69eba7dc6

Localisation updates from https://translatewiki.net.

Change-Id: Ie3a32793aa3d108fbef8534bbfe3d25419f365d2

Localisation updates from https://translatewiki.net.

Change-Id: I5f2f62795f2efc8b375d1ea2a0d413908792ecc9

Merge "Make ContribsPager properties private again"

Make ContribsPager properties private again

Bug: T212771
Follows-Up: I2295a3fefe874a59458e6ea7edd2cfdd9e912096
Depends-On: I847cb6863be2b6b196b2a992c3dde13e0b21462a
Change-Id: I6b7d0c502ab0219de1b04c8e78e4aaffe43ea0d0

Localisation updates from https://translatewiki.net.

Change-Id: Ie96ca9a369fc56dfee770db77cd81378a9f9beda

Merge "DeletedContribsPager: Add getters for properties accessed by Flow"

Merge "Add getters for properties accessed by Flow"

Remove unnecessary calc operation in .less file

Change-Id: I3db79e58e6d590da0fc2b168d784e1f666c2337b

Merge "Always load styles for Special:Block page"

Merge "Temporarily make ContribsPager properties public to unblock Flow"

Temporarily make ContribsPager properties public to unblock Flow

This partially "reverts" Ic2367ca by explicitely marking the properties
Flow depends on as public. Before, these properties have not been
declared, but dynamically created. PHP marks dynamically created
properties as public by default.

Note that before Ic2367ca it was not even documented that these properties
exist. Flow was using undocumented code.

This revert is done to unblock the currently broken Beta cluster and
give us more time to find cleaner solutions. Some much more sustainable
solutions are discussed in T212771.

Bug: T212771
Change-Id: I2295a3fefe874a59458e6ea7edd2cfdd9e912096

DeletedContribsPager: Add getters for properties accessed by Flow

Bug: T212771
Follows-Up: I35dad85740cae47877e9eb62d179b1412099aad1
Change-Id: I9d324217cacc2d731945975b7066d4c17bd915fb

Don't hardcode guzzle dependancies

Change-Id: I09eb6c9a70c269f14e1e8309fdc572769c4fc07d

Add getters for properties accessed by Flow

Bug: T212771
Follows-Up: Ic2367ca7e8d8b2b7ae4f80ad04081a0db2821de5
Change-Id: I35dad85740cae47877e9eb62d179b1412099aad1

Localisation updates from https://translatewiki.net.

Change-Id: I78a173ff93a983f41c8baf5f3b1e739d1aa1f602

Always load styles for Special:Block page

Load the styles regardless of whether the user has
JavaScript.

Bug: T212880
Change-Id: If103aebd1ca1368ddee7322bf52f7d9d35bd950c

Merge "Remove duplicate assertion in SpecialBlockTest"

Merge "Special:Block: Align partial input field better with rest of interface"

Merge "Fix type for UserMailer::send $body param"

Merge "Add check for editing restriction widget on Special:Block"

Special:Block: Align partial input field better with rest of interface

Also include sizing foundational variables to deal with `em` rendering quirks,
aimed to be replaced by central ones in future.

Change-Id: Iba65f68b233becba6054ca068a4cf7aaee0d8e4c

Remove duplicate assertion in SpecialBlockTest

Change-Id: I55d0112a8e3acf06fdfca979552a2746ae928c4f

Fix type for UserMailer::send $body param

It is documented to be an array, so add it to the type

Change-Id: I92c56fd92f2c09f75d9370aab711b7446c31beb5

Merge "watcheditem: Remove unused import/use statement"

Add check for editing restriction widget on Special:Block

If partial blocks is not enabled, editing restriction widget
will not exist, so check before binding events.

Bug: T212808
Change-Id: I33fd7efdc24c026338e25d7c0751d31bc9a7baba

watcheditem: Remove unused import/use statement

Namespace imported but never used hence removed as it's not
necessary.

Change-Id: I788852cdbd310f4c891d1f438aac691719386044

Do not pass $archiveName two times to OldLocalFile

Check the first dozen lines in ImportableUploadRevisionImporter::import().
In all cases the $archiveName is passed as a parameter to
OldLocalFile::newFromArchiveName(), and available via
OldLocalFile::getArchiveName().

I used https://codesearch.wmflabs.org/search/?q=uploadOld to make sure
this method is not called anywhere else. I consider it "package private"
and suggest to not apply the deprecation policy on this particular code.

Change-Id: Ibe9fb55d7e0d6e87698a56f7cfbf31d3d44ba70c

Merge "Convert Special:AllMessages to use OOUI"

Merge "mailmap: list new address not detected by updateCredits.php"

Merge "Update CREDITS file to include new contributors"

Convert Special:AllMessages to use OOUI

Moved form from pager and
added new HTMLSelectLanguageField.

Bug: T117749
Bug: T134425
Change-Id: I46dc6cc8f7ddf8552a726202df136cbbff66588c

Merge "SkinApi: Do not load 'mediawiki.searchSuggest'"

Merge "Make sql.php exit status more informative"

Merge "Unbreak Pbkdf2PasswordTest"

Merge "filerepo: Replace confusing substr() with rtrim()"

Localisation updates from https://translatewiki.net.

Change-Id: Ife6a31781d2ab05655f80a1c45bdd71b008f75c5

Merge "filerepo: Avoid two pointless substr() in File"

Merge "Make use of \File::getArchiveRel to avoid code duplication"

Merge "user: Ensure returned user groups are sorted"

Add force option to password policy

Adds a way to set an array of options for a password policy. Currently
there is one option, 'forceChange', which forces the user to change
their password (if it fails the given check) before logging in.

Bug: T118774
Change-Id: I28c31fc4eae08c3ac44eff3a05f5e785ce4b9e01

Tag markers should be wrapped in parentheses on diff page

Bug: T212613
Change-Id: I87ba8a12f520bcad7354dc9ed1c9f46aa02c715f

SkinApi: Do not load 'mediawiki.searchSuggest'

Bug: T212752
Change-Id: Ia6fba6934d0dfc528ad07a159bc5fbf03e9e9f24

Merge "Sync up with Parsoid parserTests.txt"