Merge pull request #2069

wrincewind · web-flow · commit 352684bc3ffe · 2025-07-20T14:15:03.000+01:00
Fix ARC not respecting unbreakable items
diff --git a/src/main/java/wayoftime/bloodmagic/common/tile/TileAlchemicalReactionChamber.java b/src/main/java/wayoftime/bloodmagic/common/tile/TileAlchemicalReactionChamber.java
@@ -381,10 +381,10 @@ public void consumeInventory(int inputCount, boolean consumeInput, boolean break
 		ItemStack toolStack = getItem(ARC_TOOL_SLOT);
 		if (!toolStack.isEmpty())
 		{
-			if (toolStack.isDamageableItem())
+			if (toolStack.getItem().isDamageable(toolStack))
 			{
 				int unbreakingLevel = EnchantmentHelper.getItemEnchantmentLevel(Enchantments.UNBREAKING, toolStack);
-				if (unbreakingLevel == 0 || level.random.nextInt(unbreakingLevel + 1) == 0)
+				if (toolStack.isDamageableItem() && (unbreakingLevel == 0 || level.random.nextInt(unbreakingLevel + 1) == 0))
 				{
 					toolStack.setDamageValue(toolStack.getDamageValue() + 1);
 					if (toolStack.getDamageValue() >= toolStack.getMaxDamage() && breakTool)

Original file line number	Diff line number	Diff line change
`@@ -381,10 +381,10 @@ public void consumeInventory(int inputCount, boolean consumeInput, boolean break`
`381`	`381`	`ItemStack toolStack = getItem(ARC_TOOL_SLOT);`
`382`	`382`	`if (!toolStack.isEmpty())`
`383`	`383`	`{`
`384`		`- if (toolStack.isDamageableItem())`
	`384`	`+ if (toolStack.getItem().isDamageable(toolStack))`
`385`	`385`	`{`
`386`	`386`	`int unbreakingLevel = EnchantmentHelper.getItemEnchantmentLevel(Enchantments.UNBREAKING, toolStack);`
`387`		`- if (unbreakingLevel == 0 \|\| level.random.nextInt(unbreakingLevel + 1) == 0)`
	`387`	`+ if (toolStack.isDamageableItem() && (unbreakingLevel == 0 \|\| level.random.nextInt(unbreakingLevel + 1) == 0))`
`388`	`388`	`{`
`389`	`389`	`toolStack.setDamageValue(toolStack.getDamageValue() + 1);`
`390`	`390`	`if (toolStack.getDamageValue() >= toolStack.getMaxDamage() && breakTool)`