Teleport player on death

So I am working on my KitPvP plugin and I want it so when a player gets killed they will respawn where I set the respawn location. Not at the server spawn.

This is what i currently have

Main set respawn point.

Code:java

    if (cmd.getName().equalsIgnoreCase("setdeath")) {
        if (p.hasPermission("cpvp.admin")) {
            settings.getData().set("death.world", p.getLocation().getWorld().getName());
            settings.getData().set("death.x", p.getLocation().getX());
            settings.getData().set("death.y", p.getLocation().getY());
            settings.getData().set("death.z", p.getLocation().getZ());
            settings.saveData();
            p.sendMessage(ChatColor.AQUA + "[PVP] " + ChatColor.GREEN + "Set Death Point!");
            return true;
    }
    } 

Respawn event that doesnt work.

Code:java

    @EventHandler
    public void onPlayerRespawn(PlayerRespawnEvent e){
        Player p = e.getPlayer();
        if(plugin.pvp.contains(p)){
            World w = Bukkit.getServer().getWorld(plugin.settings.getData().getString("death.world"));
            double x = plugin.settings.getData().getDouble("death.x");
            double y = plugin.settings.getData().getDouble("death.y");
            double z = plugin.settings.getData().getDouble("death.z");
            //p.teleport(new Location(w, x, y, z));
            Location loc = new Location (w, x, y, z);
        e.setRespawnLocation(loc);
    }
}

 

I don't see why the p.teleport() method shouldn't work.

Try making a new location with the coordinates that you provides + the world and the use the code:

Code:java

p.teleport(location);

If that doesn't work, tell me and I'll try to help you out!

 

Mattkx4 So i used this and it didnt work. I just respawned back at the server spawn not the location i set with /setdeath

Code:java

    @EventHandler(priority=EventPriority.HIGH)
    public void onPlayerRespawn(PlayerRespawnEvent e){
        Player p = e.getPlayer();
        if(plugin.pvp.contains(p)){
            World w = Bukkit.getServer().getWorld(plugin.settings.getData().getString("death.world"));
            double x = plugin.settings.getData().getDouble("death.x");
            double y = plugin.settings.getData().getDouble("death.y");
            double z = plugin.settings.getData().getDouble("death.z");
            Location loc = new Location (w, x, y, z);
            p.teleport(loc);
        //e.setRespawnLocation(loc);
    }
}

 

TheAJ471
I believe it may be a problem with the coordinates you're giving. I'm not for sure though, since I can't see your entire project!

Look to my response on this post for info on how to make a good config file!
http://forums.bukkit.org/threads/why-wont-my-config-yml-load-the-data-it-should.248008/#post-2353931

 

I am also in the process of making a kitpvp plugin. I have setup re-spawns for my plugin! Here is an example of my class:

Code:java

package me.chasertw123.kitpvp.misc;
 
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
 
import me.chasertw123.kitpvp.KitPvP;
 
public class Respawn implements Listener {
 
private static KitPvP plugin;
 
    public Respawn(KitPvP plugin) {Respawn.plugin = plugin;}
 
    public static void setRespawn(Player p) {
        plugin.getConfig().set("Respawn.X", p.getLocation().getX());
        plugin.getConfig().set("Respawn.Y", p.getLocation().getY());
        plugin.getConfig().set("Respawn.Z", p.getLocation().getZ());
        plugin.getConfig().set("Respawn.Yaw", p.getLocation().getYaw());
        plugin.getConfig().set("Respawn.Pitch", p.getLocation().getPitch());
        plugin.saveConfig();
        Messages.sendMessage(p, ChatColor.GREEN + "You have set the respawn point!");
    }
 
    public static void Spawn(Player p) {
        Double X = plugin.getConfig().getDouble("Respawn.X");
        Double Y = plugin.getConfig().getDouble("Respawn.Y");
        Double Z = plugin.getConfig().getDouble("Respawn.Z");
        Float Yaw = (float) plugin.getConfig().getDouble("Respawn.Yaw");
        Float Pitch = (float) plugin.getConfig().getDouble("Respawn.Pitch");
 
        Location loc = new Location(p.getWorld(), X, Y, Z, Yaw, Pitch);
 
        p.teleport(loc);
    }
}
 

If you have any questions feel free to ask! Hope this helps!

 

try delaying it by 1 tick. It might be a problem because the code for the other plugin says to teleport you to the server spawn. Maybe IDK. try it...

 

willy00 So how would I do this and add it in?

