Could not pass Player OnDeathEvent to my plugin

Hi guys,
Im trying to teleport players to a certain spot when they die.
I keep getting a stacktrace in which bukkit cant pass on the death event to my plugin.

The error: http://gyazo.com/0f7db57147c3f5c5ac683a7db2d2a976


Heres the code:

Code:java

 @EventHandler
public void onPlayerDeath(PlayerDeathEvent e) { // checks for death of players
 
final Player p = e.getEntity();
boolean joincheck = false;
for (int i = 1; i <= total; i++) {
Player spot = savedPlayers[I];[/I]
[I] if (spot.equals(p)) {[/I]
[I] joincheck = true;[/I]
 
[I] }[/I]
 
[I] }[/I]
[I] if (joincheck == true && gameRunning == true) {[/I]
[I] int tpx = getConfig().getInt("SpawnX");[/I]
[I] int tpy = getConfig().getInt("SpawnY");[/I]
[I] int tpz = getConfig().getInt("SpawnZ");[/I]
[I] World world = p.getWorld();[/I]
[I] Location tpa = new Location(world, tpx, tpy, tpz);[/I]
[I] p.teleport(tpa);[/I]
[I] p.getInventory().clear();[/I]
[I] p.getInventory().setHelmet(new ItemStack(Material.LEATHER_HELMET, 1));[/I]
[I] p.getInventory().setChestplate(new ItemStack(Material.LEATHER_CHESTPLATE, 1));[/I]
[I] p.getInventory().setLeggings(new ItemStack(Material.LEATHER_LEGGINGS, 1));[/I]
[I] p.getInventory().setBoots(new ItemStack(Material.LEATHER_BOOTS, 1));[/I]
[I] p.getInventory().addItem(new ItemStack(Material.IRON_SWORD, 1));[/I]
 
[I] }[/I]
 
[I] }[/I]
[I][/I]

It isnt a listening issue or event log issue because i have other events like PlayerMoveEvents, registered just fine.


Thanks Everyone!

 

theodorehip Put the full main class in pastebin.

 

It's over 800 lines or so, you sure you want to look through all of it?

 

theodorehip
Yes

 

Actually I fixed it guys, thanks, i switched out the OnDeath Event with a RespawnEvent, i dont know what i was thinking. Ill delete this thread in a few minutes.
Thanks guys.

 

theodorehip
Don't delete it, others may use it.

 

I suppose so, i just jumped the gun because the thread really wasnt used for what it should have been. But i can leave it up.

 

I want to see freaking 800 lines of code.
plz I liek coed

 

theodorehip instead of a Player[], you might want to use a Set<UUID>. It's much faster than looping over an array, plus there is no real need to store the whole Player object.