Page MenuHomePhabricator

Change user rights for autoconfirmed/autopatrolled on itwikivoyage
Closed, ResolvedPublic

Description

Hello. In it.wikivoyage we need two new features about user rights:

  1. change wgRemoveGroups right for autopatrolled group (grant e revoke by admin)
  2. create a new group "Patroller" with the features: autopatrol, rollback

'autoconfirmed' => array( 'patrol' => true, 'upload' => false ),
'patroller' => array( 'patrol' => true, 'autopatrol' => true, 'rollback' => true ),

wgAddGroups:
'+itwikivoyage' => array(

'sysop' => array( 'patroller', 'autopatrolled' ),

wgRemoveGroups:
'+itwikivoyage' => array(

'sysop' => array( 'patroller', 'autopatrolled' ),

Version: wmf-deployment
Severity: enhancement
URL: https://it.wikivoyage.org/w/index.php?title=Wikivoyage:Lounge&oldid=216550#Flag_di_patroller_e_autoverificato

Details

Reference
bz45638

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:13 AM
bzimport set Reference to bz45638.

-> "Wikimedia/Site requests"; +shell keyword

  • Bug 44764 has been marked as a duplicate of this bug. ***

Are you sure you want to set:

'autoconfirmed' => array( 'patrol' => true, 'upload' => false ),

If you decide on such a solution, the only difference between 'autoconfirmed' and 'patroller' would be 'rollback' => true set for the latter.

I would suppose you only want to have patrollers able to patrol edits, and 'autopatrolled' having 'autopatrol' => true; this is the current setting per bug 43524.

Now I had a closer look at this configuration, I see that there are a few wikis that actually use it (cawiki, enwiki, jawiki, nlwiki, ptwiki, svwiki and all the Italian wikis), so there's no need to answer this :-)

I'll fix it later today with a few other commits related to config changes for various language editions of Wikivoyage.

Changed merged and pushed to the live cluster.

Sorry for my personal mistake, the real meaning of the request of the community consist on setting the role authorization as follow:

'autoconfirmed' => array( 'patrol' => false, 'upload' => false ),
'autopatrolled' => array( 'patrol' => false, 'upload' => false ),

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

sorry:
'autopatrolled' => array( 'patrol' => false, 'autopatrol' => true, 'upload' => false ),

"'autopatrol' => true, " is implied?

In short what asked is, relative to current situation:

  1. revert gerrit change 56118 as regards autoconfirmed;
  2. add "patrol" right to "autopatrolled" group.

And nothing else. I've checked the discussion and, despite the confusion caused by the Italian translators, this seems to be what actually agreed upon.

Fixed in Gerrit change 57519

https://gerrit.wikimedia.org/r/57519 (Gerrit Change Ia6b835dd57e4d38194f0645cc5a41edb2615fb92) | change APPROVED and MERGED [by Reedy]

https://gerrit.wikimedia.org/r/57519 (Gerrit Change Ia6b835dd57e4d38194f0645cc5a41edb2615fb92) | change APPROVED and MERGED [by Reedy]

  • Bug 47451 has been marked as a duplicate of this bug. ***

Related URL: https://gerrit.wikimedia.org/r/60106 (Gerrit Change I82226e9cf5e2eb0ad4533377982097d1cfbd6dd0)

https://gerrit.wikimedia.org/r/60106 (Gerrit Change I82226e9cf5e2eb0ad4533377982097d1cfbd6dd0) | change APPROVED and MERGED [by Reedy]