fix jei item transfer size limit

GlodBlock · GlodBlock · commit 167c0c6a381f · 2023-07-29T10:12:04.000+08:00
diff --git a/build.gradle b/build.gradle
@@ -51,7 +51,7 @@ dependencies {
     deobfCompile 'curse.maven:modular-machinery-270790:2761302' // 1.11.1
     deobfCompile 'curse.maven:codechicken-lib-1-8-242818:2779848' //ccl
     deobfCompile 'curse.maven:gregtech-ce-unofficial-557242:3784798' // gt
-    deobfCompile 'curse.maven:ae2-extended-life-570458:4429430' //pae2
+    deobfCompile 'curse.maven:ae2-extended-life-570458:4646658' //pae2
     deobfCompile 'curse.maven:dynamistics-383632:3056455' // dy
     compileOnly 'curse.maven:opencomputers-223008:4630537' //oc
     runtime 'curse.maven:opencomputers-223008:4630537' //oc
diff --git a/src/main/java/com/glodblock/github/client/container/ContainerExtendedFluidPatternTerminal.java b/src/main/java/com/glodblock/github/client/container/ContainerExtendedFluidPatternTerminal.java
@@ -427,7 +427,7 @@ else if (!slot.getStack().isEmpty()) {
     }
 
     @Override
-    public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {
+    public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {
         if (part instanceof PartExtendedFluidPatternTerminal) {
             ((PartExtendedFluidPatternTerminal) part).onChangeCrafting(inputs, outputs, combine);
         }
diff --git a/src/main/java/com/glodblock/github/client/container/ContainerFluidPatternEncoder.java b/src/main/java/com/glodblock/github/client/container/ContainerFluidPatternEncoder.java
@@ -156,7 +156,7 @@ public void doAction(EntityPlayerMP player, InventoryAction action, int slotId,
     }
 
     @Override
-    public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {
+    public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {
         AeStackInventory<IAEItemStack> craftingSlot = tile.getCraftingSlots();
         AeStackInventory<IAEItemStack> outputSlot = tile.getOutputSlots();
         for (int index : inputs.keySet()) {
@@ -165,9 +165,9 @@ public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs
                 craftingSlot.setStack(index, AEItemStack.fromItemStack(items[0]));
             }
         }
-        int bound = Math.min(outputSlot.getSlotCount(), outputs.length);
+        int bound = Math.min(outputSlot.getSlotCount(), outputs.size());
         for (int index = 0; index < bound; index ++) {
-            outputSlot.setStack(index, AEItemStack.fromItemStack(outputs[index]));
+            outputSlot.setStack(index, AEItemStack.fromItemStack(outputs.get(index)));
         }
     }
 
diff --git a/src/main/java/com/glodblock/github/client/container/ContainerFluidPatternTerminal.java b/src/main/java/com/glodblock/github/client/container/ContainerFluidPatternTerminal.java
@@ -203,7 +203,7 @@ private static IAEItemStack[] collectInventory(Slot[] slots) {
     }
 
     @Override
-    public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {
+    public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {
         if (Ae2Reflect.getPart(this) instanceof PartFluidPatternTerminal) {
             ((PartFluidPatternTerminal) Ae2Reflect.getPart(this)).onChangeCrafting(inputs, outputs, combine);
         }
diff --git a/src/main/java/com/glodblock/github/common/part/PartExtendedFluidPatternTerminal.java b/src/main/java/com/glodblock/github/common/part/PartExtendedFluidPatternTerminal.java
@@ -32,6 +32,7 @@
 import net.minecraftforge.items.IItemHandlerModifiable;
 
 import javax.annotation.Nonnull;
+import java.util.List;
 
 public class PartExtendedFluidPatternTerminal extends PartExpandedProcessingPatternTerminal {
 
@@ -125,7 +126,7 @@ public void onChangeInventory(IItemHandler inv, int slot, InvOperation mc, ItemS
         super.onChangeInventory(inv, slot, mc, removedStack, newStack);
     }
 
-    public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {
+    public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {
         IItemHandler crafting = this.getInventoryByName("crafting");
         IItemHandler output = this.getInventoryByName("output");
         IItemList<IAEItemStack> storageList = this.getInventory(Util.ITEM) == null ?
@@ -145,9 +146,9 @@ public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outp
                 final ItemStack item = fuzzyFind[x];
                 ((AppEngInternalInventory) crafting).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);
             }
-            bound = Math.min(output.getSlots(), outputs.length);
+            bound = Math.min(output.getSlots(), outputs.size());
             for (int x = 0; x < bound; x++) {
-                final ItemStack item = outputs[x];
+                final ItemStack item = outputs.get(x);
                 ((AppEngInternalInventory) output).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);
             }
         }
diff --git a/src/main/java/com/glodblock/github/common/part/PartFluidPatternTerminal.java b/src/main/java/com/glodblock/github/common/part/PartFluidPatternTerminal.java
@@ -35,6 +35,7 @@
 import net.minecraftforge.items.IItemHandlerModifiable;
 
 import javax.annotation.Nonnull;
+import java.util.List;
 
 public class PartFluidPatternTerminal extends PartPatternTerminal {
 
@@ -168,7 +169,7 @@ public void putPattern(IAEItemStack[] inputs, IAEItemStack[] outputs) {
         }
     }
 
-    public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {
+    public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {
         IItemHandler crafting = this.getInventoryByName("crafting");
         IItemHandler output = this.getInventoryByName("output");
         IItemList<IAEItemStack> storageList = this.getInventory(Util.ITEM) == null ?
@@ -188,9 +189,9 @@ public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outp
                 final ItemStack item = fuzzyFind[x];
                 ((AppEngInternalInventory) crafting).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);
             }
-            bound = Math.min(output.getSlots(), outputs.length);
+            bound = Math.min(output.getSlots(), outputs.size());
             for (int x = 0; x < bound; x++) {
-                final ItemStack item = outputs[x];
+                final ItemStack item = outputs.get(x);
                 ((AppEngInternalInventory) output).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);
             }
         }
diff --git a/src/main/java/com/glodblock/github/integration/jei/ExtendedFluidPatternTerminalRecipeTransferHandler.java b/src/main/java/com/glodblock/github/integration/jei/ExtendedFluidPatternTerminalRecipeTransferHandler.java
@@ -33,10 +33,7 @@ public IRecipeTransferError transferRecipe(@Nonnull ContainerExtendedFluidPatter
        }
 
        if (doTransfer && container.part instanceof PartExtendedFluidPatternTerminal) {
-           PartExtendedFluidPatternTerminal tile = (PartExtendedFluidPatternTerminal)container.part;
            RecipeTransferBuilder transfer = new RecipeTransferBuilder(
-                   tile.getInventoryByName("crafting").getSlots(),
-                   tile.getInventoryByName("output").getSlots(),
                    recipeLayout)
                    .clearEmptySlot(true)
                    .putFluidFirst(container.fluidFirst)
diff --git a/src/main/java/com/glodblock/github/integration/jei/FluidPatternEncoderRecipeTransferHandler.java b/src/main/java/com/glodblock/github/integration/jei/FluidPatternEncoderRecipeTransferHandler.java
@@ -31,10 +31,7 @@ public IRecipeTransferError transferRecipe(@Nonnull ContainerFluidPatternEncoder
             return new RecipeTransferErrorTooltip(I18n.format(NameConst.TT_PROCESSING_RECIPE_ONLY));
         }
         if (doTransfer) {
-            TileFluidPatternEncoder tile = container.getTile();
             RecipeTransferBuilder transfer = new RecipeTransferBuilder(
-                    tile.getCraftingSlots().getSlotCount(),
-                    tile.getOutputSlots().getSlotCount(),
                     recipeLayout)
                     .clearEmptySlot(true)
                     .putFluidFirst(false)
diff --git a/src/main/java/com/glodblock/github/integration/jei/FluidPatternTerminalRecipeTransferHandler.java b/src/main/java/com/glodblock/github/integration/jei/FluidPatternTerminalRecipeTransferHandler.java
@@ -42,10 +42,7 @@ else if (!container.isCraftingMode() && recipeLayout.getRecipeCategory().getUid(
                 }
             } catch (IOException ignore) {
             }
-            PartFluidPatternTerminal tile = (PartFluidPatternTerminal) Ae2Reflect.getPart(container);
             RecipeTransferBuilder transfer = new RecipeTransferBuilder(
-                    tile.getInventoryByName("crafting").getSlots(),
-                    tile.getInventoryByName("output").getSlots(),
                     recipeLayout)
                     .clearEmptySlot(!craftMode)
                     .putFluidFirst(container.fluidFirst)
diff --git a/src/main/java/com/glodblock/github/integration/jei/RecipeTransferBuilder.java b/src/main/java/com/glodblock/github/integration/jei/RecipeTransferBuilder.java
@@ -19,8 +19,7 @@ public class RecipeTransferBuilder {
     private static final int MAX_ITEMS = 16;
 
     private final Int2ObjectArrayMap<ItemStack[]> in;
-    private final int bound;
-    private final ItemStack[] out;
+    private final List<ItemStack> out;
     private final IRecipeLayout recipe;
     private List<ItemStack[]> itemsIn;
     private List<FluidStack> fluidIn;
@@ -29,10 +28,9 @@ public class RecipeTransferBuilder {
     private boolean noNull = true;
     private boolean fluidFirst = false;
 
-    public RecipeTransferBuilder(int maxInput, int maxOutput, IRecipeLayout recipe) {
+    public RecipeTransferBuilder(IRecipeLayout recipe) {
         this.in = new Int2ObjectArrayMap<>();
-        this.bound = maxInput;
-        this.out = new ItemStack[maxOutput];
+        this.out = new ArrayList<>();
         this.recipe = recipe;
         this.itemsIn = new ArrayList<>();
         this.itemOut = new ArrayList<>();
@@ -88,7 +86,7 @@ private void split() {
     }
 
     private void setItemIn(int offset) {
-        int bound = Math.min(this.bound, this.itemsIn.size() + offset);
+        int bound = this.itemsIn.size() + offset;
         for (int index = offset; index < bound; index ++) {
             int i = index - offset;
             if (this.itemsIn.get(i) != null && this.itemsIn.get(i).length > 0) {
@@ -98,7 +96,7 @@ private void setItemIn(int offset) {
     }
 
     private void setFluidIn(int offset) {
-        int bound = Math.min(this.bound, this.fluidIn.size() + offset);
+        int bound = this.fluidIn.size() + offset;
         for (int index = offset; index < bound; index ++) {
             int i = index - offset;
             if (this.fluidIn.get(i) != null) {
@@ -108,11 +106,11 @@ private void setFluidIn(int offset) {
     }
 
     private void setOutputs() {
-        for (int index = 0; index < this.out.length; index ++) {
+        for (int index = 0; index < this.out.size(); index ++) {
             if (index < this.itemOut.size()) {
-                this.out[index] = this.itemOut.get(index);
+                this.out.add(this.itemOut.get(index));
             } else if (index - this.itemOut.size() < this.fluidOut.size()) {
-                this.out[index] = ItemFluidPacket.newStack(this.fluidOut.get(index - this.itemOut.size()));
+                this.out.add(ItemFluidPacket.newStack(this.fluidOut.get(index - this.itemOut.size())));
             }
         }
     }
@@ -145,7 +143,7 @@ public RecipeTransferBuilder build() {
         return this;
     }
 
-    public ItemStack[] getOutput() {
+    public List<ItemStack> getOutput() {
         return this.out;
     }
 
diff --git a/src/main/java/com/glodblock/github/interfaces/PatternConsumer.java b/src/main/java/com/glodblock/github/interfaces/PatternConsumer.java
@@ -3,8 +3,10 @@
 import it.unimi.dsi.fastutil.ints.Int2ObjectMap;
 import net.minecraft.item.ItemStack;
 
+import java.util.List;
+
 public interface PatternConsumer {
 
-    void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean compress);
+    void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean compress);
 
 }
diff --git a/src/main/java/com/glodblock/github/network/CPacketLoadPattern.java b/src/main/java/com/glodblock/github/network/CPacketLoadPattern.java
@@ -11,17 +11,19 @@
 import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
 import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
 import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;
+import scala.actors.threadpool.Arrays;
 
 import javax.annotation.Nullable;
+import java.util.List;
 
 public class CPacketLoadPattern implements IMessage {
 
-    private ItemStack[] output;
+    private List<ItemStack> output;
     private Int2ObjectMap<ItemStack[]> crafting;
     private boolean compress;
     private static final int SLOT_SIZE = 16;
 
-    public CPacketLoadPattern(Int2ObjectMap<ItemStack[]> crafting, ItemStack[] output, boolean compress) {
+    public CPacketLoadPattern(Int2ObjectMap<ItemStack[]> crafting, List<ItemStack> output, boolean compress) {
         this.crafting = crafting;
         this.output = output;
         this.compress = compress;
@@ -38,11 +40,12 @@ public void toBytes(ByteBuf buf) {
         for (int index : crafting.keySet()) {
             writeItemArray(msg, crafting.get(index), index + "#");
         }
-        writeItemArray(msg, output, "o");
+        writeItemArray(msg, output.toArray(new ItemStack[0]), "o");
         Util.writeNBTToBytes(buf, msg);
     }
 
     @Override
+    @SuppressWarnings("unchecked")
     public void fromBytes(ByteBuf buf) {
         crafting = new Int2ObjectArrayMap<>();
         compress = buf.readBoolean();
@@ -52,7 +55,7 @@ public void fromBytes(ByteBuf buf) {
                 crafting.put(i, readItemArray(msg, i + "#"));
             }
         }
-        output = readItemArray(msg, "o");
+        output = Arrays.asList(readItemArray(msg, "o"));
     }
 
     private void writeItemArray(NBTTagCompound nbt, ItemStack[] itemList, String key) {

Original file line number	Diff line number	Diff line change
`@@ -427,7 +427,7 @@ else if (!slot.getStack().isEmpty()) {`
`427`	`427`	`}`
`428`	`428`
`429`	`429`	`@Override`
`430`		`- public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {`
	`430`	`+ public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {`
`431`	`431`	`if (part instanceof PartExtendedFluidPatternTerminal) {`
`432`	`432`	`((PartExtendedFluidPatternTerminal) part).onChangeCrafting(inputs, outputs, combine);`
`433`	`433`	`}`
Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ public void doAction(EntityPlayerMP player, InventoryAction action, int slotId,`
`156`	`156`	`}`
`157`	`157`
`158`	`158`	`@Override`
`159`		`- public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {`
	`159`	`+ public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {`
`160`	`160`	`AeStackInventory<IAEItemStack> craftingSlot = tile.getCraftingSlots();`
`161`	`161`	`AeStackInventory<IAEItemStack> outputSlot = tile.getOutputSlots();`
`162`	`162`	`for (int index : inputs.keySet()) {`
`@@ -165,9 +165,9 @@ public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs`
`165`	`165`	`craftingSlot.setStack(index, AEItemStack.fromItemStack(items[0]));`
`166`	`166`	`}`
`167`	`167`	`}`
`168`		`- int bound = Math.min(outputSlot.getSlotCount(), outputs.length);`
	`168`	`+ int bound = Math.min(outputSlot.getSlotCount(), outputs.size());`
`169`	`169`	`for (int index = 0; index < bound; index ++) {`
`170`		`- outputSlot.setStack(index, AEItemStack.fromItemStack(outputs[index]));`
	`170`	`+ outputSlot.setStack(index, AEItemStack.fromItemStack(outputs.get(index)));`
`171`	`171`	`}`
`172`	`172`	`}`
`173`	`173`
Original file line number	Diff line number	Diff line change
`@@ -203,7 +203,7 @@ private static IAEItemStack[] collectInventory(Slot[] slots) {`
`203`	`203`	`}`
`204`	`204`
`205`	`205`	`@Override`
`206`		`- public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {`
	`206`	`+ public void acceptPattern(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {`
`207`	`207`	`if (Ae2Reflect.getPart(this) instanceof PartFluidPatternTerminal) {`
`208`	`208`	`((PartFluidPatternTerminal) Ae2Reflect.getPart(this)).onChangeCrafting(inputs, outputs, combine);`
`209`	`209`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,7 @@`
`32`	`32`	`import net.minecraftforge.items.IItemHandlerModifiable;`
`33`	`33`
`34`	`34`	`import javax.annotation.Nonnull;`
	`35`	`+import java.util.List;`
`35`	`36`
`36`	`37`	`public class PartExtendedFluidPatternTerminal extends PartExpandedProcessingPatternTerminal {`
`37`	`38`
`@@ -125,7 +126,7 @@ public void onChangeInventory(IItemHandler inv, int slot, InvOperation mc, ItemS`
`125`	`126`	`super.onChangeInventory(inv, slot, mc, removedStack, newStack);`
`126`	`127`	`}`
`127`	`128`
`128`		`- public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {`
	`129`	`+ public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {`
`129`	`130`	`IItemHandler crafting = this.getInventoryByName("crafting");`
`130`	`131`	`IItemHandler output = this.getInventoryByName("output");`
`131`	`132`	`IItemList<IAEItemStack> storageList = this.getInventory(Util.ITEM) == null ?`
`@@ -145,9 +146,9 @@ public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outp`
`145`	`146`	`final ItemStack item = fuzzyFind[x];`
`146`	`147`	`((AppEngInternalInventory) crafting).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);`
`147`	`148`	`}`
`148`		`- bound = Math.min(output.getSlots(), outputs.length);`
	`149`	`+ bound = Math.min(output.getSlots(), outputs.size());`
`149`	`150`	`for (int x = 0; x < bound; x++) {`
`150`		`- final ItemStack item = outputs[x];`
	`151`	`+ final ItemStack item = outputs.get(x);`
`151`	`152`	`((AppEngInternalInventory) output).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);`
`152`	`153`	`}`
`153`	`154`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@`
`35`	`35`	`import net.minecraftforge.items.IItemHandlerModifiable;`
`36`	`36`
`37`	`37`	`import javax.annotation.Nonnull;`
	`38`	`+import java.util.List;`
`38`	`39`
`39`	`40`	`public class PartFluidPatternTerminal extends PartPatternTerminal {`
`40`	`41`
`@@ -168,7 +169,7 @@ public void putPattern(IAEItemStack[] inputs, IAEItemStack[] outputs) {`
`168`	`169`	`}`
`169`	`170`	`}`
`170`	`171`
`171`		`- public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outputs, boolean combine) {`
	`172`	`+ public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, List<ItemStack> outputs, boolean combine) {`
`172`	`173`	`IItemHandler crafting = this.getInventoryByName("crafting");`
`173`	`174`	`IItemHandler output = this.getInventoryByName("output");`
`174`	`175`	`IItemList<IAEItemStack> storageList = this.getInventory(Util.ITEM) == null ?`
`@@ -188,9 +189,9 @@ public void onChangeCrafting(Int2ObjectMap<ItemStack[]> inputs, ItemStack[] outp`
`188`	`189`	`final ItemStack item = fuzzyFind[x];`
`189`	`190`	`((AppEngInternalInventory) crafting).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);`
`190`	`191`	`}`
`191`		`- bound = Math.min(output.getSlots(), outputs.length);`
	`192`	`+ bound = Math.min(output.getSlots(), outputs.size());`
`192`	`193`	`for (int x = 0; x < bound; x++) {`
`193`		`- final ItemStack item = outputs[x];`
	`194`	`+ final ItemStack item = outputs.get(x);`
`194`	`195`	`((AppEngInternalInventory) output).setStackInSlot(x, item == null ? ItemStack.EMPTY : item);`
`195`	`196`	`}`
`196`	`197`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,10 +42,7 @@ else if (!container.isCraftingMode() && recipeLayout.getRecipeCategory().getUid(`
`42`	`42`	`}`
`43`	`43`	`} catch (IOException ignore) {`
`44`	`44`	`}`
`45`		`- PartFluidPatternTerminal tile = (PartFluidPatternTerminal) Ae2Reflect.getPart(container);`
`46`	`45`	`RecipeTransferBuilder transfer = new RecipeTransferBuilder(`
`47`		`- tile.getInventoryByName("crafting").getSlots(),`
`48`		`- tile.getInventoryByName("output").getSlots(),`
`49`	`46`	`recipeLayout)`
`50`	`47`	`.clearEmptySlot(!craftMode)`
`51`	`48`	`.putFluidFirst(container.fluidFirst)`