SkinApi: Do not load 'mediawiki.searchSuggest'

Bug: T212752
Change-Id: Ia6fba6934d0dfc528ad07a159bc5fbf03e9e9f24

Merge "Sync up with Parsoid parserTests.txt"

Merge "Add missing property declarations to LocalFileTest"

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 391acbfe0c93d632727ea08cd2a10b4dba8c257f

Change-Id: I0e8c154eaf118f9cd5b58d657f64d0829a65c8f0

filerepo: Replace confusing substr() with rtrim()

I just looked at this code for ten minutes to understand what
sub-string is extracted here. Not only are the numbers hard to
understand: With 0 and -1 the last character is removed – that's
really all this does. Even more problematic is the fact there is
no check, hint, not even a comment explaining *which* character is
removed. If – for whatever reason – the code above returns a string
that does *not* end with a slash, the unconditional substr() call
must destroy this string.

rtrim() shows the character.

I checked and these strings are all guaranteed to use forward
slashes, never backwards (Windows) slashes.

Change-Id: I2e17fd583982920bb8a0ca73035094099e5e5d31

filerepo: Avoid two pointless substr() in File

In both cases the slash was added one line above, just to be cut off
again.

Change-Id: I15ff6b47821033f123f555f02c42fc8984ad9800

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

Before, the string "archive/" was hard-coded in many places.

Change-Id: I62e4471711d183f37e89dc02f3709187164faf0e

Add missing property declarations to LocalFileTest

Change-Id: I518a7a939042110f7ca67b0697e35e2e35105a08

Merge "Remove unused line"

logging: Correct $logId default value in LogPager.php

$logId holds the log entry ID of some specific log and the datatype
is an integer (int) which is correct but looking at the constructor,
it does take a bool (false) which contradicts the docs above. So per
@Thiemo's advice after review of prev PS, it would be better to replace
false with 0 and leave the docs untouched.

Change-Id: Ieceb79a6cbaa248e1b7a7d3c1d51eb26bb3a2d84

Merge "Move exif messages to a separate i18n file"

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

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

This reverts commit bd2a718cf09cc5826d5a1ef27b1fbb21741e4342.

Bug: T212766
Change-Id: Ic453d7e672ad6c1e39f33f7efd104483f95f5b89

Unbreak Pbkdf2PasswordTest

Due to https://github.com/sebastianbergmann/phpunit/issues/3459 it was
looking for a wrong function name and was skipping the test even when PHP
support was present.

Change-Id: I2508f192a76275286e95bd6a06e4628d98b11737

Remove unused line

Change-Id: I4bc6ec69b84b5f29e3c4c7833697f4d2be82e29a

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

mailmap: list new address not detected by updateCredits.php

Depends-On: Id54c1832313f489d9d1857d03f1663137d04ef60
Change-Id: I48073307b0fa76b159d358084601ba803d27bad2

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

Update CREDITS file to include new contributors

Ran the updateCredits.php script to include new contributors of
MW and its extensions into the CREDITS file after a while. Keeping
the contributor list updated is useful.

Change-Id: Id54c1832313f489d9d1857d03f1663137d04ef60

Move exif messages to a separate i18n file

Bug: T167762
Change-Id: I17900ff07cd9b15f00c7118b4206cf1cdc074469

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

user: Ensure returned user groups are sorted

Without it, Special:UserRights sometimes fails with a bogus conflict error
just because groups are somehow ordered differently.

Bug: T164211
Change-Id: I9c7f51338e0849d9e134dc780eb13c542960c655

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