Last modified: 2008-12-14 00:42:24 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 T18375, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 16375 - FlaggedRevs blocks/disables Patroller Extension
FlaggedRevs blocks/disables Patroller Extension
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
FlaggedRevs (Other open bugs)
unspecified
All All
: Normal enhancement with 1 vote (vote)
: ---
Assigned To: Aaron Schulz
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-11-17 21:57 UTC by Melancholie
Modified: 2008-12-14 00:42 UTC (History)
1 user (show)

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


Attachments

Description Melancholie 2008-11-17 21:57:38 UTC
FlaggedRevs blocks or disables the Patrolling extension. That is all right for the main namespaces where the FlaggedRevs replace the Patroller, but when FlaggedRevs are enabled no patrolling of all the remaining namespaces is possible anymore.

It is desired to patrol the project namespace, user pages, talk pages, category pages etc. That isn't covered by the FlaggedRevs (not by default), so disabling the Patroller feature for all namespaces will result in lowered usefulness.

Do not *reduce* helpfulness of wiki features with FlaggedRevs, please.

Error message is:
 Permissions Errors

 You do not have permission to do that, for the following reason:
 The action you have requested is limited to users in the group: Sysops.
Comment 1 Aaron Schulz 2008-11-17 22:42:36 UTC
Fixed in r43650
Comment 2 Mike.lifeguard 2008-11-17 22:45:32 UTC
(In reply to comment #1)
> Fixed in r43650
> 

Does that also fix the inability to patrol new pages for wikis with FlaggedRevs which previously had new page patrolling (but not patrolling on every diff)?
Comment 3 Melancholie 2008-11-19 19:46:33 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > Fixed in r43650
> > 
> 
> Does that also fix the inability to patrol new pages for wikis with FlaggedRevs
> which previously had new page patrolling (but not patrolling on every diff)?
> 

This open question, plus: Will auto-patrolling for patrollers be re-enabled with this too?

Currently, changes - even done by SysOps - suddenly are marked in RecentChanges with <span class="unpatrolled">!</span> (red) if the page/article is not yet flagged. Actually there should be another exclamation mark for FlaggedRevs (<span class="unflagged">!</span>), let's say orange coloured (or .unflagged red but .unpatrolled orange); it just should have a different CSS class to properly differentiate FlaggedRevs stuff from Patroller stuff.
Comment 4 Melancholie 2008-12-10 03:48:19 UTC
Not sure whether it will be fixed when code is finally live, some day, but:
http://de.wiktionary.org/wiki/Spezial:Logbuch/patrol sometimes shows some automatically patrolled entries; but only those of certain users (only sporadically, without r43650 being live). I am wondering why this happens, as either no pages/edits should be patrolled actually (buggy state; this) or all (fixed state; r43650)! Most changes are not listed as patrolled there anymore (what probably was this bug).
Comment 5 Melancholie 2008-12-12 00:44:06 UTC
REOPEN, as all other namespaces still are not patrollable anymore (although r44485 is live).

Please see comments above, especially comment #4.

The option to auto-patrol own edits is missing in Special:Preferences (sysop, even when sighter status got removed)!
Even if not being a sighter, but patroller (sysop), no red exclamation marks are shown anymore (no rcid).
Comment 6 Aaron Schulz 2008-12-12 02:54:29 UTC
Fixed in r44489. Not too interested in the other stuff.
Comment 7 Melancholie 2008-12-13 23:41:12 UTC
Now the red exclamation marks do appear again for non-content namespaces :-)
But patrolling doesn't work properly!

But:
The patrol link is different from the former one. Now it is over all other comparison stuff (in the middle). Before FlaggedRevs it had been in the right column (near undo/rollback). Is this part of FlaggedRevs? Probably yes, as *all* sighters now have this link, not just sysops like before. OK, that is not the actual problem ;-) The actual problem is that the link does not have any effect for me (sighter+sysop)!

See http://de.wiktionary.org/w/index.php?title=Wiktionary:Teestube&curid=67156&diff=898366&oldid=898298&rcid=915951

When clicking on the link it will take me to the "successfully patrolled" page, but the red exclamation mark just doesn't disappear from RecentChanges! Furthermore, [[wikt:de:Special:Log/patrol]] doesn't show any patrolling of that. I still believe that comment #4 is important!
Comment 8 Aaron Schulz 2008-12-13 23:58:48 UTC
Fixed in r44552. RC interfaces were refactored a while a back and broke this. Using the more direct functions now.
Comment 9 Melancholie 2008-12-14 00:04:40 UTC
Fast ;-)
Thanks a lot :-)

Just to give you some more information, as I am wondering why also this bug was different for each user:

Further note that a user (just being sighter) mentioned that he saw red exclamation marks before the "Wiktionary:Teestube" at 22:18 (13th), but I didn't see those at that time, only one of an IP edit that was done at 00:30 (14th Dec.) [note that the other user now also just sees the red mark only for the latest change]. But the older changes actually weren't patrolled (seems to), see [[wikt:de:Special:Log/patrol]]. Related to comment #4? >>> For some users it seemed to work a little bit better, sometimes.(this is what I am still wondering about)
Comment 10 Aaron Schulz 2008-12-14 00:20:39 UTC
Perhaps it was just not logged or something?
Comment 11 Melancholie 2008-12-14 00:39:05 UTC
Might be so, but I the red exclamation mark also disappeared for the (latest) IP edit(s) when a sighter edited the page afterwards (did test this)!
Comment 12 Melancholie 2008-12-14 00:42:24 UTC
I think the issue of comment #11 has yet to be fixed also.?
Wrong behaviour of auto-patrolling?

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


Navigation
Links