Page MenuHomePhabricator

Load current protection reason on action=unprotect
Open, MediumPublic

Description

So admins don't need to select or input reason again on protection changes.


Version: 1.22.0
Severity: enhancement

Details

Reference
bz46987

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:20 AM
bzimport set Reference to bz46987.
bzimport added a subscriber: Unknown Object (MLST).
liangent created this task.Apr 7 2013, 1:15 PM

Like on reblock, where the reason is also prefilled in the reason field

Its already visible in the protection log below - is this really needed?

The parallel to action=unprotect is really Special:Unblock, which likewise doesn't load the existing block reason, as unblocking (and unprotecting) shouldn't want the existing rationale pre-supplied. There is an argument to be made that action=protect should come preloaded an already-existing rationale, much as Special:Block does, but the reality is that, whereas Special:Block is where blocks are modified and is separate from Special:Unblock, unprotect and protect are the same thing.