IconMenu problems

Hey bukkit forums, I'm making a kit selector for my kitpvp plugin with IconMenu, I got it all set up, but when they click an item, it doesn't perform the command that it's supposed to, it just lets you put the item in your inventory
IconMenu class

Code:java

package me.Fhbgsdhkfbl;
 
import java.util.Arrays;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.HandlerList;
import org.bukkit.event.Listener;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.Plugin;
 
public class IconMenu implements Listener {
    private me.Fhbgsdhkfbl.Main plugin;
    public IconMenu(me.Fhbgsdhkfbl.Main plugin){
        this.plugin = plugin;
    }
 
    private String name;
    private int size;
    private OptionClickEventHandler handler;
 
    private String[] optionNames;
    private ItemStack[] optionIcons;
 
    public IconMenu(String name, int size, OptionClickEventHandler handler, Plugin plugin) {
        this.name = name;
        this.size = size;
        this.handler = handler;
        this.optionNames = new String[size];
        this.optionIcons = new ItemStack[size];
    }
 
    public IconMenu setOption(int position, ItemStack icon, String name, String... info) {
        optionNames[position] = name;
        optionIcons[position] = setItemNameAndLore(icon, name, info);
        return this;
    }
 
    public void open(Player player) {
        Inventory inventory = Bukkit.createInventory(player, size, name);
        for (int i = 0; i < optionIcons.length; i++) {
            if (optionIcons[i] != null) {
                inventory.setItem(i, optionIcons[i]);
            }
        }
        player.openInventory(inventory);
    }
 
    public void destroy() {
        HandlerList.unregisterAll(this);
        handler = null;
        plugin = null;
        optionNames = null;
        optionIcons = null;
    }
 
    @EventHandler(priority=EventPriority.MONITOR)
    void onInventoryClick(InventoryClickEvent event) {
        if (event.getInventory().getTitle().equals(name)) {
            event.setCancelled(true);
            int slot = event.getRawSlot();
            if (slot >= 0 && slot < size && optionNames[slot] != null) {
                Plugin plugin = this.plugin;
                OptionClickEvent e = new OptionClickEvent((Player)event.getWhoClicked(), slot, optionNames[slot]);
                handler.onOptionClick(e);
                if (e.willClose()) {
                    final Player p = (Player)event.getWhoClicked();
                    Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
                        public void run() {
                            p.closeInventory();
                        }
                    }, 1);
                }
                if (e.willDestroy()) {
                    destroy();
                }
            }
        }
    }
 
    public interface OptionClickEventHandler {
        public void onOptionClick(OptionClickEvent event);     
    }
 
    public class OptionClickEvent {
        private Player player;
        private int position;
        private String name;
        private boolean close;
        private boolean destroy;
 
        public OptionClickEvent(Player player, int position, String name) {
            this.player = player;
            this.position = position;
            this.name = name;
            this.close = true;
            this.destroy = false;
        }
 
        public Player getPlayer() {
            return player;
        }
 
        public int getPosition() {
            return position;
        }
 
        public String getName() {
            return name;
        }
 
        public boolean willClose() {
            return close;
        }
 
        public boolean willDestroy() {
            return destroy;
        }
 
        public void setWillClose(boolean close) {
            this.close = close;
        }
 
        public void setWillDestroy(boolean destroy) {
            this.destroy = destroy;
        }
    }
 
    private ItemStack setItemNameAndLore(ItemStack item, String name, String[] lore) {
        ItemMeta im = item.getItemMeta();
            im.setDisplayName(name);
            im.setLore(Arrays.asList(lore));
        item.setItemMeta(im);
        return item;
    }
 
}[/i][/i]

interact event and iconmenu event

Code:java

IconMenu menu = new IconMenu("Kit Selector", 27, new IconMenu.OptionClickEventHandler() {
        @Override
        public void onOptionClick(IconMenu.OptionClickEvent event) {
            Player p = event.getPlayer();
            switch (event.getPosition()) {
            case 0:
                p.performCommand("pvp");
                break;
            case 1:
                p.performCommand("archer");
                break;
            case 2:
                Bukkit.dispatchCommand(p, "soldier");
                break;
            }
 
            event.setWillClose(true);
        }
    }, plugin)
    .setOption(0, new ItemStack(Material.DIAMOND_SWORD, 1), "Knight", "Click me for the Knight kit!", "Default Kit")
    .setOption(1, new ItemStack(Material.BOW, 1), "Archer", "Click me for the Archer kit!")
    .setOption(2, new ItemStack(Material.IRON_SWORD, 1), "Soldier", "Click me for the Soldier kit!");
    @EventHandler
    public void onKitGui(PlayerInteractEvent e) {
        Player player = e.getPlayer();
        Action a = e.getAction();
        if (a.equals(Action.RIGHT_CLICK_BLOCK) || (a.equals(Action.RIGHT_CLICK_AIR)) && !player.getItemInHand().getType().equals(Material.DIAMOND)) {
            return;
        }
        if (a.equals(Action.RIGHT_CLICK_BLOCK) || (a.equals(Action.RIGHT_CLICK_AIR)) && player.getItemInHand().getType().equals(Material.DIAMOND)) {
            menu.open(player);
                return;
            }
    }

 

Even though it lets you put the item in your inventory, does it still give you the items you wanted? If so the problem may be that the associated event was not cancelled properly. I'm guessing you want the inventory to be closed directly after the player clicks on the item so how about cancelling the InventoryClickEvent and then running

Code:java

e.getPlayer().closeInventory();

Source: https://forums.bukkit.org/threads/closing-custom-inventories-by-clicking-on-items.164825/

Sidenote:

Code:java

private me.Fhbgsdhkfbl.Main plugin;

*shudder*
I hope that's just rudimentary obfuscation for posting purposes.