Last modified: 2013-05-02 20:14:37 UTC
I suggest to hide the edit summary in the page Undelete when shown to
non-sysops, so they will be able to see the deletion log, who edited the page
and when - but not the content of the edits, or the edit summaries.
Then I have three suggestions:
1. To grant the permission "deletedhistory" to every user (*).
2. To grant the permission "deletedhistory" to every registered user (user).
3. To grant the permission "deletedhistory" to every autoconfirmed user
I will create a patch for the edit summary, and a patch for each suggestion, soon.
Created attachment 1489 [details]
Patch for edit summaries
This patch edits the following files:
* includes/SpecialUndelete.php - uses an "if" to decide whether to show the
summary or not.
* languages/Messages.php - updates "undeletehistorynoadmin" to be more exact,
and to say that they also cannot view the edit summaries.
* languages/MessagesHe.php - same as Messages.php, and also matches
"viewdeletedpage" to "undeletepage" - not connected, but should be done for
Now I will create a patch for every option.
Created attachment 1490 [details]
Patch for permissions - option 1
Created attachment 1491 [details]
Patch for permissions - option 2
Created attachment 1492 [details]
Patch for permissions - option 3
Please check in the first patch, then choose among the second, third and fourth.
*** Bug 5237 has been marked as a duplicate of this bug. ***
Because I think (*) means anonymous users (IPs), I consider my bug
5237 ["Reactivate the permission for public history views of
deleted revisions (but without edit summaries)"] as a full
duplicate of this one.
(In reply to comment #7)
> Because I think (*) means anonymous users (IPs), I consider my bug
> 5237 ["Reactivate the permission for public history views of
> deleted revisions (but without edit summaries)"] as a full
> duplicate of this one.
"*" means every user.
Yes, I wanted to write "...means *also* anonymous users..."
So option 1 (Patch #2) would be my favourite.
This seems inordinately stagnant. I assume the patches won't apply cleanly any longer. As this is perhaps a controversial change (and was discussed only for enwiki), I'd suggest closing this pending further discussion. If then there is still a desire to implement this, rewrite things.
I don't think there's really any use for this now, and if there is, there would still need to be a way to assign the right to see summaries to groups other than sysop. Can be reopened if there is a reason to do this in the future.