How to create an Reload command?

Hey, what is false with this code? look:
I will make an reload function, for my plugin .. thanks

Code:

  } else if (args.length == 1 && args[0].equalsIgnoreCase("Reload")) {
if (player != null); 
if (player.hasPermission("promotion.reload")) {
    this.reloadConfig();
    this.saveConfig();
    player.sendMessage(ChatColor.GREEN + "[AutoPromotion] Config reloaded!"); 
    System.out.println("[AutoPromotion] Config reloaded!");
    return true;
}    

 

Code:Java

} else if (args.length == 1 && args[0].equalsIgnoreCase("Reload")) {
     if (!player instanceof Player) {
     // If player is not an instanceof Player, then it's the console sending the command.
     this.reloadConfig();
     this.saveConfig();
     Bukkit.getLogger().info("[AutoPromotion] Config Reloaded! ");
     // Returns true because the console can execute anything.
     return true;
     } else if (player.hasPermission("promotion.reload")) {
          // If it's not the console, then it's a player. Check their permission and execute if they have.
          this.reloadConfig();
          this.saveConfig();
          player.sendMessage(ChatColor.GREEN + "[AutoPromotion] Config reloaded!");
          System.out.println("[AutoPromotion] Config reloaded!");
          return true;
          // Return true if this plugin executed this command.
     } else {
          return false;
          // Returns false because the player didn't have permission, so it wasn't executed.
     }
}

 

THANKS

Ahh, that doesn't really works, the console say Config reloaded, but it do nothing -_-

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

 

Code:Java

 
} else if (args.length == 1 && args[0].equalsIgnoreCase("Reload")) {
     if (player instanceof Player) {
          if (player.hasPermission("promotion.reload")) {
               this.reloadConfig();
               this.saveConfig();
               player.sendMessage("[AutoPromotion] Config Reloaded! ");
               // Return true if this plugin executed this command.
               return true;
          }
     } else {
          this.reloadConfig();
          this.saveConfig();
          Bukkit.getLogger().info(ChatColor.GREEN + "[AutoPromotion] Config reloaded!");
          System.out.println("[AutoPromotion] Config reloaded!");
          return true;
          // Return true if this plugin executed this command.
          // Returns true because the console can execute anything.
     }
     return false;
}

Try that.

 

Yes, it says config was reloaded, but it' does not save changes.

 

did you make those chances to the confi after reloading, but before save?

 

Yes, i changed what in the Config, but it doesnt work, you can try it with my Plugin, that doesn't work ... the /Promote Reload command ://

 

I'm having the same issue. I have reloadconfig then saveconfig (just like above) but it doesn't seem to work. The config doesn't reload...

 

Yes ... exactly, damn! xD It will not work -.- i've made a new command like /Promote Reload, with this.saveConfig & this.ReloadConfig(); xD but it doesn't work :/

 

Try and get rid of the saveConfig() and just reload a config with changes.

 

Save, then load

Edit: idk what I was thinking, try reloading without saving

 

Did you mean

Code:

 this.saveConfig(); & this.loadConfig(); 

?

 

yopu need to call "getConfig()" again after loading, are you guys doing that?

 

Wait, do you want to reload the plugin file, or the config(s) ?
If you want to reload the file itself(.jar) then this can be a sulution

Code:java

plugin.getPluginLoader().disablePlugin(plugin);
plugin.getPluginLoader().enablePlugin(plugin);

Or if it is the config(s), then

Code:java

plugin.reloadConfig();

What I use for the file reload command:

Code:java

public class Reload implements CommandExecutor {
    public static pluginname plugin;
    public Reload(pluginname instance)
    {
        plugin = instance;
    }
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        if(cmd.getName().equalsIgnoreCase("reload-pluginname")) {
            if (args.length > 0) {
                sender.sendMessage(ChatColor.RED + "To many arguments.");
                return false;
            }
            if (sender instanceof Player) {
                Player player = (Player)sender;
                if (hasReload(player)) {
                    plugin.getPluginLoader().disablePlugin(plugin);
                    plugin.getPluginLoader().enablePlugin(plugin);
                    sender.sendMessage(ChatColor.YELLOW + "Reloaded "+ ChatColor.GRAY + plugin.getDescription().getFullName());
                } else {
                    sender.sendMessage(ChatColor.RED + "You do not have access to that.");
                }
            } else {
                plugin.getPluginLoader().disablePlugin(plugin);
                plugin.getPluginLoader().enablePlugin(plugin);
                sender.sendMessage(ChatColor.YELLOW + "Reloaded "+ ChatColor.GRAY + plugin.getDescription().getFullName());
            }
            return true;
        }
        return false;
    }
 
    public boolean hasReload(Player player) {
        if (player.hasPermission("pluginname.reload")) {
            return true;
        } else if  (player.hasPermission("pluginname.*")) {
            return true;
        }
        return false;
    }
}

What I use for the config reload:

Code:java

public class Reload implements CommandExecutor {
    public static pluginnameplugin;
    public Reload(pluginnameinstance)
    {
        plugin = instance;
    }
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        if(cmd.getName().equalsIgnoreCase("reload-pluginname")){
            if (args.length > 0) {
                sender.sendMessage(ChatColor.RED + "Too many arguments!");
                return false;
            }
            PluginDescriptionFile pdffile = plugin.getDescription();
            if (sender instanceof Player) { 
                Player player = (Player)sender;
                if (hasReload(player)) { 
                    plugin.reloadConfig();
            sender.sendMessage(ChatColor.AQUA + pdffile.getFullName() + ChatColor.GRAY + " reloaded files succesfully");
                    System.out.println("[pluginname] "+ (sender.getName() + " reloaded" + (pdffile.getFullName())));
            } else { 
                    sender.sendMessage(ChatColor.RED + "You do not have access to that." );
                }
            } else { 
                plugin.reloadConfig();
        plugin.log.info("[pluginname] " + pdffile.getName() + " was succesfully reloaded");
            }
            return true;
        }
public boolean hasReload(Player player) {
        if (player.hasPermission("pluginname.reload")) {
            return true;
        } else if  (player.hasPermission("pluginname.*")) {
            return true;
        }
        return false;
    }

 

Thanks! will try that now :-P