InventoryClickEvent being called 3 times

Hi there,

So there is part of my plugin with a custom inventory that is being used as a menu, and there are certain items that do different things in the menu. The listener for InventoryClickEvent is working fine, but its called 3 times.

Here is the code:

Code:java

@EventHandler
public void onInventoryClick(InventoryClickEvent e){
Player player =  e.getWhoClicked();
if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase(ChatColor.DARK_RED + "Attack Power")){
e.setCancelled(true);
player.sendMessage(ChatColor.AQUA + "[ShadowCraft RPG] " + ChatColor.GREEN + "Attack Power increases your physical damage.");
}
if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase(ChatColor.GREEN + "Skill Up")){
e.setCancelled(true);
player.sendMessage(ChatColor.AQUA + "[ShadowCraft RPG] " + ChatColor.WHITE + "Your " + ChatColor.DARK_RED + "ATTACK POWER " + ChatColor.WHITE + "has been increased!");
}
}

When they click the items, the user is sent the message 3 times each...

 

tyler53

Full code? Is there any chance it is being registered multiple times?

 

That was basically full code, this is the full class minus imports:

Code:java

public class InventoryListener implements Listener{
 
@EventHandler
public void onInventoryClick(InventoryClickEvent e){
Player player = (Player) e.getWhoClicked();
if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase(ChatColor.DARK_RED + "Attack Power")){
e.setCancelled(true);
player.sendMessage(ChatColor.AQUA + "[ShadowCraft RPG] " + ChatColor.GREEN + "Attack Power increases your physical damage.");
}
if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase(ChatColor.GREEN + "Skill Up")){
e.setCancelled(true);
player.sendMessage(ChatColor.AQUA + "[ShadowCraft RPG] " + ChatColor.WHITE + "Your " + ChatColor.DARK_RED + "ATTACK POWER " + ChatColor.WHITE + "has been increased!");
}
}
 
}

And the listener as set up in my main file:

Code:java

Bukkit.getServer().getPluginManager().registerEvents(new InventoryListener(), this);