Fixed blast mining trying to drop non-items

mcMMO-Dev · Sep 14, 2024 · 9b046bd · 9b046bd
1 parent 88cacf8
commit 9b046bd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Changelog.txt b/Changelog.txt
@@ -1,6 +1,7 @@
 Version 2.2.020
     (Codebase) Reworked Roll implementation (See notes)
     (Codebase) Added unit test coverage for Roll
+    Fixed blast mining trying to drop non-items (thanks IAISI)
     Fixed error spam in mcMMO potion matching logic (see notes)
     Fixed a bug where Roll was modifying damage unnecessarily
 

diff --git a/src/main/java/com/gmail/nossr50/skills/mining/MiningManager.java b/src/main/java/com/gmail/nossr50/skills/mining/MiningManager.java
@@ -204,7 +204,7 @@ public void blastMiningDropProcessing(float yield, EntityExplodeEvent event) {
             if (isDropIllegal(blockState.getType()))
                 continue;
 
-            if (Probability.ofPercent(50).evaluate()) {
+            if (blockState.getType().isItem() && Probability.ofPercent(50).evaluate()) {
                 ItemUtils.spawnItem(getPlayer(),
                         Misc.getBlockCenter(blockState),
                         new ItemStack(blockState.getType()),