chasertw123 I dont really know how I would use this :/ I set it up for my plugin but now what

Mattkx4 I dont think it would be though because i did almost the same thing to save the coordinates for joining and quiting the game

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

 

TheAJ471 It is very simple you just call the method you created.

 

I have a way to do this; however, it will force a player to respawn - it won't give them the option: is that okay?

 

Wizehh Yes that is definatly ok. That actually sounds sort of better in a way.

 

Here you are*:

PHP:

@EventHandler
public void onPlayerDeath(final PlayerDeathEvent e) {
    new BukkitRunnable() {
        public void run() {
            try {
                Object nmsPlayer = e.getEntity().getClass().getMethod("getHandle").invoke(e.getEntity());
                Object con = nmsPlayer.getClass().getDeclaredField("playerConnection").get(nmsPlayer);
                Class < ? > EntityPlayer = Class.forName(nmsPlayer.getClass().getPackage().getName() + ".EntityPlayer");
                Field minecraftServer = con.getClass().getDeclaredField("minecraftServer");
                minecraftServer.setAccessible(true);
                Object mcserver = minecraftServer.get(con);
                Object playerlist = mcserver.getClass().getDeclaredMethod("getPlayerList").invoke(mcserver);
                Method moveToWorld = playerlist.getClass().getMethod("moveToWorld", EntityPlayer, int.class, boolean.class);
                moveToWorld.invoke(playerlist, nmsPlayer, 0, false);
            } catch (Exception ex) {}
        }
    }.runTaskLater(plugin, 2);
}

I did not write this.
This requires NMS, so make sure you import from the Craftbukkit JAR.

 

Wizehh Ok but where would this teleport them. How would I add in the coordinates and world from the command and use them in this?

chasertw123 This didnt work. I just tested it out

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

 

Can you post how you used my code?

 

chasertw123 So I have this like you said.

Code:java

package me.TheAJ471.cpvp;
 
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
 
import me.TheAJ471.cpvp.Main;
 
public class Respawn implements Listener {
 
private static Main plugin;
 
    public Respawn(Main plugin) {Respawn.plugin = plugin;}
 
    public static void setRespawn(Player p) {
        plugin.settings.getData().set("Respawn.X", p.getLocation().getX());
        plugin.settings.getData().set("Respawn.Y", p.getLocation().getY());
        plugin.settings.getData().set("Respawn.Z", p.getLocation().getZ());
        plugin.settings.getData().set("Respawn.Yaw", p.getLocation().getYaw());
        plugin.settings.getData().set("Respawn.Pitch", p.getLocation().getPitch());
        plugin.settings.saveData();
        p.sendMessage(ChatColor.GREEN + "You have set the respawn point!");
    }
 
    public static void Spawn(Player p) {
        Double X = plugin.settings.getData().getDouble("Respawn.X");
        Double Y = plugin.settings.getData().getDouble("Respawn.Y");
        Double Z = plugin.settings.getData().getDouble("Respawn.Z");
        Float Yaw = (float) plugin.settings.getData().getDouble("Respawn.Yaw");
        Float Pitch = (float) plugin.settings.getData().getDouble("Respawn.Pitch");
 
        Location loc = new Location(p.getWorld(), X, Y, Z, Yaw, Pitch);
 
        p.teleport(loc);
    }
}
 

Then i have this when the player types the command /setdeath

Code:java

Respawn.setRespawn(p);

Then I have this when the player death event happens

Code:java

Respawn.Spawn(p);

 

OK so what you can do is replace:

Code:java

public Respawn(Main plugin) {Respawn.plugin = plugin;}

With:

Code:java

@SuppressWarnings("static-access")
public Respawn(Main plugin) {this.plugin = plugin;}

Also replace Main with the name of your main class.

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

 

Cant you just use an event for this?

 

Right below this line:

PHP:

 moveToWorld.invoke(playerlist, nmsPlayer, 0, false);

you could teleport them somewhere:

PHP:

e.getEntity().teleport(world, x, y, z);

 

Best would be to add some debug messages to be sure your code is properly executed. (the PlayerRespawnEvent)
Because you need to be sure that the event actually fires and that your array
"plugin.pvp" does contain the player who is about to respawn.
So do some chat messages to check that.

This should be entirely sufficient to let the player respawn where you want:

Code:

e.setRespawnLocation(loc);

Wizehh Using NMS for this is overkill in my opinion. The PlayerRespawnEvent has everything to accomplish this task.