-
Notifications
You must be signed in to change notification settings - Fork 552
Closed
Description
I've moved the group setting method setGroup from Utils to AuthGroupHandler. There are some major issues with this feature:
- The enum has statuses
REGISTEREDandNOT_LOGGED_IN. Unclear what the difference is; remove one or clarify the difference. - Move the group properties together. Some are currently in SecuritySettings and others are in HooksSettings.
- Properly handle if the group is empty. This should happen inside of the class itself, not everywhere before calling the method. Currently with the empty defaults from
Settingsit's causing a lot of spam in console with PermissionBukkit.
Search for TODO #761 in the code.
Reactions are currently unavailable