Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove suppress permission for flow #1135

Closed
wants to merge 1 commit into from
Closed

Remove suppress permission for flow #1135

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Dec 27, 2016

No description provided.

@JohnFLewis
Copy link
Contributor

Any particular reason?

@ghost
Copy link
Author

ghost commented Dec 27, 2016

Deletion - which bureaucrats will hold - should be enough.

@ghost
Copy link
Author

ghost commented Dec 27, 2016

Besides, I don't want to purposely be adding user rights to groups that I will never have access to

@JohnFLewis
Copy link
Contributor

JohnFLewis commented Dec 27, 2016

Though keeping in mind that the oversight group will not rarely be used except requested essentially means the chances of this being used are near zero. The removal doesn't really gain anything but makes it harder in the extremely exceptional case it is ever deemed necessary meaning a steward has to assign it globally which just kills any sort of accountability really.

So on the local scale, no gain. On the global scale, a loss. Net positivity is negative so this isn't really worth merging.

@JohnFLewis JohnFLewis closed this Dec 27, 2016
@JohnFLewis
Copy link
Contributor

Also an interesting scenario!

You didn't add the permissions to the oversight group - the extension did. By removing them, you're adding code so you're not really removing anything but instead adding 😉

@ghost ghost deleted the patch-2 branch December 28, 2016 00:00
@ghost
Copy link
Author

ghost commented Dec 28, 2016

@JohnFLewis The extension not only added the suppression right to the oversight group - it also created a brand new "Suppressor" user group that has this right and nothing else. It isn't really necessary to have the right in 2 different groups - neither of which I have access to - so what code would overwrite and remove the new extra group? Also, what is the purpose of the "Flow Bot" group?

@ghost
Copy link
Author

ghost commented Dec 28, 2016

@Reception123 Can you look at this if @JohnFLewis doesn't get to it?

@Reception123
Copy link
Contributor

@DeltaQuad-DQ I would but I don't think removing the default groups created by the extension is possible, nor would I know how to do it if you could.

@ghost
Copy link
Author

ghost commented Dec 28, 2016

@JohnFLewis per above, do you know if removing the "Suppressor" user group is possible, since the sole permission that it has is already included in the oversight group?

@JohnFLewis
Copy link
Contributor

You simply remove all permissions from the group.

@ghost
Copy link
Author

ghost commented Dec 28, 2016

But the group would still technically exist, right? Or could it be renamed to "null group" or something?

@ghost ghost mentioned this pull request Dec 28, 2016
@ghost
Copy link
Author

ghost commented Dec 28, 2016

@JohnFLewis this failed entirely. The extra group still exists on the wiki.

@The-Voidwalker
Copy link
Member

You might want to use:
unset( $wgGroupPermissions['suppress'] );

If it still exists, you can set the following to hide the group from menus:
$wgImplicitGroups[] = 'suppress';

FYI, $wgFlowGroupPermissions doesn't exist.

@ghost
Copy link
Author

ghost commented Dec 29, 2016

@The-Voidwalker do you have any idea how to fix https://phabricator.miraheze.org/T1211

JohnFLewis added a commit that referenced this pull request Dec 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants