getConfig.set giving error

Hi,
Its my first plugin.
Server error log:

Code:

00:00:46 [INFO] juniorjpdj issued server command: /parkour set test
00:00:46 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'park
our' in plugin Parkour v0.1
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:19
2)
        at org.bukkit.craftbukkit.v1_6_R2.CraftServer.dispatchCommand(CraftServe
r.java:527)
        at net.minecraft.server.v1_6_R2.PlayerConnection.handleCommand(PlayerCon
nection.java:979)
        at net.minecraft.server.v1_6_R2.PlayerConnection.chat(PlayerConnection.j
ava:890)
        at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java
:837)
        at net.minecraft.server.v1_6_R2.Packet3Chat.handle(SourceFile:49)
        at org.spigotmc.netty.NettyNetworkManager.b(NettyNetworkManager.java:230
)
        at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java
:116)
        at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
        at org.spigotmc.netty.NettyServerConnection.b(NettyServerConnection.java
:125)
        at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:5
92)
        at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:2
39)
        at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:4
81)
        at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java
:413)
        at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:5
82)
Caused by: java.lang.IllegalArgumentException: File cannot be null
        at org.apache.commons.lang.Validate.notNull(Validate.java:192)
        at org.bukkit.configuration.file.YamlConfiguration.loadConfiguration(Yam
lConfiguration.java:170)
        at org.bukkit.plugin.java.JavaPlugin.reloadConfig(JavaPlugin.java:117)
        at org.bukkit.plugin.java.JavaPlugin.getConfig(JavaPlugin.java:111)
        at pl.interia.juniorjpdj.parkour.Komenda.onCommand(Komenda.java:20)
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
        ... 15 more
>

Command class:

Code:java

package pl.interia.juniorjpdj.parkour;
 
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
 
public class Komenda extends Parkour implements CommandExecutor {
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if(cmd.getName().equalsIgnoreCase("parkour")) {
            if (sender instanceof Player){
                if (args.length == 0){
                    sender.sendMessage("[Parkour] Napisz jakis argument pajacu!");
                } else if((args[0].equals("set")) && (args.length == 2)) {
                    Player player = (Player) sender;
                    Location loc = player.getLocation();
                    getConfig().set("parkour-" + args[1], loc2str(loc));
                    saveConfig();
                    sender.sendMessage("[Parkour] Zapisano pozycje teleportu do parkouru " + args[1] + " !");
                } else if((args[0].equals("tp")) && (args.length == 2)) {
                    Player player = (Player) sender;
                    getConfig().set("player-" + player.getName(), args[1]);
                    player.teleport(str2loc(getConfig().getString("parkour-" + args[1])));
                    saveConfig();
                    sender.sendMessage("[Parkour] Przeteleportowano na parkour " + args[1] + " !");
                } else {
                    sender.sendMessage("[Parkour] Wez naucz sie skladni, nawet komendy napisac nie umiesz...");
                }
            } else {
                sender.sendMessage("[Parkour] Prosze korzystac z tej komendy w grze szanowny adminie na konsoli :D");
            }
                return true;
        } else {
            return false;
        }
    }
}

Oh, sry.
I forget Parkour class...

Code:java

package pl.interia.juniorjpdj.parkour;
 
import java.util.logging.Logger;
 
import org.bukkit.Location;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Parkour extends JavaPlugin {
 
    protected static final Logger Log = Logger.getLogger("Minecraft");
 
    public void onEnable(){
        getServer().getPluginManager().registerEvents(new Detektor(), this);
        getCommand("parkour").setExecutor(new Komenda());
        saveConfig();
        Log.info("[Parkour] Moj pierwszy gowniany plugin zostal zaladowany!");
    }
 
    public void onDisable(){
        Log.info("[Parkour] No juz, juz sie wylaczam...");
    }
 
    public String loc2str(Location loc){
        return loc.getWorld().getName()+":"+loc.getBlockX()+":"+loc.getBlockY()+":"+loc.getBlockZ()+":"+loc.getYaw()+":"+loc.getPitch();
    }
 
    public Location str2loc(String str){
        String tmp[] = str.split("\\:");
        Location loc = new Location(getServer().getWorld(tmp[0]),Double.parseDouble(tmp[1]),Double.parseDouble(tmp[2]),Double.parseDouble(tmp[3]),Float.parseFloat(tmp[4]),Float.parseFloat(tmp[5]));
        return loc;   
    }
 
}

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

 

