Even the Console Doesn't Have Access to the Command

I'm trying to make multiple Yaml files in my plugin's section, but every time I try to set something in a file different from the config.yml, it says I do not have permission to execute this command. I am certain it is not because of the actual permission nodes the player has, because of these reasons:
1. I could execute the same command before when my code used the getConfig method instead of the getCustomConfig method(I'll show you it later).
2. When I try to use the console, it says I don't have access to the command, and "If I believe this is an error, I can contact the server administrators"
Here is my code:
When I execute the command

Code:java

static Main plugin;
 
    public SetspawnCommand(Main instance) {
        plugin = instance;
    }
 
    @Override
    public boolean onCommand(CommandSender s, Command cmd, String l,
            String[] args) {
        Player player = (Player) s;
        if ((Methods.hasPermission(player, "spawnpoints.admin")) || player.isOp()) {
            if (l.equalsIgnoreCase("setspawn") && args.length == 0) {
 
                s.sendMessage(Main.Prefix + ChatColor.GOLD + " The "
                        + ChatColor.RED + "spawn" + ChatColor.GOLD
                        + " SpawnPoint has been set!");
                Location loc = player.getLocation();
                String w = player.getLocation().getWorld().getName().toString();
                plugin.getData().set("spawn" + "." + "X",
                        Double.valueOf(loc.getX()));
                plugin.getData().set("spawn" + "." + "Y",
                        Double.valueOf(loc.getY()));
                plugin.getData().set("spawn" + "." + "Z",
                        Double.valueOf(loc.getZ()));
                plugin.getData().set("spawn" + "." + "W", w);
                plugin.saveData();
            } else {
                s.sendMessage(Main.Prefix + ChatColor.RED
                        + " Something went wrong!");
            }
 
        } else {
            s.sendMessage(Main.Prefix + ChatColor.RED + " You don't have "
                    + ChatColor.GOLD + "permission" + ChatColor.RED
                    + " for this command!");
        }
        return false;
    }

Here are my methods for the separate data file:

Code:java

public static Plugin me;
public void onEnable() {
     me = this;
   }
public FileConfiguration getData() {
        if (dFile == null) {
            reloadDataFile();
        }
        return dFile;
    }
    public void reloadDataFile() {
        if (dataFile == null) {
            dataFile = new File(Main.me.getDataFolder(), "data.yml");
        }
        dFile = YamlConfiguration.loadConfiguration(dataFile);
 
 
        InputStream defConfigStream = Main.me.getResource("data.yml");
        if (defConfigStream != null) {
            YamlConfiguration defConfig = YamlConfiguration.loadConfiguration(defConfigStream);
            dFile.setDefaults(defConfig);
        }
 
        if (dFile == null || dataFile == null) {
            return;
        }
        try {
            getData().save(dataFile);
        } catch (IOException ex) {
            Main.me.getLogger().log(Level.SEVERE, "Could not save config to " + dataFile, ex);
        }
    }
    public void saveData() {
        if (dataFile == null) {
            dataFile = new File(Main.me.getDataFolder(), "data.yml");
        }
        if (!dataFile.exists()) {           
            Main.me.saveResource("data.yml", false);
        }
    }

I don't have access even when I'm opped on my server, and when I type the command into the console, I get this:

Code:

I'm sorry, but you do not have permission to perform this command. Please contact the server administrators if you believe that this is in error.

What I'm thinking is that somehow my plugin is trying to do something that Bukkit's security prevents it from doing, because it thinks its malicious code... But I have no idea. If you find the problem, please tell me. Thanks!

 

Two things:
1. You should use hash maps.
2. Based solely on the fact that you're setting a spawn, around you command you should add:

Code:java

if (sender instanceof (Player) {}

What that does, if I remember correctly, is it checks if the sender is a player and fires it.

 

You're being very vague when you say "You should use hash maps". Hashmaps for what? And just casting and saying if the sender is a player won't help in this circumstance. My problem is related to permissions, not the sender.

Anyone find an error?

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

 

Can you paste your plugin.yml, and the Methods class?

 

Casting the player checker is needed if your setting a home, the console is not a player, therefore not an entity.

Also, You should replace your current permission checker with this:

Code:java

if (sender.hasPermission("spawnpoints.admin")) {} elseif (!sender.hasPermission("spawnpoints.admin")) return false; 

 

Code:

name: Spawns
version: 0.1
main: me.abandonedminecart.spawns.main
commands:
  setspawn:
    description: Set spawn
    aliases: [setsp, ss]
    permission: spawnpoints.admin
    usage: /setspawn
permissions:
  spawnpoints.*:
    description: commands under 1 permission
    default: false
    children:
      spawnpoints.admin: true
  spawnpoints.admin:
    description: All admin permissions
    default: false

Code:java

public static boolean hasPermission(Player player, String permission) {
if (player.hasPermission(permission)) {
return true;
}
return false;
}

 

Remove the permission node from the command, so that your plugin.yml looks like this for the command:

Code:

commands:
  setspawn:
    description: Set spawn
    aliases: [setsp, ss]
    usage: /setspawn

Then it should check itself within the plugin.

That all being said, I don't know why you made a hasPermission method that just calls a boolean. You can simplify that method down to:

Code:java

public static boolean hasPermissions(Player player, String permission) {
    return player.hasPermission(permission);
}

But overall, it isn't even necessary.

 

I want to do that, but that's more boilerplate overhead code. Is there a way to allow console by default while also specifying the permission node?

 

console has all perms

 

How can console use this command when you cast him to player?

 

krisdestruction AbandonedMinecart

Set default to 'op'. BUT the console isn't a 'Player' instance. You still can't run /setspawn from the console and have it work.