Exclude implicitly frozen inventories from Gui#getInventories

This resolves all issues where inventory interactions would ignore the Gui's frozen state

Author: NichtStudioCode

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

@@ -395,15 +395,26 @@ protected int putIntoFirstInventory(UpdateReason updateReason, ItemStack itemSta
      * @param ignored the inventories to ignore
      * @return a map of all inventories and their visible slots
      */
-    private SequencedMap<Inventory, Set<Integer>> getAllInventorySlots(Inventory... ignored) {
+    private SequencedMap<Inventory, Set<Integer>> getAllActiveInventorySlots(Inventory... ignored) {
+        if (isFrozen())
+            return Collections.emptySortedMap();
+        
         HashMap<Inventory, Set<Integer>> slots = new HashMap<>();
         Set<Inventory> ignoredSet = Arrays.stream(ignored).collect(Collectors.toSet());
 
+        slotLoop:
         for (SlotElement element : slotElements) {
             if (element == null)
                 continue;
 
-            element = element.getHoldingElement();
+            // follow gui links to holding element, break out if any intermediate gui is frozen
+            while (element instanceof SlotElement.GuiLink(Gui gui, int slot)) {
+                if (gui.isFrozen())
+                    continue slotLoop;
+                
+                element = gui.getSlotElement(slot);
+            }
+            
             if (element instanceof SlotElement.InventoryLink invElement) {
                 Inventory inventory = invElement.inventory();
                 if (ignoredSet.contains(inventory))
@@ -421,10 +432,10 @@ private SequencedMap<Inventory, Set<Integer>> getAllInventorySlots(Inventory...
     @Override
     public SequencedCollection<? extends Inventory> getInventories(Inventory... ignored) {
         if (!ignoreObscuredInventorySlots)
-            return Collections.unmodifiableSequencedCollection(getAllInventorySlots(ignored).sequencedKeySet());
+            return Collections.unmodifiableSequencedCollection(getAllActiveInventorySlots(ignored).sequencedKeySet());
 
         ArrayList<Inventory> inventories = new ArrayList<>();
-        for (Map.Entry<Inventory, Set<Integer>> entry : getAllInventorySlots(ignored).entrySet()) {
+        for (Map.Entry<Inventory, Set<Integer>> entry : getAllActiveInventorySlots(ignored).entrySet()) {
             Inventory inventory = entry.getKey();
             Set<Integer> slots = entry.getValue();
             inventories.add(new ObscuredInventory(inventory, slot -> !slots.contains(slot)));

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

@@ -626,6 +626,7 @@ default void setGui(char key, Gui gui) {
      * sorted by their {@link Inventory#getGuiPriority()}, with the highest priorities coming first.
      * If {@link #isIgnoreObscuredInventorySlots()} is true, the obscured slots will not be visible
      * in the returned inventories.
+     * If the gui or any nested gui containing an inventory is frozen, those slots will also not be visible.
      *
      * @param ignored the inventories to ignore
      * @return a sequenced collection of all inventories visible in this gui

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

@@ -277,7 +277,7 @@ public void handleDrag(IntSet slots, ClickType mode) {
     }
 
     /**
-     * Gets a list of all active (non-frozen, non-animating) inventory slots that are linked to by
+     * Gets a list of all non-frozen inventory slots that are linked to by
      * the specified window slots.
      *
      * @param slots The window slots
@@ -292,7 +292,7 @@ private List<InventorySlot> getActiveInventorySlots(IntSet slots) {
             if (path.isEmpty())
                 continue;
 
-            // skip frozen and animating slots
+            // skip frozen slots
             for (SlotElement element : path) {
                 if (element instanceof SlotElement.GuiLink guiLink) {
                     if (guiLink.gui().isFrozen())
@@ -333,8 +333,6 @@ private boolean distributeItems(UpdateReason updateReason, List<InventorySlot> s
         return changed;
     }
 
-    // TODO: respect frozen / animating state
-    // TODO: prioritize the content inventory where the cursor is
     public void handleCursorCollect(Click click) {
         Player player = click.player();