Merge "Restore gray coloring for autocomments"

Merge "Don't link wikilinks in section heading autocomments"

Merge "Restore old HTML structure for history section links"

Merge "RevisionStore: Avoid exception on prev/next of deleted revision"

Restore gray coloring for autocomments

The main criticism of T165189 has been the "sea of blue" it causes. Given that
the most important part is the actual comment, other elements like the auto-
comment should be deemphasized (per Anomie).

This won't bring back exactly the same color scheme as before, as now the
arrow will also be colored gray instead of blue.

I don't think this is a permanent solution, and we should continue discussing
and iterating on how to address the prominence vs. accessibility issues.

It was also noted that the current gray is not fully accessible (T125657),
which should be addressed in a separate patch.

Bug: T165189
Change-Id: Ic8903a539017a3fd3d47627f8485caed09fce482

Don't link wikilinks in section heading autocomments

Previously, a manually constructed autocomment in the form of
/* [[Some link]] */

would create a link to the the section, and then the "Some link" page.

After T165189 was implemented, the entire autocomment is now a link to
the section, so we're creating links inside of a link...which is
problematic. In most contexts (history and watchlist particularly), the
section link is more important than the title in the section heading, so
that's what we'll favor here.

It's worth noting that this situation is a manually created edge case.
Even if the section heading is a wikilink, the edit summary will
autofill a section autocomment without the double brackets.

We'll now render the double brackets ([[...]]) and not link them. This
is what the user literally typed, and matches the existing practice of
rendering templates in section headings with their literal syntax. And
as a bonus, it's still possible for user scripts such as wikEdDiff to
turn the rendered double brackets into a real link if users want.

Bug: T165189
Change-Id: Ib10679edd76c72a60d7e1c89fc8454166e34c463

Restore old HTML structure for history section links

A follow-up to 0a8e16d7cfe6c thanks to Anomie's code review.

The section title is now inside <span dir="auto"> and <span
class="autocomment">, as before. $wgLang->getDirMark() between the arrow
and the text was restored.

Given the comment
 /* External links */ removed bogus entries
the HTML before 0a8e16d7cfe6c looked like:
 <a href="#External_links">→</a>‎<span dir="auto"><span
 class="autocomment">External links: </span> removed bogus entries</span>
after this change, it will look like:
 <span dir="auto"><span class="autocomment"><a
 href="#External_links">→‎External links</a>: </span> removed bogus
 entries</span>

The issue of having links be inside other links will be addressed in a
separate patch.

Bug: T165189
Change-Id: I31d87a87ccaf50de58fdd0621c46133b2881b490

Merge "Fix unexpected return type of User::idFromName()"

Merge "Delete unused variable"

Merge "mail: Correct return value of sendPersonalised() method"

Merge "Delete always true condition"

Merge "Delete unused variable"

Delete unused variable

Found by PHPStan.

Change-Id: If985b06fcc9e37b8747da69d1aac63a5ee944aef

Merge "media: Add missing return to MediaTransformOutput::streamFile"

mail: Correct return value of sendPersonalised() method

If email is not successfully sent, return Status (with appropriate
status error) and also, if successfully sent, return Status::newGood().

Change-Id: I602e30641b88614087d9aff83cf8c93e32af7b81

Merge "Delete always true condition"

Merge "[MCR] Introduce SlotRoleHandler and SlotRoleRegistry"

media: Add missing return to MediaTransformOutput::streamFile

Change-Id: Icf3999ae1a70dc2e702680d9b8c542c0f16f3a17

Merge "htmlform: Fix multiple PHPDoc annotations in htmlform module"

Merge "block: Fix PHPDoc comments to match function signature (or method)"

Fix unexpected return type of User::idFromName()

The user_id is an unsigned integer in the database. But not all database
abstractions we use are guaranteed to return integer values as PHP
integers. Sometimes it's a string and needs an integer cast first.

Want proof? Search for usages of this method. Almost all add an (int)
cast. This is weird and should not be necessary.

Change-Id: If1d706f73350fca5b3a0f1e0de59e4518162445b

htmlform: Fix multiple PHPDoc annotations in htmlform module

- Add @param annotation for $params variable for __construct() method
  in the HTMLExpiryField class.
- Add @throws annotation for getOptionsOOUI() method that throws a
  MWException and remove "@return array" as this method only throw an
  exception now.

Change-Id: I292f6ae04b840f6f6f74b7c92f834f056659fbe2

Merge "debug: Correct $level PHPDoc param datatype in SyslogHandler.php"

export: Fix return value of write() function in DumpOutput.php

