Fix several clone methods

Author: NichtStudioCode

invui/src/main/java/xyz/xenondevs/invui/gui/AbstractGui.java

@@ -1186,6 +1186,9 @@ protected void applyModifiers(G gui) {
         public S clone() {
             try {
                 var clone = (AbstractBuilder<G, S>) super.clone();
+                clone.background = MutableProperty.of(background.get());
+                clone.frozen = MutableProperty.of(frozen.get());
+                clone.ignoreObscuredInventorySlots = MutableProperty.of(ignoreObscuredInventorySlots.get());
                 if (structure != null)
                     clone.structure = structure.clone();
                 if (modifiers != null)

invui/src/main/java/xyz/xenondevs/invui/gui/AbstractPagedGui.java

@@ -278,8 +278,10 @@ public PagedGui<C> build() {
         @Override
         public PagedGui.Builder<C> clone() {
             var clone = (AbstractBuilder<C>) super.clone();
-            clone.pageChangeHandlers = new ArrayList<>(pageChangeHandlers);
+            clone.content = MutableProperty.of(new ArrayList<>(content.get()));
+            clone.page = MutableProperty.of(page.get());
             clone.pageCountChangeHandlers = new ArrayList<>(pageCountChangeHandlers);
+            clone.pageChangeHandlers = new ArrayList<>(pageChangeHandlers);
             return clone;
         }
 

invui/src/main/java/xyz/xenondevs/invui/gui/AbstractScrollGui.java

@@ -341,6 +341,8 @@ public ScrollGui<C> build() {
         @Override
         public ScrollGui.Builder<C> clone() {
             var clone = (AbstractBuilder<C>) super.clone();
+            clone.content = MutableProperty.of(new ArrayList<>(content.get()));
+            clone.line = MutableProperty.of(line.get());
             clone.scrollHandlers = new ArrayList<>(scrollHandlers);
             clone.lineCountChangeHandlers = new ArrayList<>(lineCountChangeHandlers);
             return clone;

invui/src/main/java/xyz/xenondevs/invui/gui/TabGuiImpl.java

@@ -247,6 +247,8 @@ public TabGui build() {
         @Override
         public TabGui.Builder clone() {
             var clone = (Builder) super.clone();
+            clone.tabs = MutableProperty.of(new ArrayList<>(tabs.get()));
+            clone.tab = MutableProperty.of(tab.get());
             clone.tabChangeHandlers = new ArrayList<>(tabChangeHandlers);
             return clone;
         }

invui/src/main/java/xyz/xenondevs/invui/item/ItemBuilder.java

@@ -138,7 +138,7 @@ public ItemStack get() {
      */
     @Override
     public ItemStack get(Locale locale) {
-        return buildCache.computeIfAbsent(locale, this::build);
+        return buildCache.computeIfAbsent(locale, this::build).clone();
     }
 
     /**

invui/src/main/java/xyz/xenondevs/invui/window/AbstractWindow.java

@@ -892,6 +892,8 @@ public void open(Player viewer) {
         public S clone() {
             try {
                 var clone = (AbstractBuilder<W, S>) super.clone();
+                clone.closeable = MutableProperty.of(closeable.get());
+                clone.windowState = MutableProperty.of(windowState.get());
                 clone.openHandlers = new ArrayList<>(openHandlers);
                 clone.closeHandlers = new ArrayList<>(closeHandlers);
                 clone.outsideClickHandlers = new ArrayList<>(outsideClickHandlers);

invui/src/main/java/xyz/xenondevs/invui/window/AnvilWindowImpl.java

@@ -136,7 +136,7 @@ public static final class BuilderImpl
         implements AnvilWindow.Builder
     {
 
-        private final List<Consumer<? super String>> renameHandlers = new ArrayList<>();
+        private List<Consumer<? super String>> renameHandlers = new ArrayList<>();
         private Supplier<? extends Gui> upperGuiSupplier = () -> Gui.empty(3, 1);
         private MutableProperty<Boolean> textFieldAlwaysEnabled = MutableProperty.of(DEFAULT_TEXT_FIELD_ALWAYS_ENABLED);
         private MutableProperty<Boolean> resultAlwaysValid = MutableProperty.of(DEFAULT_RESULT_ALWAYS_VALID);
@@ -192,6 +192,14 @@ public AnvilWindow build(Player viewer) {
             return window;
         }
 
+        @Override
+        public AnvilWindow.Builder clone() {
+            var clone = (BuilderImpl) super.clone();
+            clone.textFieldAlwaysEnabled = MutableProperty.of(textFieldAlwaysEnabled.get());
+            clone.resultAlwaysValid = MutableProperty.of(resultAlwaysValid.get());
+            clone.renameHandlers = new ArrayList<>(renameHandlers);
+            return clone;
+        }
     }
 
 }

invui/src/main/java/xyz/xenondevs/invui/window/BrewingWindowImpl.java

@@ -153,6 +153,14 @@ public BrewingWindow build(Player viewer) {
             return window;
         }
 
+        @Override
+        public BrewingWindow.Builder clone() {
+            var clone = (BuilderImpl) super.clone();
+            clone.brewProgress = MutableProperty.of(brewProgress.get());
+            clone.fuelProgress = MutableProperty.of(fuelProgress.get());
+            return clone;
+        }
+        
     }
 
 }

invui/src/main/java/xyz/xenondevs/invui/window/CartographyWindowImpl.java

@@ -15,6 +15,7 @@
 
 import java.awt.image.BufferedImage;
 import java.util.Collections;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.function.Supplier;
@@ -189,6 +190,15 @@ public CartographyWindow build(Player viewer) {
             return window;
         }
 
+        @Override
+        public CartographyWindow.Builder clone() {
+            var clone = (BuilderImpl) super.clone();
+            clone.icons = MutableProperty.of(new HashSet<>(icons.get()));
+            clone.view = MutableProperty.of(view.get());
+            clone.canvas = canvas != null ? canvas.clone() : null;
+            return clone;
+        }
+        
     }
 
 }

invui/src/main/java/xyz/xenondevs/invui/window/CrafterWindowImpl.java

@@ -120,8 +120,8 @@ static final class BuilderImpl
 
         private Supplier<? extends Gui> craftingGuiSupplier = () -> Gui.empty(3, 3);
         private Supplier<? extends Gui> resultGuiSupplier = () -> Gui.empty(1, 1);
-        private final List<BiConsumer<? super Integer, ? super Boolean>> slotToggleHandlers = new ArrayList<>();
-        private final List<MutableProperty<Boolean>> slots = CollectionUtils.newList(9, i -> MutableProperty.of(DEFAULT_SLOT_DISABLED_STATE));
+        private List<BiConsumer<? super Integer, ? super Boolean>> slotToggleHandlers = new ArrayList<>();
+        private List<MutableProperty<Boolean>> slots = CollectionUtils.newList(9, i -> MutableProperty.of(DEFAULT_SLOT_DISABLED_STATE));
 
         @Override
         public CrafterWindow.Builder setCraftingGui(Supplier<? extends Gui> guiSupplier) {
@@ -186,6 +186,14 @@ public CrafterWindowImpl build(Player viewer) {
             return window;
         }
 
+        @Override
+        public CrafterWindow.Builder clone() {
+            var clone = (BuilderImpl) super.clone();
+            clone.slotToggleHandlers = new ArrayList<>(slotToggleHandlers);
+            clone.slots = CollectionUtils.newList(CRAFTING_SLOTS, i -> MutableProperty.of(slots.get(i).get()));
+            return clone;
+        }
+        
     }
 
 }