Page MenuHomePhabricator

Auto-accept pending revision when user self-reverts to reviewed version
Open, MediumPublic

Description

With this edit, the user reverted their edit to the previous, reviewed version, but it was not automatically accepted. This might be because the user didn't use the "undo" feature (I'm assuming tools like Twinkle use it under the hood and just change the edit summary), but it would be nice to be able to detect this.

Event Timeline

Change 329545 had a related patch set uploaded (by Cen.temp):
Refactor maybeMakeEditReviewed to handle all self-reverts, avoid db hits

https://gerrit.wikimedia.org/r/329545

Cenarium triaged this task as Medium priority.Feb 9 2017, 4:49 AM

Low if this were only that, but I've addressed some related issues in the patch set, the code needed some refactoring.

Removing task assignee due to inactivity, as this open task has been assigned to the same person for more than two years (see the emails sent to the task assignee on Oct27 and Nov23). Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome.
(See https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator.)