Last modified: 2011-03-13 18:05:49 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 T15751, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 13751 - Rollback to a flagged revision should flag the new revision automatically
Rollback to a flagged revision should flag the new revision automatically
Status: RESOLVED WONTFIX
Product: MediaWiki extensions
Classification: Unclassified
FlaggedRevs (Other open bugs)
unspecified
All All
: Lowest enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on: 27582
Blocks:
  Show dependency treegraph
 
Reported: 2008-04-15 14:16 UTC by Mike.lifeguard
Modified: 2011-03-13 18:05 UTC (History)
2 users (show)

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


Attachments

Description Mike.lifeguard 2008-04-15 14:16:01 UTC
When there are edits that get reverted with rollback, and the revision being reverted to is flagged to some level, the top revision should be automatically and silently flagged to the same level, as it is identical to the earlier, flagged, revision.
Comment 1 Aaron Schulz 2008-04-15 14:38:32 UTC
This is already done, unless the user doesn't have the right permissions. Otherwise, there are some exploits.

However, I'll probably make it at least basic reviewed.
Comment 2 Nemo 2010-11-30 10:25:08 UTC
Folloquing bug 13978 comment 30 I reopen this bug.
Comment 3 Aaron Schulz 2010-11-30 19:17:12 UTC
(In reply to comment #2)
> Folloquing bug 13978 comment 30 I reopen this bug.

How is this different then that bug? It is a wontfix for the same reason since it's the same request (autoreview on rollback). Comment #1 from 2008 should be ignored.

If this is for an "autoreview on rollback" right to be available to user groups, then that would not be this bug.
Comment 4 Nemo 2010-11-30 19:53:38 UTC
(In reply to comment #3)
> How is this different then that bug? 

It applies to rollback only and not to reverts. RobLa said the summary was too wide.

> It is a wontfix for the same reason since
> it's the same request (autoreview on rollback). Comment #1 from 2008 should be
> ignored.
> 
> If this is for an "autoreview on rollback" right to be available to user
> groups, then that would not be this bug.

As I said in bug 13978 comment 22, I may agree, but I would prefer such right to be implicit in "rollback" right: otherwise, it should be added to all sysop and rollbacker groups and to global rollbacker, so I don't see the point of creating a new right.
Comment 5 Aaron Schulz 2010-11-30 20:38:09 UTC
What RobLa, Chad, and I discussed was always "autoreview on rollback" (not just any revert) and that is still a wontfix.

A new right, that when granted, lets someone do that is possible though.
Comment 6 Nemo 2010-11-30 20:49:20 UTC
Can you explain why you want to create a new right?
Comment 7 Aaron Schulz 2010-11-30 20:56:33 UTC
(In reply to comment #6)
> Can you explain why you want to create a new right?

I'm not saying *I* want to, I'm saying it is acceptable if the behavior is really desired.

We don't want the software hardcoded to autoreview edits just because someone has 'rollback'. 'rollback' and 'autoreview' should be kept separate rights. Having one right do part of what another right does like this is usually bad design and historically always backfires later on.
Comment 8 Nemo 2010-11-30 21:26:57 UTC
(In reply to comment #7)
> We don't want the software hardcoded to autoreview edits just because someone
> has 'rollback'. 'rollback' and 'autoreview' should be kept separate rights.

That's exactly what we're asking, because we don't want to add "autoreview" to all rollbackers.

> Having one right do part of what another right does like this is usually bad
> design and historically always backfires later on.

I really don't understand this. "Autoreview" is about the flag of a new revision the user creates when he edits the page, isn't it? And according to http://www.mediawiki.org/wiki/Extension:FlaggedRevs#User_rights those revisions (the page says "edits", I suppose it's a typo) are marked as sighted.
But when you use rollback you restore the most recent revision apart from the rollbacked one[s]; currently, the "new" revision, identical to the restored one, is not flagged, but even if you had "autoreview" or "autoreview on rollback" (as you propose) that revision would be only "sighted", wouldn't it? 
And why should a "sighted" recent version suddenly become unflagged, or even a "good" recent revision suddenly become only "sighted"? What has changed after the rollback? 
Why should e.g. rollbacked vandalism require additional verification of the retored revisions?
Comment 9 Aaron Schulz 2010-12-02 21:20:34 UTC
What if a wiki community doesn't want rollback to make the result page revision have the same flags as the stable one for people with no actual review rights? Maybe they don't trust such people to only revert unacceptable content.

I don't want to "hard code" this behavior because it's a huge pain to pull it out if people on a wiki don't want it (requiring going to a configuration based strategy and transitioning over the existing wikis). I'd rather keep rollback and review-related rights separate in the "hard code" of the software, but leave it open configuration as needed.
Comment 10 Aaron Schulz 2010-12-02 21:23:51 UTC
Also, one could always make a "restorereviewtags" permission that fully uses the old tags of the previous stable version (thus a rollback to a "quality" revision itself becomes "quality").
Comment 11 Nemo 2010-12-02 21:26:27 UTC
(In reply to comment #9)
> What if a wiki community doesn't want rollback to make the result page revision
> have the same flags as the stable one for people with no actual review rights?
> Maybe they don't trust such people to only revert unacceptable content.

I still don't understand: even if you rollback an acceptable edit, why should the previous revision change flag?
Comment 12 Aaron Schulz 2010-12-02 21:36:55 UTC
(In reply to comment #11)
> (In reply to comment #9)
> > What if a wiki community doesn't want rollback to make the result page revision
> > have the same flags as the stable one for people with no actual review rights?
> > Maybe they don't trust such people to only revert unacceptable content.
> 
> I still don't understand: even if you rollback an acceptable edit, why should
> the previous revision change flag?

If someone rolls back good edits to a "sighted" version, and this new version (with the same text as the last "sighted" version) is also marked "sighted" automatically, then stable version is considered "up to date" and does not show up as having anything to review (e.g. Special:PendingChanges). People may or *may not* want this behavior for users just because they have rollback.

There are probably additional reasons people might not want rollback and and reviewing rights to be mixed in a hard coded way; such things have a nasty habit of revealing themselves after code is already in use for a while.
Comment 13 Nemo 2010-12-03 09:41:58 UTC
(In reply to comment #12)
> If someone rolls back good edits to a "sighted" version, and this new version
> (with the same text as the last "sighted" version) is also marked "sighted"
> automatically, then stable version is considered "up to date" and does not show
> up as having anything to review (e.g. Special:PendingChanges). People may or
> *may not* want this behavior for users just because they have rollback.

Well, if I understand correctly your point, this may be a problem only if the restored revision has some problems which the rollbacked good edit was trying to address, because in that case something has changed (the evaluation of the restored revision may change because we know something new about it); but such a use of rollback is considered an abuse everywhere, as far as I know: rollback should basically be used only for vandalism.

> There are probably additional reasons people might not want rollback and and
> reviewing rights to be mixed in a hard coded way; such things have a nasty
> habit of revealing themselves after code is already in use for a while.

As I said above, unless some wiki uses rollback in a different way than the most common and correct one (which would be very strange, but I understand that we can't assume it won't happen) there shouldn't be any problem. 
If you think that the technical tool of rollback is likely to be used with different meanings, then I think that you could create that "rollback-restorereviewtags" right, or even an additional "rollback-autoreview" right, and add it (or them) to the default rights of sysops and rollbackers, so that wikis which want to use rollback in a different way can do so, but most wikis are not annoyed by unexpected and inconsistent behaviours; this would close this bug.
Comment 14 Aaron Schulz 2010-12-03 18:56:48 UTC
(In reply to comment #13)
> Well, if I understand correctly your point, this may be a problem only if the
> restored revision has some problems which the rollbacked good edit was trying
> to address, because in that case something has changed (the evaluation of the
> restored revision may change because we know something new about it); but such
> a use of rollback is considered an abuse everywhere, as far as I know: rollback
> should basically be used only for vandalism.

Well, that too. But I was thinking of the fact that a good edit was reverted and "lost" unless someone decides to look through the page history, notices it, and restores it.
Comment 15 Aaron Schulz 2011-02-20 10:07:32 UTC
Closing this since 'rollback' won't imply 'autoreview on rollback' per above.

New bugs can be filed for things like:
(a) Site config requests to give a user group (like Rollbackers) 'autoreview'
(b) Requests for a new right that makes rollbacks autoreview the page (if rolled back to the stable version)
Comment 16 Nemo 2011-02-20 10:28:30 UTC
(In reply to comment #15)
> (b) Requests for a new right that makes rollbacks autoreview the page (if
> rolled back to the stable version)

Bug 27582.

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


Navigation
Links