Solved InventoryClickEvent null proplem

this is the code:

Code:java

public void OnInventoryClick(InventoryClickEvent event) {
 
//simple materials
Material[] ids = {Material.BOW,Material.ARROW,Material.IRON_SWORD,Material.ENDER_PEARL,Material.DIAMOND_CHESTPLATE};
int view = InventoryView.OUTSIDE;
 
if (event != null) {
String log = "view: "
+ event.getView().getTitle() +" "+ view + " slot num: "
+ event.getSlot();
System.out.println(log);
for(Material i : ids){
if(i==event.getCurrentItem().getType())
{
event.setCancelled(true);
System.out.println("canceled event to "+i.toString());
break;
}
}
if(event.getCurrentItem().getType() == Material.STICK){
if(event.getCurrentItem().getEnchantments().isEmpty())
{
System.out.println("no enchantments");
}else{
System.out.println("has enchantments canceling event");
event.setCancelled(true);
}
}
}
}
 

if i click (while in inventory) anywhere out side the box i get null error
as you can see i tried checking if the event is null but no help


thanks


error:
2013-09-15 16:09:23 [SEVERE] Could not pass event InventoryClickEvent to LockItem v0.0.1
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:427)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java:1372)
at net.minecraft.server.v1_6_R2.Packet102WindowClick.handle(SourceFile:31)
at net.minecraft.server.v1_6_R2.NetworkManager.b(NetworkManager.java:296)
at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java:116)
at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
at net.minecraft.server.v1_6_R2.DedicatedServerConnection.b(SourceFile:30)
at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:590)
at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:226)
at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:486)
at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java:419)
at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:582)
Caused by: java.lang.NullPointerException
at io.github.fokolo.lockitem.logger(lockitem.java:23)
at io.github.fokolo.inventoryclicks.OnInventoryClick(inventoryclicks.java:15)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425)
... 14 more

half solved i edited the post but i still need help with the null problem

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

 

fokolo
1. What is on line 23?
2. What is your debug message output?
3. event.getCurrentItem() returns null when you don't click a slot in the inventory.

 

Technius
1.im checking if the stick is enchanted
2. debug states if the stick enchanted or not
3. thank you! you fixed this!

 

I've had this same problem...
A null check won't be enough, because the item in the slot is technically air.

So how I fixed it was just to add an if statement before all of your listener code

Code:java

if(event.getCurrentItem() != null && event.getCurrentItem().getType() != Material.AIR)

 

Appljuze
It's fine for his purpose since he is checking material types anyways.

 

I guess it depends on where he puts the if statement

 

thank you both i dont need it to check air too so its fine for my purpose

but thanks
SOLVED