Option to associate a rev id to a RC log entry, allowing unpatrolled status
authorcenarium <cenarium.sysop@gmail.com>
Sat, 7 Nov 2015 19:24:33 +0000 (20:24 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Mon, 21 Dec 2015 22:14:00 +0000 (22:14 +0000)
commit0160b410bd514681b1637ccab2b7c79158fbfb29
treef11886a822ff7ca0698cc3931a6f9a93ed53cdc6
parent169b1c3537b7e488784db1dc860fdae40140ee44
Option to associate a rev id to a RC log entry, allowing unpatrolled status

This provides a mechanism to associate a revision id to an action.
For example in core, it makes sense for moves and uploads, which both
generate null revisions (also protections, but it isn't interesting if one
has patrolling in mind).

Crucially, in that case an unpatrolled status is allowed for the RC item.
So if the performer of the action is not autopatrolled, it will be displayed
as unpatrolled, and if the performer is autopatrolled, it will record an
autopatrol action.

When one associates a rev id to a type of action, one should also implement
a mechanism to patrol said action, since getting the diff for the associated
revision is not user friendly and works only if RC patrol is enabled.
This is done for uploads in If71af58719a4461f12d125455b7bef07164525ca (with
a new file patrol) and for moves in Ie0fa417feaf930c096b69521fc54d57aecd6cd51
(within RC patrol).

Extensions might possess other such actions that could benefit from patrolling.

Bug: T122089
Change-Id: I694424eca32b69e277f89d4c15183870983d0993
includes/changes/OldChangesList.php
includes/changes/RecentChange.php
includes/logging/LogEntry.php