removed '/bp user/group remove' adding minus permissions if permissio…

…n not present; fixes #206
weaondara · Jul 25, 2019 · a8387e2 · a8387e2
1 parent a9ae1fd
commit a8387e2
Showing 1 changed file with 0 additions and 30 deletions.
diff --git a/src/main/java/net/alpenblock/bungeeperms/CommandHandler.java b/src/main/java/net/alpenblock/bungeeperms/CommandHandler.java
@@ -601,11 +601,6 @@ private boolean handleUserCommandsPermRemove(Sender sender, String[] args)
                 pm().removeUserPerm(user, perm);
                 sender.sendMessage(Lang.translate(MessageType.USER_REMOVED_PERM, perm, user.getName()));
             }
-            else if (!user.getExtraPerms().contains("-" + perm))
-            {
-                pm().addUserPerm(user, "-" + perm);
-                sender.sendMessage(Lang.translate(MessageType.USER_REMOVED_PERM, perm, user.getName()));
-            }
             else
             {
                 sender.sendMessage(Lang.translate(MessageType.USER_NEVER_HAD_PERM, user.getName(), perm));
@@ -622,11 +617,6 @@ else if (!user.getExtraPerms().contains("-" + perm))
                     pm().removeUserPerServerPerm(user, server, perm);
                     sender.sendMessage(Lang.translate(MessageType.USER_REMOVED_PERM_SERVER, perm, user.getName(), server));
                 }
-                else if (!srv.getPerms().contains("-" + perm))
-                {
-                    pm().addUserPerServerPerm(user, server, "-" + perm);
-                    sender.sendMessage(Lang.translate(MessageType.USER_REMOVED_PERM_SERVER, perm, user.getName(), server));
-                }
                 else
                 {
                     sender.sendMessage(Lang.translate(MessageType.USER_NEVER_HAD_PERM_SERVER, user.getName(), perm, server));
@@ -641,11 +631,6 @@ else if (!srv.getPerms().contains("-" + perm))
                     pm().removeUserPerServerWorldPerm(user, server, world, perm);
                     sender.sendMessage(Lang.translate(MessageType.USER_REMOVED_PERM_SERVER_WORLD, perm, user.getName(), server, world));
                 }
-                else if (!w.getPerms().contains("-" + perm))
-                {
-                    pm().addUserPerServerWorldPerm(user, server, world, "-" + perm);
-                    sender.sendMessage(Lang.translate(MessageType.USER_REMOVED_PERM_SERVER_WORLD, perm, user.getName(), server, world));
-                }
                 else
                 {
                     sender.sendMessage(Lang.translate(MessageType.USER_NEVER_HAD_PERM_SERVER_WORLD, user.getName(), perm, server, world));
@@ -1382,11 +1367,6 @@ private boolean handleGroupCommandsPermRemove(Sender sender, String[] args)
                 pm().removeGroupPerm(group, perm);
                 sender.sendMessage(Lang.translate(MessageType.GROUP_REMOVED_PERM, perm, group.getName()));
             }
-            else if (!group.getPerms().contains("-" + perm))
-            {
-                pm().addGroupPerm(group, "-" + perm);
-                sender.sendMessage(Lang.translate(MessageType.GROUP_REMOVED_PERM, perm, group.getName()));
-            }
             else
             {
                 sender.sendMessage(Lang.translate(MessageType.GROUP_NEVER_HAD_PERM, group.getName(), perm));
@@ -1405,11 +1385,6 @@ else if (!group.getPerms().contains("-" + perm))
                     pm().removeGroupPerServerPerm(group, server, perm);
                     sender.sendMessage(Lang.translate(MessageType.GROUP_REMOVED_PERM_SERVER, perm, group.getName(), server));
                 }
-                else if (!perserverperms.contains("-" + perm))
-                {
-                    pm().addGroupPerServerPerm(group, server, "-" + perm);
-                    sender.sendMessage(Lang.translate(MessageType.GROUP_REMOVED_PERM_SERVER, perm, group.getName(), server));
-                }
                 else
                 {
                     sender.sendMessage(Lang.translate(MessageType.GROUP_NEVER_HAD_PERM_SERVER, group.getName(), perm, server));
@@ -1425,11 +1400,6 @@ else if (!perserverperms.contains("-" + perm))
                     pm().removeGroupPerServerWorldPerm(group, server, world, perm);
                     sender.sendMessage(Lang.translate(MessageType.GROUP_REMOVED_PERM_SERVER_WORLD, perm, group.getName(), server, world));
                 }
-                else if (!perserverworldperms.contains("-" + perm))
-                {
-                    pm().addGroupPerServerWorldPerm(group, server, world, "-" + perm);
-                    sender.sendMessage(Lang.translate(MessageType.GROUP_REMOVED_PERM_SERVER_WORLD, perm, group.getName(), server, world));
-                }
                 else
                 {
                     sender.sendMessage(Lang.translate(MessageType.GROUP_NEVER_HAD_PERM_SERVER_WORLD, group.getName(), perm, server, world));