Why my plugin override all protection plugins!!!???

Please someone help this is my block break event!

Code:java

    @EventHandler(priority = EventPriority.NORMAL)
    public void blockbreak(BlockBreakEvent e){
        Player p = e.getPlayer();
        SoundsUtility sounds = new SoundsUtility();
 
        if(!(p.getInventory().firstEmpty() == -1)){
            if(!p.getGameMode().equals(GameMode.CREATIVE)){
                for(ItemStack items : e.getBlock().getDrops()){
                    if (p.getItemInHand().getEnchantments() != null) {
                        ItemStack item = p.getItemInHand();
 
                        Map<Enchantment, Integer> enchantments = item.getEnchantments();
                        if (enchantments.containsKey(Enchantment.LOOT_BONUS_BLOCKS)) {
                            int level = enchantments.get(Enchantment.LOOT_BONUS_BLOCKS);
                            for(int i = 0; i <= level; i++){
                                p.getInventory().addItem(items);
                                p.updateInventory();
                            }
                        } else {
                            p.getInventory().addItem(items);
                        }
                    }
                }
                e.getBlock().setType(Material.AIR);
                e.setCancelled(true);
            }
        } else {
            for(ItemStack items : e.getBlock().getDrops()){
                if (p.getItemInHand().getEnchantments() != null) {
                    ItemStack item = p.getItemInHand();
 
                    Map<Enchantment, Integer> enchantments = item.getEnchantments();
                    if (enchantments.containsKey(Enchantment.LOOT_BONUS_BLOCKS)) {
                        int level = enchantments.get(Enchantment.LOOT_BONUS_BLOCKS);
                        for(int i = 0; i <= level; i++){
                            sounds.playSound(p, p.getLocation(), "note.hat", 1, 1);
                            e.getBlock().getWorld().dropItemNaturally(e.getBlock().getLocation(), items);
 
                            e.getBlock().setType(Material.AIR);
                            e.setCancelled(true);
                        }
                    }
                }
            }
        }
    }

 

You should add that at the beginning of your event:

Code:java

 
if (event.isCancelled())
    return;

 

Panjab Still does override protection plugins D:

 

That means the event hasn't been cancelled by your protection plugin, my suggestion would be to move down to the next priority until you noticed a difference.

 

Watto my protection plugin is world guard and i think they cancel the event don't they .... and i tried LOWEST priority and still nothing ...
Edit: I also look on this code https://github.com/OzairP/AutoInven...m/ozairpatel/autoinventory/EventListener.java and it looks soooooo simple there but in my plugin it just overrides ...
Maybe my Main class will help a bit:

Code:java

package net.killacraft;
 
import net.killacraft.events.BreakBlockEvent;
 
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class AutoInventory extends JavaPlugin{
    public void onEnable(){
        registerEvents();
    }
    public void registerEvents(){
        PluginManager pm = getServer().getPluginManager();
 
        pm.registerEvents(new BreakBlockEvent(this), this);
    }
}
 

 

You have to set the priority to at least to HIGHER in order for WorldGuard to be able to cancle it before it reaches your plugin. Then ignore the event if it already is cancelled.
Also why do you even need to have those lines in there?
e.getBlock().setType(Material.AIR);
e.setCancelled(true);

You are tring to recreate how the server would handle the event? Why not just not cancle it and let the server actually handle it itself?

 

Ever heard of the EventHandler's value 'ignoreCancelled'? Try setting that to true.

EnchantedMiners HIGHEST is the one that is fired last (not including MONITOR because it is not meant to influence the event, it should just be used for logging and such)

 

Watto Panjab thanks for the help but i just looked up WorldGuard API and find out that there was something called canBuild() xD so easier thanks any ways

 

EnchantedMiners Using that might make a needless WorldGuard dependency, depending on how you do it, and will still override other protection plugins. Use a higher priority, and include ignoreCancelled as well

 

AdamQpzm thanks! but this plugin is a private plugin for my server so i only use WorldGuard to protect land so, but thanks for the comment!