Last modified: 2011-04-14 15:11:42 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T16211, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 14211 - Revert option to ignore bots
Revert option to ignore bots
Status: NEW
Product: MediaWiki
Classification: Unclassified
Page editing (Other open bugs)
unspecified
All All
: Low enhancement with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on: 11181
Blocks:
  Show dependency treegraph
 
Reported: 2008-05-21 14:41 UTC by Carl Fürstenberg
Modified: 2011-04-14 15:11 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Carl Fürstenberg 2008-05-21 14:41:25 UTC
As reverts is made to revert vandalism, I'm asking for an optional option to make revertions to ignore bot edits, both if an bot has made an edit in between the vandal edits, and also if a bot has made edits after vandal edit.
Comment 1 Roan Kattouw 2008-05-21 14:44:02 UTC
Recommending WONTFIX.

What rollback does is basically just pick an old revision to revert to, grab the text of that revision and save it (it probably should duplicate text_id's here, I'm not aware that it does), making it a much more efficient version of undo. If you want anything more sophisticated than that, use the undo feature or build the new revision text yourself.
Comment 2 Roan Kattouw 2008-05-21 15:37:21 UTC
(In reply to comment #1)
> Recommending WONTFIX.
> 
> What rollback does is basically just pick an old revision to revert to, grab
> the text of that revision and save it (it probably should duplicate text_id's
> here, I'm not aware that it does), making it a much more efficient version of
> undo. If you want anything more sophisticated than that, use the undo feature
> or build the new revision text yourself.
> 

Oops, big misunderstanding here, recommendation revoked.

I was under the impression the reporter wanted to revert vandal edits but not bot edits in between, while he really just wants to adjust the query we use to find out which revision to revert to, which would be a simple rc_bot = 0. The effect would be that

Vandal edit
Vandal edit
Bot edit*
Vandal edit
Benign edit**

Currently, rollback will roll this back to revision *, the proposal is to have it rollback to ** instead. Could also be implemented as an 'ignorebots' parameter to action=rollback (both for index.php and api.php), so we can either display two rollback links or have a user preference.
Comment 3 Roan Kattouw 2008-05-21 22:10:08 UTC
I wanted to write up a quick patch for this when I discovered the revision table doesn't have a rev_bot field. To filter bot edits, we'd have to JOIN against the recentchanges table, which means we'll never know whether a certain edit was a bot edit if it was made longer than $wgMaxAge ago. In the most common usecase (SineBot signing vandalism) that shouldn't be a problem, though, but this feature will still be impossible to implement completely reliably (i.e. in some cases it can be hard to predict offhand which revision "rollback ignoring bots" will revert to) unless we do a schema change.
Comment 4 Platonides 2008-11-06 14:42:40 UTC
Depends on bug 11181 adding a rev_bot field.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links