diff --git a/src/main/java/lol/hyper/toolstats/events/AnvilEvent.java b/src/main/java/lol/hyper/toolstats/events/AnvilEvent.java index edf9ebe..248d3f4 100755 --- a/src/main/java/lol/hyper/toolstats/events/AnvilEvent.java +++ b/src/main/java/lol/hyper/toolstats/events/AnvilEvent.java @@ -39,7 +39,7 @@ public class AnvilEvent implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onAnvilEvent(PrepareAnvilEvent event) { // only listen if the token system is enabled if (!toolStats.config.getBoolean("tokens.enabled")) { diff --git a/src/main/java/lol/hyper/toolstats/events/BlockBreak.java b/src/main/java/lol/hyper/toolstats/events/BlockBreak.java index ae62b20..58bd5c0 100755 --- a/src/main/java/lol/hyper/toolstats/events/BlockBreak.java +++ b/src/main/java/lol/hyper/toolstats/events/BlockBreak.java @@ -45,11 +45,8 @@ public class BlockBreak implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.MONITOR) + @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) public void onBreak(BlockBreakEvent event) { - if (event.isCancelled()) { - return; - } Player player = event.getPlayer(); if (!toolStats.configTools.checkWorld(player.getWorld().getName())) { return; diff --git a/src/main/java/lol/hyper/toolstats/events/BlockDispenseEvent.java b/src/main/java/lol/hyper/toolstats/events/BlockDispenseEvent.java index 9c53815..63e31b4 100755 --- a/src/main/java/lol/hyper/toolstats/events/BlockDispenseEvent.java +++ b/src/main/java/lol/hyper/toolstats/events/BlockDispenseEvent.java @@ -45,7 +45,7 @@ public class BlockDispenseEvent implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onDispense(BlockDispenseLootEvent event) { Player player = event.getPlayer(); if (player == null) { diff --git a/src/main/java/lol/hyper/toolstats/events/ChunkPopulate.java b/src/main/java/lol/hyper/toolstats/events/ChunkPopulate.java index e8af879..1496026 100755 --- a/src/main/java/lol/hyper/toolstats/events/ChunkPopulate.java +++ b/src/main/java/lol/hyper/toolstats/events/ChunkPopulate.java @@ -44,7 +44,7 @@ public class ChunkPopulate implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onPopulate(ChunkPopulateEvent event) { if (event.getChunk().getWorld().getEnvironment() != World.Environment.THE_END) { return; diff --git a/src/main/java/lol/hyper/toolstats/events/CraftItem.java b/src/main/java/lol/hyper/toolstats/events/CraftItem.java index 65f9065..4a791fd 100755 --- a/src/main/java/lol/hyper/toolstats/events/CraftItem.java +++ b/src/main/java/lol/hyper/toolstats/events/CraftItem.java @@ -44,11 +44,8 @@ public class CraftItem implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onCraft(CraftItemEvent event) { - if (event.isCancelled()) { - return; - } Player player = (Player) event.getWhoClicked(); if (!toolStats.configTools.checkWorld(player.getWorld().getName())) { return; diff --git a/src/main/java/lol/hyper/toolstats/events/CreativeEvent.java b/src/main/java/lol/hyper/toolstats/events/CreativeEvent.java index 7fd4830..539affe 100755 --- a/src/main/java/lol/hyper/toolstats/events/CreativeEvent.java +++ b/src/main/java/lol/hyper/toolstats/events/CreativeEvent.java @@ -23,6 +23,7 @@ import net.kyori.adventure.text.Component; import org.bukkit.GameMode; import org.bukkit.entity.Player; import org.bukkit.event.EventHandler; +import org.bukkit.event.EventPriority; import org.bukkit.event.Listener; import org.bukkit.event.inventory.InventoryCreativeEvent; import org.bukkit.inventory.ItemStack; @@ -42,7 +43,7 @@ public class CreativeEvent implements Listener { this.toolStats = toolStats; } - @EventHandler + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onCreativeEvent(InventoryCreativeEvent event) { Player player = (Player) event.getWhoClicked(); if (!toolStats.configTools.checkWorld(player.getWorld().getName())) { diff --git a/src/main/java/lol/hyper/toolstats/events/EntityDamage.java b/src/main/java/lol/hyper/toolstats/events/EntityDamage.java index de92feb..784a593 100755 --- a/src/main/java/lol/hyper/toolstats/events/EntityDamage.java +++ b/src/main/java/lol/hyper/toolstats/events/EntityDamage.java @@ -46,12 +46,8 @@ public class EntityDamage implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.MONITOR) + @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) public void onDamage(EntityDamageByEntityEvent event) { - if (event.isCancelled()) { - return; - } - if (!(event.getEntity() instanceof LivingEntity mobBeingAttacked)) { return; } diff --git a/src/main/java/lol/hyper/toolstats/events/EntityDeath.java b/src/main/java/lol/hyper/toolstats/events/EntityDeath.java index 927759e..72c2383 100755 --- a/src/main/java/lol/hyper/toolstats/events/EntityDeath.java +++ b/src/main/java/lol/hyper/toolstats/events/EntityDeath.java @@ -43,7 +43,7 @@ public class EntityDeath implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onDeath(EntityDeathEvent event) { LivingEntity livingEntity = event.getEntity(); if (livingEntity instanceof Player) { diff --git a/src/main/java/lol/hyper/toolstats/events/GenerateLoot.java b/src/main/java/lol/hyper/toolstats/events/GenerateLoot.java index d9c2919..535e25b 100755 --- a/src/main/java/lol/hyper/toolstats/events/GenerateLoot.java +++ b/src/main/java/lol/hyper/toolstats/events/GenerateLoot.java @@ -46,7 +46,7 @@ public class GenerateLoot implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onGenerateLoot(LootGenerateEvent event) { InventoryHolder inventoryHolder = event.getInventoryHolder(); if (inventoryHolder == null) { diff --git a/src/main/java/lol/hyper/toolstats/events/InventoryClose.java b/src/main/java/lol/hyper/toolstats/events/InventoryClose.java index 41f9811..1591671 100755 --- a/src/main/java/lol/hyper/toolstats/events/InventoryClose.java +++ b/src/main/java/lol/hyper/toolstats/events/InventoryClose.java @@ -50,7 +50,7 @@ public class InventoryClose implements Listener { this.toolStats = toolStats; } - @EventHandler + @EventHandler(ignoreCancelled = true) public void onClose(InventoryCloseEvent event) { if (toolStats.generateLoot.generatedInventory.isEmpty()) { return; @@ -106,7 +106,7 @@ public class InventoryClose implements Listener { }, null, 1); } - if (holder instanceof Container container) { + if (holder instanceof Container) { Chunk chestChunk = chestLocation.getChunk(); Bukkit.getRegionScheduler().runDelayed(toolStats, chestLocation.getWorld(), chestChunk.getX(), chestChunk.getZ(), scheduledTask -> { BlockState blockState = chestLocation.getWorld().getBlockAt(chestLocation).getState(); diff --git a/src/main/java/lol/hyper/toolstats/events/InventoryOpen.java b/src/main/java/lol/hyper/toolstats/events/InventoryOpen.java index 662e12c..614a27c 100644 --- a/src/main/java/lol/hyper/toolstats/events/InventoryOpen.java +++ b/src/main/java/lol/hyper/toolstats/events/InventoryOpen.java @@ -42,12 +42,8 @@ public class InventoryOpen implements Listener { this.toolStats = toolStats; } - @EventHandler + @EventHandler(ignoreCancelled = true) public void onOpen(InventoryOpenEvent event) { - if (event.isCancelled()) { - return; - } - Inventory inventory = event.getInventory(); InventoryHolder holder = inventory.getHolder(); boolean isBlockInventory = holder instanceof BlockInventoryHolder || holder instanceof DoubleChest; diff --git a/src/main/java/lol/hyper/toolstats/events/PickupItem.java b/src/main/java/lol/hyper/toolstats/events/PickupItem.java index 7813934..44f8df3 100755 --- a/src/main/java/lol/hyper/toolstats/events/PickupItem.java +++ b/src/main/java/lol/hyper/toolstats/events/PickupItem.java @@ -47,11 +47,8 @@ public class PickupItem implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onPickup(EntityPickupItemEvent event) { - if (event.isCancelled()) { - return; - } Entity entity = event.getEntity(); if (entity instanceof Player player) { if (!toolStats.configTools.checkWorld(player.getWorld().getName())) { diff --git a/src/main/java/lol/hyper/toolstats/events/PlayerDrop.java b/src/main/java/lol/hyper/toolstats/events/PlayerDrop.java index a83610c..dda8c97 100755 --- a/src/main/java/lol/hyper/toolstats/events/PlayerDrop.java +++ b/src/main/java/lol/hyper/toolstats/events/PlayerDrop.java @@ -34,7 +34,7 @@ public class PlayerDrop implements Listener { this.toolStats = toolStats; } - @EventHandler + @EventHandler(ignoreCancelled = true) public void onDrop(PlayerDropItemEvent event) { Player player = event.getPlayer(); if (!toolStats.configTools.checkWorld(player.getWorld().getName())) { diff --git a/src/main/java/lol/hyper/toolstats/events/PlayerFish.java b/src/main/java/lol/hyper/toolstats/events/PlayerFish.java index ea90ade..43bd513 100755 --- a/src/main/java/lol/hyper/toolstats/events/PlayerFish.java +++ b/src/main/java/lol/hyper/toolstats/events/PlayerFish.java @@ -46,11 +46,8 @@ public class PlayerFish implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.MONITOR) + @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) public void onFish(PlayerFishEvent event) { - if (event.isCancelled()) { - return; - } // only listen to when a player catches a fish if (event.getState() != PlayerFishEvent.State.CAUGHT_FISH) { return; diff --git a/src/main/java/lol/hyper/toolstats/events/PlayerInteract.java b/src/main/java/lol/hyper/toolstats/events/PlayerInteract.java index b5e5473..11ac781 100755 --- a/src/main/java/lol/hyper/toolstats/events/PlayerInteract.java +++ b/src/main/java/lol/hyper/toolstats/events/PlayerInteract.java @@ -54,7 +54,7 @@ public class PlayerInteract implements Listener { this.toolStats = toolStats; } - @EventHandler + @EventHandler(ignoreCancelled = true) public void onInteract(PlayerInteractEvent event) { if (event.getAction() != Action.RIGHT_CLICK_BLOCK) { return; @@ -106,7 +106,7 @@ public class PlayerInteract implements Listener { } } - @EventHandler + @EventHandler(ignoreCancelled = true) public void onInteract(PlayerInteractEntityEvent event) { Entity clicked = event.getRightClicked(); Player player = event.getPlayer(); diff --git a/src/main/java/lol/hyper/toolstats/events/PlayerMove.java b/src/main/java/lol/hyper/toolstats/events/PlayerMove.java index 0201f25..988aee7 100755 --- a/src/main/java/lol/hyper/toolstats/events/PlayerMove.java +++ b/src/main/java/lol/hyper/toolstats/events/PlayerMove.java @@ -39,7 +39,7 @@ public class PlayerMove implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.MONITOR) + @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) public void onMove(PlayerMoveEvent event) { Player player = event.getPlayer(); if (!toolStats.configTools.checkWorld(player.getWorld().getName())) { diff --git a/src/main/java/lol/hyper/toolstats/events/SheepShear.java b/src/main/java/lol/hyper/toolstats/events/SheepShear.java index 3e39ab8..8cf1c3a 100755 --- a/src/main/java/lol/hyper/toolstats/events/SheepShear.java +++ b/src/main/java/lol/hyper/toolstats/events/SheepShear.java @@ -39,7 +39,7 @@ public class SheepShear implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.MONITOR) + @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) public void onShear(PlayerInteractEntityEvent event) { if (event.isCancelled()) { return; diff --git a/src/main/java/lol/hyper/toolstats/events/ShootBow.java b/src/main/java/lol/hyper/toolstats/events/ShootBow.java index ad89f31..11a7221 100755 --- a/src/main/java/lol/hyper/toolstats/events/ShootBow.java +++ b/src/main/java/lol/hyper/toolstats/events/ShootBow.java @@ -38,7 +38,7 @@ public class ShootBow implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.MONITOR) + @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) public void onShoot(EntityShootBowEvent event) { Entity shooter = event.getEntity(); // only listen for players diff --git a/src/main/java/lol/hyper/toolstats/events/VillagerTrade.java b/src/main/java/lol/hyper/toolstats/events/VillagerTrade.java index 041ab3d..485f100 100755 --- a/src/main/java/lol/hyper/toolstats/events/VillagerTrade.java +++ b/src/main/java/lol/hyper/toolstats/events/VillagerTrade.java @@ -47,9 +47,9 @@ public class VillagerTrade implements Listener { this.toolStats = toolStats; } - @EventHandler(priority = EventPriority.HIGHEST) + @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true) public void onTrade(InventoryClickEvent event) { - if (event.isCancelled() || event.getCurrentItem() == null) { + if (event.getCurrentItem() == null) { return; } Inventory inventory = event.getClickedInventory();