Solved PlayerInteractEvent Detection Issue

I have two samples of my code. The first sample is the bit I'm having trouble with. It will not detect the event at all. I stripped it down to the basics for testing. There is virtually no difference between this code and the second fragment, but only the second works.

Code:java

@EventHandler(ignoreCancelled = true)
    public void onClick(PlayerInteractEvent event) {
        boolean cancelled = event.isCancelled();
        if (cancelled)
            event.setCancelled(false);
        boolean itemUsed = false;
        Player player = event.getPlayer();
        player.sendMessage("InteractEventTriggered");
        if (player.equals(summoner)) {
            Action act = event.getAction();
            if (act == Action.RIGHT_CLICK_AIR
                    || act == Action.RIGHT_CLICK_BLOCK) {
                player.sendMessage("Right Click");
 
            } else if (act == Action.LEFT_CLICK_AIR
                    || act == Action.LEFT_CLICK_BLOCK) {
                player.sendMessage("Left Click");
            }
            if (itemUsed) {
                event.setCancelled(true);
                PlayerInteractEvent.getHandlerList().unregister(this);
            }
        }
        if(cancelled)
            event.setCancelled(cancelled);
    }

This is the code that I derived the first bit from. They are the same. This event will always be called when the first is called, so that means the event will be canceled, but I cant seem to detect it after it is canceled.

Code:java

@EventHandler
    public void onClick(PlayerInteractEvent event) {
        boolean itemUsed = false;
        Player player = event.getPlayer();
        Action act = event.getAction();
        if (act == Action.RIGHT_CLICK_AIR || act == Action.RIGHT_CLICK_BLOCK) {
            ItemStack item = player.getItemInHand();
            if (item.hasItemMeta()) {
                ItemMeta im = item.getItemMeta();
                if (im.hasDisplayName()) {
                    int pclass = player.getMetadata("RPGClass").get(0).asInt();
                    String dispName = im.getDisplayName();
                    switch (dispName) {
                    case MagicItems.STAFF_NAME:
                        magicItems.classStaff(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.TIME_TURNER_NAME:
                        magicItems.timeTurner(player, pclass, true);
                        itemUsed = true;
                        break;
                    case MagicItems.WEATHER_DIAL_NAME:
                        magicItems.weatherDial(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.BLINDING_SAC_NAME:
                        magicItems.blindingSac(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.PLAYER_PORTAL_NAME:
                        magicItems.playerPortal(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.VANISHING_DUST_NAME:
                        magicItems.vanishingDust(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.IRON_BAR_NAME:
                        magicItems.ironBar(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.BOMB_NAME:
                        magicItems.bomb(player, pclass);
                        itemUsed = true;
                        break;
                    case MagicItems.DRAGON_ESSENCE_NAME:
                        magicItems.dragonEssence(player, pclass);
                        itemUsed = true;
                        break;
                    }
                }
            }
        } else if (act == Action.LEFT_CLICK_AIR
                || act == Action.LEFT_CLICK_BLOCK) {
            ItemStack item = player.getItemInHand();
            if (item.hasItemMeta()) {
                ItemMeta im = item.getItemMeta();
                if (im.hasDisplayName()) {
                    int pclass = player.getMetadata("RPGClass").get(0).asInt();
                    String dispName = im.getDisplayName();
                    switch (dispName) {
                    case MagicItems.TIME_TURNER_NAME:
                        magicItems.timeTurner(player, pclass, false);
                        itemUsed = true;
                        break;
                    }
                }
            }
        }
        if (itemUsed) {
            event.setCancelled(true);
        }
    }

Can anyone fathom why the first will not work?

Btw, the second fragment of code works every time.

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

 

Put debug messages, make sure all events are registered.

 

Post your entire class.

 

tommycake50

I did put debug messages in the first bit of code. I did mean to comment that the event only goes off when clicking blocks, but not air for the first.

TheSpherret

Code:

package com.yahoo.prosfis.RPGClasses.ClassListeners;
 
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.Sound;
import org.bukkit.block.Block;
import org.bukkit.entity.LivingEntity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.util.BlockIterator;
 
import com.yahoo.prosfis.RPGClasses.MagicItems;
import com.yahoo.prosfis.RPGClasses.Spells;
 
public class MoveEntityListener implements Listener {
 
    private Player summoner;
    private LivingEntity target;
    private Spells spells;
 
    public MoveEntityListener(Player summoner, LivingEntity target,
            Spells spells) {
        this.summoner = summoner;
        this.target = target;
        this.spells = spells;
    }
 
    @EventHandler(ignoreCancelled = true)
    public void onClick(PlayerInteractEvent event) {
        boolean cancelled = event.isCancelled();
        if (cancelled)
            event.setCancelled(false);
        boolean itemUsed = false;
        Player player = event.getPlayer();
        player.sendMessage("InteractEventTriggered");
        if (player.equals(summoner)) {
            Action act = event.getAction();
            if (act == Action.RIGHT_CLICK_AIR
                    || act == Action.RIGHT_CLICK_BLOCK) {
                player.sendMessage("Right Click");
               
            } else if (act == Action.LEFT_CLICK_AIR
                    || act == Action.LEFT_CLICK_BLOCK) {
                player.sendMessage("Left Click");
            }
            if (itemUsed) {
                event.setCancelled(true);
                PlayerInteractEvent.getHandlerList().unregister(this);
            }
        }
        if(cancelled)
            event.setCancelled(cancelled);
    }
}

Ignore the many imports. They are needed for the rest of the code that I deleted for the moment.

Gah... I had a misunderstanding of how ignoreCancelled works. I was under the impression that it meant to ignore that the event was cancelled, but it means not to proceed if the event was canceled. Thanks for the replys.

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

 

Did you register the event Listener class?