Fix /permission clear

This commit is contained in:
Melledy
2023-10-09 07:45:58 -07:00
parent 94432f4222
commit eeeb628406

View File

@@ -14,9 +14,6 @@ public class PermissionCommand implements CommandHandler {
if (args.getTarget() == null) { if (args.getTarget() == null) {
this.sendMessage(sender, "Error: Targeted player not found or offline"); this.sendMessage(sender, "Error: Targeted player not found or offline");
return; return;
} else if (args.size() < 2) {
this.sendMessage(sender, "Error: Not enough arguments");
return;
} }
String type = args.get(0).toLowerCase(); String type = args.get(0).toLowerCase();
@@ -25,13 +22,17 @@ public class PermissionCommand implements CommandHandler {
switch (type) { switch (type) {
case "add" -> { case "add" -> {
// Add permission // Add permission
args.getTarget().getAccount().addPermission(permission); if (!permission.isEmpty()) {
args.getTarget().getAccount().addPermission(permission);
}
// Send message // Send message
this.sendMessage(sender, "Added permission for " + args.getTarget().getName()); this.sendMessage(sender, "Added permission for " + args.getTarget().getName());
} }
case "remove" -> { case "remove" -> {
// Remove permission // Remove permission
args.getTarget().getAccount().removePermission(permission); if (!permission.isEmpty()) {
args.getTarget().getAccount().removePermission(permission);
}
// Send message // Send message
this.sendMessage(sender, "Removed permission for " + args.getTarget().getName()); this.sendMessage(sender, "Removed permission for " + args.getTarget().getName());
} }
@@ -42,7 +43,7 @@ public class PermissionCommand implements CommandHandler {
this.sendMessage(sender, "Cleared permissions for " + args.getTarget().getName()); this.sendMessage(sender, "Cleared permissions for " + args.getTarget().getName());
} }
default -> { default -> {
this.sendMessage(sender, "Error: Invalid argument"); this.sendMessage(sender, "Error: Invalid arguments");
} }
} }
} }