Solved Soup plugin trouble

Hello, I've been trying to make a soup plugin like mcPvP's but I haven't been successful. Whenever I use the soup it doesn't disappear. Please help.

 

@WasabiLove does the potion effect fire?

 

SuperOmegaCow
what do you mean by fire? If you mean work than yes it does.

 

I think you need to do setItemInHand(Material.AIR)

 

monkeymanboy still doesn't work.

 

Try this;

Code:java

player.getInventory().getItemInHand().setAmount(player.getInventory().getItemInHand().getAmount()-1);

Btw, I suggest adding health to the player instead of giving them the "Heal" potion effect.

 

Still doesn't work and I was never able to add health it always says getHealth() was ambiguous for entity type Player

 

I don't know, sorry But to fix the getHealth() was ambigiuous.... etc Import Bukkit.jar as well as CraftBukkit.jar (Make sure to make Bukkit.jar a priority)
http://dl.bukkit.org/downloads/bukkit/

 

Try:

Code:java

player.getInventory().removeItem(player.getItemInHand());

To fix the health issue cast the entity into a Damageable:

Code:java

Damageable damageable = player;
damageable.setHealth(damageable.getHealth() + 3.0D);

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

 

AstramG Great the soup works now thank you for that AstramG, but the soup still doesn't disappear

AstramG Wait I just noticed an error once the player eats enough soup to get up to say 19 health the server says the health has to be between 0 and 20

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

 

Do a check to see if their health is allowed to go that high and if it's over 17 in your case then just set their health to their max health.

 

AstramG So like this:

Code:java

if(damageable.getHealth() == 18){
damageable.setMaxHealth();
}
 

AstramG Also what should I do about them still not disappearing

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

 

I don't know what to do about disappearing but this is what you should do for health:

Code:java

if (damageable.getHealth() + 3.0D > damageable.getMaxHealth()) {
  damageable.setHealth(damageable.getMaxHealth());
} else {
  damageable.setHealth(damageable.getHealth() + 3.0D);
}

 

AstramG
I still get this error:

Code:

22:44:10 [SEVERE] Could not pass event PlayerInteractEvent to TheThreeKingdoms v6.0
org.bukkit.event.EventException
    at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:427)
    at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
    at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
    at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
    at org.bukkit.craftbukkit.v1_6_R2.event.CraftEventFactory.callPlayerInteractEvent(CraftEventFactory.java:190)
    at net.minecraft.server.v1_6_R2.PlayerInteractManager.interact(PlayerInteractManager.java:373)
    at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java:628)
    at net.minecraft.server.v1_6_R2.Packet15Place.handle(SourceFile:58)
    at net.minecraft.server.v1_6_R2.NetworkManager.b(NetworkManager.java:296)
    at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java:116)
    at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
    at net.minecraft.server.v1_6_R2.DedicatedServerConnection.b(SourceFile:30)
    at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:590)
    at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:226)
    at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:486)
    at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java:419)
    at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:582)
Caused by: java.lang.IllegalArgumentException: Health must be between 0 and 20.0
    at org.bukkit.craftbukkit.v1_6_R2.entity.CraftLivingEntity.setHealth(CraftLivingEntity.java:75)
    at me.WasabiLover.TheThreeKingdoms.Soups.onPlayerInteractEvent(Soups.java:29)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
    at java.lang.reflect.Method.invoke(Method.java:597)
    at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425)
    ... 16 more

AstramG I get this error if I click the soup when I have full health

AstramG Never mind I fixed it, but it would be a real help if you could help me fix the soups disappear. But if you don't want to it's fine. Cause you've been a real help.

AstramG
Wait sorry about this but I didn't notice it before, but the code you gave me doesn't exactly work here is my code:

Code:java

 @EventHandler
public void onPlayerInteractEvent(PlayerInteractEvent event){
 
Player player = event.getPlayer();
 
if(player.getItemInHand().getTypeId() == Material.MUSHROOM_SOUP.getId()){
 
Damageable damageable = player;
 
if(!(damageable.getHealth() == 20)){
 
damageable.setHealth(damageable.getHealth() + 3.0D);
 
if (damageable.getHealth() + 3.0D > damageable.getMaxHealth()) {
damageable.setHealth(damageable.getMaxHealth());
} else {
damageable.setHealth(damageable.getHealth() + 3.0D);
}
 
if(player.getInventory().getItemInHand().getTypeId() == Material.MUSHROOM_SOUP.getId()){
player.getItemInHand().setAmount(0);
}
}
}
}

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