Perhaps you used Tabs instead of spaces in the config file?

 

Code:java

                    getConfig().set("parkour-" + args[1], loc2str(loc));

Your problem occurs at this line, with the exception:

Code:java

Caused by: java.lang.IllegalArgumentException: File cannot be null

Are you sure your configuration file exists and can be "seen" by your plugin?

 

File exists and plugin made it....
Its blank, but when i edited it and added some variables its same error :/

PS. Sry, my english isnt good.

 

Don't extend the main class in your command executor...instead pass the instance of the main class to the executor, and use the reference to get the config.

 

Even when i put str2loc and loc2str in command class its no working.

 

Well, it's because those methods are in the main class, luv. Like I said, pass the instance of the main class to the executor. Preferably through a constructor when instantiating the executor.

 

...

New version - same error :/

Command class:

Code:java

package pl.interia.juniorjpdj.parkour;
 
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Komenda extends JavaPlugin implements CommandExecutor {
 
@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
if(cmd.getName().equalsIgnoreCase("parkour")) {
if (sender instanceof Player){
if (args.length == 0){
sender.sendMessage("[Parkour] Napisz jakis argument pajacu!");
} else if((args[0].equals("set")) && (args.length == 2)) {
Player player = (Player) sender;
Location loc = player.getLocation();
//Parkour.parkourloc.put(args[1], new double[] {loc.getX(), loc.getY(), loc.getZ(), loc.getYaw(), loc.getPitch()});
getConfig().set("parkour." + args[1] + ".x", loc.getX());
getConfig().set("parkour." + args[1] + ".y", loc.getY());
getConfig().set("parkour." + args[1] + ".z", loc.getZ());
getConfig().set("parkour." + args[1] + ".yaw", loc.getYaw());
getConfig().set("parkour." + args[1] + ".pitch", loc.getPitch());
saveConfig();
sender.sendMessage("[Parkour] Zapisano pozycje teleportu do parkouru " + args[1] + " !");
} else if((args[0].equals("tp")) && (args.length == 2)) {
Player player = (Player) sender;
Parkour.lastparkour.put(player.getName(), args[1]);
//double tmp[] = Parkour.parkourloc.get(args[1]);
Location tp = new Location(player.getLocation().getWorld(), getConfig().getDouble("parkour." + args[1] + ".x"), getConfig().getDouble("parkour." + args[1] + ".y"), getConfig().getDouble("parkour." + args[1] + ".z"), (float) getConfig().getDouble("parkour." + args[1] + ".yaw"), (float) getConfig().getDouble("parkour." + args[1] + ".pitch"));
player.teleport(tp);
sender.sendMessage("[Parkour] Przeteleportowano na parkour " + args[1] + " !");
} else {
sender.sendMessage("[Parkour] Wez naucz sie skladni, nawet komendy napisac nie umiesz...");
}
} else {
sender.sendMessage("[Parkour] Prosze korzystac z tej komendy w grze szanowny adminie na konsoli :D");
}
return true;
} else {
return false;
}
}
}

Main class (called Parkour):

Code:java

package pl.interia.juniorjpdj.parkour;
 
import java.util.HashMap;
import java.util.logging.Logger;
 
import org.bukkit.plugin.java.JavaPlugin;
 
