Last modified: 2010-08-06 10:13:15 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 T26678, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 24678 - Add administrator, bureaucrat exceptions to foundationwiki
Add administrator, bureaucrat exceptions to foundationwiki
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Site requests (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-08-05 20:36 UTC by Cary Bass
Modified: 2010-08-06 10:13 UTC (History)
3 users (show)

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


Attachments

Description Cary Bass 2010-08-05 20:36:40 UTC
Please add the following section to 'wgAddGroups' and 'wgRemoveGroups' in
InitialiseSettings.php

'+foundationwiki' => array(
   'sysop' => array( 'inactive', 'confirmed'
   ),
   'bureaucrat' => array( 'bot', 'sysop', 'bureaucrat', 'import', 'transwiki', 'inactive', 'confirmed'
   ),
),
Comment 1 MZMcBride 2010-08-05 20:42:51 UTC
I don't understand the purpose or need for an "inactive" user group.

"confirmed" is apparently being implemented to give upload rights to brand-new accounts. There is probably a better way to implement this.
Comment 2 Cary Bass 2010-08-05 20:46:28 UTC
Those usergroups do, however, exist, and there are no reasons not to have these exceptions to rights assignment on foundationwiki; which is what this bug is about.
Comment 3 Casey Brown 2010-08-05 21:04:31 UTC
(In reply to comment #1)
> "confirmed" is apparently being implemented to give upload rights to brand-new
> accounts. There is probably a better way to implement this.

Hmm, is this necessary?

Under groupOverrides in InitialiseSettings, we have:

 // Fishbowls
    'fishbowl' => array(
        '*' => array(
            'edit' => false,
            'createaccount' => false 
        ),
        'user' => array(
            'move' => true,
            'upload' => true,
        ),
        'inactive' => array(
          // for show only
        ),
    ),

That should give upload and move (the normal "autoconfirmed" rights) to all users in fishbowl wikis without needing to put them in a "confirmed" group first.  If that's not working, then we need to fix that.
Comment 4 Roan Kattouw 2010-08-05 21:16:42 UTC
Cary is requesting add/rm for groups that already exist, so discussion about their usefulness is irrelevant here.

Changes are live.
Comment 5 Roan Kattouw 2010-08-05 21:58:10 UTC
(In reply to comment #3)
> (In reply to comment #1)
> > "confirmed" is apparently being implemented to give upload rights to brand-new
> > accounts. There is probably a better way to implement this.
> 
> Hmm, is this necessary?
> 
> Under groupOverrides in InitialiseSettings, we have:
[snip]
> That should give upload and move (the normal "autoconfirmed" rights) to all
> users in fishbowl wikis without needing to put them in a "confirmed" group
> first.  If that's not working, then we need to fix that.
It's working, it's just that there are more rights in the confirmed group: collectionsaveasuserpage, collectionsaveascommunitypage, autoconfirmed (= right to edit semiprotected pages) and skipcaptcha, so the group isn't technically unnecessary. The inactive group is, but that seems to be acknowledged by the comment.

Since there's nothing obviously superfluous or useless going on here, I'm gonna give this a rest unless and until it's decided something should be done about this situation (which, admittedly, is a little weird) and I'm asked to change something.
Comment 6 Cary Bass 2010-08-06 00:38:28 UTC
please update, I had the names of user groups wrong.  Please change:

'import', 'transwiki', 'inactive', 'confirmed'

to 

'importer', 'transwiki importer', 'inactive user', 'confirmed user'

Fault me for thinking the actual user group names were consistent with what I believed to have seen on other wikis.  Cary
Comment 7 MZMcBride 2010-08-06 03:25:35 UTC
(In reply to comment #6)
> please update, I had the names of user groups wrong.

Are you sure?

$ sql foundationwiki_p;
mysql> select distinct(ug_group) from user_groups;
+------------+
| ug_group   |
+------------+
| bot        | 
| bureaucrat | 
| import     | 
| inactive   | 
| oversight  | 
| steward    | 
| sysop      | 
| transwiki  | 
+------------+
8 rows in set (0.00 sec)
Comment 8 Cary Bass 2010-08-06 03:51:13 UTC
Oddly enough, those four are still grayed out.  Not sure if this is being overridden somewhere else.
Comment 9 MZMcBride 2010-08-06 04:02:34 UTC
Yeah, something's borked. http://wikimediafoundation.org/wiki/Special:ListGroupRights has (0), (1), etc. under "Administrators" and "Bureaucrats".
Comment 10 Roan Kattouw 2010-08-06 10:13:15 UTC
Fixed. I had accidentally added the AddGroups lines to groupOverrides instead.

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


Navigation
Links