make sure we check for canceled events

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")) {

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;

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) {

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;

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;

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())) {

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;
         }

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) {

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) {

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();

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;

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())) {

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())) {

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;

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();

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())) {

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;

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

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();