Posted September 25, 201510 yr How can i display a different GUI from a Vanilla Block? For example, i want to display this GUI when clicking on the crafting table where the additional 4 slots should be automatically filled with the current armor. Is there any way to make this? And if there is, how can i make it (any links to a tutorial or a previous post like this?) Don't blame me if i always ask for your help. I just want to learn to be better
September 25, 201510 yr Replace the crafting inventory with your own. Your own should have all the extra slots handled. @SubscribeEvent public void onEvent(GuiOpenEvent event) { if(event.gui instanceof TheInventoryYouWantToReplace) { event.gui = new YourNewInventory(Minecraft.getMinecraft().thePlayer); } } Development of Plugins [2012 - 2014] Development of Mods [2012 - Current]
September 26, 201510 yr Author I don't understand what the YourNewInventory class should looks like. I'm following this example to create a GUI http://www.minecraftforge.net/wiki/Containers_and_GUIs but i can't figure out why you pass only the player to the constructor (since the class takes a TileEntity as second parameter) Don't blame me if i always ask for your help. I just want to learn to be better
September 26, 201510 yr I don't understand what the YourNewInventory class should looks like. I'm following this example to create a GUI http://www.minecraftforge.net/wiki/Containers_and_GUIs but i can't figure out why you pass only the player to the constructor (since the class takes a TileEntity as second parameter) By all means my above post was an example of code taken from my mod Armed - I had to give an EntityPlayer instance on the client for my GUI. Use your own constructor. Development of Plugins [2012 - 2014] Development of Mods [2012 - Current]
September 26, 201510 yr Author Thanks for your help I've created the GUI and now I have this handler package blaze.gui; import net.minecraft.entity.player.EntityPlayer; import net.minecraft.entity.player.InventoryPlayer; import net.minecraft.init.Blocks; import net.minecraft.inventory.Container; import net.minecraft.inventory.IInventory; import net.minecraft.inventory.InventoryCraftResult; import net.minecraft.inventory.InventoryCrafting; import net.minecraft.inventory.Slot; import net.minecraft.inventory.SlotCrafting; import net.minecraft.item.ItemStack; import net.minecraft.item.crafting.CraftingManager; import net.minecraft.util.BlockPos; import net.minecraft.world.World; public class ContainerNewWorkbench extends Container { /** The crafting matrix inventory (3x3). */ public InventoryCrafting craftMatrix = new InventoryCrafting(this, 3, 3); public IInventory craftResult = new InventoryCraftResult(); private World worldObj; private BlockPos pos; public ContainerNewWorkbench(InventoryPlayer playerInventory, World worldIn, BlockPos position) { this.worldObj = worldIn; this.pos = position; this.addSlotToContainer(new SlotCrafting(playerInventory.player, this.craftMatrix, this.craftResult, 0, 124, 35)); int i; int j; /** * SLOTS CRAFTING 0-9 */ for (i = 0; i < 3; ++i) { for (j = 0; j < 3; ++j) { this.addSlotToContainer(new Slot(this.craftMatrix, j + i * 3, 30 + j * 18, 17 + i * 18)); } } /** * SLOTS 9-35 */ for (i = 0; i < 3; ++i) { for (j = 0; j < 9; ++j) { this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, 8 + j * 18, 84 + i * 18)); } } /** * SLOTS 0-9 */ for (i = 0; i < 9; ++i) { this.addSlotToContainer(new Slot(playerInventory, i, 8 + i * 18, 142)); } /** * START ADDING THE ADDITIONAL 4 SLOTS, THEY WILL BE THE SLOTS FOR THE HELMET, THE CHESTPLATE AND THE LEGGINGS AND THE BOOTS. * THIS SLOTS MUST BE AUTOMATICALLY FILLED WITH THE CURRENT ARMOR WEARED, ALSO IN CRAFTING * IF THE PLAYER CRAFT AN ARMOR THIS WILL AUTOMATICALLY BE PUTTED HERE IF EMPTY, SO THE PLAYER * WILL WEAR IT AS SOON AS HE CRAFTED IT * SLOTS 36-39 */ for (i = 0; i < 4; ++i) { this.addSlotToContainer(new Slot(playerInventory, 36 + i, 8 + 9 * 18, 84 + i * 18)); } this.onCraftMatrixChanged(this.craftMatrix); } /** * Callback for when the crafting matrix is changed. */ public void onCraftMatrixChanged(IInventory inventoryIn) { this.craftResult.setInventorySlotContents(0, CraftingManager.getInstance().findMatchingRecipe(this.craftMatrix, this.worldObj)); } /** * Called when the container is closed. */ public void onContainerClosed(EntityPlayer playerIn) { super.onContainerClosed(playerIn); if (!this.worldObj.isRemote) { for (int i = 0; i < 9; ++i) { ItemStack itemstack = this.craftMatrix.getStackInSlotOnClosing(i); if (itemstack != null) { playerIn.dropPlayerItemWithRandomChoice(itemstack, false); } } } } public boolean canInteractWith(EntityPlayer playerIn) { return this.worldObj.getBlockState(this.pos).getBlock() != Blocks.crafting_table ? false : playerIn.getDistanceSq((double)this.pos.getX() + 0.5D, (double)this.pos.getY() + 0.5D, (double)this.pos.getZ() + 0.5D) <= 64.0D; } /** * Take a stack from the specified inventory slot. */ public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) { ItemStack itemstack = null; Slot slot = (Slot)this.inventorySlots.get(index); if (slot != null && slot.getHasStack()) { ItemStack itemstack1 = slot.getStack(); itemstack = itemstack1.copy(); if (index == 0) { if (!this.mergeItemStack(itemstack1, 10, 51, true)) { return null; } slot.onSlotChange(itemstack1, itemstack); } else if (index >= 10 && index < 37) { if (!this.mergeItemStack(itemstack1, 37, 51, false)) { return null; } } else if (index >= 37 && index < 46) { if (!this.mergeItemStack(itemstack1, 10, 37, false)) { return null; } } else if (index >= 47 && index < 51) { if (!this.mergeItemStack(itemstack1, 10, 47, false)) { return null; } } else if (!this.mergeItemStack(itemstack1, 10, 51, false)) { return null; } if (itemstack1.stackSize == 0) { slot.putStack((ItemStack)null); } else { slot.onSlotChanged(); } if (itemstack1.stackSize == itemstack.stackSize) { return null; } slot.onPickupFromSlot(playerIn, itemstack1); } return itemstack; } /** * Called to determine if the current slot is valid for the stack merging (double-click) code. The stack passed in * is null for the initial slot that was double-clicked. */ public boolean canMergeSlot(ItemStack itemstack, Slot slot) { return slot.inventory != this.craftResult && super.canMergeSlot(itemstack, slot); } } But there are a lot of problems with the additional slots. The 4 new slots should be automatically filled with the current armor weared. Actually they do but reversed (video below). Also i want that in that 4 slots only armor pieces can be putted (or everything else that can be weared (like skulls or pumpkins). Also if i craft an armor piece this should go in the correspective new slot, so the player will instantly wear it How can i do that? Another serious problem is that if i try to get something from that slot it disappear, gone forever, but if i try to put in ther it duplicates https://www.youtube.com/watch?v=cojKwg4i1T8&feature=youtu.be Don't blame me if i always ask for your help. I just want to learn to be better
September 26, 201510 yr Author Ok, quick update, if i click on the extra slot it gives me this error [20:33:38] [server thread/FATAL] [FML]: Exception caught executing FutureTask: java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 48, Size: 46 java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 48, Size: 46 at java.util.concurrent.FutureTask.report(Unknown Source) ~[?:1.8.0_51] at java.util.concurrent.FutureTask.get(Unknown Source) ~[?:1.8.0_51] at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:710) [FMLCommonHandler.class:?] at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726) [MinecraftServer.class:?] at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669) [MinecraftServer.class:?] at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171) [integratedServer.class:?] at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540) [MinecraftServer.class:?] at java.lang.Thread.run(Unknown Source) [?:1.8.0_51] Caused by: java.lang.IndexOutOfBoundsException: Index: 48, Size: 46 at java.util.ArrayList.rangeCheck(Unknown Source) ~[?:1.8.0_51] at java.util.ArrayList.get(Unknown Source) ~[?:1.8.0_51] at net.minecraft.inventory.Container.slotClick(Container.java:473) ~[Container.class:?] at net.minecraft.network.NetHandlerPlayServer.processClickWindow(NetHandlerPlayServer.java:1039) ~[NetHandlerPlayServer.class:?] at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:46) ~[C0EPacketClickWindow.class:?] at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:110) ~[C0EPacketClickWindow.class:?] at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:24) ~[PacketThreadUtil$1.class:?] at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) ~[?:1.8.0_51] at java.util.concurrent.FutureTask.run(Unknown Source) ~[?:1.8.0_51] at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:709) ~[FMLCommonHandler.class:?] ... 5 more Don't blame me if i always ask for your help. I just want to learn to be better
September 26, 201510 yr seriously u should start to learn to look at vanilla and learn from it. look how vanilla renders the player inventory maybe and then add 4 slots?
September 27, 201510 yr Author I've looked into vanilla inventory container class, so now the extra slot works as normal armor slots The only problem is that when i click in it it gives an outOfBoundException, and also the item disappear (but when reopening the GUI is in inventory) So this is the Container class right now package blaze.container; import net.minecraft.entity.player.EntityPlayer; import net.minecraft.entity.player.InventoryPlayer; import net.minecraft.init.Blocks; import net.minecraft.inventory.Container; import net.minecraft.inventory.IInventory; import net.minecraft.inventory.InventoryCraftResult; import net.minecraft.inventory.InventoryCrafting; import net.minecraft.inventory.Slot; import net.minecraft.inventory.SlotCrafting; import net.minecraft.item.ItemArmor; import net.minecraft.item.ItemStack; import net.minecraft.item.crafting.CraftingManager; import net.minecraft.util.BlockPos; import net.minecraft.world.World; import net.minecraftforge.fml.relauncher.Side; import net.minecraftforge.fml.relauncher.SideOnly; public class ContainerNewWorkbench extends Container { /** The crafting matrix inventory (3x3). */ public InventoryCrafting craftMatrix = new InventoryCrafting(this, 3, 3); public IInventory craftResult = new InventoryCraftResult(); private World worldObj; private BlockPos pos; public ContainerNewWorkbench(final InventoryPlayer playerInventory, World worldIn, BlockPos position) { this.worldObj = worldIn; this.pos = position; this.addSlotToContainer(new SlotCrafting(playerInventory.player, this.craftMatrix, this.craftResult, 0, 124, 35)); int i; int j; for (i = 0; i < 3; ++i) { for (j = 0; j < 3; ++j) { this.addSlotToContainer(new Slot(this.craftMatrix, j + i * 3, 30 + j * 18, 17 + i * 18)); } } for (i = 0; i < 3; ++i) { for (j = 0; j < 9; ++j) { this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, 8 + j * 18, 84 + i * 18)); } } for (i = 0; i < 9; ++i) { this.addSlotToContainer(new Slot(playerInventory, i, 8 + i * 18, 142)); } for(i = 0; i < 4; i++) { final int k = i; this.addSlotToContainer(new Slot(playerInventory, playerInventory.getSizeInventory() - 1 - i, 8 + 9 * 18, 84 + i * 18) { /** * Returns the maximum stack size for a given slot (usually the same as getInventoryStackLimit(), but 1 * in the case of armor slots) */ public int getSlotStackLimit() { return 1; } /** * Check if the stack is a valid item for this slot. Always true beside for the armor slots. */ public boolean isItemValid(ItemStack stack) { if (stack == null) return false; return stack.getItem().isValidArmor(stack, k, playerInventory.player); } @SideOnly(Side.CLIENT) public String getSlotTexture() { return ItemArmor.EMPTY_SLOT_NAMES[k]; } }); } this.onCraftMatrixChanged(this.craftMatrix); } /** * Callback for when the crafting matrix is changed. */ public void onCraftMatrixChanged(IInventory inventoryIn) { this.craftResult.setInventorySlotContents(0, CraftingManager.getInstance().findMatchingRecipe(this.craftMatrix, this.worldObj)); } /** * Called when the container is closed. */ public void onContainerClosed(EntityPlayer playerIn) { super.onContainerClosed(playerIn); if (!this.worldObj.isRemote) { for (int i = 0; i < 9; ++i) { ItemStack itemstack = this.craftMatrix.getStackInSlotOnClosing(i); if (itemstack != null) { playerIn.dropPlayerItemWithRandomChoice(itemstack, false); } } } } public boolean canInteractWith(EntityPlayer playerIn) { return this.worldObj.getBlockState(this.pos).getBlock() != Blocks.crafting_table ? false : playerIn.getDistanceSq((double)this.pos.getX() + 0.5D, (double)this.pos.getY() + 0.5D, (double)this.pos.getZ() + 0.5D) <= 64.0D; } /** * Take a stack from the specified inventory slot. */ public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) { ItemStack itemstack = null; Slot slot = (Slot)this.inventorySlots.get(index); if (slot != null && slot.getHasStack()) { ItemStack itemstack1 = slot.getStack(); itemstack = itemstack1.copy(); if (index == 0) { if (!this.mergeItemStack(itemstack1, 10, 46, true)) { return null; } slot.onSlotChange(itemstack1, itemstack); } else if (index >= 10 && index < 37) { if (!this.mergeItemStack(itemstack1, 37, 46, false)) { return null; } } else if (index >= 37 && index < 46) { if (!this.mergeItemStack(itemstack1, 10, 37, false)) { return null; } } else if (!this.mergeItemStack(itemstack1, 10, 46, false)) { return null; } else if (itemstack.getItem() instanceof ItemArmor && !((Slot)this.inventorySlots.get(36 + ((ItemArmor)itemstack.getItem()).armorType)).getHasStack()) { int j = 36 + ((ItemArmor)itemstack.getItem()).armorType; if (!this.mergeItemStack(itemstack1, j, j + 1, false)) { return null; } } if (itemstack1.stackSize == 0) { slot.putStack((ItemStack)null); } else { slot.onSlotChanged(); } if (itemstack1.stackSize == itemstack.stackSize) { return null; } slot.onPickupFromSlot(playerIn, itemstack1); } return itemstack; } /** * Called to determine if the current slot is valid for the stack merging (double-click) code. The stack passed in * is null for the initial slot that was double-clicked. */ public boolean canMergeSlot(ItemStack itemstack, Slot slot) { return slot.inventory != this.craftResult && super.canMergeSlot(itemstack, slot); } } And this is the behavior described above https://www.youtube.com/watch?v=X8t39b5utVI and this is the error that it gives in eclipse [15:16:38] [server thread/FATAL] [FML]: Exception caught executing FutureTask: java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 46, Size: 46 java.util.concurrent.ExecutionException: java.lang.IndexOutOfBoundsException: Index: 46, Size: 46 at java.util.concurrent.FutureTask.report(Unknown Source) ~[?:1.8.0_51] at java.util.concurrent.FutureTask.get(Unknown Source) ~[?:1.8.0_51] at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:710) [FMLCommonHandler.class:?] at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726) [MinecraftServer.class:?] at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:669) [MinecraftServer.class:?] at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:171) [integratedServer.class:?] at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:540) [MinecraftServer.class:?] at java.lang.Thread.run(Unknown Source) [?:1.8.0_51] Caused by: java.lang.IndexOutOfBoundsException: Index: 46, Size: 46 at java.util.ArrayList.rangeCheck(Unknown Source) ~[?:1.8.0_51] at java.util.ArrayList.get(Unknown Source) ~[?:1.8.0_51] at net.minecraft.inventory.Container.slotClick(Container.java:473) ~[Container.class:?] at net.minecraft.network.NetHandlerPlayServer.processClickWindow(NetHandlerPlayServer.java:1039) ~[NetHandlerPlayServer.class:?] at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:46) ~[C0EPacketClickWindow.class:?] at net.minecraft.network.play.client.C0EPacketClickWindow.processPacket(C0EPacketClickWindow.java:110) ~[C0EPacketClickWindow.class:?] at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:24) ~[PacketThreadUtil$1.class:?] at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) ~[?:1.8.0_51] at java.util.concurrent.FutureTask.run(Unknown Source) ~[?:1.8.0_51] at net.minecraftforge.fml.common.FMLCommonHandler.callFuture(FMLCommonHandler.java:709) ~[FMLCommonHandler.class:?] ... 5 more Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr Author Any idea? I really don't understand why it gives me the out of bound exception since the inv size is greater than the selected slot Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr GuiOpenEvent is fired Client-Only. Which means If you want to open a different Container u need to send a packet and let the server open the gui (because the server will inform the client)
September 28, 201510 yr Author How can i send that packet? I'm asking you this since it's the first time i work with GUI and events in general The GUI works right now, except for the 4 extra slots, they accept only respective armor items but i can't pick up anything from that slots, it disappear and it gives me that error. It's like that slots doesn't exist for the game Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr Author Thanks I'll check it Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr Author I've tried doing this in the main mod init method NetworkRegistry.INSTANCE.registerGuiHandler(this, new GUINewWorkbenchHandler()); With this handler package blaze.gui; import blaze.container.ContainerNewWorkbench; import net.minecraft.entity.player.EntityPlayer; import net.minecraft.util.BlockPos; import net.minecraft.world.World; import net.minecraftforge.fml.common.network.IGuiHandler; public class GUINewWorkbenchHandler implements IGuiHandler { //returns an instance of the Container you made earlier @Override public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) { System.out.println("SERVER: " + id); return new ContainerNewWorkbench(player.inventory, world, new BlockPos(x, y , z)); } //returns an instance of the Gui you made earlier @Override public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) { System.out.println("CLIENT: " + id); return new GUINewWorkbench(player.inventory, world, new BlockPos(x, y , z)); } } But it doesn't work Also the handler code is never called Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr Author I'm opening the GUI from this Event Handler @SubscribeEvent public void onEvent(GuiOpenEvent event) { if(event.gui instanceof GuiCrafting) { event.gui = new GUINewWorkbench(Minecraft.getMinecraft().thePlayer.inventory, Minecraft.getMinecraft().theWorld); } } So basically right-clicking on the crafting table will open my custom GUI (and it actually does). This could be the reason why the GUIHandler is never called i guess Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr GuiOpenEvent is fired Client-Only. Which means If you want to open a different Container u need to send a packet and let the server open the gui (because the server will inform the client)
September 28, 201510 yr Author So i need to do this? BL.network.sendToServer(new NetworkMessage("craftingTable")); Where NetworkMessage is my IMessage class package blaze.proxy; import io.netty.buffer.ByteBuf; import net.minecraft.util.IThreadListener; import net.minecraft.world.WorldServer; import net.minecraftforge.fml.common.network.ByteBufUtils; import net.minecraftforge.fml.common.network.simpleimpl.IMessage; import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler; import net.minecraftforge.fml.common.network.simpleimpl.MessageContext; public class NetworkMessage implements IMessage { private String text; public NetworkMessage() { } public NetworkMessage(String text) { this.text = text; } @Override public void fromBytes(ByteBuf buf) { text = ByteBufUtils.readUTF8String(buf); } @Override public void toBytes(ByteBuf buf) { ByteBufUtils.writeUTF8String(buf, text); } public static class Handler implements IMessageHandler<NetworkMessage, IMessage> { @Override public IMessage onMessage(final NetworkMessage message, final MessageContext ctx) { IThreadListener mainThread = (WorldServer) ctx.getServerHandler().playerEntity.worldObj; mainThread.addScheduledTask(new Runnable() { @Override public void run() { System.out.println(String.format("Received %s from %s", message.text, ctx.getServerHandler().playerEntity.getDisplayName())); } }); return null; } } } and it's registered like this network = NetworkRegistry.INSTANCE.newSimpleChannel("BL_Channel"); network.registerMessage(NetworkMessage.Handler.class, NetworkMessage.class, 0, Side.SERVER); I've tried this but nothing changed EDIT: in the EventReceiver class @SubscribeEvent public void onEvent(GuiOpenEvent event) { if(event.gui instanceof GuiCrafting) { BL.network.sendToServer(new NetworkMessage("craftingTable")); event.gui = new GUINewWorkbench(Minecraft.getMinecraft().thePlayer.inventory, Minecraft.getMinecraft().theWorld); } } Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr Author The tutorial says: call NetworkRegistry.INSTANCE.newSimpleChannel("myChannel"). Store the result in a static field in your Mod class. DONE register your Packet, call registerMessage(MyMessageHandler.class, MyMessage.class, packetID, receivingSide) on the wrapper you obtained. DONE Implementing the packet class and the packet handler. DONE, it's the same class of the tutorial send IMessage instances to various targets. DONE. If i have a println in the run method it prints The only thing i really don't understand is this The IMessageHandler simply requires one method, onMessage. Do whatever you want your packet to do in this method. Since i've never worked before with packets and all this stuff i don't know what to place in that method. A new instance of the container? A new one of the GUIHandler? I don't really know Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr show thje whole thing where u send the package. i guess its inside ur event handler. have u registered the event handler? show that
September 28, 201510 yr Author Main mod file (where i register the packet handler) public static SimpleNetworkWrapper network; @EventHandler public void preInit(FMLPreInitializationEvent event) { network = NetworkRegistry.INSTANCE.newSimpleChannel("BL_Channel"); network.registerMessage(NetworkMessage.Handler.class, NetworkMessage.class, 0, Side.SERVER); ... } In the ClientProxy MinecraftForge.EVENT_BUS.register(new EventReceiver()); The NetworkMessage class (with Handler inner class) package blaze.proxy; import io.netty.buffer.ByteBuf; import net.minecraft.util.IThreadListener; import net.minecraft.world.WorldServer; import net.minecraftforge.fml.common.network.ByteBufUtils; import net.minecraftforge.fml.common.network.simpleimpl.IMessage; import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler; import net.minecraftforge.fml.common.network.simpleimpl.MessageContext; public class NetworkMessage implements IMessage { private String text; public NetworkMessage() { } public NetworkMessage(String text) { this.text = text; } @Override public void fromBytes(ByteBuf buf) { text = ByteBufUtils.readUTF8String(buf); } @Override public void toBytes(ByteBuf buf) { ByteBufUtils.writeUTF8String(buf, text); } public static class Handler implements IMessageHandler<NetworkMessage, IMessage> { @Override public IMessage onMessage(final NetworkMessage message, final MessageContext ctx) { IThreadListener mainThread = (WorldServer) ctx.getServerHandler().playerEntity.worldObj; mainThread.addScheduledTask(new Runnable() { @Override public void run() { //TODO What's going on here?? } }); return null; } } } The container class package blaze.gui.container; import net.minecraft.entity.player.EntityPlayer; import net.minecraft.entity.player.InventoryPlayer; import net.minecraft.init.Blocks; import net.minecraft.inventory.Container; import net.minecraft.inventory.IInventory; import net.minecraft.inventory.InventoryCraftResult; import net.minecraft.inventory.InventoryCrafting; import net.minecraft.inventory.Slot; import net.minecraft.inventory.SlotCrafting; import net.minecraft.item.ItemArmor; import net.minecraft.item.ItemStack; import net.minecraft.item.crafting.CraftingManager; import net.minecraft.util.BlockPos; import net.minecraft.world.World; import net.minecraftforge.fml.relauncher.Side; import net.minecraftforge.fml.relauncher.SideOnly; public class ContainerNewWorkbench extends Container { /** The crafting matrix inventory (3x3). */ public InventoryCrafting craftMatrix = new InventoryCrafting(this, 3, 3); public IInventory craftResult = new InventoryCraftResult(); private World worldObj; private BlockPos pos; public ContainerNewWorkbench(final InventoryPlayer playerInventory, World worldIn, BlockPos position) { this.worldObj = worldIn; this.pos = position; this.addSlotToContainer(new SlotCrafting(playerInventory.player, this.craftMatrix, this.craftResult, 0, 124, 35)); int i; int j; /** * SLOT 0-8 */ for (i = 0; i < 3; ++i) { for (j = 0; j < 3; ++j) { this.addSlotToContainer(new Slot(this.craftMatrix, j + i * 3, 30 + j * 18, 17 + i * 18)); } } /** * SLOT 9-35 */ for (i = 0; i < 3; ++i) { for (j = 0; j < 9; ++j) { this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, 8 + j * 18, 84 + i * 18)); } } /** * SLOT 36-44 */ for (i = 0; i < 9; ++i) { this.addSlotToContainer(new Slot(playerInventory, i, 8 + i * 18, 142)); } /** * 45-48 */ for(i = 0; i < 4; i++) { final int k = i; this.addSlotToContainer(new Slot(playerInventory, playerInventory.getSizeInventory() - 1 - i, 8 + 9 * 18, 84 + i * 18) { /** * Returns the maximum stack size for a given slot (usually the same as getInventoryStackLimit(), but 1 * in the case of armor slots) */ public int getSlotStackLimit() { return 1; } /** * Check if the stack is a valid item for this slot. Always true beside for the armor slots. */ public boolean isItemValid(ItemStack stack) { if (stack == null) return false; return stack.getItem().isValidArmor(stack, k, playerInventory.player); } @SideOnly(Side.CLIENT) public String getSlotTexture() { return ItemArmor.EMPTY_SLOT_NAMES[k]; } }); } this.onCraftMatrixChanged(this.craftMatrix); } /** * Callback for when the crafting matrix is changed. */ public void onCraftMatrixChanged(IInventory inventoryIn) { this.craftResult.setInventorySlotContents(0, CraftingManager.getInstance().findMatchingRecipe(this.craftMatrix, this.worldObj)); } /** * Called when the container is closed. */ public void onContainerClosed(EntityPlayer playerIn) { super.onContainerClosed(playerIn); if (!this.worldObj.isRemote) { for (int i = 0; i < 9; ++i) { ItemStack itemstack = this.craftMatrix.getStackInSlotOnClosing(i); if (itemstack != null) { playerIn.dropPlayerItemWithRandomChoice(itemstack, false); } } } } public boolean canInteractWith(EntityPlayer playerIn) { return this.worldObj.getBlockState(this.pos).getBlock() != Blocks.crafting_table ? false : playerIn.getDistanceSq((double)this.pos.getX() + 0.5D, (double)this.pos.getY() + 0.5D, (double)this.pos.getZ() + 0.5D) <= 64.0D; } /** * Take a stack from the specified inventory slot. */ public ItemStack transferStackInSlot(EntityPlayer playerIn, int index) { ItemStack itemstack = null; Slot slot = (Slot)this.inventorySlots.get(index); if (slot != null && slot.getHasStack()) { ItemStack itemstack1 = slot.getStack(); itemstack = itemstack1.copy(); /** * FROM THE CRAFT RESULT SLOT */ if (index == 0) { if (!this.mergeItemStack(itemstack1, 10, 46, true)) { return null; } slot.onSlotChange(itemstack1, itemstack); } /** * FROM THE INV */ else if (index >= 10 && index < 37) { System.out.println("INV"); if (!this.mergeItemStack(itemstack1, 37, 46, false)) { return null; } } /** * FROM THE HOTBAR */ else if (index >= 37 && index < 46) { System.out.println("HOTBAR"); if (!this.mergeItemStack(itemstack1, 10, 37, false)) { return null; } } /** * FROM THE ARMOR */ else if ((index >= 47 && index < 50) && itemstack.getItem() instanceof ItemArmor && !((Slot)this.inventorySlots.get(47 + ((ItemArmor)itemstack.getItem()).armorType)).getHasStack()) { System.out.println("ARMOR"); int j = 47 + ((ItemArmor)itemstack.getItem()).armorType; if (!this.mergeItemStack(itemstack1, j, j + 1, false)) { return null; } } /** * FROM CRAFTING GRID */ else if (!this.mergeItemStack(itemstack1, 10, 46, false)) { System.out.println("GRID!"); return null; } if (itemstack1.stackSize == 0) { slot.putStack((ItemStack)null); } else { slot.onSlotChanged(); } if (itemstack1.stackSize == itemstack.stackSize) { return null; } slot.onPickupFromSlot(playerIn, itemstack1); } return itemstack; } /** * Called to determine if the current slot is valid for the stack merging (double-click) code. The stack passed in * is null for the initial slot that was double-clicked. */ public boolean canMergeSlot(ItemStack itemstack, Slot slot) { return slot.inventory != this.craftResult && super.canMergeSlot(itemstack, slot); } } The GUI Class package blaze.gui; import blaze.gui.container.ContainerNewWorkbench; import net.minecraft.client.gui.inventory.GuiContainer; import net.minecraft.client.renderer.GlStateManager; import net.minecraft.client.resources.I18n; import net.minecraft.entity.player.InventoryPlayer; import net.minecraft.util.BlockPos; import net.minecraft.util.ResourceLocation; import net.minecraft.world.World; import net.minecraftforge.fml.relauncher.Side; import net.minecraftforge.fml.relauncher.SideOnly; @SideOnly(Side.CLIENT) public class GUINewWorkbench extends GuiContainer { private static final ResourceLocation craftingTableGuiTextures = new ResourceLocation("blaze:textures/gui/container/crafting_table.png"); public GUINewWorkbench(InventoryPlayer playerInv, World worldIn) { this(playerInv, worldIn, BlockPos.ORIGIN); } public GUINewWorkbench(InventoryPlayer playerInv, World worldIn, BlockPos blockPosition) { super(new ContainerNewWorkbench(playerInv, worldIn, blockPosition)); } /** * Draw the foreground layer for the GuiContainer (everything in front of the items). Args : mouseX, mouseY */ protected void drawGuiContainerForegroundLayer(int mouseX, int mouseY) { this.fontRendererObj.drawString(I18n.format("container.crafting", new Object[0]), 28, 6, 4210752); this.fontRendererObj.drawString(I18n.format("container.inventory", new Object[0]), 8, this.ySize - 96 + 2, 4210752); } /** * Args : renderPartialTicks, mouseX, mouseY */ protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY) { GlStateManager.color(1.0F, 1.0F, 1.0F, 1.0F); this.mc.getTextureManager().bindTexture(craftingTableGuiTextures); int k = (this.width - this.xSize) / 2; int l = (this.height - this.ySize) / 2; this.drawTexturedModalRect(k, l, 0, 0, 194, this.ySize); } } The GUIHandler Class package blaze.gui; import blaze.gui.container.ContainerNewWorkbench; import net.minecraft.entity.player.EntityPlayer; import net.minecraft.util.BlockPos; import net.minecraft.world.World; import net.minecraftforge.fml.common.network.IGuiHandler; public class GUINewWorkbenchHandler implements IGuiHandler { public GUINewWorkbenchHandler() { // TODO Auto-generated constructor stub } //returns an instance of the Container you made earlier @Override public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) { System.out.println("SERVER: " + id); return new ContainerNewWorkbench(player.inventory, world, new BlockPos(x, y , z)); } //returns an instance of the Gui you made earlier @Override public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z) { System.out.println("CLIENT: " + id); return new GUINewWorkbench(player.inventory, world, new BlockPos(x, y , z)); } } If i missed something please tell me and i will provide any missing class EDIT: in the EventReceiver class @SubscribeEvent public void onEvent(GuiOpenEvent event) { if(event.gui instanceof GuiCrafting) { BL.network.sendToServer(new NetworkMessage("craftingTable")); event.gui = new GUINewWorkbench(Minecraft.getMinecraft().thePlayer.inventory, Minecraft.getMinecraft().theWorld); } } Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr Author Yes, i'm calling from the init method by doing this @EventHandler public void init(FMLInitializationEvent event) { if(event.getSide() == Side.CLIENT) { RenderItem renderItem = Minecraft.getMinecraft().getRenderItem(); .... proxy.registerRenderThings(renderItem); } } Don't blame me if i always ask for your help. I just want to learn to be better
September 28, 201510 yr u should consider reading a tutorial about the proxies and why u need to use them. the simple reason to use proxies is to NOT use event.getSide()
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.