Last modified: 2008-11-29 19:16:19 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 T18495, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 16495 - FlaggedRevs should not unavoidably disable core RC patrolling
FlaggedRevs should not unavoidably disable core RC patrolling
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
FlaggedRevs (Other open bugs)
unspecified
All All
: Normal minor with 1 vote (vote)
: ---
Assigned To: Aaron Schulz
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2008-11-29 17:30 UTC by Happy-melon
Modified: 2008-11-29 19:16 UTC (History)
0 users

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

Attachments
adds a configuration variable to optionally allow normal RC patrolling to continue (1.69 KB, patch)
2008-11-29 18:45 UTC, Happy-melon 		Details
Add an attachment (proposed patch, testcase, etc.)

Description Happy-melon 2008-11-29 17:30:15 UTC 
An implementation proposed on en.wiki ([[Wikipedia:Flagged revisions/Trial]]) proposes to use $wgFlaggedRevsOverride = false; $wgFlaggedRevsReviewForDefault = true; to effectively limit the availability of FlaggedRevs features to specific articles.  Wanting to only 'enable' FlaggedRevs on a subset of a namespace is an acceptable and probably fairly common desire.  However, setting $wgFlaggedRevsNamespaces = array( NS_MAIN ); disables RCPatrolling using the core interface across the entire namespace. I can't see any pressing reason why RCPatrol values necessarily need to be synchronised with reviewed revisions; at the very least it should be possible to 'unlink' the two with a configuration variable.
Comment 1 Happy-melon 2008-11-29 18:45:47 UTC 
Created attachment 5544 [details]
adds a configuration variable to optionally allow normal RC patrolling to continue

Adds a new configuration variable, default true, which when false guts the function FlaggedRevsHooks::stripPatrolRights. UNTESTED
Comment 2 Aaron Schulz 2008-11-29 19:16:19 UTC 
Should be fixed in r44045
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links