Adds option whether to show kits that players don't have permission for or not

Author: InstanceLabs

API/src/main/java/com/comze_instancelabs/minigamesapi/Classes.java

@@ -22,6 +22,7 @@
 import com.comze_instancelabs.minigamesapi.util.AClass;
 import com.comze_instancelabs.minigamesapi.util.IconMenu;
 import com.comze_instancelabs.minigamesapi.util.Util;
+import com.comze_instancelabs.minigamesapi.util.Validator;
 import com.shampaggon.crackshot.CSUtility;
 
 public class Classes {
@@ -44,6 +45,10 @@ public void openGUI(final String p) {
 		final Classes cl = this;
 		IconMenu iconm;
 		int mincount = pli.getAClasses().keySet().size();
+		if (!Validator.isPlayerOnline(p)) {
+			return;
+		}
+		Player player = Bukkit.getPlayerExact(p);
 		if (lasticonm.containsKey(p)) {
 			iconm = lasticonm.get(p);
 		} else {
@@ -64,11 +69,16 @@ public void onOptionClick(IconMenu.OptionClickEvent event) {
 					event.setWillClose(true);
 				}
 			}, plugin);
-			
+
 			int c = 0;
 			for (String ac : pli.getAClasses().keySet()) {
 				AClass ac_ = pli.getAClasses().get(ac);
 				if (ac_.isEnabled()) {
+					if (!pli.show_classes_without_usage_permission) {
+						if (!kitPlayerHasPermission(ac_.getInternalName(), player)) {
+							continue;
+						}
+					}
 					int slot = c;
 					if (pli.getClassesConfig().getConfig().isSet("config.kits." + ac_.getInternalName() + ".slot")) {
 						slot = pli.getClassesConfig().getConfig().getInt("config.kits." + ac_.getInternalName() + ".slot");
@@ -82,7 +92,7 @@ public void onOptionClick(IconMenu.OptionClickEvent event) {
 			}
 		}
 
-		iconm.open(Bukkit.getPlayerExact(p));
+		iconm.open(player);
 		lasticonm.put(p, iconm);
 	}
 

API/src/main/java/com/comze_instancelabs/minigamesapi/PluginInstance.java

@@ -71,6 +71,7 @@ public class PluginInstance {
 	public boolean color_background_wool_of_signs;
 	boolean last_man_standing = true;
 	boolean old_reset = false;
+	public boolean show_classes_without_usage_permission = true;
 
 	public HashMap<String, ArrayList<String>> cached_sign_states = new HashMap<String, ArrayList<String>>();
 
@@ -114,6 +115,7 @@ public void reloadVariables() {
 		spectator_mode_1_8 = plugin.getConfig().getBoolean("config.effects.1_8_spectator_mode");
 		last_man_standing = plugin.getConfig().getBoolean("config.last_man_standing_wins");
 		old_reset = plugin.getConfig().getBoolean("config.use_old_reset_method");
+		show_classes_without_usage_permission = plugin.getConfig().getBoolean("config.show_classes_without_usage_permission");
 
 		// Cache sign configuration
 		for (String state : ArenaState.getAllStateNames()) {

API/src/main/java/com/comze_instancelabs/minigamesapi/commands/CommandHandler.java

@@ -780,6 +780,11 @@ public boolean setKit(PluginInstance pli, CommandSender sender, String[] args, S
 			String all = "";
 			for (AClass k : pli.getAClasses().values()) {
 				if (k.isEnabled()) {
+					if (!pli.show_classes_without_usage_permission) {
+						if (!pli.getClassesHandler().kitPlayerHasPermission(k.getInternalName(), p)) {
+							continue;
+						}
+					}
 					all += k.getInternalName() + ", ";
 				}
 			}

API/src/main/java/com/comze_instancelabs/minigamesapi/config/DefaultConfig.java

@@ -41,6 +41,7 @@ public static void init(JavaPlugin plugin, boolean custom) {
 			config.addDefault("config.use_credits_instead_of_money_for_kits", false);
 			config.addDefault("config.reset_inventory_when_players_leave_server", true);
 			config.addDefault("config.color_background_wool_of_signs", false);
+			config.addDefault("config.show_classes_without_usage_permission", true);
 
 			config.addDefault("config.rewards.economy", true);
 			config.addDefault("config.rewards.economy_reward", 10);