Solved Checking if a world is in the config

I'm trying to check if the world is in a string list in the config with this:

Code:java

    public boolean isWorldBlocked(String world, String plugin){
        for(String w : getConfig().getStringList("List." + plugin + ".Worlds")){
            if (w.equalsIgnoreCase(world) || w.equalsIgnoreCase("all")){
                return true;
            }
        }
        return false;
    }

But It doesn't seem to be working, it always returns false.

My config:

Code:java

List:
  Plugin:
    Message: 'This plugin is not enabled in %w'
    Worlds: all

---

Oh and, by the way, how do I set more than 1 word in a string list in a config, for example:
getConfig().set("List." + plugin + ".Worlds", "World1"); and then
getConfig().set("List." + plugin + ".Worlds", "World2");
so it looks like this in the config:
Worlds: [world1, world2]

 

I think all is not a string in your config.

 

I have this on my onEnable():

Code:java

                getConfig().addDefault("List.Plugin.Worlds", "all");

 

You've misspelt "Plugin" on line 2 (it's capitalised in your config). Also, that's not how you format a list. I suggest you read this page for more information; it should answer your other question too.

 

XXLuigiMario, Do getConfig().set("List.Plugin.Worlds" + p.getWorld().getName())? or something like that.

 

I've not misspelled it, it's a variable...

 

Try to replace this

Code:java

 getConfig().addDefault("List.Plugin.Worlds", "all");

with

Code:java

ArrayList<String> list = new ArrayList<String>();
list.add("all");
getConfig().addDefault("List.Plugin.Worlds", list);

 

Nvm, I fixed it.
Although I would like to know how to format it like I asked in the OP. (I know it's possible, i've seen plugins do it)
So I'll leave this open.

Just saw your reply, I did this: List<String> world = Arrays.asList("all");

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

 

Oh wow, sorry; I don't know what I was thinking. Now I feel pretty stupid

Anyway, I don't think whatever that other plugin created is a list (that's not to say it isn't possible to do, though). Lists look like this in a config:

Code:

List:
  Plugin:
    Worlds:
    - World1
    - World2

Here's how I'd add to the above:

Code:java

List<String> theList = getConfig().getStringList("List.Plugin.Worlds");
theList.add("World1");
getConfig().set("List.Plugin.Worlds", theList);
saveConfig();

 

dsouzamatt thanks for your reply, I already fixed that, but I don't know what's better, if the thing you posted or what I'm doing (read my previous post)

 

There's no real difference as far as I know, just make sure you remember to add saveConfig() to the end of your code if you use it.

 

Ok, I won't forget that