Teleporting into the ground

I am making a minigame plugin where people teleport to a lobby, start, and end. See the bottom for what the error is.

Config File (open) Config File (close)

Code:

arenas:
  Test:
    world: world
    joinX: 11.0
    joinY: 70.0
    joinZ: -21.0
    startX: 15.0
    startY: 70.0
    startZ: -21.0
    endX: 18.0
    endY: 70.0
    endZ: -22.0
    maxPlayers: 1

ArenaManager Class (open) ArenaManager Class (close)

Code:java

package me.br456.WizardWar;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.entity.Player;
 
public class ArenaManager {
 
    SettingsManager settings = SettingsManager.getInstance();
    private static ArenaManager am = new ArenaManager();
 
    public static ArenaManager getManager() {
        return am;
    }
 
    public Arena getArena(String name) {
        for(Arena a : Arena.arenas) {
            if(a.getName().equals(name)) {
                return a;
            }
        }
        return null;
    }
 
    @SuppressWarnings("deprecation")
    public void addPlayer(Player player, String name) {
        if(getArena(name) != null) {
            Arena arena = getArena(name);
 
            if(!arena.isFull()) {
 
                if(!arena.isInGame()) {
 
                    ItemSaver.addInv(player.getName(), player.getInventory());
 
                    player.getInventory().clear();
                    player.setHealth(20);
                    player.setFireTicks(0);
 
                    player.teleport(arena.getLobby());
                    arena.getPlayers().add(player.getName());
                    int playersLeft = arena.getMaxPlayers() - arena.getPlayers().size();
 
                    arena.sendMessage(ChatColor.GRAY + "[" + ChatColor.YELLOW + "Wizard" + ChatColor.GRAY + "]" + ChatColor.WHITE +
                            player.getName() + " has joined! (" + arena.getPlayers().size() + "/" + arena.getMaxPlayers() + ")");
 
                    if(playersLeft == 0) {
                        startArena(name);
                    }
 
                } else {
                    player.sendMessage(ChatColor.RED + "The game has already started!");
                }
            } else {
                player.sendMessage(ChatColor.RED + "The arena is full!");
            }
        } else {
            player.sendMessage("Arena doesn't exist!");
        }
    }
 
    @SuppressWarnings("deprecation")
    public void removePlayer(Player player, String name) {
        if(getArena(name) != null) {
 
            Arena arena = getArena(name);
 
            if(arena.getPlayers().contains(player.getName())) {
                player.getInventory().clear();
 
                player.getInventory().setContents(ItemSaver.getInv(player.getName()));
                player.getInventory().setArmorContents(ItemSaver.getArmor(player.getName()));
 
                player.setHealth(20);
                player.setFireTicks(0);
 
                player.teleport(arena.getEnd());
                arena.getPlayers().remove(player.getName());
 
                arena.sendMessage(ChatColor.GRAY + "[" + ChatColor.YELLOW + "Wizard" + ChatColor.GRAY + "]" + ChatColor.WHITE +
                        player.getName() + " has left! (" + arena.getPlayers().size() + "/" + arena.getMaxPlayers() + ")");
            } else {
                player.sendMessage("You aren't in the arena!");
            }
        } else {
            player.sendMessage("Arena doesn't exist!");
        }
    }
 
    public void startArena(String name) {
        if (getArena(name) != null) {           
            Arena arena = getArena(name);
 
            arena.sendMessage(ChatColor.GREEN + "The game has started!");
            arena.setInGame(true);
 
            for (String s: arena.getPlayers()) {                 
                Bukkit.getPlayer(s).teleport(arena.getStart());
                Bukkit.getPlayer(s).sendMessage("Welcome to the arena!");
            }           
        }   
    }
 
    @SuppressWarnings("deprecation")
    public void endArena(String name) {
        if(getArena(name) != null) {
 
            Arena arena = getArena(name);
 
            arena.sendMessage(ChatColor.GREEN + "The game is over");
            arena.setInGame(false);
 
            for (String s: arena.getPlayers()) {             
                Player player = Bukkit.getPlayer(s);
                player.getInventory().clear();
 
                player.getInventory().setContents(ItemSaver.getInv(player.getName()));
                  player.getInventory().setArmorContents(ItemSaver.getArmor(player.getName()));
 
                  player.setHealth(20);
                  player.setFireTicks(0);
 
                  player.teleport(arena.getEnd());
 
                arena.getPlayers().remove(player.getName());
 
            }
        }
    }
 
    public void loadArenas() {
 
        FileConfiguration config = settings.getData();
        if(config.getConfigurationSection("arenas") == null) return;
 
        for (String keys: config.getConfigurationSection("arenas").getKeys(false)) {
            String worldName = settings.getData().getString("arenas." + keys + ".world");
            World world = Bukkit.getWorld(worldName);
 
            double joinX = config.getDouble("arenas." + keys + ".joinX");
            double joinY = config.getDouble("arenas." + keys + ".joinY");
            double joinZ = config.getDouble("arenas." + keys + ".joinZ");
            Location joinLocation = new Location(world, joinX, joinY, joinZ);
 
            double startX = config.getDouble("arenas." + keys + ".startX");
            double startY = config.getDouble("arenas." + keys + ".startY");
            double startZ = config.getDouble("arenas." + keys + ".startZ");
 
            Location startLocation = new Location(world, startX, startY, startZ);
 
            double endX = config.getDouble("arenas." + keys + ".endX");
            double endY = config.getDouble("arenas." + keys + ".endX");
            double endZ = config.getDouble("arenas." + keys + ".endX");
 
            Location endLocation = new Location(world, endX, endY, endZ);
 
            int maxPlayers = config.getInt("arenas." + keys + ".maxPlayers");
 
            Arena arena = new Arena(keys, joinLocation, startLocation, endLocation, maxPlayers);
 
            Arena.arenas.add(arena);
        }
    }
 
    public void createArena(String name, Location lobby, Location start, Location end, int maxPlayers) {
 
        Arena arena = new Arena(name, lobby, start, end, maxPlayers);
 
        settings.getData().set("arenas." + name, null);
 
        String path = "arenas." + name + ".";
 
        settings.getData().set(path + "world", lobby.getWorld().getName());
 
        settings.getData().set(path + "joinX", lobby.getX());
        settings.getData().set(path + "joinY", lobby.getY());
        settings.getData().set(path + "joinZ", lobby.getZ());
 
        settings.getData().set(path + "startX", start.getX());
        settings.getData().set(path + "startY", start.getY());
        settings.getData().set(path + "startZ", start.getZ());
 
        settings.getData().set(path + "endX", end.getX());
        settings.getData().set(path + "endY", end.getY());
        settings.getData().set(path + "endZ", end.getZ());
 
        settings.getData().set(path + "maxPlayers", maxPlayers);
 
        settings.saveData();
 
        Arena.arenas.add(arena);
    }
}
 

When I remove a player, they get teleported to the bottom of the world and at a different X and Z location(Using removePlayer method). When I teleport to the lobby(addPlayer method) this works fine. Why does this happen?

 

br456
What is the Location that you get from arena.getEnd()?

 

alex123099
It returns the location "end" from the arena object. The arena object is created in the create arena or load arena method.

 

br456 Just to make sure, when you get teleported, is your coordinates (0,0,0)? If it is, that's the default location you get teleported to in the event that there's an error with your defined location.