From 42e6b1a03f978dbae7239ffb40d3af37ac1f21e8 Mon Sep 17 00:00:00 2001
From: JFronny <git@akbvopenfl1.hopto.org>
Date: Mon, 23 Sep 2024 14:55:35 +0200
Subject: [PATCH] chore: clean up logging

---
 src/main/java/io/gitlab/jfronny/googlechat/GoogleChat.java  | 6 +++---
 .../jfronny/googlechat/server/TranslatableContainer.java    | 4 +---
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/main/java/io/gitlab/jfronny/googlechat/GoogleChat.java b/src/main/java/io/gitlab/jfronny/googlechat/GoogleChat.java
index cd40b4d..645c559 100644
--- a/src/main/java/io/gitlab/jfronny/googlechat/GoogleChat.java
+++ b/src/main/java/io/gitlab/jfronny/googlechat/GoogleChat.java
@@ -72,7 +72,7 @@ public class GoogleChat implements ModInitializer {
                     translated.append(translateIfNeeded(sibling, direction, false));
                 }
             }
-            if (GoogleChatConfig.Advanced.debugLogs) LOGGER.info("Translated " + sourceString + " to " + toString(translated));
+            if (GoogleChatConfig.Advanced.debugLogs) LOGGER.info("Translated {0} to {1}", sourceString, toString(translated));
             if (GoogleChatConfig.General.translationTooltip) {
                 return t.copy().styled(style -> addHover(style, Text.literal("Translated: ").append(translated)));
             } else if (translated.getStyle().getHoverEvent() == null) {
@@ -113,7 +113,7 @@ public class GoogleChat implements ModInitializer {
             } else if (t instanceof PlainTextContent.Literal tx) {
                 return new PlainTextContent.Literal(translateIfNeeded(tx.string(), direction, false));
             } else {
-//                LOGGER.info("Unhandled text type: " + source.getClass() + " (" + source + ")");
+//                LOGGER.info("Unhandled text type: {0} ({1}})", source.getClass().toString(), source.toString());
                 return t;
             }
         });
@@ -149,7 +149,7 @@ public class GoogleChat implements ModInitializer {
                 //noinspection unchecked
                 return m.group(1) + svc.translate(m.group(2), sourceLang, targetLang) + m.group(3);
             } catch (Throwable e) {
-                LOGGER.error("Could not translate text: " + source, e);
+                LOGGER.error("Could not translate text: {0}", e, source);
                 return source;
             }
         });
diff --git a/src/main/java/io/gitlab/jfronny/googlechat/server/TranslatableContainer.java b/src/main/java/io/gitlab/jfronny/googlechat/server/TranslatableContainer.java
index ddb26c3..de27c51 100644
--- a/src/main/java/io/gitlab/jfronny/googlechat/server/TranslatableContainer.java
+++ b/src/main/java/io/gitlab/jfronny/googlechat/server/TranslatableContainer.java
@@ -5,8 +5,6 @@ import net.minecraft.text.Text;
 
 import java.util.concurrent.CompletableFuture;
 
-import static io.gitlab.jfronny.libjf.LibJf.LOGGER;
-
 public sealed interface TranslatableContainer<T, S extends TranslatableContainer<T, S>> {
     S translate(TranslationDirection direction);
 
@@ -26,7 +24,7 @@ public sealed interface TranslatableContainer<T, S extends TranslatableContainer
 
     static Text translateAndLog(final Text source, final TranslationDirection direction) {
         var translated = GoogleChat.translateIfNeeded(source, direction, true);
-        if (GoogleChatConfig.Advanced.debugLogs) LOGGER.info("Applied C2S translation from {0} to {1}", source, translated);
+        if (GoogleChatConfig.Advanced.debugLogs) GoogleChat.LOGGER.info("Applied C2S translation from {0} to {1}", source, translated);
         return translated;
     }
 }