User "Bawolff" posted a comment on MediaWiki.r91561.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91561#c19266
Commit summary:

(Bug 19725) Do not include suppressed edits in the "View X deleted edits" 
message, and when doing prefix search of special:undelete.

I'm not 100% sure this is the right thing to do, see the bug for the details. 
But basically this doesn't include an edit in the count if its text is hidden 
and its hidden from admins. (Not sure if it should not be included only if 
everything is hidden). Its also weird to show people different things depending 
if they have suppress rights, without really indicating that.

Minor db note: This causes the query to no longer use a covering index. I don't 
think that matters but just thought i'd mention.

p.s. The upload page show deleted edits link is broken right now, (from before) 
I'll fix in a follow-up.

Comment:

adding tag 1.18 since relevant bug was a 1.18 blocker. (My personal opinion is 
its not really that important of a bug, and don't overly care either way if it 
gets backported)

_______________________________________________
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to