Quick Question!

Plugins Plugin Development
1

I have this piece of code:

@Subscribe
    public void onBreak(PlayerBreakBlockEvent event) {
        
        Player player = event.getPlayer();
        
        if(player.hasPermission("mafia.break")) {
            event.setCancelled(true);
        } else {
            event.getBlock().replaceWith(event.getBlock().getType());
        }    
    }

Will this disallow players without the mafia.break permission to break blocks?

2

Actually it will always place the block preventing anyone from breaking. The problem lies in the

event.setCancelled(true);

This will cause the block not to be broken (since you cancelled the event). The proper way would be something like this:

@Subscribe
public void onBreak(PlayerBreakBlockEvent event) {
  Player player = event.getPlayer();

  if(player.hasPermission("mafia.break")) {
    //TODO: Whatever you wanted.
  } else {
    event.setCancelled(true);
  }
}
3

@LordLambda, thanks. Preview of my code:

@Subscribe
    public void onBreak(PlayerBreakBlockEvent event) {

        Player player = event.getPlayer();

        if(!(player.hasPermission("mafia.break"))) {
            event.setCancelled(true);
        }
    }
4

That would definitely work.

5

You can take away one set of brackets there, bit easier to read

1 Like