From 6abe86ddd7f5a3cc364d1565100b4809f7d7c113 Mon Sep 17 00:00:00 2001 From: BuildTools Date: Wed, 2 Jan 2019 18:31:24 -0500 Subject: [PATCH] Make /quests top argument optional, fixes #614 --- src/main/java/me/blackvein/quests/Quests.java | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/src/main/java/me/blackvein/quests/Quests.java b/src/main/java/me/blackvein/quests/Quests.java index c815c01eb..311e19826 100644 --- a/src/main/java/me/blackvein/quests/Quests.java +++ b/src/main/java/me/blackvein/quests/Quests.java @@ -320,7 +320,7 @@ public class Quests extends JavaPlugin implements ConversationAbandonedListener commands.put(Lang.get("COMMAND_EDITOR"), 1); // editor commands.put(Lang.get("COMMAND_EVENTS_EDITOR"), 1); // events commands.put(Lang.get("COMMAND_STATS"), 1); // stats - commands.put(Lang.get("COMMAND_TOP"), 2); // top [number] + commands.put(Lang.get("COMMAND_TOP"), 2); // top {number} commands.put(Lang.get("COMMAND_INFO"), 1); // info commands.put(Lang.get("COMMAND_JOURNAL"), 1); // journal adminCommands.put(Lang.get("COMMAND_QUESTADMIN_STATS"), 2); // stats [player] @@ -1378,15 +1378,19 @@ public class Quests extends JavaPlugin implements ConversationAbandonedListener } private boolean questsTop(final CommandSender cs, String[] args) { - if (args.length == 1 || args.length > 2) { + if (args.length > 2) { cs.sendMessage(ChatColor.YELLOW + Lang.get("COMMAND_TOP_USAGE")); } else { int topNumber; - try { - topNumber = Integer.parseInt(args[1]); - } catch (NumberFormatException e) { - cs.sendMessage(ChatColor.YELLOW + Lang.get("inputNum")); - return true; + if (args.length == 1) { + topNumber = 5; // default + } else { + try { + topNumber = Integer.parseInt(args[1]); + } catch (NumberFormatException e) { + cs.sendMessage(ChatColor.YELLOW + Lang.get("inputNum")); + return true; + } } if (topNumber < 1) { cs.sendMessage(ChatColor.YELLOW + Lang.get("inputPosNum"));