Solved How to fix my code for inventory menu.

When ever I click on the block it doesn't work. I can take the block also.

Code:java

      @EventHandler
      public void onInventoryClick(InventoryClickEvent e){
 
          ItemStack comp = new ItemStack(Material.NETHER_STAR);
 
          if (!e.getInventory().getName().equalsIgnoreCase(inv.getName())) return;
          if (e.getCurrentItem().getType() != null){
                return;
            }
          if (e.getCurrentItem().getItemMeta().getDisplayName().contains("Vote")) {
              e.setCancelled(true);
              e.getWhoClicked().damage(1);
              e.getWhoClicked().closeInventory();
              //e.getWhoClicked().openInventory(vote);
          }
      }

 

I'm going to assume it's because

Code:java

 
 if (e.getCurrentItem().getType() != null){
                return;
}
 

Are you sure you don't mean

Code:java

 
 if (e.getCurrentItem().getType() == null){
                return;
}
 

 

How have a little problem.

Code:java

if (e.getCurrentItem().getItemMeta().getDisplayName().contains("Vote")) {
    e.setCancelled(true);
    e.getWhoClicked().damage(1);
    e.getWhoClicked().closeInventory();
    //e.getWhoClicked().openInventory(vote);
}

This condition never enter because...

Code:java

if (e.getCurrentItem().getType() != null){
    return;
}

This condition es allways accepted.

If i'm not wrong all the items have a type.

 

Registered events?

 

?

Xephiro thanks it worked!

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