add more error logging

1 files changed, 16 insertions, 2 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/features/garden/visitor/GardenVisitorSupercraft.kt b/src/main/java/at/hannibal2/skyhanni/features/garden/visitor/GardenVisitorSupercraft.kt
index e2343af02..ba5706f1d 100644
--- a/src/main/java/at/hannibal2/skyhanni/features/garden/visitor/GardenVisitorSupercraft.kt
+++ b/src/main/java/at/hannibal2/skyhanni/features/garden/visitor/GardenVisitorSupercraft.kt
@@ -4,6 +4,7 @@ import at.hannibal2.skyhanni.data.SackAPI.getAmountInSacks
 import at.hannibal2.skyhanni.events.GuiContainerEvent
 import at.hannibal2.skyhanni.events.InventoryCloseEvent
 import at.hannibal2.skyhanni.events.garden.visitor.VisitorOpenEvent
+import at.hannibal2.skyhanni.test.command.ErrorManager
 import at.hannibal2.skyhanni.utils.ChatUtils
 import at.hannibal2.skyhanni.utils.ItemUtils.itemName
 import at.hannibal2.skyhanni.utils.NEUInternalName
@@ -50,11 +51,24 @@ class GardenVisitorSupercraft {
         val visitor = event.visitor
         visitor.offer?.offerItem ?: return
         for ((internalName, amount) in visitor.shoppingList) {
-            if (isSupercraftEnabled) getSupercraftForSacks(internalName, amount)
+            if (isSupercraftEnabled) {
+                try {
+                    getSupercraftForSacks(internalName, amount)
+                } catch (e: NoSuchElementException) {
+                    ErrorManager.logErrorWithData(
+                        e,
+                        "Failed to calculate supercraft recipes for visitor",
+                        "internalName" to internalName,
+                        "amount" to amount,
+                        "visitor" to visitor.visitorName,
+                        "visitor.offer?.offerItem" to visitor.offer?.offerItem,
+                    )
+                }
+            }
         }
     }
 
-    fun getSupercraftForSacks(internalName: NEUInternalName, amount: Int) {
+    private fun getSupercraftForSacks(internalName: NEUInternalName, amount: Int) {
         val ingredients = NEUItems.getRecipes(internalName)
             // TODO describe what this line does
             .first { !it.ingredients.first().internalItemId.contains("PEST") }