Replaced e.printStackTrace() with ErrorManager. #1065

6 files changed, 15 insertions, 10 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/utils/ClipboardUtils.kt b/src/main/java/at/hannibal2/skyhanni/utils/ClipboardUtils.kt
index b1d8eb68b..0c55b894c 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/ClipboardUtils.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/ClipboardUtils.kt
@@ -46,7 +46,7 @@ object ClipboardUtils {
                 getClipboard()?.setContents(StringSelection(text), null)
             } catch (e: Exception) {
                 if (step == 3) {
-                    ErrorManager.logError(e, "Error while trying to access the clipboard.")
+                    ErrorManager.logErrorWithData(e, "Error while trying to access the clipboard.")
                 } else {
                     copyToClipboard(text, step + 1)
                 }
@@ -65,7 +65,7 @@ object ClipboardUtils {
             }
         } catch (e: Exception) {
             return if (step == 3) {
-                ErrorManager.logError(e, "Error while trying to access the clipboard.")
+                ErrorManager.logErrorWithData(e, "Error while trying to access the clipboard.")
                 null
             } else {
                 readFromClipboard(step + 1)
diff --git a/src/main/java/at/hannibal2/skyhanni/utils/InventoryUtils.kt b/src/main/java/at/hannibal2/skyhanni/utils/InventoryUtils.kt
index 1da7170ac..4774e4e7a 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/InventoryUtils.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/InventoryUtils.kt
@@ -65,7 +65,7 @@ object InventoryUtils {
             val booleanField = storage.javaClass.getDeclaredField("enableStorageGUI3")
             booleanField.get(storage) as Boolean
         } catch (e: Throwable) {
-            ErrorManager.logError(e, "Could not read NEU config to determine if the neu storage is emabled.")
+            ErrorManager.logErrorWithData(e, "Could not read NEU config to determine if the neu storage is emabled.")
             false
         }
     }
diff --git a/src/main/java/at/hannibal2/skyhanni/utils/LorenzColor.kt b/src/main/java/at/hannibal2/skyhanni/utils/LorenzColor.kt
index 39c877db0..fdac1a4a8 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/LorenzColor.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/LorenzColor.kt
@@ -53,7 +53,7 @@ enum class LorenzColor(private val chatColorCode: Char, private val color: Color
             EnumDyeColor.PURPLE -> DARK_PURPLE
             EnumDyeColor.YELLOW -> YELLOW
             else -> {
-                ErrorManager.logError(
+                ErrorManager.logErrorWithData(
                     Exception("Unknown dye color: $this"),
                     "Unknown dye color: $this"
                 )
diff --git a/src/main/java/at/hannibal2/skyhanni/utils/NEUItems.kt b/src/main/java/at/hannibal2/skyhanni/utils/NEUItems.kt
index 1ca6e2a70..62f49f42f 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/NEUItems.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/NEUItems.kt
@@ -139,11 +139,13 @@ object NEUItems {
         getItemStackOrNull() ?: run {
             getPriceOrNull() ?: return@run fallbackItem
             if (ignoreItemsFilter.match(this.asString())) return@run fallbackItem
-            ErrorManager.logError(
+            ErrorManager.logErrorWithData(
                 IllegalStateException("Something went wrong!"),
                 "Encountered an error getting the item for §7$this§c. " +
                     "This may be because your NEU repo is outdated. Please ask in the SkyHanni " +
-                    "Discord if this is the case"
+                    "Discord if this is the case.",
+                "Item name" to this.asString(),
+                "repo commit" to manager.latestRepoCommit
             )
             fallbackItem
         }
diff --git a/src/main/java/at/hannibal2/skyhanni/utils/OSUtils.kt b/src/main/java/at/hannibal2/skyhanni/utils/OSUtils.kt
index 0cb4e043e..812bf085f 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/OSUtils.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/OSUtils.kt
@@ -13,11 +13,14 @@ object OSUtils {
             try {
                 Desktop.getDesktop().browse(URI(url))
             } catch (e: IOException) {
-                ErrorManager.logError(e, "Error opening website: $url")
+                ErrorManager.logErrorWithData(
+                    e, "Error while opening website.",
+                    "url" to url
+                )
             }
         } else {
             copyToClipboard(url)
-            ChatUtils.error("Web browser is not supported! Copied url to clipboard.")
+            ErrorManager.skyHanniError("Cannot open website, web browser is not supported! Copied url to clipboard.")
         }
     }
 
diff --git a/src/main/java/at/hannibal2/skyhanni/utils/ParkourHelper.kt b/src/main/java/at/hannibal2/skyhanni/utils/ParkourHelper.kt
index 14787a980..2e4f16ba6 100644
--- a/src/main/java/at/hannibal2/skyhanni/utils/ParkourHelper.kt
+++ b/src/main/java/at/hannibal2/skyhanni/utils/ParkourHelper.kt
@@ -43,7 +43,7 @@ class ParkourHelper(
 
     fun render(event: LorenzRenderWorldEvent) {
         if (locations.isEmpty()) {
-            ErrorManager.logError(
+            ErrorManager.logErrorWithData(
                 IllegalArgumentException("locations is empty"),
                 "Trying to render an empty parkour"
             )
@@ -125,7 +125,7 @@ class ParkourHelper(
                 }
             }
         } catch (e: Throwable) {
-            ErrorManager.logError(e, "Error while rendering a parkour")
+            ErrorManager.logErrorWithData(e, "Error while rendering a parkour")
         }
     }