Secondary Config Returning Null

Hello there. I've encountered a problem with my plugin. I've downloaded a config manager to create multiple files and it seems to work great (I'm not 100% sure it's up to date, but I don't think the code for files in general has change in a while). However, there's one very big problem: my strings are null when read by the plugin. Anyone know why?

Here are the links to my config manager classes that I've downloaded, and below is my RespawnListener class.
http://hastebin.com/rurixomate.coffee
http://hastebin.com/kubelexemo.avrasm

My RespawnListener:

Code:java

package me.pookeythekid.WorldSpawn.Listeners;
 
import java.util.ArrayList;
 
import me.pookeythekid.WorldSpawn.Main;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerRespawnEvent;
 
public class RespawnListener implements Listener {
 
    public Main M;
 
    public RespawnListener(Main instance) {
        M = instance;
    }
 
 
    ArrayList<String> deathWorlds = new ArrayList<String>();
    ArrayList<String> worldPairList = new ArrayList<String>();
 
 
    @EventHandler(priority=EventPriority.HIGHEST, ignoreCancelled = true)
    public void onPlayerRespawn(final PlayerRespawnEvent e) {
 
        if (M.getConfig().getBoolean("usePlugin")) {
 
            respawnPlayer(e.getPlayer(), e);
 
            e.getPlayer().sendMessage("You have respawned.");
 
        }
    }
 
 
 
    public void respawnPlayer(Player p, PlayerRespawnEvent e) {
 
        deathWorlds.clear();
        worldPairList.clear();
 
 
        String[] worldPairs = M.getConfig().getString("worlds").split(",");
 
        for (String s : worldPairs) {
 
            String[] indiWorlds = s.split("-");
            deathWorlds.add(indiWorlds[0]);
            worldPairList.add(s);
 
        }
 
        if (deathWorlds.contains(p.getWorld().getName())) {
 
            for (String s : worldPairs) {
 
                String[] indiWorlds = s.split("-");
 
                p.sendMessage(indiWorlds[0]);
                p.sendMessage(indiWorlds[1]);
 
                if (worldPairList.contains(indiWorlds[0] + "-" + indiWorlds[1])) {
 
                    if (!(M.worldsConf.getConfigurationSection(indiWorlds[1]).equals(null))) {
 
                        Location loc = e.getRespawnLocation();
 
                        loc.setWorld(Bukkit.getServer().getWorld(indiWorlds[1]));
                        loc.setX(M.worldsConf.getDouble(indiWorlds[1] + ".x"));
                        loc.setY(M.worldsConf.getDouble(indiWorlds[1] + ".y"));
                        loc.setZ(M.worldsConf.getDouble(indiWorlds[1] + ".z"));
                        loc.setPitch(M.worldsConf.getLong(indiWorlds[1] + ".Pitch"));
                        loc.setYaw(M.worldsConf.getLong(indiWorlds[1] + ".Yaw"));
 
                        e.setRespawnLocation(loc);
 
                        break;
 
                    } else if (M.worldsConf.getConfigurationSection(indiWorlds[1]).equals(null)) {
 
                        if (M.getConfig().getBoolean("respawnBed")
                                && e.getPlayer().getBedSpawnLocation() != null)
 
                            e.setRespawnLocation(e.getPlayer().getBedSpawnLocation());
 
                        else if (!(M.getConfig().getBoolean("respawnBed")))
 
                            e.setRespawnLocation(Bukkit.getServer().getWorld(indiWorlds[1]).getSpawnLocation());
 
                    }
 
                }
 
            }
 
        }
 
    }
 
}
 

I know that it's the "indiWorlds[1] + ".x"" parts that are returning null because when I respawn, I'm teleported to not the world's spawnpoint, but to x: 0, y: 0, z: 0, Pitch: 0, Yaw: 0. I'm assuming that they're returning null because they're number types, so if a number is null then it equals 0.

If anyone can help with this, that'd be great. Thanks.

 

pookeythekid
Well can you tell us which strings specifically are null? What are you doing when you get an NPE in your log (post the stacktrace also)

 

pookeythekid

null != 0

 

AoH_Ruthless Sorry, did I use the term "returning null" incorrectly? I'm not getting a stack trace because the plugin's doing as it's being told, and it just recognizes the strings as 0 and/or nothing.
Here's my config that it's reading from if it helps.

Code:

# +----------------------------------------------------+
# <        These are spawnpoints of worlds that        >
# <      players will respawn into when they die.      >
# <      Note that any of these worlds will only      >
# <      take function if they are listed in the      >
# <        'worlds' section in the main config.        >
# +----------------------------------------------------+
 
world:
  x: 308.4252674931878
  y: 60.0
  z: -112.96386428373135
  Pitch: 23.450804
  Yaw: -35.32099

MRPS If you're getting a number that is nothing, I'm pretty sure you're going to receive a zero.

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

 

null is nothing, 0 is an integer/short/byte which is of 32, 16, or 8 bit allocation

 

pookeythekid
Null and 0 have a similar connotative sense in everyday language, but not always in Java. In a coding sense, null means it literally doesn't exist. 0, while associated with the word 'nothing' (or its synonyms) is something 'tangible' for lack of a better term.

But yes, a double, float, integer, etc will return 0 in the scenario you have.

Code:java

String[] worldPairs = M.getConfig().getString("worlds").split(",");

I don't see that anywhere in your config, hence it is null.

 

pookeythekid I believe this is true, since Bukkit's configuration system returns a double when using getDouble() with a config, instead of a Double (see this)

 

AoH_Ruthless Oh yes, I gave you my second config, respawnworlds.yml. I'll give you my main config.

Main Config:

Code:

#######################
## ::-------------:: ##
## :: WorldSpawns :: ##
## ::-------------:: ##
#######################
 
 
####################
## :: Settings :: ##
####################
 
# Should this plugin even take effect?
# This is for using or not using this plugin without deleting and
# re-installing it all the time. Set to "true" to say yes. Set to
# "false" to say no. For those of you plugin developers who understand
# listeners, this plugin's PlayerRespawnEvent listener is set to "HIGHEST"
# priority. This is to override any other plugins' respawning handlers.
# For everyone -- not just fancy plugin developers -- set this to "false"
# if you wish for your other plugins to handle respawning, or for regular
# Minecraft to respawn players at the default world's spawnpoint. Note that
# this plugin will not be disabled entirely; settings below may still take
# effect when a player respawns. To completely disable this plugin, you'd
# have to delete it.
usePlugin: true
 
# If a player has a bed spawn, there is no spawnpoint for the world in which
# they are respawning, and/or the usePlugin setting above is disabled, should
# that player respawn at their home bed?
respawnBed: false
 
 
 
####################
## ::  Worlds  :: ##
####################
 
# This is the list of worlds in which players will die and respawn in.
# Notice that WorldSpawns will not handle player respawning in worlds
# that are not listed here; either other plugins or default Minecraft
# will handle respawning instead. Items in this list are sparated by
# commas. Do not use spaces!!! Within each item, there are two worlds.
# As you can see, they are arranged like this: <world>-<world>
# The world BEFORE the "-" is the world that the player will DIE in.
# The world AFTER the "-" is the world that the player will RESPAWN in.
# If a world does not have a spawnpoint and a player respawns into it,
# the world's default spawnpoint will be used. Although impractical,
# please do not list the same pair of worlds twice, and do not list
# three worlds in one world group.
worlds: world-world,world_nether-world,world_the_end-world

BTW Thanks for the description of the difference between null and 0.

 

pookeythekid
"However, there's one very big problem: my strings are null when read by the plugin. Anyone know why?" - I still don't understand what exactly is null. Please elaborate.

 

AoH_Ruthless In my description (inaccurate compared to yours), null either means 0, or it means that the program is somehow not reaching the config, and therefore the strings are not 'tangible'. Otherwise, the program is reaching the strings, but is having strange difficulties interpreting them, so instead of dealing with the problems it spits out a zero. This is my guess, I really don't know what's going on.

 

pookeythekid
Did you debug your code?

 

AoH_Ruthless
If you mean looking over it a thousand times and trying several different methods for hours on end, yes I did.

Edit: I also know for sure that the listener is firing off because I sent myself a message when I respawned, and that worked just fine.

 

pookeythekid
That isn't what debugging is.

Code:java

@EventHandler
public void onEvent(PlayerTakesCrapEvent e) {
    System.out.println("1");
    if (x) {
        System.out.println("2");
        if (y) {
            System.out.println("3");
        }
    }
}

Then in your console, see how far your code gets and adjust accordingly.

 

AoH_Ruthless
Um... Isn't that just checking if the plugin works?

 

pookeythekid
That's what debugging is ... seeing how far your code gets so you know exactly which line(s) of your code are wrong.

 

AoH_Ruthless
I... don't understand... Are you saying that it's checking for a glitch in the class itself, not in the code that I write?

 

pookeythekid
This isn't a very complicated concept.

Debugging your source code simply means you are looking for errors in your code. You don't know why your code isn't working or even where the issue is, so you can use debugging to pinpoint the location of the error.
It is a tool used to assist the coder in fixing mistakes they make in the code they write.

 

AoH_Ruthless
Oh, well then yes, I have debugged it. I have spent much time pinpointing the error, and that's why I created this thread; the error(s) are in the lines that get the second config file that I have.

AoH_Ruthless
Have you read the config managers that I gave in the main post of this thread? I've read them over and over... The MyConfig class makes sense to me, nothing seems wrong. The MyConfigManager class, well... As much as I can understand about it makes sense, but I don't know the whole thing well enough to declare if it's wrong or not.

Edit: Um... nevermind, the links are gone... It's my first time using Hastebin, so I'm not surprised that I don't know why.
Here's the code for the classes, since they're evidently not in the links I gave.
MyConfig class:

Code:java

package me.pookeythekid.WorldSpawn.ConfigManager;
 
import java.io.File;
import java.io.InputStream;
import java.util.List;
import java.util.Set;
 
import org.bukkit.configuration.ConfigurationSection;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.plugin.java.JavaPlugin;
 
public class MyConfig
{
    private int comments;
    private MyConfigManager manager;
 
    private File file;
    private FileConfiguration config;
 
    public MyConfig(InputStream configStream, File configFile, int comments, JavaPlugin plugin)
    {
        this.comments = comments;
        this.manager = new MyConfigManager(plugin);
        this.file = configFile;
        this.config = YamlConfiguration.loadConfiguration(configStream);
    }
 
    public Object get(String path) {return this.config.get(path);}
 
    public Object get(String path, Object def) {return this.config.get(path, def);}
 
    public String getString(String path) {return this.config.getString(path);}
 
    public String getString(String path, String def) {return this.config.getString(path, def);}
 
    public int getInt(String path) {return this.config.getInt(path);}
 
    public int getInt(String path, int def) {return this.config.getInt(path, def);}
 
    public boolean getBoolean(String path) {return this.config.getBoolean(path);}
 
    public boolean getBoolean(String path, boolean def) {return this.config.getBoolean(path, def);}
 
    public void createSection(String path) {this.config.createSection(path);}
 
    public ConfigurationSection getConfigurationSection(String path) {return this.config.getConfigurationSection(path);}
 
    public double getDouble(String path) {return this.config.getDouble(path);}
 
    public double getDouble(String path, double def) {return this.config.getDouble(path, def);}
 
    public float getLong(String path) {return this.config.getLong(path);}
 
    public float getLong(String path, long def) {return this.config.getLong(path, def);}
 
    public List<?> getList(String path) {return this.config.getList(path);}
 
    public List<?> getList(String path, List<?> def) {return this.config.getList(path, def);}
 
    public boolean contains(String path) {return this.config.contains(path);}
 
    public void removeKey(String path) {this.config.set(path, null);}
 
    public void set(String path, Object value) {this.config.set(path, value);}
 
    public void set(String path, Object value, String comment)
    {
        if(!this.config.contains(path))
        {
            this.config.set(manager.getPluginName() + "_COMMENT_" + comments, " " + comment);
            comments++;
        }
        this.config.set(path, value);
    }
 
    public void set(String path, Object value, String[] comment)
    {
        for(String comm : comment)
        {
            if(!this.config.contains(path))
            {
                this.config.set(manager.getPluginName() + "_COMMENT_" + comments, " " + comm);
                comments++;
            }
        }
        this.config.set(path, value);
    }
 
    public void setHeader(String[] header)
    {
        manager.setHeader(this.file, header);
        this.comments = header.length + 2;
        this.reloadConfig();
    }
 
    public void reloadConfig() {this.config = YamlConfiguration.loadConfiguration(manager.getConfigContent(file));}
 
    public void saveConfig()
    {
        String config = this.config.saveToString();
        manager.saveConfig(config, this.file);
    }
 
    public Set<String> getKeys() {return this.config.getKeys(false);}
}

MyConfigManager class:

Code:java

package me.pookeythekid.WorldSpawn.ConfigManager;
import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.ByteArrayInputStream;
import java.io.File;
import java.io.FileOutputStream;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.nio.charset.Charset;
 
import org.bukkit.plugin.java.JavaPlugin;
 
public class MyConfigManager
{
    private JavaPlugin plugin;
 
    public MyConfigManager(JavaPlugin plugin)
    {
        this.plugin = plugin;
    }
 
    public MyConfig getNewConfig(String fileName, String[] header)
    {
        File file = this.getConfigFile(fileName);
 
        if(!file.exists())
        {
            this.prepareFile(fileName);
            if(header != null && header.length != 0)
                this.setHeader(file, header);
        }
 
        MyConfig config = new MyConfig(this.getConfigContent(fileName), file, this.getCommentsNum(file), plugin);
        return config;
    }
 
    public MyConfig getNewConfig(String fileName)
    {
        return this.getNewConfig(fileName, null);
    }
 
    private File getConfigFile(String file)
    {
        if(file.isEmpty() || file == null)
            return null;
 
        File configFile;
 
        if(file.contains("/"))
        {
            if(file.startsWith("/"))
                configFile = new File(plugin.getDataFolder() + file.replace("/", File.separator));
            else configFile = new File(plugin.getDataFolder() + File.separator + file.replace("/", File.separator));
        }
        else configFile = new File(plugin.getDataFolder(), file);
 
        return configFile;
    }
 
    public void prepareFile(String filePath, String resource)
    {
        File file = this.getConfigFile(filePath);
 
        if(file.exists())
            return;
 
        try
        {
            file.getParentFile().mkdirs();
            file.createNewFile();
 
            if(resource != null)
                if(!resource.isEmpty())
                    this.copyResource(plugin.getResource(resource), file);
 
        }
        catch (IOException e){e.printStackTrace();}
    }
 
    public void prepareFile(String filePath)
    {
        this.prepareFile(filePath, null);
    }
 
    public void setHeader(File file, String[] header)
    {
        if(!file.exists())
            return;
 
        try
        {
            String currentLine;
            StringBuilder config = new StringBuilder("");
            BufferedReader reader = new BufferedReader(new FileReader(file));
 
            while((currentLine = reader.readLine()) != null)
                config.append(currentLine + "\n");
 
            reader.close();
            config.append("# +----------------------------------------------------+ #\n");
 
            for(String line : header)
            {
                if(line.length() > 50)
                    continue;
 
                int lenght = (50 - line.length()) / 2;
                StringBuilder finalLine = new StringBuilder(line);
 
                for(int i = 0; i < lenght; i++)
                {
                    finalLine.append(" ");
                    finalLine.reverse();
                    finalLine.append(" ");
                    finalLine.reverse();
                }
 
                if(line.length() % 2 != 0)
                    finalLine.append(" ");
 
                config.append("# < " + finalLine.toString() + " > #\n");
            }
            config.append("# +----------------------------------------------------+ #");
 
            BufferedWriter writer = new BufferedWriter(new FileWriter(file));
            writer.write(this.prepareConfigString(config.toString()));
            writer.flush();
            writer.close();
        }
        catch (IOException e){e.printStackTrace();}
    }
 
    public InputStream getConfigContent(File file)
    {
        if(!file.exists())
            return null;
        try
        {
            int commentNum = 0;
 
            String addLine;
            String currentLine;
            String pluginName = this.getPluginName();
 
            StringBuilder whole = new StringBuilder("");
            BufferedReader reader = new BufferedReader(new FileReader(file));
 
            while((currentLine = reader.readLine()) != null)
            {
                if(currentLine.startsWith("#"))
                {
                    addLine = currentLine.replaceFirst("#", pluginName + "_COMMENT_" + commentNum + ":");
                    whole.append(addLine + "\n");
                    commentNum++;
                }
                else whole.append(currentLine + "\n");
            }
 
            String config = whole.toString();
            InputStream configStream = new ByteArrayInputStream(config.getBytes(Charset.forName("UTF-8")));
 
            reader.close();
            return configStream;
        }
        catch (IOException e){e.printStackTrace();return null;}
    }
 
    private int getCommentsNum(File file)
    {
        if(!file.exists())
            return 0;
        try
        {
            int comments = 0;
            String currentLine;
 
            BufferedReader reader = new BufferedReader(new FileReader(file));
 
            while((currentLine = reader.readLine()) != null)
                if(currentLine.startsWith("#"))
                    comments++;
 
            reader.close();
            return comments;
        }
        catch (IOException e){e.printStackTrace();return 0;}
    }
 
    public InputStream getConfigContent(String filePath)
    {
        return this.getConfigContent(this.getConfigFile(filePath));
    }
 
    private String prepareConfigString(String configString)
    {
        int lastLine = 0;
        int headerLine = 0;
 
        String[] lines = configString.split("\n");
        StringBuilder config = new StringBuilder("");
 
        for(String line : lines)
        {
            if(line.startsWith(this.getPluginName() + "_COMMENT"))
            {
                String comment = "#" + line.trim().substring(line.indexOf(":") + 1);
 
                if(comment.startsWith("# +-"))
                {
                    if(headerLine == 0)
                    {
                        config.append(comment + "\n");
                        lastLine = 0;
                        headerLine = 1;
                    }
                    else if(headerLine == 1)
                    {
                        config.append(comment + "\n\n");
                        lastLine = 0;
                        headerLine = 0;
                    }
                }
                else
                {
                    String normalComment;
                    if(comment.startsWith("# ' "))
                        normalComment = comment.substring(0, comment.length() - 1).replaceFirst("# ' ", "# ");
                    else normalComment = comment;
 
                    if(lastLine == 0)
                        config.append(normalComment + "\n");
                    else if(lastLine == 1)
                        config.append("\n" + normalComment + "\n");
 
                    lastLine = 0;
                }
            }
            else
            {
                config.append(line + "\n");
                lastLine = 1;
            }
        }
        return config.toString();
    }
 
    public void saveConfig(String configString, File file)
    {
        String configuration = this.prepareConfigString(configString);
 
        try
        {
            BufferedWriter writer = new BufferedWriter(new FileWriter(file));
            writer.write(configuration);
            writer.flush();
            writer.close();
 
        }
        catch (IOException e){e.printStackTrace();}
    }
 
    public String getPluginName()
    {
        return plugin.getDescription().getName();
    }
 
    private void copyResource(InputStream resource, File file)
    {
        try
        {
            OutputStream out = new FileOutputStream(file);
 
            int length;
            byte[] buf = new byte[1024];
 
            while((length = resource.read(buf)) > 0)
                out.write(buf, 0, length);
 
            out.close();
            resource.close();
        }
        catch (Exception e) {e.printStackTrace();}
    }
}

AoH_Ruthless
Okay, I've tried everything I can possibly think of. I'm positive that it's the config managers that I'm using that are not working somehow. I've looked them over many times, I don't see anything that is wrong as far as my understanding of it, and how much sense the code makes to me. But I have a new solution, possibly one involving much less code. I'm going to learn how to create a text file in the plugin's folder inside of the server, and I'm going to learn to efficiently use BufferedWriters and BufferedReaders to use as a worlds file.

Thanks for the help anyway.

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

 

pookeythekid
This is what you have:

Code:java

if (!(M.worldsConf.getConfigurationSection(indiWorlds[1]).equals(null))) {
 
Location loc = e.getRespawnLocation();
 
loc.setWorld(Bukkit.getServer().getWorld(indiWorlds[1]));
loc.setX(M.worldsConf.getDouble(indiWorlds[1] + ".x"));
loc.setY(M.worldsConf.getDouble(indiWorlds[1] + ".y"));
loc.setZ(M.worldsConf.getDouble(indiWorlds[1] + ".z"));
loc.setPitch(M.worldsConf.getLong(indiWorlds[1] + ".Pitch"));
loc.setYaw(M.worldsConf.getLong(indiWorlds[1] + ".Yaw"));
 
e.setRespawnLocation(loc);

Try something like:

Code:java

if (M.worldsConf.getConfigurationSection(indiWorlds[1]) != null) {
 
World world = Bukkit.getServer().getWorld(indiWorlds[1]));
double x = M.worldsConf.getDouble(indiWorlds[1] + ".x");
double y = M.worldsConf.getDouble(indiWorlds[1] + ".y");
double z = M.worldsConf.getDouble(indiWorlds[1] + ".z");
double pitch = M.worldsConf.getDouble(indiWorlds[1] + ".Pitch"); // Doubles are better than longs, but you should make a method to get floats in your config class.
double yaw = M.worldsConf.getDouble(indiWorlds[1] + ".Yaw");
 
Location loc = new Location(world,x,y,z,yaw,pitch);
e.setRespawnLocation(loc);

 

AoH_Ruthless
Lol I did make the method to get longs in the config class. I figured I'd do that because I always use the getLong() method from Bukkit, and that usually works. Thanks for the code, I'll try it out. I'm almost sure I've literally tried every other possible way of doing it.

AoH_Ruthless
By the way, here's my most recent way of doing it:

Code:java

// I have worldIsNull in another class now
// WC means WorldChecks, my new class.
if (!(WC.worldIsNull(indiWorlds[1])) {
 
loc.setWorld(Bukkit.getServer().getWorld(indiWorlds[1]);
loc.setX(M.worldsConf.getDouble(indiWolrds[1] + ".x");
// and so on until getting the Z
loc.setPitch(M.worldsConf.getLong(indiWorlds[1] + ".Pitch");
// same with Yaw
 
e.setRespawnLocation(loc);

Edit: Also, I should learn BufferedReaders and Writers anyway. May come in handy sometime when I need a text file.

AoH_Ruthless
.... It makes so much sense now.... It IS the config manager that's broken! I sent myself a message when I respawned: "p.sendMessage(M.worldsConf.getString("world.y"));"
It sent me: "MemorySection[path='world.y', root='YamlConfiguration']"
That's why it's been so darn confusing!!! If the code gets the coordinates from the config, it does get something; it's not null, so it passes the null check. But that weird message I received can't be interpreted as a double, so the program assume that it's 0, and that's why it keeps teleporting me to x: 0, y: 0, z: 0, Pitch: 0, Yaw: 0, but I haven't worked out why the worlds part doesn't work yet... The worlds are retrieved from the main config, the one that doesn't use the config manager classes.

But anyway! That's why this whole thing has been so dang confusing, because it's not getting nothing, but it's getting the wrong thing.

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

 

pookeythekid
Actually since world.y is a number you can't be using getString().

 

AoH_Ruthless
... Oh... Well, I figured using getString() would work because it would convert the number to a string, instead of me having to use toString(). I'll try the latter, and then tell you what I got.

Edit: Oh, that's why I used getString(). There is no option to do getDouble("world.y").toString() with my config manager, not much surprise that it makes an error when I send a player a double. But I'm going to learn to use text documents anyway. I'll probably need text documents in the future, and this whole second config thing isn't working well at all for me.

 

pookeythekid
String.valueOf(double)

 

AoH_Ruthless
... *facepalm* Thanks.... I'll try that...

AoH_Ruthless
Well, still kind of proves my point. I got a message saying: "0.0". That must mean zero. : P
Oh yes, and I also sent myself a loc.toString(). It said: "Location{world=CraftWorld{name=world},x=0.0,y=0.0,z=0.0,pitch=0.0,yaw=0.0}". That would explain why only the world name seems to work.

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

 

pookeythekid
A location cannot be compared to string like that.
Post your current code.

 

AoH_Ruthless
Here's my whole class if it helps more.

Code:java

package me.pookeythekid.WorldSpawn.Listeners;
 
import java.util.ArrayList;
 
import me.pookeythekid.WorldSpawn.Main;
import me.pookeythekid.WorldSpawn.PersonalAPI.WorldChecks;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerRespawnEvent;
 
public class RespawnListener implements Listener {
 
    public Main M;
 
    public RespawnListener(Main instance) {
        M = instance;
    }
 
    public WorldChecks WC;
 
    public RespawnListener(WorldChecks instance) {
 
        WC = instance;
 
    }
 
 
    ArrayList<String> deathWorlds = new ArrayList<String>();
    ArrayList<String> worldPairList = new ArrayList<String>();
 
 
    @EventHandler(priority=EventPriority.HIGHEST, ignoreCancelled = true)
    public void onPlayerRespawn(final PlayerRespawnEvent e) {
 
        if (M.getConfig().getBoolean("usePlugin")) {
 
            respawnPlayer(e);
 
            e.getPlayer().sendMessage("You have respawned.");
 
        }
    }
 
 
 
    public void respawnPlayer(PlayerRespawnEvent e) {
 
        Player p = e.getPlayer();
 
        deathWorlds.clear();
        worldPairList.clear();
 
 
        String[] worldPairs = M.getConfig().getString("worlds").split(",");
 
        for (String s : worldPairs) {
 
            String[] indiWorlds = s.split("-");
            deathWorlds.add(indiWorlds[0]);
            worldPairList.add(s);
 
        }
 
        if (deathWorlds.contains(p.getWorld().getName())) {
 
            for (String s : worldPairs) {
 
                String[] indiWorlds = s.split("-");
 
                p.sendMessage(indiWorlds[0]);
                p.sendMessage(indiWorlds[1]);
 
                if (worldPairList.contains(indiWorlds[0] + "-" + indiWorlds[1])) {
 
                    if (M.worldsConf.getConfigurationSection(indiWorlds[1]) != null) {
 
                        World world = Bukkit.getServer().getWorld(indiWorlds[1]);
                        double x = M.worldsConf.getDouble(indiWorlds[1] + ".x");
                        double y = M.worldsConf.getDouble(indiWorlds[1] + ".y");
                        double z = M.worldsConf.getDouble(indiWorlds[1] + ".z");
 
                        Location loc = new Location(world, x, y, z);
                        e.setRespawnLocation(loc);
                        p.sendMessage(loc.toString());
                        p.sendMessage(String.valueOf(M.worldsConf.getDouble("world.y")));
 
                        break;
 
                    } else if (WC.worldIsNull(indiWorlds[1])) {
 
                        if (M.getConfig().getBoolean("respawnBed")
                                && e.getPlayer().getBedSpawnLocation() != null) {
 
                            e.setRespawnLocation(e.getPlayer().getBedSpawnLocation());
                            break;
 
                        } else if (!(M.getConfig().getBoolean("respawnBed"))) {
 
                            e.setRespawnLocation(Bukkit.getServer().getWorld(indiWorlds[1]).getSpawnLocation());
                            break;
 
                        }
 
                    }
 
                }
 
            }
 
        }
 
    }
 
}
 

The p.sendMessage(indiWorlds[1]) and p.sendMessage(indiWorlds[2]) was just a check for me to be certain that the world was working. I haven't taken it out yet, but it doesn't work. I've tried setting it to a different respawn world in my config, and I'm positive that the code is set up so that it should work, but I respawned in the same, default world. I figure if it doesn't know how to handle a world that's being read wrong, it chooses the default world.

 

pookeythekid
What does your "worlds" string in your regular config look like?

 

AoH_Ruthless

Code:

worlds: world-world,world_nether-world,world_the_end-world