fix: little bit of tidy up

Author: The-Code-Monkey

src/main/java/com/tcm/MineTale/block/workbenches/entity/AbstractWorkbenchEntity.java

@@ -31,7 +31,7 @@ public abstract class AbstractWorkbenchEntity extends BlockEntity implements Men
     protected double scanRadius = 5.0;
 
     // Slot Mapping: 0-1 Inputs, 2 Fuel, 3-6 Outputs
-    protected final SimpleContainer inventory = new SimpleContainer(7);
+    protected final SimpleContainer inventory = new SimpleContainer(Constants.TOTAL_SLOTS);
     protected int progress = 0;
     protected int maxProgress = 200;
 

src/main/java/com/tcm/MineTale/block/workbenches/entity/FurnaceWorkbenchEntity.java

@@ -228,7 +228,12 @@ private void pullFromNearbyChests() {
             for (int i = 0; i < chest.getContainerSize(); i++) {
                 ItemStack stack = chest.getItem(i);
                 if (isOre(stack) || isWood(stack)) {
-                    int inputSlot = inventory.getItem(Constants.INPUT_1).isEmpty() ? Constants.INPUT_1 : (inventory.getItem(Constants.INPUT_2).isEmpty() ? Constants.INPUT_2 : -1);
+                    int inputSlot = -1;
+                    if (inventory.getItem(Constants.INPUT_1).isEmpty()) {
+                        inputSlot = Constants.INPUT_1;
+                    } else if (inventory.getItem(Constants.INPUT_2).isEmpty()) {
+                        inputSlot = Constants.INPUT_2;
+                    }
                     if (inputSlot == -1) return;
                     inventory.setItem(inputSlot, stack.split(1));
                     chest.setChanged();

src/main/java/com/tcm/MineTale/block/workbenches/menu/AbstractWorkbenchContainerMenu.java

@@ -166,19 +166,24 @@ public ItemStack quickMoveStack(Player player, int index) {
             itemStack = itemStack2.copy();
 
             // From Furnace to Player
-            if (index < 7) {
-                if (!this.moveItemStackTo(itemStack2, 7, 43, true)) {
+            int containerSlots = Constants.TOTAL_SLOTS;
+            int playerStart = containerSlots;
+            int playerEnd = playerStart + 36;
+
+            // From Furnace to Player
+            if (index < containerSlots) {
+                if (!this.moveItemStackTo(itemStack2, playerStart, playerEnd, true)) {
                     return ItemStack.EMPTY;
                 }
             } 
             // From Player to Furnace
             else {
                 // If it's fuel, try fuel slot
                 if (isFuel(itemStack2)) {
-                    if (!this.moveItemStackTo(itemStack2, 0, 1, false)) return ItemStack.EMPTY;
+                    if (!this.moveItemStackTo(itemStack2, Constants.FUEL_SLOT, Constants.FUEL_SLOT + 1, false)) return ItemStack.EMPTY;
                 } 
                 // Otherwise, try inputs
-                else if (!this.moveItemStackTo(itemStack2, 1, 3, false)) {
+                else if (!this.moveItemStackTo(itemStack2, Constants.INPUT_1, Constants.INPUT_2 + 1, false)) {
                     return ItemStack.EMPTY;
                 }
             }

src/main/java/com/tcm/MineTale/block/workbenches/menu/CampfireWorkbenchMenu.java

@@ -1,6 +1,7 @@
 package com.tcm.MineTale.block.workbenches.menu;
 
 import com.tcm.MineTale.registry.ModMenuTypes;
+import com.tcm.MineTale.util.Constants;
 
 import net.minecraft.world.Container;
 import net.minecraft.world.SimpleContainer;
@@ -9,7 +10,7 @@
 import net.minecraft.world.inventory.SimpleContainerData;
 
 public class CampfireWorkbenchMenu extends AbstractWorkbenchContainerMenu {
-    private static final int containerSize = 7;
+    private static final int containerSize = Constants.TOTAL_SLOTS;
     private static final int containerDataSize = 4;
 
     /**