I made a pvp kits plugin, that plugin have 3 kits [ pvp - archer - ninja ] pvp kit and archer kit is working, but ninja kit does not working .. if i type /ninja he is say you do not have permission, but i have permission! please anyone tell me what is trouble Code: package me.pepsigam3r.pvp; import org.bukkit.ChatColor; import org.bukkit.Color; import org.bukkit.GameMode; import org.bukkit.Location; import org.bukkit.Material; import org.bukkit.Sound; import org.bukkit.command.Command; import org.bukkit.command.CommandSender; import org.bukkit.enchantments.Enchantment; import org.bukkit.entity.Player; import org.bukkit.event.EventHandler; import org.bukkit.event.EventPriority; import org.bukkit.event.Listener; import org.bukkit.event.block.Action; import org.bukkit.event.player.PlayerDropItemEvent; import org.bukkit.event.player.PlayerInteractEvent; import org.bukkit.inventory.ItemStack; import org.bukkit.inventory.meta.LeatherArmorMeta; import org.bukkit.plugin.java.JavaPlugin; public class main extends JavaPlugin implements Listener { public void onEnable() { getServer().getPluginManager().registerEvents(this, this); } public void onDisable() { } @SuppressWarnings("deprecation") @EventHandler(priority = EventPriority.HIGH) public void onPlayerInteract(PlayerInteractEvent e) { Player p = e.getPlayer(); { if (((e.getAction() == Action.RIGHT_CLICK_AIR) || (e.getAction() == Action.RIGHT_CLICK_BLOCK)) && (p.getItemInHand().getType() == Material.MUSHROOM_SOUP)) { if (p.getGameMode() == GameMode.SURVIVAL) { p.setHealth(20); p.setFoodLevel(20); p.getItemInHand().setTypeId(281); p.setHealth(20); p.setFoodLevel(20); Location loc = p.getLocation(); p.playSound(loc, Sound.BURP, 10, 1); if (p.getGameMode() == GameMode.ADVENTURE) { p.setHealth(20); p.setFoodLevel(20); p.getItemInHand().setTypeId(281); p.setHealth(20); p.setFoodLevel(20); p.playSound(loc, Sound.BURP, 10, 1); } } else { if (p.getGameMode() == GameMode.CREATIVE) { e.setCancelled(true); } } } } } @EventHandler(priority=EventPriority.HIGH, ignoreCancelled=true) public void onDrop(PlayerDropItemEvent event) { Player pe = event.getPlayer(); pe.playSound(pe.getLocation(), Sound.ITEM_PICKUP, 10, 1); } public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) { Player p = (Player) sender; ItemStack dsword = new ItemStack(Material.DIAMOND_SWORD); if (commandLabel.equalsIgnoreCase("PvP")) { if (p.hasPermission("pvp.pvp")) { } p.getActivePotionEffects().clear(); p.getInventory().clear(); dsword.addEnchantment(Enchantment.DAMAGE_ALL, 1); p.getInventory().addItem(new ItemStack[] { dsword }); p.getInventory().setHelmet(new ItemStack(Material.IRON_HELMET)); p.getInventory().setChestplate(new ItemStack(Material.IRON_CHESTPLATE)); p.getInventory().setLeggings(new ItemStack(Material.IRON_LEGGINGS)); p.getInventory().setBoots(new ItemStack(Material.IRON_BOOTS)); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.sendMessage(ChatColor.AQUA + "[" + ChatColor.YELLOW + "PvP" + ChatColor.AQUA + "]" + ChatColor.GRAY + ": " + ChatColor.DARK_AQUA + "You have chosen the PvP kit!"); return false; } else { p.sendMessage(ChatColor.AQUA + "[" + ChatColor.YELLOW + "PvP" + ChatColor.AQUA + "]" + ChatColor.GRAY + ": " + ChatColor.DARK_AQUA + "You do not have access to use that kit!"); p.playSound(p.getLocation(), Sound.ZOMBIE_PIG_ANGRY, 10, 1); } if (commandLabel.equalsIgnoreCase("Archer")) { if (p.hasPermission("pvp.archer")) { ItemStack isword = new ItemStack(Material.IRON_SWORD); ItemStack chelmet = new ItemStack(Material.CHAINMAIL_HELMET); ItemStack cchest = new ItemStack (Material.CHAINMAIL_CHESTPLATE); ItemStack cleg = new ItemStack (Material.CHAINMAIL_LEGGINGS); ItemStack cboots = new ItemStack (Material.CHAINMAIL_BOOTS); ItemStack archerbow = new ItemStack (Material.BOW); p.getActivePotionEffects().clear(); p.getInventory().clear(); isword.addEnchantment(Enchantment.KNOCKBACK, 1); p.getInventory().addItem(isword); chelmet.addEnchantment(Enchantment.PROTECTION_PROJECTILE, 3); p.getInventory().setHelmet(chelmet); cchest.addEnchantment(Enchantment.PROTECTION_PROJECTILE, 3); p.getInventory().setChestplate(cchest); cleg.addEnchantment(Enchantment.PROTECTION_PROJECTILE, 3); p.getInventory().setLeggings(cleg); cboots.addEnchantment(Enchantment.PROTECTION_PROJECTILE, 3); p.getInventory().setBoots(cboots); archerbow.addEnchantment(Enchantment.ARROW_DAMAGE, 2); archerbow.addEnchantment(Enchantment.ARROW_INFINITE, 1); p.getInventory().addItem(archerbow); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.ARROW, 1) }); p.sendMessage(ChatColor.AQUA + "[" + ChatColor.YELLOW + "PvP" + ChatColor.AQUA + "]" + ChatColor.GRAY + ": " + ChatColor.DARK_AQUA + "You have chosen the Archer kit!"); return false; } else { p.sendMessage(ChatColor.AQUA + "[" + ChatColor.YELLOW + "PvP" + ChatColor.AQUA + "]" + ChatColor.GRAY + ": " + ChatColor.DARK_AQUA + "You do not have access to use that kit!"); p.playSound(p.getLocation(), Sound.ZOMBIE_PIG_ANGRY, 10, 1); } if (commandLabel.equalsIgnoreCase("ninja")) { if (p.hasPermission("pvp.ninja")) { p.getActivePotionEffects().clear(); p.getInventory().clear(); ItemStack ssword = new ItemStack(Material.STONE_SWORD); ItemStack lhelmet = new ItemStack(Material.LEATHER_HELMET); ItemStack lchest = new ItemStack(Material.LEATHER_HELMET); ItemStack lleg = new ItemStack(Material.LEATHER_HELMET); ItemStack lboots = new ItemStack(Material.LEATHER_HELMET); LeatherArmorMeta hm = (LeatherArmorMeta) lhelmet.getItemMeta(); LeatherArmorMeta cm = (LeatherArmorMeta) lchest.getItemMeta(); LeatherArmorMeta lm = (LeatherArmorMeta) lleg.getItemMeta(); LeatherArmorMeta bm = (LeatherArmorMeta) lboots.getItemMeta(); ssword.addEnchantment(Enchantment.DAMAGE_ALL, 2); ssword.addEnchantment(Enchantment.KNOCKBACK, 2); ssword.addEnchantment(Enchantment.DURABILITY, 2); p.getInventory().addItem(ssword); lhelmet.setItemMeta(hm); lchest.setItemMeta(cm); lleg.setItemMeta(lm); lboots.setItemMeta(bm); hm.setColor(Color.BLACK); cm.setColor(Color.BLACK); lm.setColor(Color.BLACK); bm.setColor(Color.BLACK); p.getInventory().setHelmet(lhelmet); p.getInventory().setChestplate(lchest); p.getInventory().setLeggings(lleg); p.getInventory().setBoots(lboots); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.getInventory().addItem(new ItemStack[] { new ItemStack(Material.MUSHROOM_SOUP, 1) }); p.sendMessage(ChatColor.AQUA + "[" + ChatColor.YELLOW + "PvP" + ChatColor.AQUA + "]" + ChatColor.GRAY + ": " + ChatColor.DARK_AQUA + "You have chosen the Ninja kit!"); return false; } else { p.sendMessage(ChatColor.AQUA + "[" + ChatColor.YELLOW + "PvP" + ChatColor.AQUA + "]" + ChatColor.GRAY + ": " + ChatColor.DARK_AQUA + "You do not have access to use that kit!"); p.playSound(p.getLocation(), Sound.ZOMBIE_PIG_ANGRY, 10, 1); } } } return false; } } ( Sorry for my bad english )
A for loop is your friend. While it's not directly related to your problem, excessive code reuse is a glaring mistake and is bad habit. Furthermore the bracket problems can be fixed by formatting the code in a IDE such as Eclipse.