From 6a187104e5f88f61069c2c034fa8a1d2375e336e Mon Sep 17 00:00:00 2001
From: "Erymanthus[#5074] | (u/)RayDeeUx"
 <51521765+RayDeeUx@users.noreply.github.com>
Date: Thu, 28 Sep 2023 03:07:33 -0400
Subject: Bugfix: Revise isSack() to rely on internal name instead (#510)

Fixed Bazaar Item category "Ink Sack" gets wrongly detected as an item sack. #510
---
 src/main/java/at/hannibal2/skyhanni/utils/ItemUtils.kt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/main/java/at/hannibal2/skyhanni/utils')

diff --git a/src/main/java/at/hannibal2/skyhanni/utils/ItemUtils.kt b/src/main/java/at/hannibal2/skyhanni/utils/ItemUtils.kt
index 4fd35a330..ee753e17d 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/ItemUtils.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/ItemUtils.kt
@@ -21,8 +21,8 @@ object ItemUtils {
 
     fun ItemStack.cleanName() = this.displayName.removeColor()
 
-    fun isSack(name: String): Boolean =
-        name.endsWith(" Sack")//TODO use item id or api or something? or dont, its working fine now
+    fun isSack(stack: ItemStack): Boolean =
+        stack?.getInternalName()?.endsWith("_SACK") ?: false
 
     fun ItemStack.getLore(): List<String> {
         val tagCompound = this.tagCompound ?: return emptyList()
-- 
cgit