From a0b81d3592883225988e058b7de87d33fa2dddf3 Mon Sep 17 00:00:00 2001 From: CalMWolfs <94038482+CalMWolfs@users.noreply.github.com> Date: Fri, 1 Mar 2024 21:51:05 +1100 Subject: Replaced e.printStackTrace() with ErrorManager. #1065 --- src/main/java/at/hannibal2/skyhanni/utils/ClipboardUtils.kt | 4 ++-- src/main/java/at/hannibal2/skyhanni/utils/InventoryUtils.kt | 2 +- src/main/java/at/hannibal2/skyhanni/utils/LorenzColor.kt | 2 +- src/main/java/at/hannibal2/skyhanni/utils/NEUItems.kt | 6 ++++-- src/main/java/at/hannibal2/skyhanni/utils/OSUtils.kt | 7 +++++-- src/main/java/at/hannibal2/skyhanni/utils/ParkourHelper.kt | 4 ++-- 6 files changed, 15 insertions(+), 10 deletions(-) (limited to 'src/main/java/at/hannibal2/skyhanni/utils') 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") } } -- cgit