Add missing property declarations to LocalFileTest

Change-Id: I518a7a939042110f7ca67b0697e35e2e35105a08

Localisation updates from https://translatewiki.net.

Change-Id: I3ea23299dd3d8608c829977c8aa1f3dfe2505d3d

build: Run minus-x to check for executable files that shouldn't be

Bug: T175794
Change-Id: Ia94b3e832ff21f9f76bd63924bf289cd2afdefbc

Merge "Revert "Cached HTML/JS shouldn't show two toggle links""

Merge "Mark constructors of IndexPager subclasses as public"

Merge "Improve documentation of wfDeprecated"

Merge "SpecialTags: Add <thead> and <tbody> in the sortable table"

Merge "SpecialMediaStatistics: Add <thead> and <tbody> in the table"

Merge "tablesorter: Simplify jQuery#removeClass usage by using array"

SpecialTags: Add <thead> and <tbody> in the sortable table

This avoids that module 'jquery.tablesorter' has to add the elements.

With change I0b446d18f47428d8c0c4aed78b75de16fe106218 this also avoids
a flash of missing sorting buttons while loading.

Change-Id: If6368af27606ebdb6f8d6e3bbe742706ec45e4e6

Merge "Use ContextSource::getConfig instead of globals"

tablesorter: Simplify jQuery#removeClass usage by using array

Since jQuery 3.3, .removeClass() supports arrays.
https://blog.jquery.com/2018/01/19/jquery-3-3-0-a-fragrant-bouquet-of-deprecations-and-is-that-a-new-feature/

Change-Id: I7302373357efe9fbd619cfeedfec364fd245b335

Use ContextSource::getConfig instead of globals

Change-Id: I26dd9b1efc0292bbe0044d823f639a2ac5e339dd

Use ContextSource::getConfig instead of globals in Skin.php

Change-Id: I2c1b8ab875ff8d671a7cad6c056fb1c23112e241

SpecialMediaStatistics: Add <thead> and <tbody> in the table

This avoids that jquery.tablesorter has to add the elements.

Change-Id: Iab7baa641113552720df022cd05d7c4970b4a886

Revert "Cached HTML/JS shouldn't show two toggle links"

This reverts commit 51edd7e9fa3a27e796498547044d063813886503.

All caches are now expired.

Bug: T195053
Change-Id: I16403b5cd57c3cc8acd3afaf6152e30ebb8d4a05

Merge "auth: Follow up on e907d4328dc3e"

Merge "Special:Search: Generate toggle buttons in HTML instead of JavaScript"

Special:Search: Generate toggle buttons in HTML instead of JavaScript

This change ensures that the toggle buttons are already present while
loading.

Depends-On: I41225ccdf8a95a7c501fb6eea99abbd08353f4ea
Change-Id: I3292cf48214b842542ba97730ad91a1e95d127fe

Improve documentation of wfDeprecated

Change-Id: I20b7724f0cf2b0221ae27412844e4019ae451b61

Deprecate PasswordPolicyChecks::checkPopularPasswordBlacklist

Change-Id: I4c90e361b7538efeb0c5efa7ca61aed2e468591c

Simplify default password policy

* Remove PasswordNotInLargeBlacklist => false, it's a no-op
* Deduplicate PasswordCannotMatchUsername
* Remove PasswordCannotBePopular. There is no point in using both
  PasswordCannotBePopular (top 10K passwords from SecLists) and
  PasswordNotInLargeBlacklist (top 100K passwords from SecLists),
  and the CDB lookup is probably not that cheap.

Change-Id: I868846931988b88f06800e42861e82e2bb0bc427

Correct $magicWords for sa.wiki

Bug: T102320
Change-Id: I4ef78dc7a41916a9af6aa259de455e3948662913

Localisation updates from https://translatewiki.net.

Change-Id: If00b12af3d4346114260c9e4312bcab489e9a622

Fix comments in language class tests

* Add `@covers LanguageGa`.
* Language code `bs` is for "Bosnian (bosanski)" and not for "Croatian
  (hrvatski)".

Change-Id: I605bdd254518dd708343e36a2dee65dd0aa17b63

Fix typo "maintainance"

Bug: T201491
Change-Id: I8724206404a461eaace3b60f91c7704bd6581ee2

Merge "Update OOUI to v0.30.0"

