Last modified: 2008-02-28 21:21:31 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 T15186, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 13186 - Marking a diff as patrolled twice creates two entries in the patrol log
Marking a diff as patrolled twice creates two entries in the patrol log
Status: RESOLVED DUPLICATE of bug 12575
Product: MediaWiki
Classification: Unclassified
History/Diffs (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-28 20:40 UTC by heuler06
Modified: 2008-02-28 21:21 UTC (History)
2 users (show)

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


Attachments

Description heuler06 2008-02-28 20:40:38 UTC
When patrolling it is possible to mark a diff two times as patrolled.

Sometimes when there is many to patrol I forget/overlook what I have marked as patrolled. So I sometimes open yet patrolled diffs. Then there is the "mark as patrolled" link and I mark it as patrolled. A new site opens with the report the diff is marked as patrolled although it has been marked a few minutes ago.

Shouldn't there be a error message like "This diff is also marked as patrolled"?

I think the problem is the rcid. Only if the rcid is known you can mark a diff as patrolled. If you know the rcid it's possible to mark a diff twice as patrolled. Normally the rcid is vanished in the RCs when diff is patrolled. But as described above I do not refresh every time after marking a diff as patrolled so the rcid is still there.

I also want to draw your attention to Bug 8697 which would be also a nice feature. :)
Comment 1 Roan Kattouw 2008-02-28 20:50:25 UTC
I don't see why this is bad.
Comment 2 heuler06 2008-02-28 20:59:07 UTC
I have forgotten to write that it is also reported twice in the patrol log.
Comment 3 Roan Kattouw 2008-02-28 21:18:16 UTC
(In reply to comment #2)
> I have forgotten to write that it is also reported twice in the patrol log.
> 

Ah. Reopening and changing summary.
Comment 4 OverlordQ 2008-02-28 21:21:31 UTC

*** This bug has been marked as a duplicate of bug 12575 ***

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


Navigation
Links