Solved Saving Location to config.yml

Hello there, I am saving a location to the config file; This part works.

When I reload the server, the data in the key gets wiped and replaced with "{}" Does any one know why?

Here is the code:

Code:java

//Setting the config           
Location loc = player.getLocation();
 
            org.bukkit.World world = loc.getWorld();
            int x = loc.getBlockX();
            int y = loc.getBlockY();
            int z = loc.getBlockZ();
            float pitch = loc.getPitch();
            float yaw = loc.getYaw();
 
            FileConfiguration config = plugin.getConfig();
 
            String jl = "JOIN_LOCATION.";
            config.set(jl+"world", world);
            config.set(jl+"x", x);
            config.set(jl+"y", y);
            config.set(jl+"z", z);
            config.set(jl+"pitch", pitch);
            config.set(jl+"yaw", yaw);
            plugin.saveConfig();

Code:java

    //Getting the config
        FileConfiguration config = plugin.getConfig();
        String jl = "JOIN_LOCATION.";
        org.bukkit.World world = (org.bukkit.World) config.get(jl+"world");
        int x = (int) config.get(jl+"x");
        int y = (int) config.get(jl+"y");
        int z = (int) config.get(jl+"z");
        float pitch = (float) config.get(jl+"pitch");
        float yaw = (float) config.get(jl+"yaw");
 
        Location jLocation = new Location(world,x,y,z,yaw,pitch);
        player.teleport(jLocation);

 

I'm writting the code , wait.

 

change the world to a UUID

config.set(jl+"world",world.getUUID())

and

Bukkit.getWorld(UUID.fromString((String)config.get(jl+"world")));

 

getConfig().set("Loc.X", jLocation.getX());
getConfig().set("Loc.Y", jLocation.getY());
getConfig().set("Loc.Z", jLocation.getZ());

And to load the location use:
Location l = player.getLocation();
l.setX(getConfig().getDouble("Loc.X"));
l.setY(getConfig().getDouble("Loc.Y"));
l.setZ(getConfig().getDouble("Loc.Z"));
player.teleport(l);

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

 

you need the world...

Some servers have multiple worlds, so it does not save the Location properly

 

I already have this in my plugin. When I reload my plugin, It then removes all the data from the JOIN_LOCATION key

to this;

This happens every time I reload/stop the server.

Couldn't get that to work on my PlayerJoinEvent

I used this code:

Code:java

FileConfiguration config = plugin.getConfig();
        String jl = "JOIN_LOCATION.";
        org.bukkit.World world = (org.bukkit.World)UUID.fromString((String) config.get(jl+"world");
        int x = (int) config.get(jl+"x");
        int y = (int) config.get(jl+"y");
        int z = (int) config.get(jl+"z");
        float pitch = (float) config.get(jl+"pitch");
        float yaw = (float) config.get(jl+"yaw");
 
        Location jLocation = new Location(world,x,y,z,yaw,pitch);
        player.teleport(jLocation);
 
// I get a error on line 3

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

 

its Bukkit.getWorld(UUID.fromString((String) config.get(jl+"world")))

read my post clearly -_-

 

Still gets removed after reloading/stopping the server.

 

@CaLxCyMru
Could you send me the whole file?

 

What do you need? onEnable()?

 

he just said he needs it all

 

Yes, But I don't want to send my whole plugin to him

 

Well. it could be that somewhere deeper in your code is the issue. So he asks for it all. Because if you now send him the onEnable and there is no error he asks for the next part. and so on.

However if you post all at once here we can look at it right now and then see where the issue is and can help you faster

 

I will post all of the methods the config is called.
onEnable()

Code:java

@Override
    public void onEnable() {
 
        getCommands();
 
        eRegister();
 
        getConfig().options().copyDefaults(true);
        saveConfig();
 
 
 
    }

Setting the JoinLocation command

Code:java

public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if (!(sender instanceof Player)) {
 
            sender.sendMessage(plugin.prefix + "You are not permitted to use this command in console!");
            return true;
        }
 
        Player player = (Player)sender;
        if(cmd.getName().equalsIgnoreCase("joinlocation")){
 
            if(!(player.hasPermission(new Permissions().joinlocation))) {
 
                player.sendMessage(plugin.prefix + "The cow goes, NOO");
 
                return true;
            }
 
            Location loc = player.getLocation();
 
            org.bukkit.World world = loc.getWorld();
            int x = loc.getBlockX();
            double y = loc.getBlockY();
            double z = loc.getBlockZ();
            float pitch = loc.getPitch();
            float yaw = loc.getYaw();
 
            FileConfiguration config = plugin.getConfig();
 
            String jl = "JOIN_LOCATION.";
            config.set(jl+"world", world.getUID());
            config.set(jl+"x", x);
            config.set(jl+"y", y);
            config.set(jl+"z", z);
            config.set(jl+"pitch", pitch);
            config.set(jl+"yaw", yaw);
            plugin.saveConfig();
 
 
                player.sendMessage(plugin.prefix + "You have set the join location to where you are standing");
 
        return true;
 
        }
 
        return false;
 
    }

