Fix AbstractWindow#handleCloseEvent not being called on disable

Author: NichtStudioCode

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

@@ -187,7 +187,7 @@ public void handleCloseEvent(Player player) {
                 throw new IllegalStateException("Window is already closed!");
 
             currentlyOpen = false;
-            remove(false);
+            remove();
 
             // handleClosed() might have already been called by open() if the window was replaced by another one
             if (!hasHandledClose) {
@@ -250,7 +250,7 @@ protected Map<Integer, SlotElement> getInvSlotElements(Inventory inventory) {
             && ((SlotElement.InventorySlotElement) element).getInventory() == inventory);
     }
 
-    public void remove(boolean closeForViewer) {
+    private void remove() {
         WindowManager.getInstance().removeWindow(this);
 
         Arrays.stream(elementsDisplayed)
@@ -266,8 +266,6 @@ public void remove(boolean closeForViewer) {
 
         Arrays.stream(getGuis())
             .forEach(gui -> gui.removeParent(this));
-        
-        if (closeForViewer) close();
     }
 
     @Override

invui-core/src/main/java/xyz/xenondevs/invui/window/WindowManager.java

@@ -26,12 +26,12 @@ public class WindowManager implements Listener {
 
     private static WindowManager instance;
 
-    private final Map<Inventory, AbstractWindow> windows = new HashMap<>();
-    private final Map<Player, AbstractWindow> openWindows = new HashMap<>();
+    private final Map<Inventory, AbstractWindow> windowsByInventory = new HashMap<>();
+    private final Map<Player, AbstractWindow> windowsByPlayer = new HashMap<>();
 
     private WindowManager() {
         Bukkit.getPluginManager().registerEvents(this, InvUI.getInstance().getPlugin());
-        InvUI.getInstance().addDisableHandler(() -> new HashSet<>(windows.values()).forEach(window -> window.remove(true)));
+        InvUI.getInstance().addDisableHandler(() -> new HashSet<>(windowsByPlayer.values()).forEach(AbstractWindow::close));
     }
 
     /**
@@ -50,7 +50,7 @@ public static WindowManager getInstance() {
      * @param window The {@link AbstractWindow} to add
      */
     public void addWindow(AbstractWindow window) {
-        windows.put(window.getInventories()[0], window);
+        windowsByInventory.put(window.getInventories()[0], window);
     }
 
     /**
@@ -60,7 +60,7 @@ public void addWindow(AbstractWindow window) {
      * @param window The {@link AbstractWindow} to remove
      */
     public void removeWindow(AbstractWindow window) {
-        windows.remove(window.getInventories()[0]);
+        windowsByInventory.remove(window.getInventories()[0]);
     }
 
     /**
@@ -71,7 +71,7 @@ public void removeWindow(AbstractWindow window) {
      */
     @Nullable
     public Window getWindow(Inventory inventory) {
-        return windows.get(inventory);
+        return windowsByInventory.get(inventory);
     }
 
     /**
@@ -82,25 +82,25 @@ public Window getWindow(Inventory inventory) {
      */
     @Nullable
     public Window getOpenWindow(Player player) {
-        return openWindows.get(player);
+        return windowsByPlayer.get(player);
     }
 
     /**
-     * Gets a set of all registered {@link Window Windows}.
+     * Gets a set of all open {@link Window Windows}.
      *
      * @return A set of all {@link Window Windows}
      */
     public Set<Window> getWindows() {
-        return new HashSet<>(windows.values());
+        return new HashSet<>(windowsByInventory.values());
     }
 
     /**
-     * Gets a set of all currently opened {@link Window Windows}.
-     *
-     * @return A set of all opened {@link Window Windows}
+     * Gets a set of all open {@link Window Windows}.
+     * @deprecated Use {@link #getWindows()} instead
      */
+    @Deprecated
     public Set<Window> getOpenWindows() {
-        return new HashSet<>(openWindows.values());
+        return getWindows();
     }
 
     @EventHandler
@@ -127,15 +127,15 @@ private void handleInventoryClose(InventoryCloseEvent event) {
             window.handleCloseEvent(player);
         }
 
-        openWindows.remove(player);
+        windowsByPlayer.remove(player);
     }
 
     @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
     private void handleInventoryOpen(InventoryOpenEvent event) {
         AbstractWindow window = (AbstractWindow) getWindow(event.getInventory());
         if (window != null) {
             window.handleOpenEvent(event);
-            openWindows.put((Player) event.getPlayer(), window);
+            windowsByPlayer.put((Player) event.getPlayer(), window);
         }
     }
 
@@ -145,7 +145,7 @@ private void handlePlayerQuit(PlayerQuitEvent event) {
         AbstractWindow window = (AbstractWindow) getOpenWindow(player);
         if (window != null) {
             window.handleCloseEvent(player);
-            openWindows.remove(player);
+            windowsByPlayer.remove(player);
         }
     }