From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Jake Potrebic <jake.m.potrebic@gmail.com>
Date: Fri, 26 Apr 2024 21:33:20 -0700
Subject: [PATCH] Don't lose removed data components in ItemMeta


diff --git a/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaItem.java b/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaItem.java
index a1faf7906f8734149538d52173fcb3401200e594..dd17d7d88ad43e0c426a7a508709fc8993d3a5c9 100644
--- a/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaItem.java
+++ b/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaItem.java
@@ -174,6 +174,13 @@ class CraftMetaItem implements ItemMeta, Damageable, Repairable, BlockDataMeta {
             return this;
         }
 
+        // Paper start - support removing component types
+        <T> Applicator remove(DataComponentType<T> type) {
+            this.builder.remove(type);
+            return this;
+        }
+        // Paper end - support removing component types
+
         DataComponentPatch build() {
             return this.builder.build();
         }
@@ -384,7 +391,9 @@ class CraftMetaItem implements ItemMeta, Damageable, Repairable, BlockDataMeta {
 
         Set<Map.Entry<DataComponentType<?>, Optional<?>>> keys = tag.entrySet();
         for (Map.Entry<DataComponentType<?>, Optional<?>> key : keys) {
-            if (!CraftMetaItem.getHandledTags().contains(key.getKey())) {
+            if (key.getValue().isEmpty()) {
+                this.unhandledTags.remove(key.getKey());
+            } else if (!CraftMetaItem.getHandledTags().contains(key.getKey())) {
                 key.getValue().ifPresentOrElse((value) -> {
                     this.unhandledTags.set((DataComponentType) key.getKey(), value);
                 }, () -> {
@@ -780,9 +789,9 @@ class CraftMetaItem implements ItemMeta, Damageable, Repairable, BlockDataMeta {
         }
 
         for (Map.Entry<DataComponentType<?>, Optional<?>> e : this.unhandledTags.build().entrySet()) {
-            e.getValue().ifPresent((value) -> {
+            e.getValue().ifPresentOrElse((value) -> {
                 itemTag.builder.set((DataComponentType) e.getKey(), value);
-            });
+            }, () -> itemTag.remove(e.getKey()));
         }
 
         CompoundTag customTag = (this.customTag != null) ? this.customTag.copy() : null;