Page MenuHomePhabricator

Undo available for non-sysops
Closed, DeclinedPublic

Description

Author: gpvos+wkp

Description:
The recent undo implementation (Bug 6925: History patching: allow old edits to be
reversed) is available to non-sysops, while the normal undo is for sysops only. Surely
this is not intended.


Version: unspecified
Severity: normal

Details

Reference
bz8064

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:31 PM
bzimport set Reference to bz8064.
bzimport added a subscriber: Unknown Object (MLST).

robchur wrote:

I see no harm in it, since the user still has to perform a manual save, so it's
not quite like rollback.

ayg wrote:

It has much more in common with manual revert (go to old revision, click a
button, enter edit summary, save) than rollback (click a single button straight
from revision/history view), so it's only logical that mere peons should get it.