b485c74503 backport (#2397)

Co-authored-by: Raoul <raoulvdberge@gmail.com>

src/main/java/com/raoulvdberge/refinedstorage/api/network/grid/IGrid.java

@@ -231,6 +231,11 @@ public interface IGrid {
      */
     boolean isActive();
 
+    /**
+     * @return the slot id where this grid is located, if applicable, otherwise -1
+     */
+    int getSlotId();
+
     static boolean isValidViewType(int type) {
         return type == VIEW_TYPE_NORMAL ||
             type == VIEW_TYPE_CRAFTABLES ||

src/main/java/com/raoulvdberge/refinedstorage/api/network/grid/IGridFactory.java

@@ -17,10 +17,11 @@ public interface IGridFactory {
      *
      * @param player the player
      * @param stack  the stack
+     * @param slotId the slot id, if applicable, otherwise -1
      * @return the grid, or null if a problem occurred
      */
     @Nullable
-    IGrid createFromStack(EntityPlayer player, ItemStack stack);
+    IGrid createFromStack(EntityPlayer player, ItemStack stack, int slotId);
 
     /**
      * Creates a grid from a block. Used when {@link #getType()} is BLOCK.

src/main/java/com/raoulvdberge/refinedstorage/api/network/grid/IGridManager.java

@@ -41,8 +41,9 @@ public interface IGridManager {
      * @param id     the grid factory id as returned from {@link #add(IGridFactory)}
      * @param player the player
      * @param stack  the stack
+     * @param slotId the slot id, if applicable, otherwise -1
      */
-    void openGrid(int id, EntityPlayerMP player, ItemStack stack);
+    void openGrid(int id, EntityPlayerMP player, ItemStack stack, int slotId);
 
     /**
      * Creates a grid.
@@ -51,8 +52,9 @@ public interface IGridManager {
      * @param player the player
      * @param stack  the stack, if there is one
      * @param pos    the block position, if there is one
+     * @param slotId the slot id, if applicable, otherwise -1
      * @return a grid, or null if an error has occurred
      */
     @Nullable
-    Pair<IGrid, TileEntity> createGrid(int id, EntityPlayer player, @Nullable ItemStack stack, @Nullable BlockPos pos);
+    Pair<IGrid, TileEntity> createGrid(int id, EntityPlayer player, @Nullable ItemStack stack, @Nullable BlockPos pos, int slotId);
 }

src/main/java/com/raoulvdberge/refinedstorage/api/network/item/INetworkItemHandler.java

@@ -15,8 +15,9 @@ public interface INetworkItemHandler {
      *
      * @param player the player that opened the network item
      * @param stack  the stack that has been opened
+     * @param slotId the slot id, if applicable, otherwise -1
      */
-    void open(EntityPlayer player, ItemStack stack);
+    void open(EntityPlayer player, ItemStack stack, int slotId);
 
     /**
      * Called when the player closes a network item.

src/main/java/com/raoulvdberge/refinedstorage/api/network/item/INetworkItemProvider.java

@@ -15,8 +15,9 @@ public interface INetworkItemProvider {
      * @param handler the network item handler
      * @param player  the player
      * @param stack   the stack
+     * @param slotId  the slot id, if applicable, otherwise -1
      * @return the network item
      */
     @Nonnull
-    INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack);
+    INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId);
 }

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridFactoryGridBlock.java

@@ -15,7 +15,7 @@ import javax.annotation.Nullable;
 public class GridFactoryGridBlock implements IGridFactory {
     @Override
     @Nullable
-    public IGrid createFromStack(EntityPlayer player, ItemStack stack) {
+    public IGrid createFromStack(EntityPlayer player, ItemStack stack, int slotId) {
         return null;
     }
 

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridFactoryPortableGrid.java

@@ -15,8 +15,8 @@ import javax.annotation.Nullable;
 public class GridFactoryPortableGrid implements IGridFactory {
     @Nullable
     @Override
-    public IGrid createFromStack(EntityPlayer player, ItemStack stack) {
+    public IGrid createFromStack(EntityPlayer player, ItemStack stack, int slotId) {
-        return new PortableGrid(player, stack);
+        return new PortableGrid(player, stack, slotId);
     }
 
     @Nullable

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridFactoryPortableGridBlock.java

@@ -15,7 +15,7 @@ import javax.annotation.Nullable;
 public class GridFactoryPortableGridBlock implements IGridFactory {
     @Override
     @Nullable
-    public IGrid createFromStack(EntityPlayer player, ItemStack stack) {
+    public IGrid createFromStack(EntityPlayer player, ItemStack stack, int slotId) {
         return null;
     }
 

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridFactoryWirelessFluidGrid.java

@@ -15,8 +15,8 @@ import javax.annotation.Nullable;
 public class GridFactoryWirelessFluidGrid implements IGridFactory {
     @Nullable
     @Override
-    public IGrid createFromStack(EntityPlayer player, ItemStack stack) {
+    public IGrid createFromStack(EntityPlayer player, ItemStack stack, int slotId) {
-        return new WirelessFluidGrid(stack);
+        return new WirelessFluidGrid(stack, slotId);
     }
 
     @Nullable

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridFactoryWirelessGrid.java

@@ -15,8 +15,8 @@ import javax.annotation.Nullable;
 public class GridFactoryWirelessGrid implements IGridFactory {
     @Nullable
     @Override
-    public IGrid createFromStack(EntityPlayer player, ItemStack stack) {
+    public IGrid createFromStack(EntityPlayer player, ItemStack stack, int slotId) {
-        return new WirelessGrid(stack);
+        return new WirelessGrid(stack, slotId);
     }
 
     @Nullable

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridManager.java

@@ -43,16 +43,16 @@ public class GridManager implements IGridManager {
 
     @Override
     public void openGrid(int id, EntityPlayerMP player, BlockPos pos) {
-        openGrid(id, player, null, pos);
+        openGrid(id, player, null, pos, -1);
     }
 
     @Override
-    public void openGrid(int id, EntityPlayerMP player, ItemStack stack) {
+    public void openGrid(int id, EntityPlayerMP player, ItemStack stack, int slotId) {
-        openGrid(id, player, stack, null);
+        openGrid(id, player, stack, null, slotId);
     }
 
-    private void openGrid(int id, EntityPlayerMP player, @Nullable ItemStack stack, @Nullable BlockPos pos) {
+    private void openGrid(int id, EntityPlayerMP player, @Nullable ItemStack stack, @Nullable BlockPos pos, int slotId) {
-        Pair<IGrid, TileEntity> grid = createGrid(id, player, stack, pos);
+        Pair<IGrid, TileEntity> grid = createGrid(id, player, stack, pos, slotId);
         if (grid == null) {
             return;
         }
@@ -73,7 +73,7 @@ public class GridManager implements IGridManager {
         // So we first send the window id in MessageGridOpen.
 
         // The order is preserved by TCP.
-        RS.INSTANCE.network.sendTo(new MessageGridOpen(player.currentWindowId, pos, id, stack), player);
+        RS.INSTANCE.network.sendTo(new MessageGridOpen(player.currentWindowId, pos, id, stack, slotId), player);
 
         player.openContainer = new ContainerGrid(grid.getLeft(), new ResizableDisplayDummy(), grid.getRight() instanceof TileBase ? (TileBase) grid.getRight() : null, player);
         player.openContainer.windowId = player.currentWindowId;
@@ -84,7 +84,7 @@ public class GridManager implements IGridManager {
 
     @Override
     @Nullable
-    public Pair<IGrid, TileEntity> createGrid(int id, EntityPlayer player, @Nullable ItemStack stack, @Nullable BlockPos pos) {
+    public Pair<IGrid, TileEntity> createGrid(int id, EntityPlayer player, @Nullable ItemStack stack, @Nullable BlockPos pos, int slotId) {
         IGridFactory factory = get(id);
 
         if (factory == null) {
@@ -96,7 +96,7 @@ public class GridManager implements IGridManager {
 
         switch (factory.getType()) {
             case STACK:
-                grid = factory.createFromStack(player, stack);
+                grid = factory.createFromStack(player, stack, slotId);
                 break;
             case BLOCK:
                 grid = factory.createFromBlock(player, pos);

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/grid/GridOpenHandler.java

@@ -23,7 +23,7 @@ public class GridOpenHandler implements Runnable {
     public void run() {
         EntityPlayer player = Minecraft.getMinecraft().player;
 
-        Pair<IGrid, TileEntity> grid = API.instance().getGridManager().createGrid(message.getGridId(), player, message.getStack(), message.getPos());
+        Pair<IGrid, TileEntity> grid = API.instance().getGridManager().createGrid(message.getGridId(), player, message.getStack(), message.getPos(), message.getSlotId());
 
         if (grid == null) {
             return;

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/item/NetworkItemHandler.java

@@ -23,7 +23,7 @@ public class NetworkItemHandler implements INetworkItemHandler {
     }
 
     @Override
-    public void open(EntityPlayer player, ItemStack stack) {
+    public void open(EntityPlayer player, ItemStack stack, int slotId) {
         boolean inRange = false;
 
         for (INetworkNode node : network.getNodeGraph().all()) {
@@ -46,7 +46,7 @@ public class NetworkItemHandler implements INetworkItemHandler {
             return;
         }
 
-        INetworkItem item = ((INetworkItemProvider) stack.getItem()).provide(this, player, stack);
+        INetworkItem item = ((INetworkItemProvider) stack.getItem()).provide(this, player, stack, slotId);
 
         if (item.onOpen(network)) {
             items.put(player, item);

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/item/NetworkItemWirelessCraftingMonitor.java

@@ -17,13 +17,13 @@ public class NetworkItemWirelessCraftingMonitor implements INetworkItem {
     private INetworkItemHandler handler;
     private EntityPlayer player;
     private ItemStack stack;
-    private int invIndex;
+    private int slotId;
 
-    public NetworkItemWirelessCraftingMonitor(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int invIndex) {
+    public NetworkItemWirelessCraftingMonitor(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId) {
         this.handler = handler;
         this.player = player;
         this.stack = stack;
-        this.invIndex = invIndex;
+        this.slotId = slotId;
     }
 
     @Override
@@ -43,7 +43,7 @@ public class NetworkItemWirelessCraftingMonitor implements INetworkItem {
             return false;
         }
 
-        player.openGui(RS.INSTANCE, RSGui.WIRELESS_CRAFTING_MONITOR, player.getEntityWorld(), invIndex, 0, 0);
+        player.openGui(RS.INSTANCE, RSGui.WIRELESS_CRAFTING_MONITOR, player.getEntityWorld(), slotId, 0, 0);
 
         drainEnergy(RS.INSTANCE.config.wirelessCraftingMonitorOpenUsage);
 

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/item/NetworkItemWirelessFluidGrid.java

@@ -19,11 +19,13 @@ public class NetworkItemWirelessFluidGrid implements INetworkItem {
     private INetworkItemHandler handler;
     private EntityPlayer player;
     private ItemStack stack;
+    private int slotId;
 
-    public NetworkItemWirelessFluidGrid(INetworkItemHandler handler, EntityPlayer player, ItemStack stack) {
+    public NetworkItemWirelessFluidGrid(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId) {
         this.handler = handler;
         this.player = player;
         this.stack = stack;
+        this.slotId = slotId;
     }
 
     @Override
@@ -43,7 +45,7 @@ public class NetworkItemWirelessFluidGrid implements INetworkItem {
             return false;
         }
 
-        API.instance().getGridManager().openGrid(WirelessFluidGrid.ID, (EntityPlayerMP) player, stack);
+        API.instance().getGridManager().openGrid(WirelessFluidGrid.ID, (EntityPlayerMP) player, stack, slotId);
 
         drainEnergy(RS.INSTANCE.config.wirelessFluidGridOpenUsage);
 

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/item/NetworkItemWirelessGrid.java

@@ -19,11 +19,13 @@ public class NetworkItemWirelessGrid implements INetworkItem {
     private INetworkItemHandler handler;
     private EntityPlayer player;
     private ItemStack stack;
+    private int slotId;
 
-    public NetworkItemWirelessGrid(INetworkItemHandler handler, EntityPlayer player, ItemStack stack) {
+    public NetworkItemWirelessGrid(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId) {
         this.handler = handler;
         this.player = player;
         this.stack = stack;
+        this.slotId = slotId;
     }
 
     @Override
@@ -43,7 +45,7 @@ public class NetworkItemWirelessGrid implements INetworkItem {
             return false;
         }
 
-        API.instance().getGridManager().openGrid(WirelessGrid.ID, (EntityPlayerMP) player, stack);
+        API.instance().getGridManager().openGrid(WirelessGrid.ID, (EntityPlayerMP) player, stack, slotId);
 
         drainEnergy(RS.INSTANCE.config.wirelessGridOpenUsage);
 

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/node/NetworkNodeCraftingMonitor.java

@@ -135,4 +135,9 @@ public class NetworkNodeCraftingMonitor extends NetworkNode implements ICrafting
             TileDataManager.setParameter(TileCraftingMonitor.TAB_PAGE, page);
         }
     }
+
+    @Override
+    public int getSlotId() {
+        return -1;
+    }
 }

src/main/java/com/raoulvdberge/refinedstorage/apiimpl/network/node/NetworkNodeGrid.java

@@ -449,6 +449,11 @@ public class NetworkNodeGrid extends NetworkNode implements IGridNetworkAware, I
         // NO OP
     }
 
+    @Override
+    public int getSlotId() {
+        return -1;
+    }
+
     @Override
     public void onCrafted(EntityPlayer player) {
         onCrafted(this, world, player);

src/main/java/com/raoulvdberge/refinedstorage/container/ContainerBase.java

@@ -61,7 +61,11 @@ public abstract class ContainerBase extends Container {
         
         for (int y = 0; y < 3; y++) {
             for (int x = 0; x < 9; x++) {
-                addSlotToContainer(new Slot(player.inventory, id, xInventory + x * 18, yInventory + y * 18));
+                if (id == disabledSlotNumber) {
+                    addSlotToContainer(new SlotLegacyDisabled(player.inventory, id, xInventory + x * 18, yInventory + y * 18));
+                } else {
+                    addSlotToContainer(new Slot(player.inventory, id, xInventory + x * 18, yInventory + y * 18));
+                }
 
                 id++;
             }
@@ -69,11 +73,13 @@ public abstract class ContainerBase extends Container {
         
         id = 0;
 
+        int disabledSlotNumber = getDisabledSlotNumber();
+
         for (int i = 0; i < 9; i++) {
             int x = xInventory + i * 18;
             int y = yInventory + 4 + (3 * 18);
 
-            if (isHeldItemDisabled() && i == player.inventory.currentItem) {
+            if (id == disabledSlotNumber) {
                 addSlotToContainer(new SlotLegacyDisabled(player.inventory, id, x, y));
             } else {
                 addSlotToContainer(new Slot(player.inventory, id, x, y));
@@ -91,8 +97,12 @@ public abstract class ContainerBase extends Container {
     public ItemStack slotClick(int id, int dragType, ClickType clickType, EntityPlayer player) {
         Slot slot = id >= 0 ? getSlot(id) : null;
 
+        int disabledSlotNumber = getDisabledSlotNumber();
+
         // Prevent swapping disabled held item with the number keys (dragType is the slot we're swapping with)
-        if (isHeldItemDisabled() && clickType == ClickType.SWAP && dragType == player.inventory.currentItem) {
+        if (disabledSlotNumber != -1 &&
+                clickType == ClickType.SWAP &&
+                dragType == disabledSlotNumber) {
             return ItemStack.EMPTY;
         }
 
@@ -158,8 +168,8 @@ public abstract class ContainerBase extends Container {
         return super.canMergeSlot(stack, slot);
     }
 
-    protected boolean isHeldItemDisabled() {
+    protected int getDisabledSlotNumber() {
-        return false;
+        return -1;
     }
 
     @Override

src/main/java/com/raoulvdberge/refinedstorage/container/ContainerCraftingMonitor.java

@@ -82,8 +82,8 @@ public class ContainerCraftingMonitor extends ContainerBase implements ICrafting
     }
 
     @Override
-    protected boolean isHeldItemDisabled() {
+    protected int getDisabledSlotNumber() {
-        return craftingMonitor instanceof WirelessCraftingMonitor;
+        return craftingMonitor.getSlotId();
     }
 
     @Override

src/main/java/com/raoulvdberge/refinedstorage/container/ContainerFilter.java

@@ -53,7 +53,7 @@ public class ContainerFilter extends ContainerBase {
     }
 
     @Override
-    protected boolean isHeldItemDisabled() {
+    protected int getDisabledSlotNumber() {
-        return true;
+        return getPlayer().inventory.currentItem;
     }
 }

src/main/java/com/raoulvdberge/refinedstorage/container/ContainerGrid.java

@@ -18,9 +18,7 @@ import com.raoulvdberge.refinedstorage.container.slot.legacy.SlotLegacyFilter;
 import com.raoulvdberge.refinedstorage.gui.IResizableDisplay;
 import com.raoulvdberge.refinedstorage.tile.TileBase;
 import com.raoulvdberge.refinedstorage.tile.config.IType;
-import com.raoulvdberge.refinedstorage.tile.grid.WirelessGrid;
 import com.raoulvdberge.refinedstorage.tile.grid.portable.IPortableGrid;
-import com.raoulvdberge.refinedstorage.tile.grid.portable.PortableGrid;
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.entity.player.EntityPlayerMP;
 import net.minecraft.inventory.IContainerListener;
@@ -271,8 +269,7 @@ public class ContainerGrid extends ContainerBase implements IGridCraftingListene
     }
 
     @Override
-    protected boolean isHeldItemDisabled() {
+    protected int getDisabledSlotNumber() {
-        // Here we check for the concrete portable grid type, not IPortableGrid, because we *CAN* move the held item in the portable grid tile
+        return grid.getSlotId();
-        return grid instanceof WirelessGrid || grid instanceof PortableGrid;
     }
 }

src/main/java/com/raoulvdberge/refinedstorage/gui/GuiHandler.java

@@ -143,7 +143,7 @@ public class GuiHandler implements IGuiHandler {
     }
 
     private WirelessCraftingMonitor getWirelessCraftingMonitor(EntityPlayer player, int invIndex) {
-        return new WirelessCraftingMonitor(player.inventory.getStackInSlot(invIndex));
+        return new WirelessCraftingMonitor(player.inventory.getStackInSlot(invIndex), invIndex);
     }
 
     private GuiCraftingMonitor getWirelessCraftingMonitorGui(EntityPlayer player, int invIndex) {

src/main/java/com/raoulvdberge/refinedstorage/item/ItemNetworkItem.java

@@ -47,7 +47,7 @@ public abstract class ItemNetworkItem extends ItemEnergyItem implements INetwork
         ItemStack stack = player.getHeldItem(hand);
 
         if (!world.isRemote) {
-            applyNetwork(stack, n -> n.getNetworkItemHandler().open(player, player.getHeldItem(hand)), player::sendMessage);
+            applyNetwork(stack, n -> n.getNetworkItemHandler().open(player, player.getHeldItem(hand), player.inventory.currentItem), player::sendMessage);
         }
 
         return ActionResult.newResult(EnumActionResult.SUCCESS, stack);

src/main/java/com/raoulvdberge/refinedstorage/item/ItemWirelessCraftingMonitor.java

@@ -33,16 +33,8 @@ public class ItemWirelessCraftingMonitor extends ItemNetworkItem {
 
     @Override
     @Nonnull
-    public INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack) {
+    public INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId) {
-        int invIndex = 0;
+        return new NetworkItemWirelessCraftingMonitor(handler, player, stack, slotId);
-        for (int i = 0; i < player.inventory.getSizeInventory(); ++i) {
-            if (player.inventory.getStackInSlot(i) == stack) {
-                invIndex = i;
-                break;
-            }
-        }
-
-        return new NetworkItemWirelessCraftingMonitor(handler, player, stack, invIndex);
     }
 
     public static Optional<UUID> getTabSelected(ItemStack stack) {

src/main/java/com/raoulvdberge/refinedstorage/item/ItemWirelessFluidGrid.java

@@ -29,8 +29,8 @@ public class ItemWirelessFluidGrid extends ItemNetworkItem {
 
     @Override
     @Nonnull
-    public INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack) {
+    public INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId) {
-        return new NetworkItemWirelessFluidGrid(handler, player, stack);
+        return new NetworkItemWirelessFluidGrid(handler, player, stack, slotId);
     }
 
     public static int getSortingType(ItemStack stack) {

src/main/java/com/raoulvdberge/refinedstorage/item/ItemWirelessGrid.java

@@ -34,8 +34,8 @@ public class ItemWirelessGrid extends ItemNetworkItem {
 
     @Override
     @Nonnull
-    public INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack) {
+    public INetworkItem provide(INetworkItemHandler handler, EntityPlayer player, ItemStack stack, int slotId) {
-        return new NetworkItemWirelessGrid(handler, player, stack);
+        return new NetworkItemWirelessGrid(handler, player, stack, slotId);
     }
 
     public static int getViewType(ItemStack stack) {

src/main/java/com/raoulvdberge/refinedstorage/item/itemblock/ItemBlockPortableGrid.java

@@ -30,7 +30,7 @@ public class ItemBlockPortableGrid extends ItemBlockEnergyItem {
         ItemStack stack = player.getHeldItem(hand);
 
         if (!world.isRemote) {
-            API.instance().getGridManager().openGrid(PortableGrid.ID, (EntityPlayerMP) player, stack);
+            API.instance().getGridManager().openGrid(PortableGrid.ID, (EntityPlayerMP) player, stack, player.inventory.currentItem);
         }
 
         return ActionResult.newResult(EnumActionResult.SUCCESS, stack);

src/main/java/com/raoulvdberge/refinedstorage/network/MessageGridOpen.java

@@ -18,12 +18,13 @@ public class MessageGridOpen implements IMessage, IMessageHandler<MessageGridOpe
     private BlockPos pos;
     @Nullable
     private ItemStack stack;
+    private int slotId;
     private int gridId;
 
     public MessageGridOpen() {
     }
 
-    public MessageGridOpen(int windowId, @Nullable BlockPos pos, int gridId, @Nullable ItemStack stack) {
+    public MessageGridOpen(int windowId, @Nullable BlockPos pos, int gridId, @Nullable ItemStack stack, int slotId) {
         if (pos == null && stack == null) {
             throw new IllegalArgumentException("Can't be both null");
         }
@@ -32,12 +33,14 @@ public class MessageGridOpen implements IMessage, IMessageHandler<MessageGridOpe
         this.pos = pos;
         this.stack = stack;
         this.gridId = gridId;
+        this.slotId = slotId;
     }
 
     @Override
     public void fromBytes(ByteBuf buf) {
         windowId = buf.readInt();
         gridId = buf.readInt();
+        slotId = buf.readInt();
 
         if (buf.readBoolean()) {
             pos = new BlockPos(buf.readInt(), buf.readInt(), buf.readInt());
@@ -52,6 +55,7 @@ public class MessageGridOpen implements IMessage, IMessageHandler<MessageGridOpe
     public void toBytes(ByteBuf buf) {
         buf.writeInt(windowId);
         buf.writeInt(gridId);
+        buf.writeInt(slotId);
 
         buf.writeBoolean(pos != null);
         if (pos != null) {
@@ -84,6 +88,10 @@ public class MessageGridOpen implements IMessage, IMessageHandler<MessageGridOpe
         return gridId;
     }
 
+    public int getSlotId() {
+        return slotId;
+    }
+
     @Override
     public IMessage onMessage(MessageGridOpen message, MessageContext ctx) {
         Minecraft.getMinecraft().addScheduledTask(new GridOpenHandler(message));

src/main/java/com/raoulvdberge/refinedstorage/network/MessageGridProcessingTransfer.java

@@ -10,7 +10,6 @@ import com.raoulvdberge.refinedstorage.util.StackUtils;
 import io.netty.buffer.ByteBuf;
 import net.minecraft.entity.player.EntityPlayerMP;
 import net.minecraft.item.ItemStack;
-import net.minecraftforge.fluids.Fluid;
 import net.minecraftforge.fluids.FluidStack;
 import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
 

src/main/java/com/raoulvdberge/refinedstorage/network/MessageNetworkItemOpen.java

@@ -25,9 +25,9 @@ public class MessageNetworkItemOpen extends MessageHandlerPlayerToServer<Message
         ItemStack stack = player.inventory.getStackInSlot(message.slotId);
 
         if (stack.getItem() instanceof ItemNetworkItem) {
-            ((ItemNetworkItem) stack.getItem()).applyNetwork(stack, n -> n.getNetworkItemHandler().open(player, player.inventory.getStackInSlot(message.slotId)), player::sendMessage);
+            ((ItemNetworkItem) stack.getItem()).applyNetwork(stack, n -> n.getNetworkItemHandler().open(player, stack, message.slotId), player::sendMessage);
         } else if (stack.getItem() == Item.getItemFromBlock(RSBlocks.PORTABLE_GRID)) { // @Hack
-            API.instance().getGridManager().openGrid(PortableGrid.ID, player, stack);
+            API.instance().getGridManager().openGrid(PortableGrid.ID, player, stack, message.slotId);
         }
     }
 

src/main/java/com/raoulvdberge/refinedstorage/tile/craftingmonitor/ICraftingMonitor.java

@@ -36,4 +36,6 @@ public interface ICraftingMonitor {
     void onTabSelectionChanged(Optional<UUID> taskId);
 
     void onTabPageChanged(int page);
+
+    int getSlotId();
 }

src/main/java/com/raoulvdberge/refinedstorage/tile/craftingmonitor/WirelessCraftingMonitor.java

@@ -28,9 +28,11 @@ public class WirelessCraftingMonitor implements ICraftingMonitor {
     private BlockPos network;
     private int tabPage;
     private Optional<UUID> tabSelected;
+    private int slotId;
 
-    public WirelessCraftingMonitor(ItemStack stack) {
+    public WirelessCraftingMonitor(ItemStack stack, int slotId) {
         this.stack = stack;
+        this.slotId = slotId;
         this.networkDimension = ItemWirelessCraftingMonitor.getDimensionId(stack);
         this.network = new BlockPos(ItemWirelessCraftingMonitor.getX(stack), ItemWirelessCraftingMonitor.getY(stack), ItemWirelessCraftingMonitor.getZ(stack));
         this.tabPage = ItemWirelessCraftingMonitor.getTabPage(stack);
@@ -138,4 +140,9 @@ public class WirelessCraftingMonitor implements ICraftingMonitor {
             RS.INSTANCE.network.sendToServer(new MessageWirelessCraftingMonitorSettings(tabSelected, tabPage));
         }
     }
+
+    @Override
+    public int getSlotId() {
+        return slotId;
+    }
 }

src/main/java/com/raoulvdberge/refinedstorage/tile/grid/WirelessFluidGrid.java

@@ -39,6 +39,7 @@ public class WirelessFluidGrid implements IGridNetworkAware {
 
     private int networkDimension;
     private BlockPos network;
+    private int slotId;
 
     private int sortingType;
     private int sortingDirection;
@@ -62,11 +63,12 @@ public class WirelessFluidGrid implements IGridNetworkAware {
         }
     };
 
-    public WirelessFluidGrid(ItemStack stack) {
+    public WirelessFluidGrid(ItemStack stack, int slotId) {
         this.networkDimension = ItemWirelessFluidGrid.getDimensionId(stack);
         this.network = new BlockPos(ItemWirelessFluidGrid.getX(stack), ItemWirelessFluidGrid.getY(stack), ItemWirelessFluidGrid.getZ(stack));
 
         this.stack = stack;
+        this.slotId = slotId;
 
         this.sortingType = ItemWirelessFluidGrid.getSortingType(stack);
         this.sortingDirection = ItemWirelessFluidGrid.getSortingDirection(stack);
@@ -294,6 +296,11 @@ public class WirelessFluidGrid implements IGridNetworkAware {
         return true;
     }
 
+    @Override
+    public int getSlotId() {
+        return slotId;
+    }
+
     @Override
     public void onClosed(EntityPlayer player) {
         INetwork network = getNetwork();

src/main/java/com/raoulvdberge/refinedstorage/tile/grid/WirelessGrid.java

@@ -39,6 +39,7 @@ public class WirelessGrid implements IGridNetworkAware {
 
     private int networkDimension;
     private BlockPos network;
+    private int slotId;
 
     private int viewType;
     private int sortingType;
@@ -63,11 +64,12 @@ public class WirelessGrid implements IGridNetworkAware {
         }
     };
 
-    public WirelessGrid(ItemStack stack) {
+    public WirelessGrid(ItemStack stack, int slotId) {
         this.networkDimension = ItemWirelessGrid.getDimensionId(stack);
         this.network = new BlockPos(ItemWirelessGrid.getX(stack), ItemWirelessGrid.getY(stack), ItemWirelessGrid.getZ(stack));
 
         this.stack = stack;
+        this.slotId = slotId;
 
         this.viewType = ItemWirelessGrid.getViewType(stack);
         this.sortingType = ItemWirelessGrid.getSortingType(stack);
@@ -300,6 +302,12 @@ public class WirelessGrid implements IGridNetworkAware {
         return true;
     }
 
+    @Override
+    public int getSlotId() {
+        return slotId;
+    }
+
+
     @Override
     public void onClosed(EntityPlayer player) {
         INetwork network = getNetwork();

src/main/java/com/raoulvdberge/refinedstorage/tile/grid/portable/PortableGrid.java

@@ -63,6 +63,7 @@ public class PortableGrid implements IGrid, IPortableGrid, IStorageDiskContainer
 
     private EntityPlayer player;
     private ItemStack stack;
+    private int slotId;
 
     private int sortingType;
     private int sortingDirection;
@@ -132,9 +133,10 @@ public class PortableGrid implements IGrid, IPortableGrid, IStorageDiskContainer
         }
     };
 
-    public PortableGrid(EntityPlayer player, ItemStack stack) {
+    public PortableGrid(EntityPlayer player, ItemStack stack, int slotId) {
         this.player = player;
         this.stack = stack;
+        this.slotId = slotId;
 
         if (player != null) {
             this.sortingType = ItemWirelessGrid.getSortingType(stack);
@@ -426,6 +428,11 @@ public class PortableGrid implements IGrid, IPortableGrid, IStorageDiskContainer
         return true;
     }
 
+    @Override
+    public int getSlotId() {
+        return slotId;
+    }
+
     @Override
     public AccessType getAccessType() {
         return AccessType.INSERT_EXTRACT;

src/main/java/com/raoulvdberge/refinedstorage/tile/grid/portable/TilePortableGrid.java

@@ -519,6 +519,11 @@ public class TilePortableGrid extends TileBase implements IGrid, IPortableGrid,
         return redstoneMode.isEnabled(world, pos);
     }
 
+    @Override
+    public int getSlotId() {
+        return -1;
+    }
+
     @Override
     @Nullable
     public IStorageCache getCache() {