How to check permissions

I am a very new bukkit plugin coder... as you can tell by this question I am asking. Where would I insert and what would I insert to check if a player has the correct permission when they type "/hat". I know is goes something like "if(player.hasPermission"BbyHelms.use");". Here is my code:​

​

 

Code:

if(player.hasPermission("node.here"){

}

Put it above the if(cmd.getName().stuff), but make sure all that stuff is inside the permission checking if statement's squiggly brackets.

 

Alright, so would it look like this?

Code:

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        Player player = (Player)sender;
        if(player.hasPermission("BbyHelms.use")){
           
        }
        if(cmd.getName().equalsIgnoreCase("hat"));
        ItemStack is = player.getInventory().getHelmet();
        player.getInventory().setHelmet(player.getItemInHand());
        player.getInventory().removeItem(player.getItemInHand());
        player.sendMessage(ChatColor.GOLD + "[BbyHelms]" + ChatColor.DARK_RED + " Enjoy your new hat!");
        player.setItemInHand(is);
        return false;
    }
}

 

Code:

if(player.hasPermission("BbyHelms.use")){
         
}

What you coded:

If player has this permission: Do nothing!

 

Firefly
Hm, how about this:

Code:

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        Player player = (Player)sender;
        if(player.hasPermission("BbyHelms.use")){
        if(cmd.getName().equalsIgnoreCase("hat"));
        ItemStack is = player.getInventory().getHelmet();
        player.getInventory().setHelmet(player.getItemInHand());
        player.getInventory().removeItem(player.getItemInHand());
        player.sendMessage(ChatColor.GOLD + "[BbyHelms]" + ChatColor.DARK_RED + " Enjoy your new hat!");
        player.setItemInHand(is);
        }
        return false;
    }
}

 

JjPwN1
That is, in fact, how it should be done. Good job!

 

For user-friendliness sake, I'd add an else statement to tell the player they don't have permission, instead of doing nothing.

 

turqmelon
To do that, would the code look like this?

Code:

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        Player player = (Player)sender;
        if(player.hasPermission("BbyHelms.use")){
        if(cmd.getName().equalsIgnoreCase("hat"));
        ItemStack is = player.getInventory().getHelmet();
        player.getInventory().setHelmet(player.getItemInHand());
        player.getInventory().removeItem(player.getItemInHand());
        player.sendMessage(ChatColor.GOLD + "[BbyHelms]" + ChatColor.DARK_RED + " Enjoy your new hat!");
        player.setItemInHand(is);
        }else;
        player.sendMessage(ChatColor.GOLD + "[BbyHelms]" + ChatColor.DARK_RED + " You don't have enough permission!");
        return false;
    }
}

 

No, since that else is linked to the if regarding the cmd.getName()

You have some brackets missing.

 

Change this:

Code:

}else;
        player.sendMessage(ChatColor.GOLD + "[BbyHelms]" + ChatColor.DARK_RED + " You don't have enough permission!");
        return false;

To:

Code:

}else{
        player.sendMessage(ChatColor.GOLD + "[BbyHelms]" + ChatColor.DARK_RED + " You don't have enough permission!");
return true;
}

If you return false, it'll display the syntax you specify in the plugin.yml. Returning true will allow you to use your own custom messages.

 

Alright, I got it. Works perfectly, learned a ton from you guys.

 

Firefly
There are two opening brackets, and two closing brackets, @turqmelon's solution should fix it.