Added commands to help

pom.xml

@@ -3,7 +3,7 @@
   <groupId>de.web.basman93</groupId>
   <artifactId>KickPlayerOnJoin</artifactId>
   <packaging>jar</packaging>
-  <version>1.0.1</version>
+  <version>1.0.3</version>
   <name>KickPlayerOnJoin</name>
   <url></url>
   <build>

src/main/java/de/web/basman93/KickPlayerOnJoin.java

@@ -3,7 +3,7 @@ package de.web.basman93;
 import java.util.ArrayList;
 import java.util.List;
 
-import org.bukkit.Color;
+import org.bukkit.ChatColor;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.configuration.file.FileConfiguration;
@@ -19,7 +19,7 @@ public class KickPlayerOnJoin extends JavaPlugin implements Listener {
 	private boolean active = true;
 	private FileConfiguration config = getConfig();
 	private List<String> playerlist = new ArrayList<String>();
-	private final String prefix = Color.SILVER + "[K" + Color.GRAY + "P" + Color.SILVER + "OJ] ";
+	private final String prefix = ChatColor.GRAY + "[K" + ChatColor.DARK_GRAY + "P" + ChatColor.GRAY + "OJ] ";
 	
 	@Override
 	public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
@@ -36,12 +36,20 @@ public class KickPlayerOnJoin extends JavaPlugin implements Listener {
 				save();
 				
 			} else if(args.length > 0 && args[0].equalsIgnoreCase("help") && cop(sender, "kpoj.help")) {
-				sender.sendMessage(prefix + "--------------- HELP ---------------");
+				sender.sendMessage(prefix + "---------------------- HELP ----------------------");
-				sender.sendMessage(prefix + "/kpoj      - shows version & status");
+				sender.sendMessage(prefix + "/kpoj              - shows version & status");
-				sender.sendMessage(prefix + "/kpoj help - shows this help info");
+				sender.sendMessage(prefix + "/kpoj version      - shows version");
-				sender.sendMessage(prefix + "/kpoj off  - deactivate this plugin");
+				sender.sendMessage(prefix + "/kpoj help         - shows this help info");
-				sender.sendMessage(prefix + "/kpoj on   - activate this plugin");
+				sender.sendMessage(prefix + "/kpoj off          - deactivate this plugin");
-				sender.sendMessage(prefix + "------------------------------------");
+				sender.sendMessage(prefix + "/kpoj on           - activate this plugin");
+				sender.sendMessage(prefix + "/kpoj add <player> - add player to whitelist");
+				sender.sendMessage(prefix + "/kpoj del <player> - removes player from whitelist");
+				sender.sendMessage(prefix + "/kpoj reload       - reload plugin");
+				sender.sendMessage(prefix + "/kpoj list         - list players on whitelist");
+				sender.sendMessage(prefix + "--------------------------------------------------");
+				
+			} else if(args.length > 0 && args[0].equalsIgnoreCase("version")) {
+				sender.sendMessage(prefix + "Version: " + getDescription().getVersion());
 				
 			} else if(args.length > 1 && args[0].equalsIgnoreCase("add") && cop(sender, "kpoj.add")) {
 				for(int i = 1; i < args.length; i++) {
@@ -67,25 +75,20 @@ public class KickPlayerOnJoin extends JavaPlugin implements Listener {
 				
 			} else if(args.length > 0 && args[0].equalsIgnoreCase("reload") && cop(sender, "kpoj.reload")) {
 				reloadConfig();
-				
 				config = getConfig();
-				
 				playerlist = config.getStringList("Player");
 				active = config.getBoolean("Check");
+				sender.sendMessage(prefix + "Config reloaded!");
 				
 			} else if(args.length > 0 && args[0].equalsIgnoreCase("list") && cop(sender, "kpoj.list")) {
-				String buffer = playerlist.isEmpty() ? "" : playerlist.get(0);
+				sender.sendMessage(prefix + "Players: " + String.join(", ", playerlist));
-				for(int i = 1; i < playerlist.size(); i++) {
-					buffer += ", " + playerlist.get(i);
-				}
-				sender.sendMessage(prefix + "Players: " + buffer);
 				
 			} else {
 				sender.sendMessage(prefix + "Version: " + getDescription().getVersion());
 				if(cop(sender, "kpoj.info"))
-					sender.sendMessage(prefix + "Active:  " + (active ? Color.GREEN + "yes" : Color.RED + "no"));
+					sender.sendMessage(prefix + "Active:  " + (active ? ChatColor.GREEN + "yes" : ChatColor.RED + "no"));
 				if(sender.hasPermission("kpoj.info") && sender instanceof Player)
-					sender.sendMessage(prefix + "Status:  " + (playerlist.contains(sender.getName()) ? Color.GREEN + "Allowed" : Color.RED + "Disallowed"));
+					sender.sendMessage(prefix + "Status:  " + (playerlist.contains(sender.getName()) ? ChatColor.GREEN + "Allowed" : ChatColor.RED + "Disallowed"));
 			}
 			return true;
 		}
@@ -105,8 +108,8 @@ public class KickPlayerOnJoin extends JavaPlugin implements Listener {
     @EventHandler
     public void onPlayerLogin(PlayerLoginEvent event) {
     	if(!playerlist.contains(event.getPlayer().getName()) && active) {
-    		event.disallow(Result.KICK_WHITELIST, "You are not whitelisted on this server!");
+    		event.disallow(Result.KICK_WHITELIST, "You are not white-listed on this server!");
-    		getServer().broadcastMessage(event.getPlayer().getName());
+    		getServer().broadcastMessage(event.getPlayer().getName() + " tried to join.");
     	}
     }