player.getLastDamageCause().getEntity(); returning the player!

Hello! I've ran into some trouble whilst trying to log the player that previously hit a player typing a command.

Here is what happens:
1. Player1 hits player2.
2. Player2 types /hub.
3: This code is ran: Bukkit.getServer().broadcastMessage(Player1.getName());
4. This message is broadcasted: Player2.

May be badly explained, but I hope you get the general picture. It's not returning what it's supposed to?
Please help me with this. Thanks in advance!

P.S: Player1 is a Player variable(Player Player1 = (Player) player.getLastDamaggeCause().getEntity();

The variable "player", is the sender of the command.

 

so then Player1 is Player2, according to that logic:

Player2 types /hub -> Player1 is assigned from the executor of command which is Player2 -> Player2's name is broadcast/

 

1. There's no need to use Bukkit.getServer().broadcastMessage(String), you can just use Bukkit.broadcastMessage(String), see here.
2. I strongly suggest including source code next time, instead of trying to describe your problem with variable names...

Now onto your problem, getEntity() returns the subject of the DamageEvent, not the damager. To get the damager, you need to check:

Code:java

Player.getLastDamageCause() instanceof EntityDamageByEntityEvent

and then run the following code to get the damager

Code:java

Player damager = ((EntityDamageByEntityEvent) player.getLastDamageCause()).getDamager();

I haven't tested this, but looking at CraftBukkit's source code, it should work. By the way, if someone knows a 'nicer' way of doing this, please show it to me!

 

I will. Thank you for your generous help.

EDIT: There seem to have been an issue on reload. It works great, thanks you for help! I removed all the information in this post since it is unnecessary.