Now using one single command for the whole plugin but with many options.

Author: ExplodingBottle

src/config.yml

@@ -51,6 +51,12 @@ helper-removed: You have removed an helper.
 non-active-world: You need to set the world ${world} as active to continue.
 near-spawn-placing: You cannot place a block that near from the spawn.
 no-drop: You are not allowed to drop items.
+command-non-existing: Please enter a correct option. Try with the help option.
+command-help-message: |+
+  ClutchesPractice Command
+  Options:
+   help: Shows this message
+   summonhelper: Allows you to summon an helper
 try-again: Try again.
 
 # prefix: The prefix to put on each message.

src/io/github/ascpialgroup/clp/ClutchesPracticePlugin.java

@@ -17,7 +17,8 @@
 
 import io.github.ascpialgroup.clp.configuration.LoadedConfiguration;
 import io.github.ascpialgroup.clp.configuration.Translations;
-import io.github.ascpialgroup.clp.listeners.CommandsHandler;
+import io.github.ascpialgroup.clp.listeners.CommandHandler;
+import io.github.ascpialgroup.clp.listeners.CommandTabCompleter;
 import io.github.ascpialgroup.clp.listeners.EventsHandlerClp;
 
 public class ClutchesPracticePlugin extends JavaPlugin {
@@ -52,14 +53,16 @@ private void loadTranslations() {
 		lTrans = new Translations();
 		lTrans.prefix = loadTranslation("prefix", false);
 		lTrans.noPermissions = loadTranslation("no-permission", true);
-		lTrans.consoleSummoning = loadTranslation("console-summoning", false);
+		lTrans.consoleSummoning = loadTranslation("console-summoning", true);
 		lTrans.punchMe = loadTranslation("punch-me", false);
 		lTrans.helperAdded = loadTranslation("helper-added", true);
 		lTrans.helperRemoved = loadTranslation("helper-removed", true);
 		lTrans.nonActiveWorld = loadTranslation("non-active-world", true);
 		lTrans.nearSpawnPlacing = loadTranslation("near-spawn-placing", true);
 		lTrans.tryAgain = loadTranslation("try-again", true);
 		lTrans.noDropping = loadTranslation("no-drop", true);
+		lTrans.commandNotExisting = loadTranslation("command-non-existing", true);
+		lTrans.commandHelpMessage = loadTranslation("command-help-message", true);
 		getLogger().log(Level.INFO, "Translations parsed.");
 	}
 
@@ -117,7 +120,8 @@ public void onEnable() {
 		loadArmorStandsConf();
 		cleanupUnusedRefs();
 		Bukkit.getPluginManager().registerEvents(new EventsHandlerClp(this, lConfig, lTrans), this);
-		getCommand("summonhelper").setExecutor(new CommandsHandler(this, lConfig, lTrans));
+		getCommand("clutchespractice").setExecutor(new CommandHandler(this, lConfig, lTrans));
+		getCommand("clutchespractice").setTabCompleter(new CommandTabCompleter());
 	}
 
 	@Override

src/io/github/ascpialgroup/clp/configuration/Translations.java

@@ -11,4 +11,6 @@ public class Translations {
 	public String tryAgain;
 	public String prefix;
 	public String noDropping;
+	public String commandNotExisting;
+	public String commandHelpMessage;
 }

…group/clp/listeners/CommandsHandler.java …lgroup/clp/listeners/CommandHandler.javasrc/io/github/ascpialgroup/clp/listeners/CommandsHandler.java renamed to src/io/github/ascpialgroup/clp/listeners/CommandHandler.java src/io/github/ascpialgroup/clp/listeners/CommandsHandler.java renamed to src/io/github/ascpialgroup/clp/listeners/CommandHandler.java

@@ -15,26 +15,31 @@
 import io.github.ascpialgroup.clp.configuration.LoadedConfiguration;
 import io.github.ascpialgroup.clp.configuration.Translations;
 
-public class CommandsHandler implements CommandExecutor {
+public class CommandHandler implements CommandExecutor {
 	private ClutchesPracticePlugin referer;
 	private LoadedConfiguration config;
 	private Translations lTrans;
 
-	public CommandsHandler(ClutchesPracticePlugin referer, LoadedConfiguration config, Translations lTrans) {
+	public CommandHandler(ClutchesPracticePlugin referer, LoadedConfiguration config, Translations lTrans) {
 		this.referer = referer;
 		this.config = config;
 		this.lTrans = lTrans;
 	}
 
 	@Override
-	public boolean onCommand(CommandSender arg0, Command arg1, String arg2, String[] arg3) {
-		if (arg1.getName().equals("summonhelper")) {
-			if (arg3.length == 0) {
-				if (!(arg0 instanceof Player)) {
-					arg0.sendMessage(lTrans.consoleSummoning);
+	public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+		if (args.length < 1) {
+			sender.sendMessage(lTrans.commandNotExisting);
+			return true;
+		}
+		switch (args[0]) {
+		case "summonhelper":
+			if (sender.hasPermission("clp.summonhelper")) {
+				if (!(sender instanceof Player)) {
+					sender.sendMessage(lTrans.consoleSummoning);
 					referer.getLogger().log(Level.INFO, "A non player has tried to summon an helper.");
 				} else {
-					Player ref = (Player) arg0;
+					Player ref = (Player) sender;
 					if (config.activeWorlds.contains(ref.getWorld().getName())) {
 						ArmorStand a = (ArmorStand) ref.getWorld().spawnEntity(ref.getLocation(),
 								EntityType.ARMOR_STAND);
@@ -48,15 +53,24 @@ public boolean onCommand(CommandSender arg0, Command arg1, String arg2, String[]
 						a.setCustomName(lTrans.punchMe);
 						a.setCustomNameVisible(true);
 						referer.registerNewArmorStandIfNotExisting(a.getUniqueId());
-						arg0.sendMessage(lTrans.helperAdded);
+						sender.sendMessage(lTrans.helperAdded);
 						referer.getLogger().log(Level.INFO, "An helper has been added.");
 					} else {
-						arg0.sendMessage(lTrans.nonActiveWorld.replace("${world}", ref.getWorld().getName()));
+						sender.sendMessage(lTrans.nonActiveWorld.replace("${world}", ref.getWorld().getName()));
 						referer.getLogger().log(Level.INFO, "Someone tried to summon an helper in a non active world.");
 					}
 
 				}
+			} else {
+				sender.sendMessage(lTrans.noPermissions.replace("${perm}", "clp.summonhelper"));
 			}
+			break;
+		case "help":
+			sender.sendMessage(lTrans.commandHelpMessage);
+			break;
+		default:
+			sender.sendMessage(lTrans.commandNotExisting);
+			break;
 		}
 		return true;
 	}

src/io/github/ascpialgroup/clp/listeners/CommandTabCompleter.java

@@ -0,0 +1,33 @@
+package io.github.ascpialgroup.clp.listeners;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandSender;
+import org.bukkit.command.TabCompleter;
+
+public class CommandTabCompleter implements TabCompleter {
+	private List<String> allPossible;
+
+	public CommandTabCompleter() {
+		allPossible = new ArrayList<>();
+		allPossible.add("help");
+		allPossible.add("summonhelper");
+	}
+
+	@Override
+	public List<String> onTabComplete(CommandSender arg0, Command arg1, String arg2, String[] arg3) {
+
+		if (arg3.length == 1) {
+			List<String> possible = new ArrayList<>();
+			allPossible.forEach(option -> {
+				if (option.startsWith(arg3[0]))
+					possible.add(option);
+			});
+			return possible;
+		}
+		return allPossible;
+	}
+
+}

src/plugin.yml

@@ -1,32 +1,37 @@
 main: io.github.ascpialgroup.clp.ClutchesPracticePlugin
 name: ClutchesPractice
-version: 1.0.1
+version: 1.0.2
 description: This plugin adds a system to learn to clutch.
 load: POSTWORLD
 author: ASCPIAL Group
 website: ascpialgroup.github.io
 
 commands:
-   summonhelper:
-      description: This command summon the helper armor stand for practice.
-      usage: /<command>
-      permission: clp.summonhelper
-      permission-message: Sorry, you need permission <permission> to do this.
-
+  clutchespractice:
+    description: This command allows you to do everything with this plugin.
+    usage: /<command>
+    permission: clp.maincommand
+    permission-message: Sorry, you need permission <permission> to do this.
+    aliases:
+      - clp
 
 permissions:
-    clp.*:
-        description: Gives access to all clutches practice commands
-        children:
-            clp.summonhelper: true
-            clp.removehelper: true
-            clp.breakblocks: true
+  clp.*:
+    description: Gives access to all clutches practice commands.
+    children:
+      clp.summonhelper: true
+      clp.removehelper: true
+      clp.breakblocks: true
+      clp.maincommand: true
     clp.summonhelper:
-        description: Allows you to summon an helper armor stand
-        default: op
+      description: Allows you to summon an helper armor stand.
+      default: op
     clp.removehelper:
-        description: Allows you to remove an helper armor stand
-        default: op
+      description: Allows you to remove an helper armor stand.
+      default: op
     clp.breakblocks:
-        description: Allows you to break blocks in ClutchesPractice active worlds.
-        default: op
+      description: Allows you to break blocks in ClutchesPractice active worlds.
+      default: op
+    clp.maincommand:
+      description: Allows you to use the main command of this plugin.
+      default: op