public class Parkour extends JavaPlugin {
 
public static HashMap<String, String> lastparkour = new HashMap<String, String>();
public static HashMap<String, double[]> parkourloc = new HashMap<String, double[]>();
 
protected static final Logger Log = Logger.getLogger("Minecraft");
 
public void onEnable(){
getServer().getPluginManager().registerEvents(new Detektor(), this);
getCommand("parkour").setExecutor(new Komenda());
saveConfig();
Log.info("[Parkour] Moj pierwszy gowniany plugin zostal zaladowany!");
}
 
public void onDisable(){
Log.info("[Parkour] No juz, juz sie wylaczam...");
}
}

Error log from console:

Code:

15:32:51 [INFO] juniorjpdj issued server command: /parkour set lol
15:32:51 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'park
our' in plugin Parkour v0.1
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:19
1)
        at org.bukkit.craftbukkit.v1_6_R2.CraftServer.dispatchCommand(CraftServe
r.java:523)
        at net.minecraft.server.v1_6_R2.PlayerConnection.handleCommand(PlayerCon
nection.java:962)
        at net.minecraft.server.v1_6_R2.PlayerConnection.chat(PlayerConnection.j
ava:880)
        at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java
:837)
        at net.minecraft.server.v1_6_R2.Packet3Chat.handle(SourceFile:49)
        at net.minecraft.server.v1_6_R2.NetworkManager.b(NetworkManager.java:296
)
        at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java
:116)
        at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
        at net.minecraft.server.v1_6_R2.DedicatedServerConnection.b(SourceFile:3
0)
        at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:5
90)
        at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:2
26)
        at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:4
86)
        at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java
:419)
        at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:5
82)
Caused by: java.lang.IllegalArgumentException: File cannot be null
        at org.apache.commons.lang.Validate.notNull(Validate.java:203)
        at org.bukkit.configuration.file.YamlConfiguration.loadConfiguration(Yam
lConfiguration.java:170)
        at org.bukkit.plugin.java.JavaPlugin.reloadConfig(JavaPlugin.java:117)
        at org.bukkit.plugin.java.JavaPlugin.getConfig(JavaPlugin.java:111)
        at pl.interia.juniorjpdj.parkour.Komenda.onCommand(Komenda.java:22)
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
        ... 15 more
15:33:00 [INFO] juniorjpdj issued server command: /parkour tp lol
15:33:00 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'park
our' in plugin Parkour v0.1
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:19
1)
        at org.bukkit.craftbukkit.v1_6_R2.CraftServer.dispatchCommand(CraftServe
r.java:523)
        at net.minecraft.server.v1_6_R2.PlayerConnection.handleCommand(PlayerCon
nection.java:962)
        at net.minecraft.server.v1_6_R2.PlayerConnection.chat(PlayerConnection.j
ava:880)
        at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java
:837)
        at net.minecraft.server.v1_6_R2.Packet3Chat.handle(SourceFile:49)
        at net.minecraft.server.v1_6_R2.NetworkManager.b(NetworkManager.java:296
)
        at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java
:116)
        at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
        at net.minecraft.server.v1_6_R2.DedicatedServerConnection.b(SourceFile:3
0)
        at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:5
90)
        at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:2
26)
        at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:4
86)
        at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java
:419)
        at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:5
82)
Caused by: java.lang.IllegalArgumentException: File cannot be null
        at org.apache.commons.lang.Validate.notNull(Validate.java:203)
        at org.bukkit.configuration.file.YamlConfiguration.loadConfiguration(Yam
lConfiguration.java:170)
        at org.bukkit.plugin.java.JavaPlugin.reloadConfig(JavaPlugin.java:117)
        at org.bukkit.plugin.java.JavaPlugin.getConfig(JavaPlugin.java:111)
        at pl.interia.juniorjpdj.parkour.Komenda.onCommand(Komenda.java:33)
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
        ... 15 more
>

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

 

You are trying to access a config which doesn't exist. Access the config from the instance of your main class, and don't extend the main class.

 

So what i need to do?
Can i use Parkour.getConfig??
Or what else?

 

