Solved RightClickEntity not working

I'm trying to detect when a player right clicks a entity with a compass in it, this is my code:

Code:

if(e instanceof ItemFrame){
            Entity entity = e.getRightClicked();
            if(entity.getType().equals(Material.COMPASS)){
                Bukkit.getServer().dispatchCommand(p, "return");
                e.setCancelled(true);
            }
        }

 

i think is because youre putting if the entity is with the compass, not the player
and thats not if(entity.getType().equals(Material.COMPASS)){
thats getItemInHand
by doing if(entity.getType().equals(Material.COMPASS)){ i think that youre setting that the entity you want to click in a compass

 

@Reynergodoy Sohh how would I fix it? Cuz p.getType will not work..

 

@MordorKing78 I think you must search for the right clicked block.

Entity entity = e.getRightClickedBlock();

Don't know if it works. Just try it

 

@Ungemonstert Uhh, An ItemFrame is a entity?

 

1) #getRightClickedBlock() isn't anything
2) It would return a Block
3) An Entity isn't a Block
4) So that's invalid code

<3

 

Oh okay, I thought it could be but when i'm think about it, it was a stupid idea ._.

 

try to check if it is with a compass in hand

 

you have to do:
p.getItemInHand == Material.COMPASS

 

@Reynergodoy p.getItemInHand returns an ItemStack not an Material. Use p.getItemInHand.getType instead

 

@I Al Istannen I want to do if a player right clicks a item in a item frame not in his hand

 

@MordorKing78 Like this?

Code:

@EventHandler
public void onInteract(PlayerInteractEntityEvent event) {
  if(event.getRightClicked().getType() == EntityType.ITEM_FRAME) {
    Player p = event.getPlayer();
    p.sendMessage("Item frame");
    
    ItemFrame frame = (ItemFrame) event.getRightClicked();
    ItemStack itemInFrame = frame.getItem();
    p.sendMessage(itemInFrame.getType().toString());
  }
}

 

that was just an example '-'

 

@I Al Istannen Thank you man! It worked

 

@MordorKing78
Glad i could help