Get EntityDamageEvent weapon

Is there any way I can get the weapon/cause of damage from EntityDamageEvent?
Get item in hand doesn't work well with bows :/

 

XxShadow_NinjaxxX Just use the getCause() method which returns a damagecause. To get the weapon, check if getDamager() is an instance of Player, cast the damager to a player and use getItemInHand().getType() to get the material which the damaging player is holding.

 

GameplayJDK
There's the problem I'm talking about.
Ex: PlayerA shoots an arrow at PlayerB and switches to a sword, when PlayerB gets shot, I get the sword, not the bow

 

XxShadow_NinjaxxX So you could assume, that if the damager is instance of an Arrow, and the projectile source (the shooter) of that arrow is instance of a player, he has launched the arrow using a bow.. Right?

 

Is there a way to tell which bow it was shot from? Or do you have to loop through the players inventory?
EDIT:
Cause there are people who just like to screw with you, and try and break plugins :/

 

XxShadow_NinjaxxX Another possibility would be to keep traxk of all projectiles that are shot, store data about the shooter, the item in his hand, the location and the projectile in a new object, and listen for projectile to compare the projectile with the ones, which are stored and get the shooter and the item that way.
Or just add a custom metadata value to the projectile which identifies the player and bow it comes from.

 

XxShadow_NinjaxxX

This is a really good question. You could use an EntityShootBowEvent, get the projectile and use setMetadata to set the projectile's source to the bow:

Code:java

@EventHandler
public void onBowShoot(EntityShootBowEvent event){
    if (event.getEntity() instanceof Player){
        event.getProjectile().setMetadata("bow", new FixedMetadataValue(plugin, event.getBow()));
    }
}

Then, in your damage event, get the projectile, and retrieve the metadata from it:

Code:java

if (arrow.hasMetadata("bow")){
    ItemStack bow = (ItemStack)arrow.getMetadata("bow").get(0).value();
}

To avoid casting to ItemStack, you could set the value of the Metadata to an Integer, which stores the index of the bow in the player's inventory. Then, you could use:

Code:java

event.getShooter().getInventory().getItem(index);

 

Is it possible to store all the bow's info in the arrow's metadata? And would that affect server speed or anything?

 

XxShadow_NinjaxxX
This method would store a reference to the bow (ItemStack). Since arrows don't last long, you don't need to worry that much. See my edit in my last post.

 

XxShadow_NinjaxxX You should be able to store as many values in metadata as you want, without causing a significant change of ingame performance.