How to cancel death message if the player was killed with a bow

I am trying to cancel the death message when the killer was using a bow. Can you tell me what is wrong with my code?

Code:

public void onPlayerDeath(PlayerDeathEvent ev){
        final Player victim = ev.getEntity();
        Bukkit.broadcastMessage("test: " + victim.getKiller());
        if(victim.getKiller() instanceof Arrow){
            final LivingEntity shooter = ((Arrow)victim.getKiller()).getShooter();
            final Player bowman = (Player) shooter;
            ev.setDeathMessage("");
        }
   
    }

 

you cannot cancel the PlayerDeathEvent. You would have to calculate the death in the EntityDamageByEntityEvent (note that the getDamage() method does not account for armor, potions and enchantments) and cancel that
misread the post

 

MrSnare

First use the PlayerDeathEvent. Then check if the bow is in killers hand. If there is a bow so something like

Code:java

evt.setDeathMessage(ChatColor.GOLD + "");

I'll post a snippet in a minute

MrSnare

Code:java

 
@EventHandler
public void PlayerDeath(PlayerDeathEvent evt){
Player player = evt.getEntity();
Player killer = player.getKiller();
if(killer != null && killer.getItemInHand() == Material.BOW){
evt.setDeathMessage(""); //Put a ChatColor or somethink to prevent bukkit think the message is empty
system.printLn("A Player just was killed with a Bow");
}
}
 

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

 

Code:java

@EventHandler
public void PlayerDeath(PlayerDeathEvent evt)
{
    Player player = evt.getEntity();
    Player killer = player.getKiller();
    if(killer != null && killer.getItemInHand() != null && killer.getItemInHand().getType() == Material.BOW)
    {
        evt.setDeathMessage("");
    }
}

check if the ItemStack in hand is not null, and if the type is bow. Also, setting the message to "" would cancel the death message

 

Also you can use, evt.setDeathMessage(null)

 

Just check the death cause and send a message : "(nothing)"

 

MrSnare
I dont know if getting shot by bow has an unique static part of message, but if, you could just check the current death-message and set it to null on match.