From af8c26a4a579596da669189fffc69e8dfc665aa0 Mon Sep 17 00:00:00 2001
From: Camotoy <20743703+Camotoy@users.noreply.github.com>
Date: Sun, 28 Nov 2021 17:18:37 -0500
Subject: [PATCH] Remove unnecessary version check for minecart furnace icon
 information

All supported protocols are now 1.17.30 or later.
---
 .../geyser/registry/populator/ItemRegistryPopulator.java      | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/core/src/main/java/org/geysermc/geyser/registry/populator/ItemRegistryPopulator.java b/core/src/main/java/org/geysermc/geyser/registry/populator/ItemRegistryPopulator.java
index b0df6d051..340c6a45c 100644
--- a/core/src/main/java/org/geysermc/geyser/registry/populator/ItemRegistryPopulator.java
+++ b/core/src/main/java/org/geysermc/geyser/registry/populator/ItemRegistryPopulator.java
@@ -465,9 +465,7 @@ public class ItemRegistryPopulator {
 
                 NbtMapBuilder componentBuilder = NbtMap.builder();
                 // Conveniently, as of 1.16.200, the furnace minecart has a texture AND translation string already.
-                // 1.17.30 moves the icon to the item properties section
-                (palette.getValue().protocolVersion() >= Bedrock_v465.V465_CODEC.getProtocolVersion() ?
-                        itemProperties : componentBuilder).putCompound("minecraft:icon", NbtMap.builder()
+                itemProperties.putCompound("minecraft:icon", NbtMap.builder()
                         .putString("texture", "minecart_furnace")
                         .putString("frame", "0.000000")
                         .putInt("frame_version", 1)