Solved setHeldItemSlot() Problem

Hey Programmers out there! I am working on a new Plugin. I want to change the Players Selected ItemBar Slot to 1 if the Slot 0 contains a sword which is not a Golden_Sword. So far it works. But when I want the Player to switch his Selected ItemBar Slot the Item in this slot automatically is the same Item as in Slot 0 ... Please Help me Here is my Code:

Code:java

    @EventHandler
    public void MineBlock(PlayerInteractEvent ev){
        Player p = ev.getPlayer();
        if((ev.getAction() == Action.RIGHT_CLICK_BLOCK ||
            ev.getAction() == Action.RIGHT_CLICK_AIR ) && (
            ev.getPlayer().getItemInHand().getType() == Material.WOOD_SWORD ||
            ev.getPlayer().getItemInHand().getType() == Material.STONE_SWORD ||
            ev.getPlayer().getItemInHand().getType() == Material.IRON_SWORD ||
            ev.getPlayer().getItemInHand().getType() == Material.DIAMOND_SWORD)){
            if (p.getInventory().getHeldItemSlot() == 0){
                p.sendMessage("TEST 1");
                if (p.getInventory().getItem(1).getType() == Material.GOLD_SWORD){
                    p.sendMessage("TEST 2");
                    ItemStack is = p.getInventory().getItem(1);
                    p.getInventory().setHeldItemSlot(1);
                    p.getInventory().setItemInHand(is);
                }
            }
        }
    }

By the way both messages appear so in the Chat will be TEST 1 and a line under this is TEST 2 so the Plugin definitely loads this part of the Plugin.

 

There's no need to use setItemInHand, that item will already be in their hand when you switch their hotbar slot.

 

nisovin The Problem is that the Item 1 gets written over from item 0 so i have two times item 0 instead of one time Item 0 and one time Item 1

 

Setting the slot from the interact event may be causing a problem. Try using the scheduler to add a 1 tick delay.

 

Thank you It worked!