Per http://php.net/manual/en/function.print.php, print always returns
1 (integer value of one) but write() doesn't actually return this value
to it's caller. So "@return bool" in this case doesn't make sense as
one will think it's returning a bool type. `write()` only takes a string
and prints it, and if we really want to return it's value (in this case),
it will be "@return int" not data of type bool.

Change-Id: I45b4a157cde2f768d77dda76d6ae0caa47e28f20

debug: Correct $level PHPDoc param datatype in SyslogHandler.php

In Monolog\Logger (Logger.php), the logger levels are constants (interger values).
Logger::DEBUG is a constant (int) with value 100 which depicts that error logging
level. Datatype of these values; DEBUG, INFO, WARNING, etc are all integers not
strings.

Change-Id: I1cc67381bc64800241f5f9b7232ffe5419388d8a

block: Fix PHPDoc comments to match function signature (or method)

- Add @param annotation for $row to newFromRow() method in Restriction
  interface declaration.
- Remove @param annotation for $option not used in loadByBlockId() method
  signature.

Change-Id: Iec49b9541ea06da9f497e45bd35df26f8124aecf

Merge "Delete always true condition"

mediawiki.util: Drop jsMessage(), deprecated since 1.20 and mostly unused

This also means that mediawiki.notify is no longer loaded.

Bug: T193901
Change-Id: Ifa222ca05ec913c140284ee0113c0d7827215130

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

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

This reverts commit b1e4ca70b8e375d52c35b46fb3157fb852948263.

Introduced a breaking change, and didn't follow the deprecation policy.

Bug: T210953
Change-Id: I11806165912d9188261d3c4c37dbae5cfd2865ec

Localisation updates from https://translatewiki.net.

Change-Id: I6e2615bdd5a2aa9be73317d83770e61bc2b6b450

Delete always true condition

The typehint doesn't allow null here.

Found by PHPStan.

Change-Id: I3e30bf49e778c42e8313a5b6de3d943ed7002c81

Delete unused variable

Found by PHPStan.

Change-Id: I416069e4199e1cd3cf7a98a32cc1cb09c1525093

Delete always true condition

Found by PHPStan.

Change-Id: I3198b9ab568f469e32b3a511d8a0506fcc030fee

Delete always true condition

The typehint doesn't allow passing null here.

Found by PHPStan.

Change-Id: I978469c0df584968ab6d1b54c9ebff05c3222054

Delete always-true condition

Found by PHPStan.

Change-Id: Ibefd9d5d1b374448650a36702db18144f5feddae

Merge "Use our fork of less.php"

Merge "Add $wgReauthenticateTime to extension.json"

Merge "SpecialRevisiondelete: Do not pass unused parameter"

Merge "Do not pass unused parameter"

Merge "Do not pass unused parameter"

Use our fork of less.php

Supports PHP 7.3, among other things

Bug: T206975
Depends-On: I06f1e43b0f12f9b56f9dff55a92b5ae4546acfcc
Change-Id: I84c5ed8ac0ae1c382ba035b37dc88ccd69494ac3

Merge "Do not pass unused parameter"

Merge "Do not pass unused parameter"

Merge "RedisBagOStuff: Do not pass unused parameter"

Merge "Remove duplicate keys from arrays"

Merge "media: Do not pass unused parameter"

Merge "Do not pass unused parameter"

Merge "export: Do not pass unused parameter"

Merge "rdbms: Pass required parameter"

SpecialRevisiondelete: Do not pass unused parameter

Found by PHPStan.

Change-Id: Iba7020f6bf793e525b4d82c286064d05609eef2f

Merge "Remove unused use in anonymous function"

Merge "Remove unused parameter"

Merge "Use correct letter case"

Merge "Remove unused use in anonymous function"

Merge "Change typehint callback to callable"

Merge "Change typehint callback to callable"

RedisBagOStuff: Do not pass unused parameter

Found by PHPStan.

Change-Id: I20fe8ca713362d115d620a22e04c5dc984ae279e

media: Do not pass unused parameter

FileRepo::getHashPath() takes a $name parameter, but File::getHashPath()
doesn't.

Found by PHPStan.

Change-Id: I9cbcfd206249f4be7b50f2b104c797606bc4406a

export: Do not pass unused parameter

Found by PHPStan.

Change-Id: I0e4971a3d5a4170ada776cd88ea664806f23c6ed

rdbms: Pass required parameter

Found by PHPStan.

Change-Id: I8fbb64f7b5058b36381bccd5bc6a9d095b9ace15

Merge "Remove unused use in anonymous function"

InfoAction: Add missing @param type

null is explicit supported and in use.

