From a60077f9e2625f62c92053791cdec279ac04d151 Mon Sep 17 00:00:00 2001
From: LittleCircleOO <51949761+LittleCircleOO@users.noreply.github.com>
Date: Sun, 8 Dec 2024 01:40:12 +0800
Subject: [PATCH] [Forge-1.20.1] p2p frequency tooltip fix (#8125)

This Pull Request contains two changes:
- Fix memory card tooltip not showing P2P frequency.
- Backport of the colorful frequency format. Show the P2P frequency in
tooltips using the colors used on the P2P tunnels back.
---
 .../igtooltip/parts/P2PStateDataProvider.java |  5 ++--
 .../appeng/items/tools/MemoryCardItem.java    |  9 ++++---
 .../java/appeng/util/helpers/P2PHelper.java   | 25 ++++++++++++++++++-
 3 files changed, 32 insertions(+), 7 deletions(-)

diff --git a/src/main/java/appeng/integration/modules/igtooltip/parts/P2PStateDataProvider.java b/src/main/java/appeng/integration/modules/igtooltip/parts/P2PStateDataProvider.java
index eab2018c8d3..dd61cea4e63 100644
--- a/src/main/java/appeng/integration/modules/igtooltip/parts/P2PStateDataProvider.java
+++ b/src/main/java/appeng/integration/modules/igtooltip/parts/P2PStateDataProvider.java
@@ -1,5 +1,6 @@
 package appeng.integration.modules.igtooltip.parts;
 
+import net.minecraft.ChatFormatting;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.nbt.Tag;
 import net.minecraft.network.chat.Component;
@@ -45,10 +46,10 @@ public void buildTooltip(P2PTunnelPart object, TooltipContext context, TooltipBu
             var freq = serverData.getShort(TAG_P2P_FREQUENCY);
 
             // Show the frequency and name of the frequency if it exists
-            var freqTooltip = Platform.p2p().toHexString(freq);
+            var freqTooltip = Platform.p2p().toColoredHexString(freq).withStyle(ChatFormatting.BOLD);
             if (serverData.contains(TAG_P2P_FREQUENCY_NAME, Tag.TAG_STRING)) {
                 var freqName = serverData.getString(TAG_P2P_FREQUENCY_NAME);
-                freqTooltip = freqName + " (" + freqTooltip + ")";
+                freqTooltip = Component.literal(freqName).append(" (").append(freqTooltip).append(")");
             }
 
             tooltip.addLine(InGameTooltip.P2PFrequency.text(freqTooltip));
diff --git a/src/main/java/appeng/items/tools/MemoryCardItem.java b/src/main/java/appeng/items/tools/MemoryCardItem.java
index 796ec8b2fd8..3c61115cf36 100644
--- a/src/main/java/appeng/items/tools/MemoryCardItem.java
+++ b/src/main/java/appeng/items/tools/MemoryCardItem.java
@@ -57,6 +57,7 @@
 import appeng.api.util.IConfigurableObject;
 import appeng.core.AELog;
 import appeng.core.localization.GuiText;
+import appeng.core.localization.InGameTooltip;
 import appeng.core.localization.PlayerMessages;
 import appeng.core.localization.Tooltips;
 import appeng.helpers.IConfigInvHost;
@@ -288,11 +289,11 @@ public void appendHoverText(ItemStack stack, Level level, List<Component> lines,
             lines.add(Tooltips.of(Component.translatable(tooltipKey)));
         }
 
-        if (data.contains("freq")) {
-            final short freq = data.getShort("freq");
-            final String freqTooltip = ChatFormatting.BOLD + Platform.p2p().toHexString(freq);
+        if (data.contains("p2pFreq")) {
+            final short freq = data.getShort("p2pFreq");
+            var freqTooltip = Platform.p2p().toColoredHexString(freq).withStyle(ChatFormatting.BOLD);
 
-            lines.add(Tooltips.of(Component.translatable("gui.tooltips.ae2.P2PFrequency", freqTooltip)));
+            lines.add(Tooltips.of(Component.translatable(InGameTooltip.P2PFrequency.getTranslationKey(), freqTooltip)));
         }
     }
 
diff --git a/src/main/java/appeng/util/helpers/P2PHelper.java b/src/main/java/appeng/util/helpers/P2PHelper.java
index d88377f4d9b..50d82da4c0b 100644
--- a/src/main/java/appeng/util/helpers/P2PHelper.java
+++ b/src/main/java/appeng/util/helpers/P2PHelper.java
@@ -20,6 +20,9 @@
 
 import com.google.common.base.Preconditions;
 
+import net.minecraft.network.chat.Component;
+import net.minecraft.network.chat.MutableComponent;
+
 import appeng.api.util.AEColor;
 
 public class P2PHelper {
@@ -28,7 +31,7 @@ public AEColor[] toColors(short frequency) {
         final AEColor[] colors = new AEColor[4];
 
         for (int i = 0; i < 4; i++) {
-            int nibble = frequency >> 4 * (3 - i) & 0xF;
+            int nibble = getFrequencyNibble(frequency, i);
 
             colors[i] = AEColor.values()[nibble];
         }
@@ -36,6 +39,10 @@ public AEColor[] toColors(short frequency) {
         return colors;
     }
 
+    private static int getFrequencyNibble(short frequency, int i) {
+        return frequency >> 4 * (3 - i) & 0xF;
+    }
+
     public short fromColors(AEColor[] colors) {
         Preconditions.checkArgument(colors.length == 4);
 
@@ -58,4 +65,20 @@ public String toHexString(short frequency) {
         return String.format("%04X", frequency);
     }
 
+    private static final String[] HEX_DIGITS = {
+            "0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "A", "B", "C", "D", "E", "F"
+    };
+
+    public MutableComponent toColoredHexString(short frequency) {
+        var parent = Component.empty();
+
+        for (var i = 0; i < 4; i++) {
+            var nibble = getFrequencyNibble(frequency, i);
+            var hex = Component.literal(HEX_DIGITS[nibble]);
+            parent.append(hex.setStyle(hex.getStyle().withColor(AEColor.values()[nibble].mediumVariant)));
+        }
+
+        return parent;
+    }
+
 }