Solved Inventory gui help

Why is it that if a player clicks any item it opens the inventory...? when it's only supposed to be a book?

Code:java

package me.winitro.kits;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerRespawnEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.java.JavaPlugin;
 
 
public class Test extends JavaPlugin implements Listener {
 
public void onEnable() {
getServer().getPluginManager().registerEvents(this, this);
}
 
private void openGUI(Player player) {
Inventory inv = Bukkit.createInventory(null, 9, ChatColor.AQUA +
"[" + ChatColor.BLUE + player.getName() + ChatColor.AQUA + "] " + ChatColor.DARK_AQUA + "Your kits");
 
ItemStack sniper = new ItemStack(Material.BOW);
ItemMeta sniperMeta = sniper.getItemMeta();
 
ItemStack archer = new ItemStack(Material.ARROW);
ItemMeta archerMeta = archer.getItemMeta();
 
ItemStack stomper = new ItemStack(Material.IRON_BOOTS);
ItemMeta stomperMeta = stomper.getItemMeta();
 
if(player.hasPermission("kits.sniper") || player.isOp()){
sniperMeta.setDisplayName(ChatColor.GREEN + "Sniper");
}else{
sniperMeta.setDisplayName(ChatColor.RED + "Sniper");
}
if(player.hasPermission("kits.archer") || player.isOp()){
archerMeta.setDisplayName(ChatColor.GREEN + "Archer");
}else{
archerMeta.setDisplayName(ChatColor.RED + "Archer");
}
if(player.hasPermission("kits.stomper") || player.isOp()){
stomperMeta.setDisplayName(ChatColor.GREEN + "Stomper");
}else{
stomperMeta.setDisplayName(ChatColor.RED + "Stomper");
}
 
sniper.setItemMeta(sniperMeta);
archer.setItemMeta(archerMeta);
stomper.setItemMeta(stomperMeta);
 
inv.setItem(0, sniper);
inv.setItem(8, archer);
inv.setItem(4, stomper);
 
player.openInventory(inv);
}
 
@EventHandler
public void onInventoryClick(InventoryClickEvent event) {
Player player = (Player) event.getWhoClicked();
if (!ChatColor.stripColor(event.getInventory().getName())
.equalsIgnoreCase("[" + player.getName() + "] " + "Your kits"))
return;
event.setCancelled(true);
 
if (event.getCurrentItem() == null
|| event.getCurrentItem().getType() == Material.AIR
|| !event.getCurrentItem().hasItemMeta()) {
player.closeInventory();
return;
}
 
switch (event.getCurrentItem().getType()) {
case BOW:
player.closeInventory();
if(player.hasPermission("kits.sniper")){
player.sendMessage("Stomper message sent");
}else{
if(!player.hasPermission("kits.sniper")){
player.sendMessage(ChatColor.DARK_RED + player.getName() + " You do not have perms to that kit");
}
}
break;
case ARROW:
player.closeInventory();
if(player.hasPermission("kits.archer")){
player.sendMessage("Archer message sent");
}else{
if(!player.hasPermission("kits.archer")){
player.sendMessage(ChatColor.DARK_RED + player.getName() + " You do not have perms to that kit");
}
}
break;
case IRON_BOOTS:
player.closeInventory();
if(player.hasPermission("kits.stomper")){
player.sendMessage("Stomper message sent");
}else{
if(!player.hasPermission("kits.stomper")){
player.sendMessage(ChatColor.DARK_RED + player.getName() + " You do not have perms to that kit");
}
}
break;
 
 
 
default:
player.closeInventory();
break;
}
}
 
@EventHandler
public void onPlayerJoin(PlayerJoinEvent event) {
Player player = event.getPlayer();
ItemStack book = new ItemStack(Material.BOOK);
ItemMeta tm = book.getItemMeta();
tm.setDisplayName( "§2[" + "§a" + player.getName() + "§2] " + "§bKit selector");
book.setItemMeta(tm);
player.getInventory().addItem(new ItemStack(book));
 
}
@EventHandler
public void onPlayerRespawn(PlayerRespawnEvent e){
Player player = e.getPlayer();
ItemStack book = new ItemStack(Material.BOOK);
ItemMeta tm = book.getItemMeta();
tm.setDisplayName( "§2[" + "§a" + player.getName() + "§2] " + "§bKit selector");
book.setItemMeta(tm);
player.getInventory().addItem(new ItemStack(book));
 
}
 
@EventHandler
public void onPlayerInteract(PlayerInteractEvent event) {
Action a = event.getAction();
ItemStack is = event.getItem();
 
if (is.getType() == Material.BOOK);
openGUI(event.getPlayer());
 
if (a == Action.PHYSICAL || is == null || is.getType() == Material.AIR)
return;
 
if (is.getType() == Material.BOOK);
openGUI(event.getPlayer());
 
}
}

 

All your ifs have semicolons after then not start brackets, try this on interact.
Try

Code:

if (event.getPlayer().getItemInHand().getType() == Material.BOOK) {
  //open inv and stuff
}

 

It didn't work.

It can still be opened with any item

Nvm it did work I just forgot to remove something thanks for the help!

EDIT by Moderator: merged posts, please use the edit button instead of double posting.