Page MenuHomePhabricator

Please remove the 'reviewer' user group from en wikinews if possible
Closed, ResolvedPublic

Description

Hi.

Currently enwikinews does not use the reviewer group (note: on en wikinews [[n:MediaWiki:Group-reviewer]] is 'Super-reviewers' so this group is known on enwikinews as 'Super-reviewer', but internally its the 'reviewer' group). We have no need for the 'validate' right it contains. It tends to confuse users, so we'd like to ask it be removed if possible. Currently [[user:Voice of All]] is the only member of this group

We understand that this group is defined internally within the flagged revisions extension, so if there is no clean way of removing it, thats ok, but we thought we would ask to see if it was possible.

Community consensus is at:

Thanks,
Bawolff


Version: unspecified
Severity: enhancement
URL: http://http://en.wikinews.org/wiki/Wikinews:Water_cooler/proposals#Proposal_to_deprecate_the_.22reviewer.22_user_right_on_en.wikinews

Details

Reference
bz23351

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:03 PM
bzimport set Reference to bz23351.
bzimport added a subscriber: Unknown Object (MLST).

jeluf wrote:

Done.

Index: InitialiseSettings.php

  • InitialiseSettings.php (revision 755)

+++ InitialiseSettings.php (working copy)
@@ -4930,6 +4930,7 @@

'confirmed' => array( 'upload' => true, 'patrol' => true ),
'patroller' => array( 'autopatrol' => true, 'patrol' => true ),
'autopatrolled' => array ( 'autopatrol' => true ),

+ 'filemover' => array( 'movefile' => true ),

),
'dawiki' => array(
	'patroller'		=> array( 'patrol' => true, 'autopatrol' => true, 'rollback' => true, ),

@@ -5450,7 +5451,7 @@

    '+commonswiki' => array(
		'bureaucrat' => array( 'ipblock-exempt', 'bot', 'sysop', 'bureaucrat'),
		'checkuser'  => array( 'ipblock-exempt' ),
  • 'sysop' => array( 'rollbacker', 'confirmed', 'patroller', 'autopatrolled' ),

+ 'sysop' => array( 'rollbacker', 'confirmed', 'patroller', 'autopatrolled', 'filemover' ),

    ),        
    '+elwiktionary' => array(
	    'bureaucrat' => array( 'interface_editors' ),

@@ -5665,7 +5666,7 @@

'+commonswiki' => array(
     'bureaucrat' => array( 'bot', 'ipblock-exempt' ),
     'checkuser' => array( 'ipblock-exempt' ),
  • 'sysop' => array( 'rollbacker', 'confirmed', 'patroller', 'autopatrolled' ),

+ 'sysop' => array( 'rollbacker', 'confirmed', 'patroller', 'autopatrolled', 'filemover' ),

),
'+cswiki' => array(
    'bureaucrat' => array( 'autopatrolled' ),

jeluf wrote:

Please ignore the diff, it's not related to this change.