Fix read/write in NEISetCraftingRecipe

Author: bziemons

common/logisticspipes/gui/popup/GuiRecipeImport.java

@@ -179,7 +179,8 @@ protected void renderGuiBackground(int mouseX, int mouseY) {
 	protected void actionPerformed(GuiButton button) {
 		int id = button.id;
 		if (id == 0) {
-			NonNullList<ItemStack> stackList = NonNullList.withSize(9, ItemStack.EMPTY);
+			NEISetCraftingRecipe packet = PacketHandler.getPacket(NEISetCraftingRecipe.class);
+			NonNullList<ItemStack> stackList = packet.getStackList();
 			int i = 0;
 			for (Canidates canidate : grid) {
 				if (canidate == null) {
@@ -188,8 +189,7 @@ protected void actionPerformed(GuiButton button) {
 				}
 				stackList.set(i++, canidate.order.get(canidate.pos).makeNormalStack());
 			}
-			NEISetCraftingRecipe packet = PacketHandler.getPacket(NEISetCraftingRecipe.class);
-			MainProxy.sendPacketToServer(packet.setStackList(stackList).setBlockPos(tile.getPos()));
+			MainProxy.sendPacketToServer(packet.setBlockPos(tile.getPos()));
 			exitGui();
 		} else if (id == 1) {
 			MainProxy.sendPacketToServer(PacketHandler.getPacket(FindMostLikelyRecipeComponents.class).setContent(list).setTilePos(tile));

common/logisticspipes/network/packets/NEISetCraftingRecipe.java

@@ -1,14 +1,11 @@
 package logisticspipes.network.packets;
 
 import net.minecraft.entity.player.EntityPlayer;
-import net.minecraft.item.Item;
 import net.minecraft.item.ItemStack;
+import net.minecraft.nbt.NBTTagCompound;
 import net.minecraft.tileentity.TileEntity;
 import net.minecraft.util.NonNullList;
 
-import lombok.Getter;
-import lombok.Setter;
-
 import logisticspipes.blocks.crafting.LogisticsCraftingTableTileEntity;
 import logisticspipes.network.abstractpackets.CoordinatesPacket;
 import logisticspipes.network.abstractpackets.ModernPacket;
@@ -21,14 +18,16 @@
 @StaticResolve
 public class NEISetCraftingRecipe extends CoordinatesPacket {
 
-	@Getter
-	@Setter
 	private NonNullList<ItemStack> stackList = NonNullList.withSize(9, ItemStack.EMPTY);
 
 	public NEISetCraftingRecipe(int id) {
 		super(id);
 	}
 
+	public NonNullList<ItemStack> getStackList() {
+		return this.stackList;
+	}
+
 	@Override
 	public void processPacket(EntityPlayer player) {
 		TileEntity tile = getTileAs(player.world, TileEntity.class);
@@ -47,37 +46,16 @@ public ModernPacket template() {
 	@Override
 	public void writeData(LPDataOutput output) {
 		super.writeData(output);
-
-		output.writeInt(stackList.size());
-
-		for (int i = 0; i < stackList.size(); i++) {
-			final ItemStack stack = stackList.get(i);
-
-			if (!stack.isEmpty()) {
-				output.writeByte(i);
-				output.writeInt(Item.getIdFromItem(stack.getItem()));
-				output.writeInt(stack.getCount());
-				output.writeInt(stack.getItemDamage());
-				output.writeNBTTagCompound(stack.getTagCompound());
-			}
-		}
-		output.writeByte(-1); // mark packet end
+		output.writeCollection(stackList, (out, stack) -> out.writeNBTTagCompound(stack.isEmpty() ? null : stack.writeToNBT(new NBTTagCompound())));
 	}
 
 	@Override
 	public void readData(LPDataInput input) {
 		super.readData(input);
-
-		byte index = input.readByte();
-
-		while (index != -1) { // read until the end
-			final int itemID = input.readInt();
-			int stackSize = input.readInt();
-			int damage = input.readInt();
-			ItemStack stack = new ItemStack(Item.getItemById(itemID), stackSize, damage);
-			stack.setTagCompound(input.readNBTTagCompound());
-			stackList.set(index, stack);
-			index = input.readByte(); // read the next slot
-		}
+		NonNullList<ItemStack> readList = input.readNonNullList(inp -> {
+			NBTTagCompound tag = inp.readNBTTagCompound();
+			return tag == null ? null : new ItemStack(tag);
+		}, ItemStack.EMPTY);
+		if (readList != null) stackList = readList;
 	}
 }

src/main/java/network/rs485/logisticspipes/util/LPDataIOWrapper.java

@@ -692,7 +692,8 @@ public <T> NonNullList<T> readNonNullList(IReadListObject<T> reader, @Nonnull T
 
 		NonNullList<T> list = NonNullList.withSize(size, fillItem);
 		for (int i = 0; i < size; i++) {
-			list.set(i, reader.readObject(this));
+			T obj = reader.readObject(this);
+			if (obj != null) list.set(i, obj);
 		}
 		return list;
 	}