I am new to Bukkit and I want some help on sth like permissions

First of all, here is my AntiBlock.java code:

Code:java

 
package me.firstflames;
 
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.permissions.Permission;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class AntiBlock extends JavaPlugin
{
    public Permission playerPermission = new Permission("playerAbility.allowed");
 
    @Override
    public void onEnable()
    {
        new BlockListener(this);
        PluginManager pm = getServer().getPluginManager();
        pm.addPermission(playerPermission);
 
    }
    @Override
    public void onDisable()
    {
 
    }
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if (cmd.getName().equalsIgnoreCase("giveme") && sender instanceof Player)
        {
            Player player = (Player) sender;
 
            if(player.hasPermission("playerAbility.allowed"))
            {
                player.setItemInHand(new ItemStack(Material.DIAMOND_AXE));
            }
            return true;
        }
        return false;
    }
}
 

here is the BlockListener.java code:

Code:java

 
package me.firstflames;
 
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockPlaceEvent;
 
 
public class BlockListener implements Listener{
 
    public BlockListener(AntiBlock plugin)
    {
        plugin.getServer().getPluginManager().registerEvents(this, plugin);
    }
 
    @EventHandler
    public void onBlockPlace(BlockPlaceEvent e){
 
        Player player = e.getPlayer();
 
        if (!player.hasPermission("playerAbility.allowed")){
 
            e.setCancelled(true);
        }      
    }  
}
 

I am following the video here:

The question is that whenever I join the server,
I don't have the ability to place a block or use the command "giveme".
Also, I can't use the command /deop
It should be the permission problem, right?
So, what is the problem in my code?

thank you.

(p.s. I have used permissionEx with default, haven't changed anything)

 

When you have a player object just do:

Code:java

Player p = /* this is our example player */;
p.hasPermission("your.example.node");[]/syntax]

 

I have tried and the code becomes

Code:java

 
public void onBlockPlace(BlockPlaceEvent e){
 
        Player player = e.getPlayer();
        player.hasPermission("playerAbilities.allowed");
 
        if (e.getBlock().getType() == Material.SAND)
        {
            if (!player.hasPermission("playerAbilities.allowed")){
                player.sendMessage(ChatColor.RED + "You cannot place " + e.getBlock().getType().toString() + " down!");
               e.setCancelled(true);
            }
        } 
}
 

It still does not work.

 

@ykn121 Why doesn't it work? We can't read your mind, although it would make life a lot easier.

 

You're not checking the return value of "hasPermission". Use an if statement.

 

@1Rogue Wow of course, don't know how I missed that...

 

sorry, I mean the plugin should make the player to get the permission "playerAbilities.allowed" such that the player can place the sandblock. However, after several testing, I believe that the code cannot give the permission to the player which the player cannot place the block. And I don't know why...

 

@ykn121 You have to put it in an if statement... As stated by @1Rogue

 

Do you mean the if-statement in line 9?

If I am correct, the hasPermission function should return true if the player got the permission "playerAbilities.allowed". I think I am concerning how to give the permission to the player.

Sorry if I misunderstand your suggestion.

 

@ykn121 You aren't doing anything with the return statement.

 

could you show me what the code should be?

I think the if-statement should be the one in line 9.

 

@ykn121 Ok, you really don't know what you're doing. Anyone I know would suggest to learn Java before learning Bukkit. Bukkit is simply an API written in Bukkit, and learning Bukkit first would be similar to learning to run before learning to walk. Basically just learn Java and once you're sufficient at it, come back to learn Bukkit.

 

For a basic if-statement in Java, I have learnt it already and I have a good background in C++. For some codes like "instanceof" or "this", probably I am not an expert.

If you are just looking at line 5, it means nothing obviously because it just returns a "false" value and that's it. It does not affect the whole program at all. Thats why I am asking whether the if-statement you mentioned has been in line 9.

Don't just focus on line 5.

 

@ykn121 Why is it there?

 

What does your "it"refer to? line 5 or line 9?

If u are talking about line 5, I would say that ...that is not my original version of codes. line 5 was originally not there. As I mentioned, I am new to Bukkit. I thought the hasPermission function can also give the permission to the "player". Thats why I added it after your suggestion.

So, lets come back to the question. The Original codes are on the top of the post. I want to know why I cannot give the permission to the "player" even I think I have followed the video...