Skip to content

Permissions - fix group handling #761

@ljacqu

Description

@ljacqu

I've moved the group setting method setGroup from Utils to AuthGroupHandler. There are some major issues with this feature:

  • The enum has statuses REGISTERED and NOT_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 Settings it's causing a lot of spam in console with PermissionBukkit.

Search for TODO #761 in the code.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions