Solved saveConfig(); Problems.

Hey, So I was working on this plugin but when I tried to add config.yml functionality it just completely broke. The thing I'm trying to do is that when someone does

/cb on

it turns it on and sets it in the config. Any ideas?

- onCommand method

Code:java

 
package xyz.cb;
 
import static org.bukkit.ChatColor.BOLD;
import static org.bukkit.ChatColor.DARK_RED;
import static org.bukkit.ChatColor.GRAY;
import static org.bukkit.ChatColor.GREEN;
import static org.bukkit.ChatColor.RED;
import static org.bukkit.ChatColor.RESET;
 
import java.util.ArrayList;
import java.util.HashMap;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerCommandPreprocessEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
 
public class Main extends JavaPlugin implements Listener {
 
    public boolean DisableCommands = false;
    public void onEnable () {
        Bukkit.getPluginManager().registerEvents(this, this);
        //getConfig().options().copyDefaults(true);
        saveDefaultConfig();
        saveConfig();
 
        System.out.println("Config Recap:");
        System.out.println("- DisableCommands: " + getConfig().getBoolean("DisableCommands"));
        System.out.println("- warnings: " + getConfig().getInt("warnings"));
        System.out.println("- kickmsg: '" + getConfig().getString("kickmsg") + "'");
        System.out.println("- warnmsg: '" + getConfig().getString("warnmsg") + "'");
 
    }
 
 
    //@SuppressWarnings("unused")
    public boolean onCommand (CommandSender sender, Command cmd, String label, String[] args) {
        if(cmd.getName().equalsIgnoreCase("cb") || cmd.getName().equalsIgnoreCase("commandblocker")) {
            if(args.length == 1) {
                if(args[0].equalsIgnoreCase("off")) {
                    DisableCommands = false;
                    getConfig().set("DisableCommands", false);
                    saveConfig();
                    reloadConfig();
                    if(DisableCommands == false && getConfig().getBoolean("DisableCommands", DisableCommands)) {
                        sender.sendMessage(GREEN + "" + BOLD + "Success! " + RESET + GRAY + "Disabled Command filter");
                        return true;
                    } else {
                        sender.sendMessage(RED + "" + BOLD + "Error: " + RESET + GRAY + "Couldn't disable command-filter...");
                        return true;
                    }
                }
                if(args[0].equalsIgnoreCase("on")) {
                    DisableCommands = true;
                    getConfig().set("DisableCommands", true);
                    saveConfig();
                    reloadConfig();
                    if(DisableCommands == true && getConfig().getBoolean("DisableCommands", DisableCommands)) {
                        sender.sendMessage(GREEN + "" + BOLD + "Success! " + RESET + GRAY + "Enabled Command filter");
                        return true;
                    } else {
                        sender.sendMessage(RED + "" + BOLD + "Error: " + RESET + GRAY + "Couldn't enable command-filter...");
                        return true;
                    }
                }
                if(args[0].equalsIgnoreCase("reload")) {
                    reloadConfig();
                    sender.sendMessage(GREEN + "Reloaded config.yml");
                }
            } else {
                sender.sendMessage(DARK_RED + "" + BOLD + "Invalid syntax: " + RESET + GRAY + "/cb <on/off/reload>");
            }
 
 
        return true;
        }
        return true;
    }
 
    public HashMap<Player, Integer> warned = new  HashMap<Player, Integer>();
 
    @EventHandler
    public void onCommand(PlayerCommandPreprocessEvent e){
       Player p = e.getPlayer();
       if (DisableCommands == true && p instanceof Player) {
          if(!e.getMessage().toLowerCase().startsWith("/cb") && !e.getMessage().toLowerCase().startsWith("/commandblocker")) {
              e.setCancelled(true);
              // p.sendMessage(RED + "" + BOLD + "Hey! " + RESET + GRAY + "Commands are disabled on this server");
              p.sendMessage(ChatColor.translateAlternateColorCodes('&', getConfig().getString("warnmsg")));
              if(warned.get(p) == null) warned.put(p, 0);
 
              ArrayList<Player> cooldown = new ArrayList<Player>();
              cooldown.add(p);
              Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
                    public void run() { 
                        cooldown.remove(p);
                    }
 
              }, 300);
 
 
              if(getConfig().getInt("warnings") != -1) {
                  int i = warned.get(p);
                  i++;
                  warned.put(p, i);
                  if(warned.get(p) > getConfig().getInt("warnings")) {
                      p.kickPlayer(ChatColor.translateAlternateColorCodes('&', getConfig().getString("kickmsg")));
                  }
              }
          }
       }
    }
}
 

 

@Xydez What broke? Is there an error message?

EDIT: You don't have to call reload config after you saveConfig();

 

The error message I made myself just says "couldn't enable command-filter", Which basically means it's not true in the config.

I looked at the config, and there I see a DisableCommands: false which should be set to true.

EDIT: ok thx, but the main problem still remains

 

@Xydez in both of your if statements, you are only using one "=" sign. For if statements, you must use "==".

 

Yeah it says DisableCommand = true; in the code

PS: Why are you "Recommended/Required" to use the edit button, Isn't it just easier to read from top-to-bottom. Is it some kind of storage thing or is it so you don't have to scroll through a giant discussion?

 

@Xydez Is your config file being updated when you run the command? Like is "DisableCommands" being set to true when you run /cb on?

 

Yes

Code:java

DisableCommands = true;
getConfig().set("DisableCommands", true);
saveConfig();

 

@Xydez Sorry I meant in the config, is the config changing?

 

@X1machinemaker1X Nope, it dosen't. The code seems to hate me.

 

@Xydez Ok, I think I know the problem but I didnt see it because they look almost the same. Some times you use "config" when you reference the configuration in the code, and sometimes you use getConfig(); Always use getConfig();

 

@X1machinemaker1X Up for the test

EDIT: The code looks a lot cleaner, But it still won't write to the config

EDIT2 :@X1machinemaker1X Updated code.

 

@Xydez Is all your code in one class? If so, can you post the entire thing? not just the onCommand() method?

 

The code is in one class. Yeah, kind of a beginner mistake but it's easier. Updated code.

 

@Xydez
Use the getConfig().options().copyDefaults(true).
And using saveDefaultConfig(); will not do anything if there are no defaults. just use saveConfig();

 

I have been looking in the wrong config folder. Sh*t.

Well, it does change in the right config folder. But it still won't disable the command-filter

 

@Xydez
Wow. *facepalm* So now the config changes when you run /cb on or /cb off?

 

Yeah, But it still won't see that DisableCommand: true is true in the config.

It's like really blind or something (This is only for /cb off.)

 

@Xydez remove the second argument from the getBoolean(path) method. And you don't need the reloadConfig() methods before saving. Also, try printing out the getBoolean("DisableCommands") just to see if its null or not

 

No need to debug anymore. The code Finally works!

All thanks to the (getConfig().getBoolean("DisableCommands") == false)

 

@Xydez
Darn, I'm kinda mad I didn't see that. If you compare two things with "==" and you have "&&" or "||" you need parentheses.

 

gg, Plugin released on <Edit by Moderator: Redacted not allowed paid resource url>