Need help - Stop player from moving item in inventory

Hi guys, I want to block every players from moving a certain item in their inventory.
This is my code:

Code:

@EventHandler
    public void onInventoryClickEven(InventoryClickEvent event) {
        ItemStack LoveGun = new ItemStack(Material.STICK);
        ItemMeta LoveGunM = LoveGun.getItemMeta();
        LoveGunM.setDisplayName(colorize(LoveGunName));
        LoveGun.setItemMeta(LoveGunM);
        if(event.getCurrentItem().equals(LoveGun)){
            event.setCancelled(true);
            event.setResult(Result.DENY);
            event.setCursor(new ItemStack(Material.AIR));
            event.getWhoClicked().closeInventory();
        }
       
        if(event.isShiftClick() && (event.getSlot() == 4)) {
            event.setCancelled(true);
            event.setResult(Result.DENY);
            event.setCursor(new ItemStack(Material.AIR));
            event.getWhoClicked().closeInventory();
        }
    }

The problem is when the player shift+click or click on a number (hotbar), the item get duplicated and then the inventory close. So they have 2 items instead of one. Can someone help me with this?

 

@Maxx_Qc
This should work. I'm pretty sure it checks for durability tho so if your item isn't unbreakable then that might be a problem.

Code:java

if(itemStack.isSimilar(anotherItemStack)) {
    // TODO Put code in here.
}

 

Umm, don't know if this will work or not.. but try to close the inventory right after the event is cancelled. Like this:

Code:

@EventHandler
  public void onInventoryClickEven(InventoryClickEvent event) {
     Player p = (Player)event.getWhoClicked();
  ItemStack LoveGun = new ItemStack(Material.STICK);
  ItemMeta LoveGunM = LoveGun.getItemMeta();
  LoveGunM.setDisplayName(colorize(LoveGunName));
  LoveGun.setItemMeta(LoveGunM);
  if(event.getCurrentItem().equals(LoveGun)){
  event.setCancelled(true);
  p.closeInventory();
  event.setResult(Result.DENY);
  event.setCursor(new ItemStack(Material.AIR));
  }
   
  if(event.isShiftClick() && (event.getSlot() == 4)) {
  event.setCancelled(true);
  p.closeInventory();
  event.setResult(Result.DENY);
  event.setCursor(new ItemStack(Material.AIR));
  }
  }

@Maxx_Qc

 

Don't attempt to use equals on item stacks as you are checking boolean if they're the same instance (which they're most likely not). Just make each item name in your game unique (Whatsoever), then check if the item has the correct display name. And all that junk about setting cursor to none is unnecessary. You just need to ice.setCancelled(true) and it does the work for you. Also if you can't make item names unique, try a == instead of .equals();