Player Join event that TP's Them to the location from the command.

Code:java

@EventHandler (priority = EventPriority.HIGHEST)
    public void onJoin(PlayerJoinEvent event) {
        Player player = event.getPlayer();       
 
        event.setJoinMessage(ChatColor.DARK_AQUA + player.getName() + ChatColor.GRAY +  " Joined");
 
        player.sendMessage(ChatColor.DARK_GRAY + "" + ChatColor.BOLD + " --------------------------------------------" );
        player.sendMessage(ChatColor.DARK_AQUA + "" + ChatColor.BOLD + "                      Viral Aftermath ");
        player.sendMessage(ChatColor.RED +  "                            Will YOU Survive?.. ");
        player.sendMessage(ChatColor.DARK_GRAY + "" + ChatColor.BOLD + " --------------------------------------------" );
 
        //Getting the config
        FileConfiguration config = plugin.getConfig();
        String jl = "JOIN_LOCATION.";
        org.bukkit.World world = Bukkit.getWorld(UUID.fromString((String)config.get(jl+"world")));
        double x = (double) config.get(jl+"x");
        double y = (double) config.get(jl+"y");
        double z = (double) config.get(jl+"z");
        float pitch = (float) config.get(jl+"pitch");
        float yaw = (float) config.get(jl+"yaw");
 
        Location jLocation = new Location(world,x,y,z,yaw,pitch);
        player.teleport(jLocation);
    }
 

config.yml

PHP:

EU1_IP: mc.hypixel.net // ingore this
EU1_PORT: 25565 // ignore this
JOIN_LOCATION:
    world:
    x:
    y:
    z:
    pitch:
    yaw:

 

bump

 

Yeah, yeah, blah, blah, send the whole code, this, that... God, it's simple. Your variables get wiped at reload/restart, and the player are already there, so the JoinEvent will not be triggered so, on the onEnable method you have to do a For loop thats RE-DO the JoinEvent for all players online.

Code:java

public void onEnable(){
for (Player player : this.getServer().getOnlinePlayers()){
FileConfiguration config = plugin.getConfig();
String jl = "JOIN_LOCATION.";
org.bukkit.World world = Bukkit.getWorld(UUID.fromString((String)config.get(jl+"world")));
double x = (double) config.get(jl+"x");
double y = (double) config.get(jl+"y");
double z = (double) config.get(jl+"z");
 
float pitch = (float) config.get(jl+"pitch");
float yaw = (float) config.get(jl+"yaw");
Location jLocation = new Location(world,x,y,z,yaw,pitch);
player.teleport(jLocation); }
}

And plus, there is no problem if there is no player online, the code is just skipped.

 

Thanks for the help man, I will try this out soon.

I thought it was something to do with the onEnable(). No idea why they wanted to see all my code. So just posted the code I thought would be affected.

samosaara I made a method in the main class, playerJoin() and put it in my onEnable() and also my PlayerJoinEvent. I still get the NPE after reloading.
Code:

Code:java

public void playerJoin(){
 
        for (Player player : this.getServer().getOnlinePlayers()){
            FileConfiguration config = plugin.getConfig();
            String jl = "JOIN_LOCATION.";
            org.bukkit.World world = Bukkit.getWorld(UUID.fromString((String)config.get(jl+"world")));
            double x = (double) config.get(jl+"x");
            double y = (double) config.get(jl+"y");
            double z = (double) config.get(jl+"z");
 
            float pitch = (float) config.get(jl+"pitch");
            float yaw = (float) config.get(jl+"yaw");
            Location jLocation = new Location(world,x,y,z,yaw,pitch);
            player.teleport(jLocation); }
    }

