Fix part builder leftover dupe (#4835)

Happens with recipes that are an exact multiple of the input material size, and with recipes that cost more than the input material size for inputs with leftovers
SlimeKnights · May 1, 2022 · 454a2d0 · 454a2d0
1 parent 935278f
commit 454a2d0
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/main/java/slimeknights/tconstruct/library/recipe/partbuilder/IPartBuilderRecipe.java b/src/main/java/slimeknights/tconstruct/library/recipe/partbuilder/IPartBuilderRecipe.java
@@ -57,8 +57,11 @@ default ItemStack getLeftover(PartBuilderContainerWrapper inventoryWrapper) {
     if (recipe != null) {
       int value = recipe.getValue();
       if (value > 1) {
-        int remainder = value - getCost() % value;
-        if (remainder > 0) {
+        int remainder = (value - getCost()) % value;
+        if (remainder < 0) {
+          remainder += value;
+        }
+        if (remainder != 0) {
           ItemStack leftover = recipe.getLeftover();
           leftover.setCount(leftover.getCount() * remainder);
           return leftover;