From 650b974484698beb0263618916b537eee08c58df Mon Sep 17 00:00:00 2001
From: dordsor21 <dordsor21@gmail.com>
Date: Fri, 15 Mar 2024 11:21:55 +0100
Subject: [PATCH] fix: correctly (de)serialise ItemType  - InternalId changes
 depending on ~~magic~~ so we should not store this value  - fixes #2589

---
 .../sk89q/worldedit/util/gson/GsonUtil.java   |  3 +++
 .../worldedit/util/gson/ItemTypeAdapter.java  | 26 +++++++++++++++++++
 .../sk89q/worldedit/world/item/ItemType.java  |  2 +-
 3 files changed, 30 insertions(+), 1 deletion(-)
 create mode 100644 worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/ItemTypeAdapter.java

diff --git a/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/GsonUtil.java b/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/GsonUtil.java
index 9b9a9a6e5a..d31750e11d 100644
--- a/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/GsonUtil.java
+++ b/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/GsonUtil.java
@@ -23,6 +23,8 @@
 import com.google.gson.GsonBuilder;
 import com.sk89q.worldedit.math.BlockVector3;
 import com.sk89q.worldedit.math.Vector3;
+import com.sk89q.worldedit.world.item.ItemType;
+import com.sk89q.worldedit.world.item.ItemTypes;
 
 /**
  * Utility methods for Google's GSON library.
@@ -41,6 +43,7 @@ public static GsonBuilder createBuilder() {
         GsonBuilder gsonBuilder = new GsonBuilder();
         gsonBuilder.registerTypeAdapter(Vector3.class, new VectorAdapter());
         gsonBuilder.registerTypeAdapter(BlockVector3.class, new BlockVectorAdapter());
+        gsonBuilder.registerTypeAdapter(ItemType.class, new ItemTypeAdapter());
         return gsonBuilder;
     }
 
diff --git a/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/ItemTypeAdapter.java b/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/ItemTypeAdapter.java
new file mode 100644
index 0000000000..5f8d6d9a20
--- /dev/null
+++ b/worldedit-core/src/main/java/com/sk89q/worldedit/util/gson/ItemTypeAdapter.java
@@ -0,0 +1,26 @@
+package com.sk89q.worldedit.util.gson;
+
+import com.google.gson.JsonDeserializationContext;
+import com.google.gson.JsonDeserializer;
+import com.google.gson.JsonElement;
+import com.google.gson.JsonObject;
+import com.google.gson.JsonParseException;
+import com.sk89q.worldedit.world.item.ItemType;
+import com.sk89q.worldedit.world.item.ItemTypes;
+
+import java.lang.reflect.Type;
+
+public final class ItemTypeAdapter implements JsonDeserializer<ItemType> {
+
+    @Override
+    public ItemType deserialize(JsonElement json, Type type, JsonDeserializationContext cont) throws JsonParseException {
+        JsonObject jsonObject = json.getAsJsonObject();
+        String id = jsonObject.get("id").getAsString();
+        ItemType itemType = ItemTypes.get(id);
+        if (itemType == null) {
+            throw new JsonParseException("Could not parse item type `" + id + "`");
+        }
+        return itemType;
+    }
+
+}
diff --git a/worldedit-core/src/main/java/com/sk89q/worldedit/world/item/ItemType.java b/worldedit-core/src/main/java/com/sk89q/worldedit/world/item/ItemType.java
index 5a1e7653ff..16a65c3086 100644
--- a/worldedit-core/src/main/java/com/sk89q/worldedit/world/item/ItemType.java
+++ b/worldedit-core/src/main/java/com/sk89q/worldedit/world/item/ItemType.java
@@ -79,7 +79,7 @@ public String getId() {
     }
 
     //FAWE start
-    private int internalId;
+    private transient int internalId;
 
     @Override
     public void setInternalId(int internalId) {