Breakup event listeners (#21)

* refactor: breakup event listeners

* misc: bump version

README.md

@@ -21,7 +21,7 @@ repositories {
 }
 
 dependencies {
-    implementation("com.github.DebitCardz:mc-chestui-plus:1.3.1")
+    implementation("com.github.DebitCardz:mc-chestui-plus:1.3.2")
 }
 ```
 ### Maven
@@ -34,7 +34,7 @@ dependencies {
 <dependency>
     <groupId>com.github.DebitCardz</groupId>
     <artifactId>mc-chestui-plus</artifactId>
-    <version>1.3.1</version>
+    <version>1.3.2</version>
 </dependency>
 
 ```

build.gradle.kts

@@ -8,7 +8,7 @@ val githubActor = project.findProperty("gpr.user") as String? ?: System.getenv("
 val githubToken = project.findProperty("gpr.key") as String? ?: System.getenv("GITHUB_TOKEN")
 
 group = "me.tech"
-version = "1.3.1"
+version = "1.3.2"
 
 repositories {
     mavenCentral()

src/main/kotlin/me/tech/mcchestui/GUI.kt

@@ -8,7 +8,9 @@
 package me.tech.mcchestui
 
 import me.tech.mcchestui.item.GUIItem
-import me.tech.mcchestui.listeners.GUIListener
+import me.tech.mcchestui.listeners.*
+import me.tech.mcchestui.listeners.item.*
+import me.tech.mcchestui.listeners.hotbar.*
 import me.tech.mcchestui.utils.GUICloseEvent
 import me.tech.mcchestui.utils.GUIDragItemEvent
 import me.tech.mcchestui.utils.GUIItemPickupEvent
@@ -138,7 +140,17 @@ class GUI(
 	 */
 	internal var templateSlots = mutableMapOf<Char, Slot>()
 
-	private val guiListener = GUIListener(this)
+	private val eventListeners = listOf(
+		GUISlotClickListener(this),
+
+		GUIItemPickupListener(this),
+		GUIItemPlaceListener(this),
+		GUIItemDragListener(this),
+
+		GUIHotbarListener(this),
+
+		GUICloseListener(this)
+	)
 
 	/**
 	 * Define weather the [GUI] has been unregistered.
@@ -147,8 +159,10 @@ class GUI(
 	internal var unregistered = false
 
 	init {
-		plugin.server.pluginManager
-			.registerEvents(guiListener, plugin)
+		eventListeners.forEach {
+			plugin.server.pluginManager
+				.registerEvents(it, plugin)
+		}
 	}
 
 	/**
@@ -308,7 +322,7 @@ class GUI(
 	 * Mark a [GUI] as unregistered and remove its [Listener].
 	 */
 	fun unregister() {
-		HandlerList.unregisterAll(guiListener)
+		eventListeners.forEach(HandlerList::unregisterAll)
 		unregistered = true
 	}
 }

src/main/kotlin/me/tech/mcchestui/listeners/GUICloseListener.kt

@@ -0,0 +1,28 @@
+package me.tech.mcchestui.listeners
+
+import me.tech.mcchestui.GUI
+import org.bukkit.event.EventHandler
+import org.bukkit.event.inventory.InventoryCloseEvent
+
+internal class GUICloseListener(gui: GUI) : GUIEventListener(gui) {
+    @EventHandler
+    internal fun InventoryCloseEvent.onClose() {
+        if(!gui.isBukkitInventory(inventory)) {
+            return
+        }
+
+        gui.onCloseInventory?.let { uiEvent ->
+            uiEvent(this, player)
+        }
+
+        if(gui.singleInstance) {
+            return
+        }
+
+        if(inventory.viewers.size > 1) {
+            return
+        }
+
+        gui.unregister()
+    }
+}

src/main/kotlin/me/tech/mcchestui/listeners/GUIEventListener.kt

@@ -0,0 +1,45 @@
+package me.tech.mcchestui.listeners
+
+import me.tech.mcchestui.GUI
+import org.bukkit.event.Listener
+import org.bukkit.event.inventory.InventoryAction
+
+internal abstract class GUIEventListener(
+    protected val gui: GUI
+) : Listener {
+    companion object {
+        /**
+         * All actions related to placing an item.
+         */
+        @JvmStatic
+        protected val PLACE_ACTIONS = listOf(
+            InventoryAction.PLACE_ONE,
+            InventoryAction.PLACE_SOME,
+            InventoryAction.PLACE_ALL,
+            InventoryAction.SWAP_WITH_CURSOR
+        )
+
+        /**
+         * All actions related to using hotkeys to place/pickup
+         * an item.
+         */
+        @JvmStatic
+        protected val HOTBAR_ACTIONS = listOf(
+            InventoryAction.HOTBAR_SWAP,
+            InventoryAction.HOTBAR_MOVE_AND_READD
+        )
+
+        /**
+         * All actions related to picking up an item.
+         */
+        @JvmStatic
+        protected val PICKUP_ACTIONS = listOf(
+            InventoryAction.PICKUP_ONE,
+            InventoryAction.PICKUP_SOME,
+            InventoryAction.PICKUP_HALF,
+            InventoryAction.PICKUP_ALL,
+            InventoryAction.SWAP_WITH_CURSOR,
+            InventoryAction.COLLECT_TO_CURSOR
+        )
+    }
+}