[Urgent!] getting from config does not work... or it does in the wrong way!

Hi guys, I am making a plugin which disables/enables pvp messages with /pvpmsg,
I am doing /pvpmsg and it gives me the text that it enables, then it disables really fast..
code:

Code:java

package Com.chip;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class PvpDisabler extends JavaPlugin implements Listener {
    String Prefix = ChatColor.GREEN + "[" + ChatColor.BLUE + "NoPvpMsg" + ChatColor.GREEN + "] ";
    String blue = ChatColor.BLUE + "";
    String red = ChatColor.RED + "";
    String green = ChatColor.GREEN + "";
    String bold = ChatColor.BOLD + "";
    String reset = ChatColor.RESET + "";       
    public void onEnable() {
        getServer().getConsoleSender().sendMessage(Prefix + "Enabled!");
        getServer().getPluginCommand("pvpmsg").setPermissionMessage(Prefix + green + "You do not have permission!");
    }
    public void onDisable() {
        getServer().getConsoleSender().sendMessage(Prefix + "Disabled!");
    }
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if(cmd.getName().equalsIgnoreCase("pvpmsg")) {
            Player player = (Player) sender;
                if(getConfig().contains(player.getWorld().getName())) {
                    getConfig().set(player.getWorld().getName(), null);
                    saveConfig();
                    player.sendMessage(Prefix + green + "Pvp message enabled in:" + red + bold + player.getWorld().getName() + reset + green + "!");
                }
                if(!getConfig().contains(player.getWorld().getName())) {
                    getConfig().set(player.getWorld().getName(), 1);
                    saveConfig();
                    player.sendMessage(Prefix + green + "Pvp message disabled in:" + red + bold + player.getWorld().getName() + reset + green + "!");
                }
            }
    return true;   
    }
    public void onPlayerDeathEvent(PlayerDeathEvent e) {
        Player player = e.getEntity();
        if (e.getEntity() instanceof Player){
            if(!player.hasPermission("pvpmessage.bypass")) {
            if(getConfig().contains(player.getWorld().toString())) {
                e.setDeathMessage(null);
            }
 
        }
    }
}
}
 

Thanks !

 

ChipDev You could try using if/else statements in your command. Your 2 if statements is basically mimicking that except that the second if statement could execute it's code if the first does something to make the test true.

 

Hmm use else.
Hah I used scratch before bukkit, Never used else's , I just used more If's .. heh k.

 

remove your second if statement and replace it with "else".
EDIT: Just your if test. Don't remove the code inside it.

Code:java

if(test)
{
 
}
else
{
 
}

The else will execute if the if test returns false

 

Please don't put urgent tags in your post title; your problems aren't more important than anybody else's. I don't mean to be harsh, but all it does is make people (at least me) want to help less.

 

getWorld().toString() won't give you the information which you need to identify world. You should get (and save to config) it's name or UUID.

 

How does one get UUID of a world?

 

Konkz World#getUID()

I'm surprised you didn't read the JavaDocs, I thought you knew better than that

 

#getUID() and #getUUID are not the same, one is for user (Unique User Identification) other is for just general use (Unique Identification) - that's why I picked him up on UUID.

Also, when I read JavaDocs I read for what I need, not everything. I gotta life you know

 

Konkz I think that's the real problem - what with 1.8 we're forgetting that UUID is actually a Java thing for identifying objects, not a Minecraft/Bukkit/Player specific and actually stands for Universally Unique IDentifier. As such, getUID returns a UUID

Pah, I don't!

 

Konkz

Code:java

world.getName();
world.getUUID();