refactor: simplify channel handling

Author: toxicity188

core/src/main/kotlin/kr/toxicity/model/manager/PlayerManagerImpl.kt

@@ -14,6 +14,7 @@ import kr.toxicity.model.util.handleFailure
 import kr.toxicity.model.util.registerListener
 import org.bukkit.entity.Player
 import org.bukkit.event.EventHandler
+import org.bukkit.event.EventPriority
 import org.bukkit.event.Listener
 import org.bukkit.event.player.PlayerJoinEvent
 import org.bukkit.event.player.PlayerQuitEvent
@@ -26,15 +27,15 @@ object PlayerManagerImpl : PlayerManager, GlobalManager {
 
     override fun start() {
         registerListener(object : Listener {
-            @EventHandler
+            @EventHandler(priority = EventPriority.HIGHEST)
             fun PlayerJoinEvent.join() {
                 if (player.isOnline) runCatching { //For fake player
                     player.register()
                 }.handleFailure {
                     "Unable to load ${player.name}'s data."
                 }
             }
-            @EventHandler
+            @EventHandler(priority = EventPriority.MONITOR)
             fun PlayerQuitEvent.quit() {
                 playerMap.remove(player.uniqueId)?.use {
                     SkinManagerImpl.removeCache(it.profile())

nms/v1_20_R4/src/main/kotlin/kr/toxicity/model/nms/v1_20_R4/NMSImpl.kt

@@ -133,10 +133,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim

nms/v1_21_R1/src/main/kotlin/kr/toxicity/model/nms/v1_21_R1/NMSImpl.kt

@@ -133,10 +133,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim

nms/v1_21_R2/src/main/kotlin/kr/toxicity/model/nms/v1_21_R2/NMSImpl.kt

@@ -133,10 +133,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim

nms/v1_21_R3/src/main/kotlin/kr/toxicity/model/nms/v1_21_R3/NMSImpl.kt

@@ -136,10 +136,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim

nms/v1_21_R4/src/main/kotlin/kr/toxicity/model/nms/v1_21_R4/NMSImpl.kt

@@ -135,10 +135,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim

nms/v1_21_R5/src/main/kotlin/kr/toxicity/model/nms/v1_21_R5/NMSImpl.kt

@@ -135,10 +135,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim

nms/v1_21_R6/src/main/kotlin/kr/toxicity/model/nms/v1_21_R6/NMSImpl.kt

@@ -136,10 +136,8 @@ class NMSImpl : NMS {
         private val slim = BetterModel.plugin().skinManager().isSlim(profile())
 
         init {
-            val pipeLine = getConnection(connection).channel.pipeline()
-            pipeLine.toMap().forEach {
-                if (it.value is Connection) pipeLine.addBefore(it.key, INJECT_NAME, this)
-            }
+            val pipeline = getConnection(connection).channel.pipeline()
+            pipeline.addBefore(pipeline.first { it.value is Connection }.key, INJECT_NAME, this)
         }
 
         override fun isSlim(): Boolean = slim