better main hand container transfer

Author: beanbag44

src/main/kotlin/com/lambda/interaction/material/container/containers/MainHandContainer.kt

@@ -64,10 +64,8 @@ object MainHandContainer : MaterialContainer(Rank.MainHand) {
                     return@inventoryRequest
                 }
 
-                val slots = player.currentScreenHandler.slots.filter { it.stack == moveStack }
-
-                if (slots.isNotEmpty()) swap(player.currentScreenHandler.slots.indexOf(slots.first()), player.inventory.selectedSlot)
-                else throw NotInInventoryException()
+                val slot = player.currentScreenHandler.slots.first { it.stack == moveStack } ?: throw NotInInventoryException()
+                swap(slot.id, player.inventory.selectedSlot)
 
                 if (hand == Hand.OFF_HAND) swapHands()
 

src/main/kotlin/com/lambda/interaction/material/transfer/SlotTransfer.kt

@@ -71,7 +71,6 @@ class SlotTransfer @Ta5kBuilder constructor(
             val nextTo = selectedTo.firstOrNull() ?: return@listen
 
             inventoryRequest {
-//                moveSlot(nextFrom.id, nextTo.id)
                 swap(nextTo.id, 1)
                 swap(nextFrom.id, 1)
                 onComplete { success() }