Player drinks potion and bottle disappears.

So, when someone drinks a POTIONBOTTLE I would like it to disappear. Any suggestions on how to make it work? I haven't had any luck in my tries so far.

 

ItemConsumeEvent?

 

@Skionz And how would I limit the itemconsumeevent strictly to potion bottles? I've been having all my trouble due to that.

 

Show your code....

 

@Aephout14 I'm trying to make it right now. I haven't been able to advance due to my flustering.

 

@mrdude123 With the getItem method

 

@Konato_K Ah, I didn't think of that. Thank you so much.

I'm really stuck. Can someone give me a small example?

 

How are you stuck?

 

@Aephout14 I can't figure it out!

 

I am probably wrong but.

Get the Item, Check If its a Water Bottle, If It Is, Remove It from the players Inventory.

 

@mrdude123 Did you mark this thread as Solved on purpose?

 

@Aephout14 No, I want it to be so that if someone drinks the potion, the bottle then disappears from their inventory.

@nverdier No, sorry about that.

 

So like a Player drinks the water bottle, Then Check If the Item In there Inventory Is Bottle (Not Water Bottle) then remove it, Or you mean like Speed and Health potions etc?

 

@Aephout14 Yes.

Code:

package me.thecerealkill3r.bottledispose;

import org.bukkit.GameMode;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.player.PlayerItemConsumeEvent;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.scheduler.BukkitRunnable;

public class Main extends JavaPlugin {

  @EventHandler
  public void onPlayerConsume(PlayerItemConsumeEvent event)
  {
    final Player player = event.getPlayer();
    Material mat = event.getItem().getType();

    if (player.getGameMode() == GameMode.CREATIVE) return;
    if (mat != Material.POTION) return;
    if (event.getItem().getAmount() > 1) return;

    new BukkitRunnable()
    {
      public void run() {
        player.setItemInHand(null);
      }
    }
    .runTaskLater(this, 1L);
 


        return;
}
}

This is what I came up with so far. If you can see the Material.POTION, how can I change that to a specific potion? That's all I need.

 

I think you need to use short values, Like doing

Code:

Material.POTION, (short) 8197

I am not sure but doing this would be a Healing Potion.

 

Code:

if (mat != Material.8197 ) return;

That doesn't work.
Neither does

Code:

 if (mat !=Material.POTION.8197) return; 

 

Thats not what I meant, Its Material.POTION (short) 8197

 

@Aephout14 It's telling me this - The operator / is undefined for the argument type(s) Material, short

 

I don't know then...

 

@mrdude123 What are you doing?!? Please post all of your code.

 

Code:

package me.thecerealkill3r.bottledispose;

import org.bukkit.GameMode;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.player.PlayerItemConsumeEvent;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.scheduler.BukkitRunnable;

public class Main extends JavaPlugin {

  @EventHandler
  public void onPlayerConsume(PlayerItemConsumeEvent event)
  {
    final Player player = event.getPlayer();
    Material mat = event.getItem().getType();

    if (player.getGameMode() == GameMode.CREATIVE) return;
    if (mat != Material.POTION(short)8197 ) return;
    if (event.getItem().getAmount() > 1) return;

    new BukkitRunnable()
    {
      public void run() {
        player.setItemInHand(null);
      }
    }
    .runTaskLater(this, 1L);
 


        return;
}
}

 

@mrdude123 You will have to compare ItemStacks instead of Materials if you want to check if it's a specific potion. Although it seems like you don't need to check for specific potions, so you would just check if the Material is POTION.

 

I'm not sure what you mean. Can you give an example?

 

@mrdude123 If you need specific values for the potion you can use ItemStack#getDurability

 

@Konato_K @mrdude123 Just use this site http://www.minecraftinfo.com/IDList.htm

 

You need to implement Listener, and register your events for anything to work. http://bukkit.org/threads/tutorial-registering-events.200289/

 

Ok. this is what I got. I made it work, but now if I drink a water bottle it's bottle disappears. Can someone show me how to exempt the waterbottle?

Plugin is supposed to get rid of bottles from potions like healing or speed, not water bottles.

Code:

package npb;

import org.bukkit.GameMode;
import org.bukkit.Material;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerItemConsumeEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;

public class Main extends JavaPlugin
  implements Listener
  {

  @EventHandler
  public void consume(final PlayerItemConsumeEvent e) {
    ItemStack item = e.getItem();
    if (item.getType().equals(Material.POTION))
      getServer().getScheduler().runTaskLater(this, new Runnable() {
        public void run() {
            if (e.getPlayer().getItemInHand().getType().equals(Material.POTION))
          if ((!e.getPlayer().getGameMode().equals(GameMode.CREATIVE)) && (
            (e.getPlayer().getItemInHand().getType().equals(Material.GLASS_BOTTLE)) || (e.getPlayer().getItemInHand().getType().equals(Material.POTION))))
            e.getPlayer().getInventory().setItemInHand(null);
        }
      }
      , 2L);
  }
}

@nverdier

Do you know?

 

@mrdude123 Just check if the item being consumed (PlayerItemConsumeEvent#getItem()) has a type of potion. If it does, then remove a glass bottle.

 

@nverdier A type of a potion? How would I do that?

I mean, what would I say? POTION. ?

 

@mrdude123 ItemStack#getType(), and compare it to Material.POTION.