Merge "Add 'suppressrevision' right to new 'oversight' grant"

Add 'suppressrevision' right to new 'oversight' grant

Change-Id: Ic7eeff94ea533670c85c9e82801f38fded9e7a30

Update OOUI to v0.30.0

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

Bug: T150071
Bug: T150781
Bug: T183271
Bug: T190581
Bug: T196208
Bug: T208644
Bug: T208821
Bug: T210039
Bug: T210914
Bug: T211154
Bug: T211223
Bug: T212064
Bug: T212068
Bug: T212159
Bug: T212186
Bug: T212299
Bug: T212389
Depends-On: I00d3fa932de968019c6ad676237ed5e5077fb961
Change-Id: I00d3fa932de968019c6ad676237ed5e5077fb962

eval.php: write error messages to stderr

Error messages should go to stderr not stdout.

Bug: T121597
Change-Id: Ibde26059982d1e621a104694343682a2032ac844

Merge "XmlSelect: Fix PHPDoc param comments for addOption() & formatOptions()"

XmlSelect: Fix PHPDoc param comments for addOption() & formatOptions()

Change-Id: I60f1bd78b3820510f9fbc497e1c703f9da7fd900

Merge "Mark protected IndexPager properties also as protected in subclasses"

Merge "Avoid expensive array_shift where possible"

Merge "Revision: Assert that $mRecord is never null in Revision"

Merge "Linker: Deprecate non-Language types for $lang of tocList() and generateTOC()"

Revision: Assert that $mRecord is never null in Revision

Bug: T187153
Change-Id: I828954855aa24114ec756e45d4dfe1468b2efa27

Merge "objectcache: add expiration check callback to WANObjectCache::getWithSetCallback"

Merge "rdbms: add trace to getLagFromPtHeartbeat() log entries"

Merge "Make thumbnail image decoding async"

Mark private properties in ContribsPager as "private"

There should not be any code accessing these. The way to set these
properties is via the constructor.

Change-Id: I8f2a8e88b8dfad53d0b16bc3f73974b894a031fb

objectcache: add expiration check callback to WANObjectCache::getWithSetCallback

This is useful when the timestamps to be checked depend on the value or are stored
in the database rather than as check keys.

Change-Id: I81ab08a943ee7d2f96a132d371965501941ed37f

Remove redundant property declarations

These are already declared and initialized in the base class. No need
to repeat this.

Change-Id: I4f83202cc9076bd68fa25490bdf8b4a2913b6b04

Add missing, dynamically declared properties in ContribsPager

These are all set in the constructor, and don't need a default because
of this.

I'm also adding documentation for the class properties in this class as
good as I can (type, possible values, meaning, and behavior).

Change-Id: Ic2367ca7e8d8b2b7ae4f80ad04081a0db2821de5

rdbms: add trace to getLagFromPtHeartbeat() log entries

Change-Id: I677610c711443b787f7cea1be2dd9f01486b1324

Linker: Deprecate non-Language types for $lang of tocList() and generateTOC()

$lang still defaults to $wgLang on unset parameter.

Change-Id: I15b65fec987641885374dfef9e1229ea405f7c30

Merge "Deprecate User::getPasswordValidity()"

Deprecate User::getPasswordValidity()

Unused, the return format does not seem useful.

Also improve the documentation of $wgPasswordPolicy
and PasswordPolicyChecks.

Change-Id: Ic01e80cfefc4cfb0eee1eccc6a66942f692278a0

Add warning if you give Html::openElement a name with a space

Bug: T201747
Change-Id: I7772f1e8c0c12bdf1fd60d62e015f2ec82d8ac90

EmailNotification should provide a getter for pageStatus

Extensions that have hooks for
UpdateUserMailerFormattedPageStatus (to provide additional
pageStatus indicators) need a way to make sure that, when their
hook is called in SendWatchlistemailNotification, they only
notifications handling their pageStatus indicator.  This allows them
to fetch the protected property.

Bug: T175104
Change-Id: I2beb5b3576f2cd739dab16b9adf613c96636edd3

registration: Add release notes for platform requirements

This adds release notes for I510de1e6d80 and I6744cc0be2.

Bug: T197535
Change-Id: I563012dd7f398c4892ec8a4ecab6144bc589d441

Merge "localisation: Make PHP cache files slimmer"

