Page MenuHomePhabricator

Remove [Mark as patrolled] link for already patrolled revisions
Closed, ResolvedPublic

Description

In the IRC channels the links to diffs include an &rcid= part, which was used to enable patrolling from it I guess. However, when I click any diff now it shows a [Mark as patrolled] link, even for edits admins and autopatrolled users. This makes it kind of pointless. And since this part was added to the links, [Mark as patrolled] links have started appearing even when there is no &rcid= (when the revision is not patrolled) from Special:Recentchanges. In othec words, the rcid part of the URL is superfluous, and should be removed because it decreases the usability of the links.

Sorry for not being able to explain it better, but I hope I got the meaning across.


Version: unspecified
Severity: enhancement

Details

Reference
bz17104

Event Timeline

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

matthew.britton wrote:

Wouldn't it be better to fix the other end of the issue, and not show the "mark as patrolled" link on diffs if the edit is already patrolled, regardless of what's in the URL?

I asked for the rcid part to be added in order that bots and editing tools (the main consumers of the RC feed) are able to patrol (and sight) pages. Before it was added, it was difficult to obtain this value when all you had was a page name or revision ID. It was particularly difficult for new page patrolling -- the only place the rcid was actually shown was on Special:Newpages, or on article pages arrived at by following links from Special:Newpages. This meant that in order to patrol a given page, the tool would have to load up special:newpages and scrape the rcid from it... unless the page wasn't in the top however many, in which case it would have to load more and more of it, or give up.

If this is removed, please provide some alternative method for bots and editing tools to obtain the rcid given a revision id.

Ah, I understand. The bug should then instead be "Remove [Mark as patrolled] link for already patrolled revisions"; changing title to reflect that. Note that it is only an issue when &rcid= is provided.

I filed it as part of the Patroller extension; however, I am not sure that's correct, as that extension does nat show up at http://no.wikipedia.org/wiki/Spesial:Versjon . Is the patrol feature a part of MediaWiki core now?

matthew.britton wrote:

Yes, it is part of core. There doesn't seem to be a "patrolling" component, though, so I put it under "User interface".