Skip to content

Commit 1f9bd0d

Browse files
committed
Validate delete/swap kit slots are within 1-9
- Use `parseSlotInRange(..., 1, 9)` in `/deletekit` and `/swapkit` - Reject out-of-range slot arguments before command execution
1 parent 68a3328 commit 1f9bd0d

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/main/java/dev/noah/perplayerkit/commands/kits/DeleteKitCommand.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
4646
return true;
4747
}
4848

49-
Integer slot = SlotArgumentParser.parseSlot(args[0]);
49+
Integer slot = SlotArgumentParser.parseSlotInRange(args[0], 1, 9);
5050
KitManager kitManager = KitManager.get();
5151
if (slot == null) {
5252
player.sendMessage(ChatColor.RED + "Usage: /deletekit <slot>");

src/main/java/dev/noah/perplayerkit/commands/kits/SwapKitCommand.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
4646
return true;
4747
}
4848

49-
Integer slot1 = SlotArgumentParser.parseSlot(args[0]);
50-
Integer slot2 = SlotArgumentParser.parseSlot(args[1]);
49+
Integer slot1 = SlotArgumentParser.parseSlotInRange(args[0], 1, 9);
50+
Integer slot2 = SlotArgumentParser.parseSlotInRange(args[1], 1, 9);
5151

5252
if (slot1 == null || slot2 == null) {
5353
player.sendMessage(ChatColor.RED + "Usage: /swapkit <slot1> <slot2>");

0 commit comments

Comments
 (0)