From 857bb339805f28080b207fb9a83aed61fcae94b6 Mon Sep 17 00:00:00 2001
From: IM23a-spirgif <spirgif@bzz.ch>
Date: Mon, 5 May 2025 18:02:25 +0200
Subject: [PATCH] Refactor message translations to use parameterized strings

---
 .../java/net/fretux/knockedback/CarryManager.java   | 10 +++++-----
 .../fretux/knockedback/PlayerExecutionHandler.java  |  4 ++--
 .../resources/assets/knockedback/lang/de_de.json    | 13 +++++++------
 3 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/src/main/java/net/fretux/knockedback/CarryManager.java b/src/main/java/net/fretux/knockedback/CarryManager.java
index 31c7acc..f4a9b42 100644
--- a/src/main/java/net/fretux/knockedback/CarryManager.java
+++ b/src/main/java/net/fretux/knockedback/CarryManager.java
@@ -25,7 +25,7 @@ public class CarryManager {
         }
         Optional<Player> existing = carrier.getPassengers().stream()
                 .filter(p -> p instanceof Player)
-                .map(p -> (Player)p)
+                .map(p -> (Player) p)
                 .findFirst();
         if (existing.isPresent()) {
             stopCarry(existing.get(), carrier);
@@ -54,8 +54,8 @@ public class CarryManager {
         if (knocked instanceof ServerPlayer spKnocked) {
             spKnocked.connection.send(mountPacket);
         }
-        carrier.sendSystemMessage(Component.translatable("message.knockedback.picked_up" + knocked.getName().getString()));
-        knocked.sendSystemMessage(Component.translatable("message.knockedback.being_carried" + carrier.getName().getString()));
+        carrier.sendSystemMessage(Component.translatable("message.knockedback.picked_up", knocked.getDisplayName()));
+        knocked.sendSystemMessage(Component.translatable("message.knockedback.being_carried", carrier.getDisplayName()));
     }
 
 
@@ -67,8 +67,8 @@ public class CarryManager {
         if (knocked instanceof ServerPlayer spKnocked) {
             spKnocked.connection.send(dismountPacket);
         }
-        carrier.sendSystemMessage(Component.translatable("message.knockedback.dropped" + knocked.getName().getString()));
-        knocked.sendSystemMessage(Component.translatable("message.knockedback.been_dropped"));
+        carrier.sendSystemMessage(Component.translatable("message.knockedback.dropped", knocked.getDisplayName()));
+        knocked.sendSystemMessage(Component.translatable("message.knockedback.been_dropped", carrier.getDisplayName()));
     }
 
     @SubscribeEvent
diff --git a/src/main/java/net/fretux/knockedback/PlayerExecutionHandler.java b/src/main/java/net/fretux/knockedback/PlayerExecutionHandler.java
index 81d0465..b4e351e 100644
--- a/src/main/java/net/fretux/knockedback/PlayerExecutionHandler.java
+++ b/src/main/java/net/fretux/knockedback/PlayerExecutionHandler.java
@@ -130,9 +130,9 @@ public class PlayerExecutionHandler {
                     new ExecutionProgressPacket(0)
             );
         }
-        executor.sendSystemMessage(Component.translatable("message.knockedback.you_executed" + knockedPlayer.getName().getString() + "!"));
+        executor.sendSystemMessage(Component.translatable("message.knockedback.you_executed", knockedPlayer.getDisplayName(), "!"));
         if (knockedPlayer instanceof ServerPlayer serverPlayer) {
-            serverPlayer.sendSystemMessage(Component.translatable("message.knockedback.you_were_executed" + executor.getName().getString() + "!"));
+            serverPlayer.sendSystemMessage(Component.translatable("message.knockedback.you_were_executed", executor.getDisplayName(), "!"));
         }
     }
 
diff --git a/src/main/resources/assets/knockedback/lang/de_de.json b/src/main/resources/assets/knockedback/lang/de_de.json
index 0de9974..ba8870c 100644
--- a/src/main/resources/assets/knockedback/lang/de_de.json
+++ b/src/main/resources/assets/knockedback/lang/de_de.json
@@ -5,17 +5,18 @@
 
   "message.knockedback.cannot_carry_while_knocked": "Du kannst niemanden tragen, während du betäubt bist!",
   "message.knockedback.cannot_execute_while_knocked": "Du kannst niemanden hinrichten, während du betäubt bist!",
-  "message.knockedback.picked_up": "%s wurde Aufgehoben",
-  "message.knockedback.being_carried": "Du wirst von %s getragen",
-  "message.knockedback.dropped": "%s wurde Abgesetzt",
-  "message.knockedback.been_dropped": "Du wurdest abgesetzt",
 
   "message.knockedback.execution_started": "Hinrichtung gestartet...",
   "message.knockedback.being_executed": "Du wirst hingerichtet!",
   "message.knockedback.execution_interrupted": "Hinrichtung abgebrochen!",
   "message.knockedback.execution_interrupted_executor": "Deine Hinrichtung wurde abgebrochen!",
-  "message.knockedback.you_executed": "Du hast %s hingerichtet!",
-  "message.knockedback.you_were_executed": "Du wurdest von %s hingerichtet!",
+  "message.knockedback.picked_up":        "%1$s wurde Aufgehoben",
+  "message.knockedback.being_carried":     "Du wirst von %1$s getragen",
+  "message.knockedback.dropped":           "%1$s wurde Abgesetzt",
+  "message.knockedback.been_dropped":      "Du wurdest von %1$s abgesetzt",
+  "message.knockedback.you_executed":      "Du hast %1$s hingerichtet!",
+  "message.knockedback.you_were_executed": "Du wurdest von %1$s hingerichtet!",
+
 
   "overlay.knockedback.recovery_time": "Zeit bis Erholung: %s s"
 }