Solved Events not working detecting Materials? Why not?

Code:java

    @EventHandler
    public void onPlayerClick(InventoryMoveItemEvent evt) {
        ItemStack Click = evt.getItem();
    if(Click.getType().equals(Material.RECORD_9)) {
        evt.setCancelled(true);
    }
    if(Click.getType().equals(Material.BLAZE_ROD)) {
        evt.setCancelled(true);
    }
 
    }
 
    @EventHandler
    public void onPlayerDrop(PlayerDropItemEvent evt) {
        Item item = evt.getItemDrop();
        if(item.getItemStack().equals(Material.RECORD_9)) {
            evt.setCancelled(true);
        }
        if(item.getItemStack().equals(Material.BLAZE_ROD)) {
            evt.setCancelled(true);
        }
    }

 

Try just

Code:java

      if(Player.getInventory.equals(Material.BLAZE_ROD)) {
      //Put code here you want to run
        }

 

Is your event registered and firing?
Is the item in question null? is it even the item you are checking for?
Does your class implement Listener?

Variables start with lowercase, not Upper.

That would check if the player inventory is a material (Blaze rod), which it will never be. It will never be anything other than the player's inventory.

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

 

Yah the event is registered but Idk why it wont run the rest of the code. No it is not. There are no errors. My class does implement listener. Variable should not be problem. TheGreenGamerHD

This is not the answer. This will never turn true...

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

 

I'd try

Code:java

if(event.getItem().getTypeId() == 1) {
  // CODE
}
 

It worked in my plugin

 

your second won't fire, because an ItemStack is NOT a Material enum, therefore you need to check it's type

Edit: Hey MayoDwarf ! I know you from the Pokeballers server and Pixelmon website

 

haha Ik you too I kinda dont play those anymore. More focused on my development jobs and so what should I do to fix it? What ayden sayd?
EDIT: TheGreenGamerHD

TheGreenGamerHD ayden1232

Would this work?:

Code:java

@EventHandler
    public void onPlayerClick(InventoryMoveItemEvent evt) {
        ItemStack Click = evt.getItem();
    if(Click.getType().equals(Material.RECORD_9)) {
        evt.setCancelled(true);
    }
    if(Click.getType().equals(Material.BLAZE_ROD)) {
        evt.setCancelled(true);
    }
 
    }
 
    @EventHandler
    public void onPlayerDrop(PlayerDropItemEvent evt) {
        Item item = evt.getItemDrop();
        if(item.getType().equals(Material.RECORD_9)) {
            evt.setCancelled(true);
        }
        if(item.getType().equals(Material.BLAZE_ROD)) {
            evt.setCancelled(true);
        }
    }
 

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

 

MayoDwarf or you can just do something like this:

Code:

@EventHandler
public void onPlayerClick (InventoryMoveItemEvent event) {
    if (!event.isCancelled())
        event.setCancelled(event.getItem().equals(Material.RECORD_9) || event.getItem().equals(Material.BLAZE_ROD));
}

 

It should...

You can use an or statement or a switch statement instead of doing if if if if if for checks.

Code:

if(item.getType() == Material.RECORD_9 || item.getType() == Material.BLAZE_ROD){
    e.setCancelled(true);
}
 
switch(item.getType()){
    default:
        break;
    case RECORD_9:
        e.setCancelled(true);
        break;
    case BLAZE_ROD:
        e.setCancelled(true);
        break;
}