Your command class does not need to extend JavaPlugin.
Replace:

Code:java

public class Komenda extends JavaPlugin implements CommandExecutor {

with:

Code:java

public class Komenda implements CommandExecutor {
    private Parkour plugin;
    public Komenda(Parkour plugin) {
        this.plugin = plugin;
    }

and:

Code:java

getCommand("parkour").setExecutor(new Komenda());

with:

Code:java

getCommand("parkour").setExecutor(new Komenda(this));

This will allow you to use plugin.getConfig(), or call any other methods using the instance of your plugin.

 

Oh, u r amazing D
It works.
But ive listener class too, and i need to use getConfig there too :/
What i need to change?

Listener class:

Code:java

package pl.interia.juniorjpdj.parkour;
 
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.block.Block;
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.PlayerMoveEvent;
 
public class Detektor implements Listener{
 
    @EventHandler(priority = EventPriority.LOWEST)
    public void spadles(PlayerMoveEvent event){
        Player player = event.getPlayer();
        Location loc = player.getLocation();
        World w = loc.getWorld();
        loc.setY(loc.getY() -3);
        Block block = w.getBlockAt(loc);
        if (Parkour.lastparkour.get(player.getName()) != null){
            if (block.getTypeId() == 7){
                player.sendMessage("[Parkour] Fraaajer, spadles!");
                String tmp = Parkour.lastparkour.get(player.getName());
                Location tp = new Location(player.getLocation().getWorld(), getConfig().getDouble("parkour." + tmp + ".x"), getConfig().getDouble("parkour." + tmp + ".y"), getConfig().getDouble("parkour." + tmp + ".z"), (float) getConfig().getDouble("parkour." + tmp + ".yaw"), (float) getConfig().getDouble("parkour." + tmp + ".pitch"));
                player.teleport(tp);
            }
        }
    }
}
 

Ref
Pls help me!

ref

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

 

OK, i fixed it by myself

 

Im still confused!, Even tho this post is really old xD,
I cannot get my string to change in config.yml using commands
using this, nothing changed!

Code:

if(cmd.getName().equalsIgnoreCase("enable"))
{
String s = getConfig().getString("enable")
getConfig().set(s, "true");
{
 

Ive also tried using
List<String> enable = getConfig().getString("enable");

That didnt work, it does not change!, What is im doing wrong!

 

Probably not saving the config, also you dont need to do "true", simply true itself is fine.

 

mactown21 Why are you taking a string argument in a method asking for true/false? Make it a boolean!

Code:java

// This in your main class //
private static YourMainClass plugin;
 
// Initialize the above in your onEnable
@Override
public void onEnable() {
    plugin = this;
}
 
public static YourMainClass getInstance() {
    return plugin;
}
 
// Verious code stuffs //
if (cmd.getName().equalsIgnoreCas("enable")) {
    Boolean bool = YourMainClass.getInstance().getConfig().getBoolean("enabled");
    if (!bool) {
        YourMainClass.getInstance().getConfig().set("enabled", true);
        YourMainClass.getInstance().saveConfig();
        return true;
    }
    sender.sendMessage("Already enabled!");
    return true;
}

 

iiHeroo Azubuso
I did something like this, I tried different ways
I got enable: 'true' & 'true': false & true: '[]' something like that, im not for sure..

Im trying to check a string on config for either true or false, this is something that i created!

Code:

      if (args[0].equalsIgnoreCase("enable"))
      {
          if(getConfig().getBoolean("enable.barapi"))
          {
              player.sendMessage(ChatColor.AQUA+""+ChatColor.BOLD+"[MiniSiTool]" + ChatColor.RESET+""+ChatColor.DARK_AQUA+" || " + ChatColor.RESET+""+ChatColor.AQUA+""+ChatColor.BOLD+"BarAPI is already enabled!");
              return true;
          }else
          {
              String s = "enable.barapi";
              getConfig().set("true", s);
              this.saveConfig();
              this.reloadConfig();
              player.sendMessage(ChatColor.AQUA+""+ChatColor.BOLD+"[MiniSiTool]" + ChatColor.RESET+""+ChatColor.DARK_AQUA+" || " + ChatColor.RESET+""+ChatColor.AQUA+""+ChatColor.BOLD+"Plugin Reloded!");
              return true;
          }
      }
      if (args[0].equalsIgnoreCase("disable"))
      {
          if(getConfig().getBoolean("enable.barapi"))
          {
              String s = "enable.barapi";
              getConfig().set("false", s);
              this.saveConfig();
              this.reloadConfig();
              player.sendMessage(ChatColor.AQUA+""+ChatColor.BOLD+"[MiniSiTool]" + ChatColor.RESET+""+ChatColor.DARK_AQUA+" || " + ChatColor.RESET+""+ChatColor.AQUA+""+ChatColor.BOLD+"Plugin Reloaded");
              return true;
          }else{
              player.sendMessage(ChatColor.AQUA+""+ChatColor.BOLD+"[MiniSiTool]" + ChatColor.RESET+""+ChatColor.DARK_AQUA+" || " + ChatColor.RESET+""+ChatColor.AQUA+""+ChatColor.BOLD+"BarAPI is already disabled!");
              return true;
          }
      }

so mainly this is being changed in config.yml or what im trying to get changed. Ive added a plugin save & reload, and its not working

enable:
barapi: true
^

 

mactown21
Look at what you're actually doing here:

Code:java

String s = "enable.barapi";
getConfig().set("true", s);

What you're doing here is setting the entry "true" in your config, to "enable.barapi", which I'm guessing is the exact opposite of what you're trying to do here! So just flip around the arguments in "getConfig().set(...);" and you should be getting what you want.

Code:java

getConfig().set("enable.barapi", true);
// Save it
 
// In another method
Boolean bool = getConfig().getBoolean("enable.barapi");
if (bool) { } else {} ...
 

 

Azubuso A little to much, ?. What if i did String s = getConfig().getString("enable.barapi");

Config:

enable:
barapi: true

 

mactown21

That's not correct spacing for YAML format. The yaml node "enable.barapi" means that barapi needs 2 or 4 spaces before it. Why 2 or 4 ? Because I've seen it done both ways. And both ways seem to work (and validate).

config.yml (open) config.yml (close)

Code:

enable:
  barapi: true

Do you need any other help with this ?

 

mactown21 Well yeah, It'd work as well, you'd just need to check if the returned string is equal to either true/false. It's just that it's a bit unconventional to use a getString method when asking for a Boolean.

 

maybe because you forgot this????
1.

Code:java

public void onEnable() {
saveDefaultConfig();
}

and 2. put a config.yml in the same directory as your plugin.yml

 

Just a little tutorial what you can make with config:

Code:Java

 
//Get values
Boolean bool = getConfig().getBoolean("Path.to.boolean");
String str = getConfig().getString("Path.to.string");
int i = getConfig().getInt("Path.to.int");
List<String> list = getConfig().getStringList("Path.to.stringlist");
 
//Set values
getConfig().set("Path.to.boolean", true);
getConfig().set("Path.to.string", "example");
getConfig().set("Path.to.int", 3);
getConfig().set("Path.to.stringlist", Arrays.asList("example", "another example", "and one more"));
 
//!!SAVE IT AFTER YOU SET THINGS TO THE CONFIG!!
saveConfig();
 

 

You are using an old version of Bukkit. Please update to the latest 1.7 builds.

/edit: Ugh, necro...

Please don't necro posts like this. Also, his problem was that he extended JavaPlugin instead of simply implementing CmmandExecutor.

On the latest versions, extending JavaPlugin more than once throws an exception itself.

 

Azubuso that was just the format that happened on here. Ik its not the correct formatting. I didn't may attention.