Solved Permission not working

I've recently been making a plugin that will tp you to people's log off location. I'm trying to add a permission node to it and all works well except for having a return message if the permission node is not in your permissions.

If I try to add an extra else statement at the bottom it won't work. I just need to know where to put the way to respond if you don't have the permission. Help?

Code:java

if(args.length == 1){
String target = args[0];
 
if (player.hasPermission("mgv.staff")) {
if(getConfig().contains(target)){
//Grab all the config values
int x = getConfig().getInt(target + ".x");
int y = getConfig().getInt(target + ".y");
int z = getConfig().getInt(target + ".z");
String world = getConfig().getString(target + ".world");
Location loc = new Location(Bukkit.getWorld(world), x, y, z); //Build the location object
 
player.teleport(loc); //Teleport the player
} else {
player.sendMessage(ChatColor.RED + "That player hasn't played before!");
}
}
else {
player.sendMessage(ChatColor.RED + "Usage: /lastlog <player>");
}
}
}
return false; 

Note the code up top is only part of the code, if you need the entire code, I will happily post it.

Thanks.

 

SimplyCode An else statement would work perfectly fine

Code:

if(player.hasPermission("perm.node"){
 
} else {
 
}

Either the Player will have the permission, or they won't.
Else is ran if they don't.

 

But where do I put the else statment in my code, as I have two other else statements at the bottom.

Thanks.

 

SimplyCode simply make the else statement for the if statement that checks for the permission.

 

I'm a little confused by the code you posted as to whether there are mistakes with the brackets, or if those are intentional and you left the code there out(you said it isn't the full code). So here's your original code(formatted, as your post's isn't), and I left comments asking where the extra brackets/mistakes are. Correct me if you took code out and I'm wrong

Code:java

if(args.length == 1) {
    String target = args[0];
 
    if(player.hasPermission("mgv.staff")) {
        if(getConfig().contains(target)) {
        //Grab all the config values
            int x = getConfig().getInt(target + ".x");
            int y = getConfig().getInt(target + ".y");
            int z = getConfig().getInt(target + ".z");
            String world = getConfig().getString(target + ".world");
            Location loc = new Location(Bukkit.getWorld(world), x, y, z); //Build the location object
 
            player.teleport(loc); //Teleport the player
            }
        else {
            player.sendMessage(ChatColor.RED + "That player hasn't played before!");
            }
        }
    else {
        player.sendMessage(ChatColor.Red + "Usage: /lastlog <player>");
        }
    }
} //this bracket is an extra, it is outside the if(args.length ==1)? Is there more code you left out or is this a mistake?
return false;

Now, assuming that the mistakes in the code are indeed mistakes, I've fixed the brackets and here's the code showing you where to add the else statement.

Code:java

if(args.length == 1) {
    String target = args[0];
 
    if(player.hasPermission("mgv.staff")) {
        if(getConfig().contains(target)) {
        //Grab all the config values
            int x = getConfig().getInt(target + ".x");
            int y = getConfig().getInt(target + ".y");
            int z = getConfig().getInt(target + ".z");
            String world = getConfig().getString(target + ".world");
            Location loc = new Location(Bukkit.getWorld(world), x, y, z); //Build the location object
 
            player.teleport(loc); //Teleport the player
            }
        else {
            player.sendMessage(ChatColor.RED + "That player hasn't played before!");
            }
        }
    else {
        player.sendMessage(ChatColor.Red + "Usage: /lastlog <player>");
        player.sendMessage(//put your you dont have permission message here)
        //this else ^ is just outside of the if(player.hasPermission), so you would put that here.
        }
    }
return false;

*That should be all correct, but I apologize if there is a small error because it is late at night