Refactor permission handling logic in GroupManager classes

Update the `addPermission` method to delegate to `setPermission` for consistency. Modify `setPermission` to handle permission state checks and removal/addition of permissions correctly, ensuring accurate toggle behavior.

Author: david

plugin/src/main/java/net/thenextlvl/service/model/group/GroupManagerGroup.java

@@ -86,9 +86,7 @@ public TriState checkPermission(String permission) {
 
     @Override
     public boolean addPermission(String permission) {
-        if (!checkPermission(permission).equals(TriState.NOT_SET)) return false;
-        group().addPermission(permission);
-        return true;
+        return setPermission(permission, true);
     }
 
     @Override
@@ -98,8 +96,11 @@ public boolean removePermission(String permission) {
 
     @Override
     public boolean setPermission(String permission, boolean value) {
-        removePermission(!value ? "-" + permission : permission);
-        return value ? addPermission(permission) : addPermission("-" + permission);
+        var state = checkPermission(permission).toBoolean();
+        if (state != null && state.equals(value)) return false;
+        removePermission(value ? "-" + permission : permission);
+        group().addPermission(!value ? "-" + permission : permission);
+        return true;
     }
 
     @Override

plugin/src/main/java/net/thenextlvl/service/model/permission/GroupManagerPermissionHolder.java

@@ -34,9 +34,7 @@ public TriState checkPermission(String permission) {
 
     @Override
     public boolean addPermission(String permission) {
-        if (!checkPermission(permission).equals(TriState.NOT_SET)) return false;
-        user().addPermission(permission);
-        return true;
+        return setPermission(permission, true);
     }
 
     @Override
@@ -46,7 +44,11 @@ public boolean removePermission(String permission) {
 
     @Override
     public boolean setPermission(String permission, boolean value) {
-        return false;
+        var state = checkPermission(permission).toBoolean();
+        if (state != null && state.equals(value)) return false;
+        removePermission(value ? "-" + permission : permission);
+        user().addPermission(!value ? "-" + permission : permission);
+        return true;
     }
 
     @Override