Solved Clear item in player hand. Bukkit bug???

Hello.

In my plugin I made it so to skip the respawn menu. I used "EntityDamageEvent" to see when the player health would get below or equal to zero, they would then be teleported to random location.

The problem is, if that player dies for example with an iron sword, after they respawn their damage(without any damage changing item in hand) will stay the same as if they have an iron sword in their hand untill they actually use any other damage changing item(like word, shovel or axe). How can I fix this?

I use bukkit-1.7.2-R0.3

This is the method I use to clear the player.

Code:Java

private void clearPlayer(Player p){
    p.setHealth(20);
    p.getInventory().clear();
    p.getEquipment().clear();
    p.setItemInHand(null);
    p.setItemOnCursor(null);
    p.getInventory().setHeldItemSlot(0);
    p.getInventory().setItemInHand(null);
    for(PotionEffect pet: p.getActivePotionEffects()){
        p.removePotionEffect(pet.getType());
    }
    p.setExp(0);
    p.setLevel(0);
    p.setFoodLevel(20);
    p.setExhaustion(0);
    p.setFallDistance(0);
    p.setFireTicks(0);
    p.setFlying(false);
    p.setGameMode(GameMode.SURVIVAL);
    p.setSaturation(20);
    p.setSneaking(false);
    p.setSprinting(false);
}

When the player dies, they get banned for 30 seconds. After the 30 seconds, they should come back as if they have been respawned(thats what I want). But somehow, their damage is equal to the last item-damage they had before they died.
This is the where I call this method.

Code:Java

@EventHandler
public void onPlayerDamage(EntityDamageEvent event){
    if(event.getEntityType() == EntityType.PLAYER){
 
        double dmg;
        double curHp;
        Player p = (Player)event.getEntity();
        dmg = event.getDamage();
        curHp = ((Player)event.getEntity()).getHealth();
 
        if (curHp - dmg <= 0){ //<------- Here I check if they would die by this hit
            event.setCancelled(true); //<------ Here I cancel the EntityDamageEvent
 
            p.setMaximumNoDamageTicks(200);
            p.setNoDamageTicks(200);
            clearPlayer(p); //<--------- Here I use my clearPlayer(Player) method(player is still online here)
 
            //The rest is not important, its a custom player data file which I am checking for things.
            ZNHardcore.pList.get(p.getName()).setStillAlive(false);
            ZNHardcore.pList.get(p.getPlayer().getName()).setKickDur(deathPenalty);
            ZNHardcore.pList.get(p.getPlayer().getName()).setDead();
            ZNHardcore.pList.get(p.getPlayer().getName()).setKick();
            p.kickPlayer(MSG_DEATH_PENALTY); //<----Here I kick the player.
 
            if (ZNHardcore.TempPList.get(p.getName()).isInBattle()){
                ZNHardcore.TempPList.get(p.getName()).showDamageScoreboard();
 
                Player mostDamager = plugin.getServer().getPlayer(ZNHardcore.TempPList.get(p.getName()).getMostDamager());
                if(mostDamager.isOnline()){
                    ZNHardcore.pList.get(mostDamager.getName()).addCredits(100);
                    ZNHardcore.pList.get(mostDamager.getName()).setpKill();
                }else{
                    ZNPlayerData tempLoadMostDamager = new ZNPlayerData(plugin);
                    tempLoadMostDamager.LoadProfileData(p);
                    tempLoadMostDamager.addCredits(100);
                    tempLoadMostDamager.setpKill();
                    tempLoadMostDamager.SaveProfile();
                }
            }
            @SuppressWarnings("unused")
            BukkitTask task = new ZNThreader(plugin, ZNHardcore.pList.get(p.getName())).runTaskTimer(this.plugin, 2, 2);
            ZNHardcore.pList.remove(p.getName());
        }
    }
}

 

I dont understand.

 

Post the full event.

 

try

Code:java

p.setItemInHand(Material.AIR);

 

eezjzbzbzb Use this.

Code:java

player.getInventory().setItemInHand(null);

 

Wat?

Just wat?

 

eezjzbzbzb Even though it's a deprecated method, try:

Code:java

player.updateInventory();

I don't exactly understand what your problem is, but maybe this helps

 

What he's doing is known to cause weird bugs.

 

Garris0n Oh, okay I wasn't quite sure what he meant, but it worth a try

 

I update the post, I will try what you guys suggested now. This is really a critical point for my plugin

EDIT: I actually thought of this idea today... I wanna try if I make the player drop their items before they get kicked and then removed those items from the map, but I couldn't figure out how to drop the player items...
So if anyone could let me know how to drop player items, I will try that aswell..


EDIT2: Some didn't understand whats the problem. To put it simply:
Plugin kicks players right before they die and resets their profile, clearing their inventory is one of the processes the profile-clearing takes. But when they come back, their damage is the same as the last weapon they had in their hand before they got kicked and profile-cleared.
So they don't have diamond sword, but do same damage as diamond sword because they had a diamond sword in their hand before they died.

 

Why don't you clear their inventory on login instead?

 

Garris0n hmmm, thats a good idea... This sounds like something I would depend on if there is really nothing else I can do.

Wruczek and Th3Br1x I just tried what you suggested, it didn't change anything...
__Sour I have already try that before, thats the first thing I tried.

I tried so many things just now, also clear inv on login, I found out that clearing the inventory somehow doesnt clear items that do damage, it removes them, but if they are in hand, the damage stays in hand while the item is gone.

Well I found a way around which is sufficient for my plugin, but the problem is still not fixed.
I made my plugin remove the playerdata file from the world its running on, so it's as if the player is new to that world.

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