Page MenuHomePhabricator

Rollback shouldn't patrol without log entry
Open, LowPublic

Description

As near as I can tell, rollback only marks edits as patrolled - it doesn't add any info to the patrol log itself, like who patrolled those edits (by virtue of clicking rollback) and at what time.

Documentation added on Meta:Help:Patrolled_edit, where else?


Version: 1.19.0
Severity: enhancement

Details

Reference
bz40624

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 12:50 AM
bzimport added a project: MediaWiki-Patrolling.
bzimport set Reference to bz40624.
bzimport added a subscriber: Unknown Object (MLST).

See bug 8697 comment 7 for a description of current behaviour.
I'm not sure this is considered a bug; the log already contains an entry for the auto-patrolling of the revision added by rollback.

Dalba added a comment.Nov 14 2014, 2:49 AM

I was going to get all unpatrolled edits of an article using api. To do so I first got all needed revisions ('action=query&prop=revisions&titles=...') and then compared the results with patrolled logs. ('action=query&list=logevents&leprop=details&letype=patrol&letitle= ...')

But because of this bug I won't be able to detect edits that are patrolled by rollback.

Do you know a better way to do this?

(In reply to Dalba from comment #2)

Do you know a better way to do this?

Patrolled edits are RC properties, ask the RC: [[mw:API:Recentchanges]].

(In reply to Nemo from comment #3)

(In reply to Dalba from comment #2)

Do you know a better way to do this?

Patrolled edits are RC properties, ask the RC: [[mw:API:Recentchanges]].

If he's only looking for a specific title, that becomes problematic on something like Wikipedia, since there's no title-specific filter for RC. There was one, briefly, in MW 1.14, but it was removed again in the next version.

On the dutch wikipedia, somebody was confused about not seeing the patrols either. I indeed think this is confusion, so I support logging the patrols separate as well.

Nemo_bis updated the task description. (Show Details)Apr 27 2015, 12:15 PM
Nemo_bis set Security to None.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 17 2015, 5:40 PM
Restricted Application added a subscriber: Luke081515. · View Herald TranscriptAug 6 2016, 12:35 PM