fixed profit per mineshaft corpse

2 files changed, 10 insertions, 11 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/CorpseTracker.kt b/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/CorpseTracker.kt
index 77aabc1bd..c54315d7e 100644
--- a/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/CorpseTracker.kt
+++ b/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/CorpseTracker.kt
@@ -34,7 +34,7 @@ object CorpseTracker {
         "Corpse Tracker",
         { BucketData() },
         { it.mining.mineshaft.corpseProfitTracker },
-        { drawDisplay(it) }
+        { drawDisplay(it) },
     )
 
     class BucketData : BucketedItemTrackerData<CorpseType>() {
diff --git a/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/MineshaftCorpseProfitPer.kt b/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/ProfitPerMineshaftCorpse.kt
index eed5f2b9d..7eebeeaab 100644
--- a/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/MineshaftCorpseProfitPer.kt
+++ b/src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/ProfitPerMineshaftCorpse.kt
@@ -5,6 +5,7 @@ import at.hannibal2.skyhanni.events.mining.CorpseLootedEvent
 import at.hannibal2.skyhanni.skyhannimodule.SkyHanniModule
 import at.hannibal2.skyhanni.utils.ChatUtils
 import at.hannibal2.skyhanni.utils.CollectionUtils.sortedDesc
+import at.hannibal2.skyhanni.utils.ItemPriceUtils.getPriceOrNull
 import at.hannibal2.skyhanni.utils.ItemUtils.itemName
 import at.hannibal2.skyhanni.utils.NEUInternalName
 import at.hannibal2.skyhanni.utils.NEUItems.getPrice
@@ -13,7 +14,7 @@ import at.hannibal2.skyhanni.utils.NumberUtil.shortFormat
 import net.minecraftforge.fml.common.eventhandler.SubscribeEvent
 
 @SkyHanniModule
-object MineshaftCorpseProfitPer {
+object ProfitPerMineshaftCorpse {
     private val config get() = SkyHanniMod.feature.mining.mineshaft
 
     @SubscribeEvent
@@ -25,14 +26,12 @@ object MineshaftCorpseProfitPer {
         val map = mutableMapOf<String, Double>()
         for ((name, amount) in loot) {
             if (name == "§bGlacite Powder") continue
-            NEUInternalName.fromItemNameOrNull(name)?.let {
-                val pricePer = it.getPrice()
-                if (pricePer == -1.0) continue
-                val profit = amount * pricePer
-                val text = "§eFound $name §8${amount.addSeparators()}x §7(§6${profit.shortFormat()}§7)"
-                map[text] = profit
-                totalProfit += profit
-            }
+            val internalName = NEUInternalName.fromItemNameOrNull(name) ?: continue
+            val pricePer = internalName.getPriceOrNull() ?: continue
+            val profit = amount * pricePer
+            val text = "§eFound $name §8${amount.addSeparators()}x §7(§6${profit.shortFormat()}§7)"
+            map[text] = profit
+            totalProfit += profit
         }
 
         val corpseType = event.corpseType
@@ -42,7 +41,7 @@ object MineshaftCorpseProfitPer {
             val keyName = it.itemName
             val price = it.getPrice()
 
-            map["$keyName: §c-${price.shortFormat()}"] = -price
+            map["§cCost: $keyName §7(§c-${price.shortFormat()}§7)"] = -price
             totalProfit -= price
         }