Page MenuHomePhabricator

Add log excerpts to revdeleted diffs
Open, NormalPublic

Description

Author: jake

Description:
Right now when an unprivileged user tries to view a deleted diff they are told that the revision has been hidden, but no why or who by. Adding a log excerpt similar to that shown when someone tries to view or create a deleted page that would show who hid the revision and the reason would greatly increase transparency.


Version: 1.16.x
Severity: normal

Details

Reference
bz21175

Event Timeline

bzimport raised the priority of this task from to Normal.
bzimport set Reference to bz21175.
bzimport added a subscriber: Unknown Object (MLST).
bzimport created this task.Oct 19 2009, 2:57 AM
aaron added a comment.Oct 28 2009, 7:55 AM

I'd assume this would then be needed on viewing deleted revisions too.

mike.lifeguard+bugs wrote:

(In reply to comment #1)

I'd assume this would then be needed on viewing deleted revisions too.

Ideally, yes.

See also bug 23730.

Nemo_bis renamed this task from Add log exerpts to revdeleted diffs to Add log excerpts to revdeleted diffs.Apr 26 2015, 6:02 PM
Nemo_bis set Security to None.
Scott added a comment.Jan 3 2016, 9:22 PM

See also bug 23730.

This is now T25730.

Restricted Application added subscribers: JEumerus, Aklapper. · View Herald TranscriptJan 3 2016, 9:22 PM
Scott added a comment.Jan 3 2016, 9:33 PM

jake wrote in the description of this bug:

Right now when an unprivileged user tries to view a deleted diff they are told that the revision has been hidden, but no why or who by.

More accurately, both privileged and unprivileged users see "Details can be found in the deletion log for this page.", which links to that log. However as you don't know when the revdel took place, it can impossible for an unprivileged user to know who did the action, and merely very difficult for a privileged user who can tediously comb through all the RevDel info.

I would suggest that as a simplistic workaround that that the time of RevDel be added to the diff message, changing the part of the current message that reads

One or both of the revisions of this diff has been removed from the public archives.

to

One or both of the revisions of this diff was removed from the public archives at 22:26, 19 November 2015.

That way any user can identify the action in the log. No change to Special:Log would need to be made.

the time of RevDel be added to the diff message

That would be useful, but still requires a (simple) message change.

Meno25 removed a subscriber: Meno25.Feb 19 2016, 5:50 PM