Solved Adding Defaults to Config w/ Command

Hi there! I am making a plugin where the main configuration file will hold a bunch of different names(I am making a school plugin and the names are the classes). The names will be string lists which will hold player names. I'm trying to create a command to where the 2nd argument will be the new class name. I am having much trouble making it. Also, can you explain how I can add a player's name if they want to join the class?

Code:

onEnable method(If needed):

Code:java

    public void onEnable(){
        logger.info("School has been enabled!");
        getCommand("class").setExecutor(new ClassCommands());
        FileConfiguration config = this.getConfig();
        config.options().header("##School Configuration File Main##");
        config.options().copyHeader(true);
        config.options().copyDefaults(true);
        this.saveConfig();
    }

Class Command:

Code:java

package net.atvci.noahtemp.schoolplugin.commands;
 
import java.util.Arrays;
 
import net.atvci.noahtemp.schoolplugin.SchoolPluginMain;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
 
public class ClassCommands implements CommandExecutor {
 
    public Player player;
    public SchoolPluginMain plugin;
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {
        if(cmd.getName().equalsIgnoreCase("class")){
            if(sender instanceof Player){
            if(args.length != 2){
                sender.sendMessage(ChatColor.RED + "Usage: /class <option> [value]");
                return true;
            }
                if(args[0].equalsIgnoreCase("create")){
                    String className = args[1].toString();
                    String[] list = {"Test"};
                    if(!(plugin.getConfig().contains("Classes." + className))){               
                        plugin.getConfig().addDefault("Classes." + className, Arrays.asList(list));
                        plugin.getConfig().options().copyDefaults(true);
                        plugin.saveConfig();
                        sender.sendMessage(ChatColor.GREEN + "You have successfully created the " + className + " class!");
                    }else{
                        sender.sendMessage(ChatColor.RED + "That class already exists!");
                    }
                }
            }
            return true;
        }
        return false;
    }
}

If any more code is needed I'll be happy to add it! Thanks for your help!

Also, I see that the log might help in this!

Log:

Code:

2013-07-30 20:39:23 [INFO] WeaselBuilds issued server command: /class create Red
2013-07-30 20:39:23 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'class' in plugin School v0.0.0.1b
    at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
    at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:189)
    at org.bukkit.craftbukkit.v1_6_R2.CraftServer.dispatchCommand(CraftServer.java:523)
    at net.minecraft.server.v1_6_R2.PlayerConnection.handleCommand(PlayerConnection.java:962)
    at net.minecraft.server.v1_6_R2.PlayerConnection.chat(PlayerConnection.java: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:30)
    at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:590)
    at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:226)
    at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:486)
    at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java:419)
    at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:582)
Caused by: java.lang.NullPointerException
    at net.atvci.noahtemp.schoolplugin.commands.ClassCommands.onCommand(ClassCommands.java:30)
    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 need to put line 27-37 in an else statement

 

WeaselBuilds
You need to instantiate plugin in your Command class

 

What should I put?

How can I do that? Can you give me code please?

EDIT: I added this to my onEnable:

Code:java

plugin = this;

And then in the ClassCommands class, removed:

Code:java

public SchoolPluginMain plugin; //This might have worked. I don't know

And now I can use this:

Code:java

SchoolPluginMain.plugin.getConfig().set("path", value);
SchoolPluginMain.plugin.saveConfig();

Thanks for your help! I got it to work now!

 

WeaselBuilds
That is really bad programming practice. You would instantiate the plugin variable by passing the plugin class through a constructor.
This goes in the command class

PHP:

Main plugin;
public Class(Main plugin){
this.plugin = plugin;
}

This will cause errors with your register line. To register it, use this in the main class

PHP:

getCommand("class").setExecutor(new Class(this));

You would replace all Main 's in the above code with the main class name / all Class 's in the above post with the class you want to register.

 

You're right but don't confuse him :/. I doubt he knows about constructors.

 

I'm not confused about constructors. I make mods, too, so I know how to do this stuff.