Solved How to make cmd freak out on inventory click.

I am having problems with my plugin that uses InventoryClickEvent and I have Item that you click on but, when you click on air, the console FREAKS out.

 

When you click on an empty slot, the clicked type is null.

Add null checks, before running the code make sure it != null.

 

Okay I will try that.

(e.getCurrentItem().getType().equals(air) || e.getCurrentItem().getType().equals(null)) {

Would this work?

didn't work and I need help with != null.

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

 

euller12
air cannot be resolved to a type...

 

euller12 I believe Material.AIR literally is null, no need for the or in the if statement.
Maybe try something like this

Code:

if (e.getCurrentItem().getType() != null){
  player.getInventory().addItem(new ItemStack(e.getCurrentItem().getType()));
}

So as you can see in the example above if you didn't check for != null, you would be trying to add null to the inventory, which would throw an NPE ofc.

 

OMG THANK YOU!!!! It worked with the help of your example Axe2760, also thanks xTrollxDudex.