diff --git a/src/main/java/com/songoda/epichoppers/hopper/levels/modules/ModuleAutoCrafting.java b/src/main/java/com/songoda/epichoppers/hopper/levels/modules/ModuleAutoCrafting.java index 950da13..67eb348 100644 --- a/src/main/java/com/songoda/epichoppers/hopper/levels/modules/ModuleAutoCrafting.java +++ b/src/main/java/com/songoda/epichoppers/hopper/levels/modules/ModuleAutoCrafting.java @@ -47,6 +47,8 @@ public class ModuleAutoCrafting extends Module { // jam check: is this hopper gummed up? if (crafterEjection) { + // TODO: Recode. Why specifically eject ingredients (allMaterials). + // And why not eject when we checked crafting is possible final List allMaterials = getRecipes(toCraft).getAllMaterials(); if (Stream.of(hopperCache.cachedInventory) .allMatch(item -> item != null && allMaterials.stream().anyMatch(mat -> mat == item.getType()))) { @@ -105,7 +107,7 @@ public class ModuleAutoCrafting extends Module { } } - // Not enough items for this recipe + // Not enough ingredients for this recipe if (amount != 0) continue; for (Map.Entry entry : slotsToAlter.entrySet()) { diff --git a/src/main/java/com/songoda/epichoppers/listeners/HopperListeners.java b/src/main/java/com/songoda/epichoppers/listeners/HopperListeners.java index 649f670..de4bdac 100644 --- a/src/main/java/com/songoda/epichoppers/listeners/HopperListeners.java +++ b/src/main/java/com/songoda/epichoppers/listeners/HopperListeners.java @@ -134,7 +134,7 @@ public class HopperListeners implements Listener { if (!allowItem) { event.setCancelled(true); if (moveInstead != null) { - // hopper code is a bit derpy - changing the item doesn't change what's removed + // hopper code is a bit derpy - changing the item doesn't change what's removed //event.setItem(moveInstead); // we need to instead cancel and manually remove the item to move source.removeItem(moveInstead);