Solved Spams death in chat?

I have written a plugin which kills a player on when they touch water but it seems to spam the chat with "Player died" when the plugin takes effect. There is no error in the console, it just spams the same message. Anyone got any idea of how I could fix this? Any help is appreciated.

 

Code?

 

Jake0oo0

Code:java

package me.faithdev.DeathWater;
 
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class DeathWater extends JavaPlugin implements Listener {
 
public void onEnable() {
getServer().getPluginManager().registerEvents(this, this);
}
 
@EventHandler
public void onPlayerMove(PlayerMoveEvent e) {
Player p = e.getPlayer();
 
if (p.getLocation().getBlock().getType() == Material.WATER
|| (p.getLocation().getBlock().getType() == Material.STATIONARY_WATER)) {
p.setHealth(0);
}
}
 
}
 

 

Faith While you are on the respawn screen, Bukkit probably thinks that you are still in the water that you died in.

 

I agree i think it still thinks ur their since ur a entity but a dead invisible one

 

Shockwave317 Compressions The only way I can think of fixing it is teleporting the player up out of water when they die, can you think of any other way?

 

You can use the isDead method for checks.

edit: this code works, just tested.

Code:

@EventHandler
    public void onPlayerMove(PlayerMoveEvent e) {
    Player p = e.getPlayer();
    if(p.isDead() == true)return;
    if (p.getLocation().getBlock().getType() == Material.WATER
    || (p.getLocation().getBlock().getType() == Material.STATIONARY_WATER)) {
    p.setHealth(0);
    }
    }

 

Code:

@EventHandler
public void onPlayerMove(PlayerMoveEvent e) {
Player p = e.getPlayer();
 
if (p.getLocation().getBlock().getType() == Material.WATER
|| (p.getLocation().getBlock().getType() == Material.STATIONARY_WATER) && p.getHealth() > 0) {
p.setHealth(0);
}
}

Tada!

 

CakePvP Thanks, appreciate it.

 

Just a tip, heres a better way to register listeners.

Code:

    @Override
    public void onEnable()
    {
        getServer().getPluginManager().registerEvents(new Listener(){
// Put listener here.
        }, this);
    }

 

LittleMonster7_
What's the point of making an anonymous class? It's much more organised if you have it in a separate class.

 

This thread was solved hours ago. Please stop commenting.

Also, Henzz, may I ask where you got that sig picture from? Would love to know.

 

Faith
Made it with Paint.NET, also may want to prefix it since you said it was already solved.