Solved EventListeners don't listen

Greetings. I have problem with EventListeners. They don't react on events:

Code:java

public void onEnable() {
...
getServer().getPluginManager().registerEvents(new ToolListener(), this);
}

ToolListener:

Code:java

@EventHandler(priority = EventPriority.HIGH)
    public void onPlayerMove(PlayerMoveEvent e) {
        if (e instanceof Player){
            if (((Player) e).getInventory().getHelmet().getItemMeta().getDisplayName() == ChatColor.YELLOW+"Glowstone Hat") {
            Player p = (Player) e;
            ((Player) e).addPotionEffect(new PotionEffect(PotionEffectType.NIGHT_VISION, 2, 1));
            }
        }
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        if (e instanceof Player) {           
            if (e.getItem().getItemMeta().getDisplayName() == ChatColor.GOLD+"Fireball Gun" && e.getAction() == Action.RIGHT_CLICK_AIR) {
 
                if (((Player) e).getInventory().contains(Material.FIREBALL)) {
                    ((Player) e).launchProjectile(Fireball.class);
                    ((Player) e).getInventory().remove(Material.FIREBALL);
 
                } else {
                    ((Player) e).sendMessage(ChatColor.RED+"Out of fireballs!");
                }
 
            }
        }
        }

What's wrong?

 

You are checking if the event is an instance of a player, which the event itself will never be, you just have to use e.getPlayer() to get the according player

 

Your putting if the event is an instance of a Player.

 

Indeed you are. Change it to if e.getEntity() instanceof Player

 

After that:
- You are first checking if the e.getItem() has a certain displayname before you event check if the player right clicks (&& means: first check the thing before the &&, and if its true the thing behind the &&)

e.getItem() can be null, because if a player stands on a pressureplate there's no item used. My tip:
First check for e.getAction (maybe you need both RIGHT_CLICK_AIR an RIGHT_CLICK_BLOCK)

 

Thanks a lot.

 

Why do that when the events already contain player in the name, so the entity will be a player regardless.