From e5d62e7e1565ad42bee191997883a23445fa3cfd Mon Sep 17 00:00:00 2001 From: SirSalad Date: Mon, 28 Feb 2022 14:15:34 -0500 Subject: Fix quest command's behavior - Return if quest id arg is invalid - Tab complete quests instead of categories - Change usage message to reflect real usage --- .../leonardobishop/quests/bukkit/command/QuestCommandHandler.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/bukkit/src/main/java/com/leonardobishop/quests/bukkit/command/QuestCommandHandler.java b/bukkit/src/main/java/com/leonardobishop/quests/bukkit/command/QuestCommandHandler.java index bab1d62a..6fdce976 100644 --- a/bukkit/src/main/java/com/leonardobishop/quests/bukkit/command/QuestCommandHandler.java +++ b/bukkit/src/main/java/com/leonardobishop/quests/bukkit/command/QuestCommandHandler.java @@ -33,6 +33,7 @@ public class QuestCommandHandler implements CommandHandler { } if (quest == null) { sender.sendMessage(Messages.COMMAND_QUEST_GENERAL_DOESNTEXIST.getMessage().replace("{quest}", args[1])); + return; } if (args[2].equalsIgnoreCase("s") || args[2].equalsIgnoreCase("start")) { qPlayer.startQuest(quest); @@ -45,13 +46,13 @@ public class QuestCommandHandler implements CommandHandler { } return; } - sender.sendMessage(ChatColor.RED + "/quests q/quest (start|cancel|track)"); + sender.sendMessage(ChatColor.RED + "/quests q/quest (start|cancel|track)"); } @Override public List tabComplete(CommandSender sender, String[] args) { if (args.length == 2) { - return TabHelper.tabCompleteCategory(args[1]); + return TabHelper.tabCompleteQuests(args[1]); } else if (args.length == 3) { return TabHelper.matchTabComplete(args[2], Arrays.asList("start", "cancel", "track")); } -- cgit v1.2.3-70-g09d2