NPE:


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

 

Give me the method of the line 197 is in (from your main class) so we can see whats giving the null pointer thing.

 

There you go. Here is the code in text format;

Code:java

public void playerJoin(){
 
        for (Player player : this.getServer().getOnlinePlayers()){
            FileConfiguration config = plugin.getConfig();
            String jl = "JOIN_LOCATION.";
            org.bukkit.World world = Bukkit.getWorld(UUID.fromString((String)config.get(jl+"world")));
            double x = (double) config.get(jl+"x");
            double y = (double) config.get(jl+"y");
            double z = (double) config.get(jl+"z");
 
            float pitch = (float) config.get(jl+"pitch");
            float yaw = (float) config.get(jl+"yaw");
            Location jLocation = new Location(world,x,y,z,yaw,pitch);
            player.teleport(jLocation); }
    }

Here is a example of my config.yml

Before I set the join location:

PHP:

EU1_IP: mc.hypixel.net
EU1_PORT: 25565
JOIN_LOCATION:
  world:
  x:
  y:
  z:
  pitch:
  yaw:


I set the join location:

PHP:

EU1_IP: mc.hypixel.net
EU1_PORT: 25565
JOIN_LOCATION:
  world: !!java.util.UUID {}
  x: 1143
  y: 6.0
  z: -73.0
  pitch: 7.5000014
  yaw: -35.700005


And this is after a restart/reload:

PHP:

EU1_IP: mc.hypixel.net
EU1_PORT: 25565
JOIN_LOCATION: {}


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

 

The plugin variable isn't initialized make sure to initialize it before using.

 

Of course! I will try this now!

samosaara Its worked. But I now get this error while reloading:



I am using this method:

Code:java

public void playerJoin(){
 
        for (Player player : this.getServer().getOnlinePlayers()){
            FileConfiguration config = this.getConfig();
            String jl = "JOIN_LOCATION.";
            org.bukkit.World world = Bukkit.getWorld(UUID.fromString((String)config.get(jl+"world")));
            double x = (double) config.get(jl+"x");
            double y = (double) config.get(jl+"y");
            double z = (double) config.get(jl+"z");
 
            float pitch =  (float) config.get(jl+"pitch");
            float yaw = (float) config.get(jl+"yaw");
            Location jLocation = new Location(world,x,y,z,yaw,pitch);
            player.teleport(jLocation);
            }
    }

And here is my onCommand() where I set the config.

Code:java

public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if (!(sender instanceof Player)) {
 
            sender.sendMessage(plugin.prefix + "You are not permitted to use this command in console!");
            return true;
        }
 
        Player player = (Player)sender;
        if(cmd.getName().equalsIgnoreCase("joinlocation")){
 
            if(!(player.hasPermission(new Permissions().joinlocation))) {
 
                player.sendMessage(plugin.prefix + "The cow goes, NOO");
 
                return true;
            }
 
            Location loc = player.getLocation();
 
            org.bukkit.World world = loc.getWorld();
            double x = loc.getBlockX();
            double y = loc.getBlockY();
            double z = loc.getBlockZ();
 
            float pitch = loc.getPitch();
            float yaw = loc.getYaw();
 
            FileConfiguration config = plugin.getConfig();
 
            String jl = "JOIN_LOCATION.";
            config.set(jl+"world", world.getUID().toString());
            config.set(jl+"x", x);
            config.set(jl+"y", y);
            config.set(jl+"z", z);
            config.set(jl+"pitch", pitch);
            config.set(jl+"yaw", yaw);
            plugin.saveConfig();
 
 
                player.sendMessage(plugin.prefix + "You have set the join location to where you are standing");
 
        return true;
 
        }
 
        return false;
 
    }

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

 

Hey, I think, if your lines still the same from this print:
I'm a bit confused not sure. I think the yaw variable needs to be double. so converts to float (float) when u need it float.
like:

Code:java

double yaw = config.getDouble(jl+"yaw");
Location jLocation = new Location(world,x,y,z,(float)yaw,pitch);

Tip: Instead of get the object with getConfig().get("path"); and converting. Get with the type already: getConfig().getTYPE("path"); Like I've done at line 1 of my code.
EDIT: Grammar fix

 

Ah okay. I will try and use that method of getting info from a config. I will try the fix now

Thanks for the help so far man!

samosaara Thanks dude, This worked. I really do appreciate the help you have given me!

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