Page MenuHomePhabricator

Revert not detected as revert within Page History
Closed, ResolvedPublic

Description

Posted on my talk page by User:NicoScribe:

I have seen your username in https://xtools.wmflabs.org/about, so I have one remark about "Page History".

When looking at https://en.wikipedia.org/w/index.php?title=Category:Enfield_Town_F.C._players&action=history, there is one incorrect edit done by one IP user (of +15 bytes) and this edit has been reverted with the next edit. But https://xtools.wmflabs.org/articleinfo/en.wikipedia.org/Category:Enfield_Town_F.C._players#top-editors is showing that the IP user has added +15 bytes, despite the note "Added text is any positive addition that wasn't reverted with the next edit". (The tool's behavior makes sense because reverted edits are not necessarily vandalisms.) (And all vandalisms are not necessarily reverted.)

Could you correct the note, please?

Event Timeline

MusikAnimal subscribed.

The regex looks right to me: https://github.com/x-tools/xtools/blob/master/app/config/semi_automated.yml#L45

Not sure why this wasn't detected as a revert. You are correct that not every reverted edit is vandalism, but we still want to discount it as addition of content, so what we have here is a bug and not an issue with the wording.

Matthewrbowker renamed this task from Wording correction on Page HIstory to Revert not detected as revert within Page History.Sep 18 2017, 5:19 PM
Matthewrbowker raised the priority of this task from Lowest to Low.