localisation: Make PHP cache files slimmer

The only difference is the scalar values are no longer encoded as an
array, but just the value itself. As a result, PHP localisation cache
files are about 2/3rds of their previous size and become slightly
smaller than CDB files.

Bug: T99740
Change-Id: Iaa5e32830dc1bb710b9e0f1a681afe91e521ece9

Localisation updates from https://translatewiki.net.

Change-Id: I6c5f1888a144fff4fd129244c43441535a47fd79

Mark constructors of IndexPager subclasses as public

I believe this is uncontroversial. The constructor or the top-level
base class is public. Subclasses are not even allowed to change that.
So what this patch effectively does is basically documentation, but it
does not change anything.

Change-Id: Icd5da81de284bc4bf382bfdfd33dab428f1fab09

Merge "Fix order on Special:Contributions when timestamps are identical"

Merge "Fix the cache timestamp for forced updates."

Mark protected IndexPager properties also as protected in subclasses

I'm intentionally only touching these protected properties and nothing
else, because I believe these are uncontroversial. They should be protected
according to modern SOLID standards, and actually are declared as
protected in the base class. What this patch effectively does is documenting
this fact, but it does not change anything.

Change-Id: I625307b58bd419c4a72e9842b2b46c08ed9b4f8e

Fix order on Special:Contributions when timestamps are identical

Please see T200259#4827781 for a longer explanation why this is needed.

Bug: T200259
Change-Id: I1678f7ec994328a96eed208483421be2f82be0ce

Merge "Include block ID in unblock log"

ApiDelete: Handle batched deletions properly

When batched deletions via the job queue were added in Ie800fb5a, the
way this was reported caused ApiDelete to report an error. Instead it
should report success with appropriate signaling to the client.

Bug: T212356
Change-Id: I1ef66277e988572c6720cf3e3cb36b18530746b4

Make thumbnail image decoding async

