Fix #586

Author: spartacus04

src/main/java/me/spartacus04/jext/listeners/DiscUpdateEvent.kt

@@ -10,9 +10,11 @@ import me.spartacus04.jext.utils.isRecordFragment
 import org.bukkit.event.EventHandler
 import org.bukkit.event.entity.EntityPickupItemEvent
 import org.bukkit.event.inventory.InventoryOpenEvent
+import org.bukkit.event.inventory.InventoryType
 import org.bukkit.event.player.PlayerJoinEvent
 import org.bukkit.inventory.Inventory
 import org.bukkit.inventory.ItemStack
+import org.bukkit.block.Crafter
 
 internal class DiscUpdateEvent : JextListener() {
     @EventHandler(ignoreCancelled = true)
@@ -26,7 +28,22 @@ internal class DiscUpdateEvent : JextListener() {
         e.item.itemStack = updateItem(e.item.itemStack)
     }
 
+    @Suppress("UnstableApiUsage")
     private fun updateInventory(inv: Inventory) {
+        // If the inventory is a crafter, we need to keep track of the disabled slots
+        val crafterArr = if(inv.type == InventoryType.CRAFTER) {
+            val disabled = arrayListOf<Int>()
+            val holder = inv.holder as? Crafter ?: return
+
+            for(i in 1..inv.size) {
+                if(holder.isSlotDisabled(i)) {
+                    disabled.add(i)
+                }
+            }
+
+            disabled
+        } else null
+
         // We can't use inv.contents.forEachIndexed because it interferes with other inventory plugins
         val contents = inv.contents
         contents.forEachIndexed { i, it ->
@@ -35,6 +52,15 @@ internal class DiscUpdateEvent : JextListener() {
             }
         }
         inv.contents = contents
+
+        // Restore the disabled slots
+        if(crafterArr != null) {
+            val holder = inv.holder as? Crafter ?: return
+
+            for(i in crafterArr) {
+                holder.setSlotDisabled(i, true)
+            }
+        }
     }
 
     private fun updateItem(itemStack: ItemStack) : ItemStack {

src/main/java/me/spartacus04/jext/listeners/PrepareCraftingEvent.kt

@@ -13,30 +13,27 @@ import org.bukkit.inventory.CrafterInventory
 internal class PrepareCraftingEvent : JextListener("1.19") {
     @EventHandler
     fun prepareCraftingEvent(e: PrepareItemCraftEvent) {
-        if (e.inventory.result == null || e.inventory.result!!.type != JEXT_FRAGMENT_OUTPUT) return
         val block = e.inventory.holder as Block
-
         printDisabledSlots(block)
+
+        if (e.inventory.result == null || e.inventory.result!!.type != JEXT_FRAGMENT_OUTPUT) return
+
         val inventory = e.inventory.matrix.clone()
 
-        printDisabledSlots(block)
         val isCustomDisc = inventory.any {
             return@any Disc.isCustomDisc(it)
         }
 
-        printDisabledSlots(block)
         // check if every disc has same namespace, if they have the same namespace return the namespace else null
         val namespace = inventory.map {
             Disc.fromItemstack(it)?.namespace
         }.distinct().singleOrNull()
 
-        printDisabledSlots(block)
         if (isCustomDisc && namespace != null) {
             e.inventory.result = DISCS[namespace]!!.discItemStack
         } else if (isCustomDisc) {
             e.inventory.result = null
         }
-        printDisabledSlots(block)
     }
 
     fun printDisabledSlots(b: Block) {