Change-Id: I495ca2eacb5359c8bf88c094db39fa44ca5866e1

WebInstallerUpgrade: Simplify return paths

* Avoid else after return.
* Combine redundant return values.

Change-Id: If87f5b2163abc7867ce83081aa6ab9e358869060

Localisation updates from https://translatewiki.net.

Change-Id: I849a483d1800b2b580f64d05f3f12453afbf7549

Fix typo 'defintion'

Bug: T201491
Change-Id: I2b6b87761e6e01b982feaccd98f08f66a1888a16

Merge "Fix return typehint"

Merge "Remove unused use in anonymous function"

Merge "Use correct letter case"

Do not pass unused parameter

Found by PHPStan.

Change-Id: I4a73279b6956b9819e609536ae6774872f1736f4

Do not pass unused parameter

Found by PHPStan.

Change-Id: I6c33aa29af02b7a5ca01e2e20103e4a873cb722a

Do not pass unused parameter

Found by PHPStan.

Change-Id: I48a613d94624ef4f8c533b98b8c128b4cc9c47f5

Do not pass unused parameter

Found by PHPStan.

Change-Id: Icf9f1c87ff2690b7a16c99418cdb1dae9d97ce25

Do not pass unused parameter

Found by PHPStan.

Change-Id: I760e7382931ac00317319bee5223b0230b24bb7d

Remove unused parameter

Found by PHPStan.

Change-Id: I0480146fd7fce1d39db904f1c4b6dfa8314a74fe

Merge "Changing "===" on secrets to hash_equals to protect from timing attacks."

Changing "===" on secrets to hash_equals to protect from timing attacks.

Bug: T207777
Change-Id: I1e12ef94f455f96b4d70af27a315414500c709ab

Merge "Message: Throw if given invalid serialized data"

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

Use correct letter case

Found by PHPStan.

Change-Id: I31619e0fb1819939b5b64ed01c6959668b86870c

Use correct letter case

Found by PHPStan.

Change-Id: Iaff53951e1e49b9bb99f7bf9da5a228765ead7f3

Remove unused use in anonymous function

Found by PHPStan.

Change-Id: I88247514703820a6afb7c9861dcf64326aa32d19

Remove unused use in anonymous function

Found by PHPStan.

Change-Id: I50d27a50806048697dcac8e24712a10a6c7e7c16

Remove unused use in anonymous function

Found by PHPStan.

Change-Id: Id2dd9996fd4d78da930ee0cc802bb07b66a5f084

Remove unused use in anonymous function

Found by PHPStan.

Change-Id: Ic00d2b7b42c7c75b673601eabe947b0f380a3399

Change typehint callback to callable

Found by PHPStan.

Change-Id: I09f21da69ad9b9357cee85a47717dbe1bca04070

Change typehint callback to callable

Found by PHPStan.

Change-Id: I77877a18131bd69996bad07f2ee1c5f3ba3ba2e7

Fix return typehint

Found by PHPStan.

Change-Id: Ia5280815f9dda25788de0438af1468adb8c915a6

Localisation updates from https://translatewiki.net.

Change-Id: Ia2a54ab49aa88e2450c66e91385ecfaac29312ef

[MCR] Introduce SlotRoleHandler and SlotRoleRegistry

These new classes provide a mechanism for defining the
behavior of slots, like the content models it supports.
This acts as an extension point for extensions that need
to define custom slots, like the MediaInfo extension
for the SDC project.

Bug: T194046
Change-Id: Ia20c98eee819293199e541be75b5521f6413bc2f

Merge "Special:Preferences: Add example value as placeholder to timezone offset"

Merge "Migrate Api modules from tag_summary table to change_tag"

Localisation updates from https://translatewiki.net.

Change-Id: Ie38cd1c54843a40b6b366ae1945403df102516fe

Fix RevisionDeleteUser rev_actor query for MySQL

MySQL uses an extremely bad plan for the update with subquery here.
For the time being let's split out the subquery.

This'll break if the user has too many revision rows, but on the other
hand all the existing queries here will probably break in the same way
so let's leave that for later.

Bug: T210628
Change-Id: Ia3eb41b43b96c506349279e19743c41530969636

Merge "build: Upgrade stylelint-config-wikimedia to 0.5.0 and make pass"

Merge "Use ParserCache in CategoryMembershipChangeJob"

Merge "Add stats for block errors on create/edit actions"

build: Upgrade stylelint-config-wikimedia to 0.5.0 and make pass

Change-Id: I648375927bace5fc21bd842268c645570759ce85

Merge "Drop ct_tag column from change_tag table"

Merge "Drop $wgChangeTagsSchemaMigrationStage"