Solved Plugin help

aaron1998ish · Mar 30, 2015

Hey im having trouble with the onJoin teleport feature im messing with:

Basically what im trying to do at the moment is if someone first joins a new map is created, then they're sent to that map created on their next login..

The code works, files are generated and players are teleported on next login, but the teleport is not following the config coordinates?

Its probably going to be an easy fix but i just cant see where ive gone wrong :/
Code:
//Solved//

TGRHavoc · Mar 30, 2015

aaron1998ish said: ↑
Hey im having trouble with the onJoin teleport feature im messing with:

Basically what im trying to do at the moment is if someone first joins a new map is created, then they're sent to that map created on their next login..

The code works, files are generated and players are teleported on next login, but the teleport is not following the config coordinates?

Its probably going to be an easy fix but i just cant see where ive gone wrong :/
Code:
@EventHandler (priority = EventPriority.HIGHEST)
public void onPlayerJoin(PlayerJoinEvent event) {
    if (!event.getPlayer().hasPlayedBefore()){
        Bukkit.getServer().createWorld(new WorldCreator(event.getPlayer().getName() + "world"));
        plugin.getConfig().set(event.getPlayer().getName() + ".WORLD", event.getPlayer().getName() + "world");
        plugin.getConfig().set(event.getPlayer().getName() + ".spawnPointX", "1");
        plugin.getConfig().set(event.getPlayer().getName() + ".spawnPointY", "70");
        plugin.getConfig().set(event.getPlayer().getName() + ".spawnPointZ", "1");
        plugin.saveConfig();
        event.getPlayer().sendMessage(ChatColor.GREEN + "Heres your new map!");
    }
  
    else{
        event.getPlayer().sendMessage(ChatColor.GREEN + "Welcome back " + event.getPlayer().getName() + " Your map is ready!");
        World w = Bukkit.getWorld(plugin.getConfig().getString(event.getPlayer().getName()+".WORLD"));
        if(w == null){
            //TODO: Error message?
            return;
        }
       
         double x = plugin.getConfig().getDouble(event.getPlayer().getName()+".spawnPointx");
         double y = plugin.getConfig().getDouble(event.getPlayer().getName()+".spawnPointy");
         double z = plugin.getConfig().getDouble(event.getPlayer().getName()+".spawnPointz");
       
        Location l = new Location(w, x, y, z);
        event.getPlayer().teleport(l);
    }
}
 
}
Click to expand...
You're setting the co-ordinates to a string in the config file ("set(event.getPlayer().getName() + ".spawnPointX", "1");") meaning that when you do getDouble(String) you're receiving the default values for a double (0).
Remove the quotation marks from the lines of code that set the "spawnPoint" variables in the config.

WladHD · Mar 30, 2015

Case sensitive... "spawnPointX" is not equals to "spawnPointx".
And dont set them to a string...

aaron1998ish · Mar 30, 2015

Thanks guys, its all working now, and the caps issue was fixed a few seconds after posting, but the main issue was the string

Log in or Sign up

Solved Plugin help

aaron1998ish

TGRHavoc

WladHD

aaron1998ish

Share This Page

Log in or Sign up

Solved Plugin help

aaron1998ish

TGRHavoc

WladHD

aaron1998ish

Share This Page

Useful Searches