Fixed strange white color for shop info sign Ghost-chu#1537

Author: Ghost-chu

src/main/java/org/maxgamer/quickshop/shop/ContainerShop.java

@@ -24,9 +24,11 @@
 import io.papermc.lib.PaperLib;
 import lombok.EqualsAndHashCode;
 import me.lucko.helper.serialize.BlockPosition;
+import net.md_5.bungee.api.chat.BaseComponent;
 import net.md_5.bungee.api.chat.ComponentBuilder;
 import net.md_5.bungee.api.chat.TextComponent;
 import net.md_5.bungee.api.chat.TranslatableComponent;
+import net.md_5.bungee.chat.ComponentSerializer;
 import org.bukkit.DyeColor;
 import org.bukkit.Location;
 import org.bukkit.Material;
@@ -740,31 +742,33 @@ public List<ComponentPackage> getSignText(@NotNull String locale) {
         }
 
         // TODO No-longer use SHOP_SIGN_PREFIX since we use modern storage method. Pending for deletion.
-        lines.add(new ComponentPackage(TextComponent.fromLegacyText(SHOP_SIGN_PREFIX + line2 + " ")));
+        lines.add(new ComponentPackage(new ComponentBuilder().color(net.md_5.bungee.api.ChatColor.BLACK).appendLegacy(line2).create()));
 
         //line 3
         if (plugin.getConfiguration().getBoolean("shop.force-use-item-original-name") || !this.getItem().hasItemMeta() || !this.getItem().getItemMeta().hasDisplayName()) {
-            TextComponent left = new TextComponent(plugin.text().of("signs.item-left").forLocale());
-            TextComponent right = new TextComponent(plugin.text().of("signs.item-right").forLocale());
+            BaseComponent[] left = TextComponent.fromLegacyText(plugin.text().of("signs.item-left").forLocale());
+            BaseComponent[] right = TextComponent.fromLegacyText(plugin.text().of("signs.item-right").forLocale());
             if (plugin.getNbtapi() == null) {
                 // NBTAPI not installed
                 lines.add(new ComponentPackage(new ComponentBuilder()
+                        .color(net.md_5.bungee.api.ChatColor.BLACK)
                         .append(left)
-                        .append(TextComponent.fromLegacyText(Util.getItemStackName(getItem())))
+                        .appendLegacy(Util.getItemStackName(getItem()))
                         .append(right)
                         .create()));
             } else {
                 // NBTAPI installed
                 lines.add(new ComponentPackage(new ComponentBuilder()
+                        .color(net.md_5.bungee.api.ChatColor.BLACK)
                         .append(left)
                         .append(new TranslatableComponent(ReflectFactory.getMaterialMinecraftNamespacedKey(getItem().getType())))
                         .append(right)
                         .create()));
             }
         } else {
-            lines.add(new ComponentPackage(new ComponentBuilder().append(TextComponent.fromLegacyText(plugin.text().of("signs.item-left").forLocale()))
-                    .append(new TextComponent(Util.getItemStackName(getItem())).toLegacyText())
-                    .append(TextComponent.fromLegacyText(plugin.text().of("signs.item-right").forLocale())).create()));
+            lines.add(new ComponentPackage(new ComponentBuilder().color(net.md_5.bungee.api.ChatColor.BLACK).appendLegacy(plugin.text().of("signs.item-left").forLocale())
+                    .append(Util.getItemStackName(getItem()))
+                    .appendLegacy(plugin.text().of("signs.item-right").forLocale()).create()));
         }
 
         //line 4
@@ -776,7 +780,10 @@ public List<ComponentPackage> getSignText(@NotNull String locale) {
         } else {
             line4 = plugin.text().of("signs.price", plugin.getShopManager().format(this.getPrice(), this)).forLocale();
         }
-        lines.add(new ComponentPackage(TextComponent.fromLegacyText(line4)));
+        lines.add(new ComponentPackage(new ComponentBuilder().color(net.md_5.bungee.api.ChatColor.BLACK).appendLegacy(line4).create()));
+
+        lines.forEach(pack -> Util.debugLog(ComponentSerializer.toString(pack.getComponents())));
+
         return lines;
     }