This is a browser hint indicating that decoding of the image can be done
asynchronously (instead of synchronously during the image's onload event).
Marking decoding as async enables the browser to display any other
non-image content that might be waiting to be rendered, without waiting
for the image to be decoding. This should allow text to appear sooner in
some situations, where the browser would have otherwise waited to
bundle its rendering with an image. The final result (non-image + image
content being displayed) will happen at the same time as before.

Given how text-heavy wiki context usually is, it makes sense to allow
users to see the text sooner in some situations, so they can start reading.

https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/decoding
https://html.spec.whatwg.org/multipage/images.html#decoding-images

Bug: T212124
Change-Id: I3d738eba016bad486194f032a47ac6312f61c34e

Merge "Add editing own JSON to editmyoptions grant"

Localisation updates from https://translatewiki.net.

Change-Id: I30701a3722177d2ded82027426fb8984dc7c8f67

Merge "API: Add User-Agent to CORS allow list"

Merge "ApiBlock: Avoid PHP warning when partial blocks are enabled but not used"

API: Add User-Agent to CORS allow list

An updated spec[1] removed the User-Agent header from the list of forbidden
headers for XHR, so some clients are now requesting to use it.

[1] https://github.com/whatwg/fetch/commit/dab09b0c483

Change-Id: I43d42c3c3a6510721ff7e5fc406db797bf08ffae

Fix the cache timestamp for forced updates.

Without this patch, the forcelinksupdate parameter of ApiPurge
was inoperational, caused by the fact that RefreshLinksJob got
the original revision's timestamp in the rootJobTimestamp parameter,
instead of the time at which the new ParserOutput was created.

See <https://phabricator.wikimedia.org/T210307#4771586> for details.

Bug: T210307
Change-Id: I281d6d0ed112b35e160775e528d363ce4770990a

Merge "Use MediaWiki\SuppressWarnings around trigger_error('') instead @"

Merge "Include bot password app ID in audit data"

ApiBlock: Avoid PHP warning when partial blocks are enabled but not used

If partial blocks are available but a sitewide block is being made,
$params['pagerestrictions'] is null and implode() raises a warning.
Since null casts to the empty array, it's easy enough to work around.

Also add a test hitting this case.

Change-Id: Id7e2559d7569031b7c1228adb0c0a14b3c1527c3

Merge "Fix regular expression"

Clean ups of dropping tag_summary table

Change-Id: I92dc906d0dfd870e6b77671e5d7cfd3f56090325

Include block ID in unblock log

Also change block log to only include the ID of the block.
Including autoblock IDs just makes analysis harder, and it is not
useful when searching for autoblocks since those created when a
blocked user tries to edit are not logged this way.

Bug: T210476
Change-Id: Iff1697be416abbb807227d0629cf41346bd379c2

Merge "travis: Add PHP 7.3 (allowed failure)"

Merge "Special:AllMessages: Improve zebra striping on hover"

Localisation updates from https://translatewiki.net.

Change-Id: Idfb2f9b39fe0eacd31663f9c769be3e88c46353e

Drop tag_summary and valid_tag tables

Bug: T209525
Change-Id: I153607239145a772f5ea47eefd3340096a494b4b

ChangesList separates content elements from presentation

Special:RecentChanges and Special:Watchlist benefit from this change

Bug: T205581
Change-Id: If46d9ed904fdc6f01decbfde12ade0c47e282171

Fix copy-paste error

It's actually adding the column on the wrong table

Change-Id: I2fd8ea50f3eb4b5da04fce2ea0348a2dc6329965

Option to load restrictions from DB_MASTER

Adds a $readLatest parameter to Title::loadRestrictions that skips
the replica DBs and goes straight to the master DB to get the latest.

TODO: should I add $readLatest to all the getRestrictions* functions?

Bug: T210983
Change-Id: I834de033fb79870de7d081701693663fb6ee93d3

Merge "HTMLForm OOUI Flatlist: Reduce `margin` slightly"

Merge "MagicWordFactory: Follow-up on I5d2166f155e31900cb40c"

MagicWordFactory: Follow-up on I5d2166f155e31900cb40c

Applied comments from I5d2166f155e31900cb40c to improve on
the unit test. makeMagicWordFactory() makes passing "null"
optional (which is default, hence some cleanup for calls).

Change-Id: Ib9a096c519f0de7286aa244bf3bb842cc11740f9

Merge "ApiComparePages: Don't error with no prev/next rev"

Merge "tests: Add PHPUnit tests for methods in MagicWordFactory::class"

tests: Add PHPUnit tests for methods in MagicWordFactory::class

* testGetContentLanguage() - covers the getContentLanguage() method.
* testGet() - covers the get() method in the MagicWordFactory class.
* testGetVariableIDs() - covers the getVariableIDs method.
* testGetSubstIDs() - covers the getSubstIDs() method.
* testGetCacheTTL() - covers the getCacheTTL() method. This covers
  both paths when there is a valid and an invalid caching hint.
* makeMagicWordFactory() - a helper function for creating an object
  of the MagicWordFactory class.
* testGetDoubleUnderscoreArray() - covers getDoubleUnderscoreArray()
  method and also calls newArray() under the hood so that is also
  covered.

Change-Id: I5d2166f155e31900cb40c22fb976e81d0d545627

HTMLForm OOUI Flatlist: Reduce `margin` slightly

When provided without label, flatlists are in general modificators
logically connected to the input above as seen on Special:BlockList
for example. Reducing the `margin` slightly helps to visually
emphasize the connection.

Bug: T177683
Change-Id: Ice666e54e440f97d177580ac258dfbed1577da0a

Upgrade wikimedia/ip-set to 2.0.0

Bug: T209756
Depends-On: I21a6032defb15b77519631b251b5e577207902a8
Change-Id: I424cf8871daf1e19b9d03511ace5f09d77746145

travis: Add PHP 7.3 (allowed failure)

Change-Id: I2205ae6b3543ec8fc094c043723c73ab1d1476b5

Merge "Message: Don't include Title objects in the serialization (part 2)"

Merge "ProfilerExcimer: allow early start"

Localisation updates from https://translatewiki.net.

Change-Id: I14d75d0d131a158d36055d612ab9139586258978

Merge "DateTimeInputWidget: Fix UI/UX glitches"

Merge "Follow-up I2266ec0: Mark new method as an internal"

Merge "Set default of sleep time in populateChangeTagDef to zero"

Set API limits for page restrictions to 10

Bug: T212142
Change-Id: Ide0f2f86df2101194155be53a4b9012ddefbbcaf

Merge "Fix sql.php --json behavior"

Merge "DateTimeInputWidget: Use 'clear' icon to clear input"