Page MenuHomePhabricator

Removing someone as reviewer creates log of the user who is removed instead of the user who removed.
Closed, DeclinedPublic

Description

Hi there;

When I added myself as a reviewer, it reported that in the recent changes and logs. When another user removed me as a test, this happened:

09:34, 12 March 2014 TBloemink (Talk | contribs | block) removed himself as reviewer to article testpagina worked upon by Hulpsint as part of course Onderwijsprogramma:Testorganisatie/Testleercursus (1)
09:33, 12 March 2014 TBloemink (Talk | contribs | block) added himself as reviewer to article testpagina worked upon by Hulpsint as part of course Onderwijsprogramma:Testorganisatie/Testleercursus (1)

Note: I did NOT remove myself.


Version: unspecified
Severity: minor

Details

Reference
bz62563

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 3:02 AM
bzimport set Reference to bz62563.
bzimport added a subscriber: Unknown Object (MLST).

Confirmed.

The extension only has message logic for remove oneself as a reviewer. This comes from the logReviewersRemoval function in /includes/rows/EPArticle.php, which (unlike the methods for logging addition and removal of articles) does not check whether it is a self-unreview.

Reedy subscribed.

Extension is being removed and archived.