Help with removing item on click!

Hey everyone so I need some help with removing items when they right click it here is my code http://pastebin.com/R6PBmqnC it doesn't seem to work when I click it nothing happens. Any help would be great thanks!

 

Try changing null to Material.AIR

 

THESABERSLAYER register the event handler.

 

SuperOmegaCow the EventHandler is registered though

Also RDNachoz I did that and nothing happened still didnt get my potion effect or the text or anything

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

 

THESABERSLAYER instead of setting the item in there hand to null, remove the itemstack, it will also help as when the player has more then one it will only lose one and not all.

 

Niknea can u please post the code to remove?

 

THESABERSLAYER create an itemstack that contains one red
Mushroom. Then use this code

Code:java

player.getInventory.removeItem(<itemstack name>); [\syntax]

 

Niknea my code works except for when there is only one mushroom left then it doesn't get rid of it

 

Can you post the code that you currently have?

 

Sour http://pastebin.com/FstwFsDv

 

Try This

Code:java

package me.MichaelSaber.Strong;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
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.plugin.java.JavaPlugin;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
public class Core extends JavaPlugin implements Listener {
 
    public void onEnable() {
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
    }
 
 
    @EventHandler
    @SuppressWarnings("deprecation")
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player player = e.getPlayer();
        if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction().equals(Action.RIGHT_CLICK_AIR)) {
            if (player.getItemInHand().getType().equals(Material.RED_MUSHROOM)) {
                player.sendMessage(ChatColor.GREEN + "You Are Now Invisble!");
                player.addPotionEffect(new PotionEffect(PotionEffectType.INVISIBILITY, 300, 300));
                ItemStack item = player.getItemInHand();
                item.setAmount(1);
                player.getInventory().removeItem(item);
                player.updateInventory();
            }
        }
    }
}

 

Sour when you take a stack it gets rid of all, how do you make it only get rid of one at a time?

 

Sour thanks it works!

 

Sour Please don't spoonfeed people entire classes (and on top of that not even say what you fixed)...Also, you can set the item in hand to null, that's how it's supposed to work.

Besides, what you're doing is calling this method. That in turn does this to call this method. Which sets the item to null.

 

An ItemStack does not necessarily mean that it contains 64 items. I set the amount to one, so it will only remove one item.

No problem.

Ya, probably shouldn't have handed it to him, but whatever.

Also, does setting an item to null not remove the whole stack? He only wanted to remove one item, not all of them.