Tile Entity GUI Not working

[DuckCreeper]

So I've been learning Tile entities and adding gui to them. I've created a tile entity with a simple gui but when i right click the block. Nothing happens.. Its a fare bit of code. But I will include some. I can post more if needed.

 

Main class:

package com.duckcreeper.steelcraft;


import net.minecraft.init.Items;
import net.minecraft.item.ItemStack;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.Mod.EventHandler;
import net.minecraftforge.fml.common.Mod.Instance;
import net.minecraftforge.fml.common.SidedProxy;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPostInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPreInitializationEvent;
import net.minecraftforge.fml.common.network.NetworkRegistry;
import net.minecraftforge.fml.common.registry.GameRegistry;

import com.duckcreeper.steelcraft.gui.SCGuiHandler;
import com.duckcreeper.steelcraft.init.SteelAchievements;
import com.duckcreeper.steelcraft.init.SteelCraftBlocks;
import com.duckcreeper.steelcraft.init.SteelCraftItems;
import com.duckcreeper.steelcraft.init.SteelTileEntities;
import com.duckcreeper.steelcraft.proxy.CommonProxy;
import com.duckcreeper.steelcraft.tileentity.SCTileEntities;


@Mod(modid = Reference.MOD_ID, name = Reference.MOD_NAME, version = Reference.VERISION)
public class SteelMain {

@SidedProxy(clientSide = Reference.CLIENT_PROXY_CLASS, serverSide = Reference.SERVER_PROXY_CLASS)
public static CommonProxy proxy;

public static final SteelTab tabSteel = new SteelTab("tabSteel");

    @Instance
    public static SteelMain instance = new SteelMain();

@EventHandler
public void preInit(FMLPreInitializationEvent event)
{
	SteelCraftBlocks.init();
	SteelCraftBlocks.register();
	SteelCraftItems.init();
	SteelCraftItems.register();	

	/** Achievement Registering */
	SteelAchievements.loadAchievements();
	SteelAchievements.registerPage();

	/** Tile Entites*/
	SCTileEntities.init();

}

@EventHandler
public void init(FMLInitializationEvent event)
{
	proxy.registerRenders();
	GameRegistry.addRecipe(new ItemStack(SteelCraftBlocks.block_turbine, 4),
		"SIS", 
            "IBI",
            "SIS", 'I', Items.iron_ingot, 'S', SteelCraftItems.steel_ingot, 'B', new ItemStack(Items.dye, 1, 15) ); 

	GameRegistry.addRecipe(new ItemStack(SteelCraftItems.duck_bill_float),
		"SIR",
		"SIS",
		" S ", 'S', SteelCraftItems.steel_ingot, 'I', Items.iron_ingot, 'R', new ItemStack(Items.dye, 1, 1) );

	/** GUI Registering*/
	NetworkRegistry.INSTANCE.registerGuiHandler(this, new SCGuiHandler());

	/** TileEntity Registering */
	SteelTileEntities.register();

}
@EventHandler
public void postInit(FMLPostInitializationEvent event)
{

}

}

 

Tile Entity

package com.duckcreeper.steelcraft.tileentity;


import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.ISidedInventory;
import net.minecraft.item.ItemFood;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.nbt.NBTTagList;
import net.minecraft.network.NetworkManager;
import net.minecraft.network.Packet;
import net.minecraft.network.play.server.S35PacketUpdateTileEntity;
import net.minecraft.tileentity.TileEntityLockable;
import net.minecraft.util.ChatComponentText;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.IChatComponent;

import com.duckcreeper.steelcraft.gui.containers.ContainerSCTileEntity;

public class SCTileEntity extends TileEntityLockable implements ISidedInventory
{
private static final int[] slots = new int[] { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14 };

private ItemStack[] inventory = new ItemStack[15];
private String customName;

public int getSizeInventory()
{
	return 15;
}

@Override
public ItemStack getStackInSlot(int par1)
{
	return this.inventory[par1];
}

@Override
public ItemStack decrStackSize(int par1, int par2)
{
	if (this.inventory[par1] != null)
	{
		ItemStack var3;

		if (this.inventory[par1].stackSize <= par2)
		{
			var3 = this.inventory[par1];
			this.inventory[par1] = null;
			this.markDirty();
			return var3;
		}
		var3 = this.inventory[par1].splitStack(par2);

		if (this.inventory[par1].stackSize == 0)
		{
			this.inventory[par1] = null;
		}

		this.markDirty();
		return var3;
	}
	return null;
}

@Override
public ItemStack getStackInSlotOnClosing(int par1)
{
	if (this.inventory[par1] != null)
	{
		ItemStack var2 = this.inventory[par1];
		this.inventory[par1] = null;
		return var2;
	}
	return null;
}

@Override
public void setInventorySlotContents(int par1, ItemStack par2ItemStack)
{
	this.inventory[par1] = par2ItemStack;

	if (par2ItemStack != null && par2ItemStack.stackSize > this.getInventoryStackLimit())
	{
		par2ItemStack.stackSize = this.getInventoryStackLimit();
	}

	this.markDirty();
}

/**
 * Reads a tile entity from NBT.
 */
public void readFromNBT(NBTTagCompound tagCompound)
{
	super.readFromNBT(tagCompound);
	NBTTagList tagList = (NBTTagList) tagCompound.getTag("Items");
	this.inventory = new ItemStack[this.getSizeInventory()];

	for (int count = 0; count < tagList.tagCount(); ++count)
	{
		NBTTagCompound nbt = (NBTTagCompound) tagList.getCompoundTagAt(count);
		int slot = nbt.getByte("Slot") & 255;

		if (slot >= 0 && slot < this.inventory.length)
		{
			this.inventory[slot] = ItemStack.loadItemStackFromNBT(nbt);
		}
	}

	if (tagCompound.hasKey("CustomName", )
	{
		this.customName = tagCompound.getString("CustomName");
	}
}

@Override
public void writeToNBT(NBTTagCompound par1NBTTagCompound)
{
	super.writeToNBT(par1NBTTagCompound);
	NBTTagList var2 = new NBTTagList();

	for (int var3 = 0; var3 < this.inventory.length; ++var3)
	{
		if (this.inventory[var3] != null)
		{
			NBTTagCompound var4 = new NBTTagCompound();
			var4.setByte("Slot", (byte) var3);
			this.inventory[var3].writeToNBT(var4);
			var2.appendTag(var4);
		}
	}

	par1NBTTagCompound.setTag("Items", var2);
}

@Override
public void onDataPacket(NetworkManager net, S35PacketUpdateTileEntity pkt)
{
	NBTTagCompound tagCom = pkt.getNbtCompound();
	this.readFromNBT(tagCom);
}

@Override
public Packet getDescriptionPacket()
{
	NBTTagCompound tagCom = new NBTTagCompound();
	this.writeToNBT(tagCom);
	return new S35PacketUpdateTileEntity(pos, getBlockMetadata(), tagCom);
}

@Override
public int getInventoryStackLimit()
{
	return 64;
}

@Override
public boolean isUseableByPlayer(EntityPlayer par1EntityPlayer)
{
	return this.worldObj.getTileEntity(pos) != this ? false : par1EntityPlayer.getDistanceSq(this.pos.getX() + 0.5D, this.pos.getY() + 0.5D, this.pos.getZ() + 0.5D) <= 64.0D;
}

@Override
public void invalidate()
{
	this.updateContainingBlockInfo();
	super.invalidate();
}

@Override
public boolean isItemValidForSlot(int par1, ItemStack par2ItemStack)
{
	return true;
}

@Override
public String getName()
{
	return this.hasCustomName() ? this.customName : "container.fridge";
}

@Override
public boolean hasCustomName()
{
	return this.customName != null && this.customName.length() > 0;
}

@Override
public IChatComponent getDisplayName()
{
	return new ChatComponentText(getName());
}

@Override
public void openInventory(EntityPlayer playerIn)
{
}

@Override
public void closeInventory(EntityPlayer playerIn)
{
}

@Override
public int getField(int id)
{
	return 0;
}

@Override
public void setField(int id, int value)
{
}

@Override
public int getFieldCount()
{
	return 0;
}

@Override
public void clear()
{
	for (int i = 0; i < inventory.length; i++)
	{
		inventory[i] = null;
	}
}

@Override
public Container createContainer(InventoryPlayer playerInventory, EntityPlayer playerIn) 
{
	return new ContainerSCTileEntity(playerInventory, this);
}

@Override
public String getGuiID() 
{
	return "0";
}

@Override
public int[] getSlotsForFace(EnumFacing side) 
{
	return slots;
}

@Override
public boolean canInsertItem(int index, ItemStack stack, EnumFacing direction) 
{
	return stack.getItem() instanceof ItemFood && !isLocked();
}

@Override
public boolean canExtractItem(int index, ItemStack stack, EnumFacing direction) 
{
	return !isLocked();
}
}

 

Tile Entities Class

package com.duckcreeper.steelcraft.tileentity;

import net.minecraftforge.fml.common.registry.GameRegistry;

public class SCTileEntities {

public static void init(){
	GameRegistry.registerTileEntity(SCTileEntity.class, "test_tile_entity");
}

}

 

Container For Tile Entity

package com.duckcreeper.steelcraft.gui.containers;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.Container;
import net.minecraft.inventory.IInventory;
import net.minecraft.inventory.Slot;
import net.minecraft.item.ItemFood;
import net.minecraft.item.ItemStack;

import com.duckcreeper.steelcraft.gui.GuiSCTileEntity;
import com.duckcreeper.steelcraft.gui.slots.SlotSCTile;
import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

public class ContainerSCTileEntity extends Container
{
private IInventory lowerChestInventory;
private int numRows;

public ContainerSCTileEntity(IInventory playerInventory, IInventory SCTileEntityInventory)
{
	this.lowerChestInventory = playerInventory;
	SCTileEntityInventory.openInventory(null);
	int var4, var5;

	int count = 0;
	for (int i = 0; i < 3; i++)
	{
		for (int j = 0; j < 5; j++)
		{
			this.addSlotToContainer(new SlotSCTile(SCTileEntityInventory, count, j * 18 + 44, i * 18 + 18));
			count++;
		}
	}

	for (int i = 0; i < 3; i++)
	{
		for (int j = 0; j < 9; ++j)
		{
			this.addSlotToContainer(new Slot(playerInventory, j + i * 9 + 9, j * 18 + 8, i * 18 + 85));
		}
	}

	for (int i = 0; i < 9; i++)
	{
		this.addSlotToContainer(new Slot(playerInventory, i, i * 18 + 8, 143));
	}
}

public boolean canInteractWith(EntityPlayer par1EntityPlayer)
{
	return this.lowerChestInventory.isUseableByPlayer(par1EntityPlayer);
}

public ItemStack transferStackInSlot(EntityPlayer par1EntityPlayer, int par2)
{
	ItemStack var3 = null;
	Slot var4 = (Slot) this.inventorySlots.get(par2);

	if (var4 != null && var4.getHasStack())
	{
		ItemStack var5 = var4.getStack();
		var3 = var5.copy();

		if (!(var5.getItem() instanceof ItemFood))
			return null;

		if (par2 < 16)
		{
			if (!this.mergeItemStack(var5, 16, this.inventorySlots.size(), true))
			{
				return null;
			}
		}
		else if (!this.mergeItemStack(var5, 0, 16, false))
		{
			return null;
		}

		if (var5.stackSize == 0)
		{
			var4.putStack((ItemStack) null);
		}
		else
		{
			var4.onSlotChanged();
		}
	}

	return var3;
}

/**
 * Callback for when the crafting gui is closed.
 */
public void onContainerClosed(EntityPlayer par1EntityPlayer)
{
	super.onContainerClosed(par1EntityPlayer);
	this.lowerChestInventory.closeInventory(par1EntityPlayer);
}

public IInventory func_85151_d()
{
	return this.lowerChestInventory;
}
}

[Ernio]

You don't have to do this: (initializing I mean)

    @Instance
    public static SteelMain instance = new SteelMain();

This is useless: (tab[0, 1, ..., n-1] is literally a number from 0 to n-1.)

private static final int[] slots = new int[] { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14 };

 

As to your problem - "we" need Block, Gui and GuiHandler classes.

1st one is basically responsible for calling opening, which should be done with GuiHandler implementation and which should then open Container AND Gui accordingly (to side).

[Draco18s]

You do not have an onBlockActivated method telling the game to open the gui

[DuckCreeper]

GUI Handler

package com.duckcreeper.steelcraft.gui;


import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;
import net.minecraftforge.fml.common.network.IGuiHandler;

import com.duckcreeper.steelcraft.gui.containers.ContainerSCTileEntity;
import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

public class SCGuiHandler implements IGuiHandler
{
@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new ContainerSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
	}
	return null;
}

@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new GuiSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
	}

	return null;
}

}

 

GUI

package com.duckcreeper.steelcraft.gui;

import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.inventory.IInventory;
import net.minecraft.util.ResourceLocation;

import com.duckcreeper.steelcraft.gui.containers.ContainerSCTileEntity;
import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

public class GuiSCTileEntity extends GuiContainer
{
private static final ResourceLocation gui = new ResourceLocation("sc:textures/gui/basic_machine.png");

public GuiSCTileEntity(IInventory playerInv, SCTileEntity te){
	super(new ContainerSCTileEntity(playerInv, te));

this.xSize = 176;
this.ySize = 166;
}

@Override
protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY) {
    }
}

 

Block

package com.duckcreeper.steelcraft.blocks;

import com.duckcreeper.steelcraft.tileentity.SCTileEntity;

import net.minecraft.block.BlockContainer;
import net.minecraft.block.material.Material;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.EnumWorldBlockLayer;
import net.minecraft.world.World;

public class basic_rebar_machine extends BlockContainer{

public basic_rebar_machine(Material materialIn) {
	super(materialIn);
}

@Override
public int getRenderType() {
	return 3;
}

@Override
public boolean isOpaqueCube() {
	return false;
}

@Override
public EnumWorldBlockLayer getBlockLayer() {
	return EnumWorldBlockLayer.CUTOUT;
}

@Override
public TileEntity createNewTileEntity(World worldIn, int meta) {
	return new SCTileEntity();
}

}

 

Hope that helps?

[Ernio]

Well, basically Draco is right (Was that a pure guess Draco?).

 

You need to override onBlockActivated method in your Block class and call player.openGui(id, Mod.instance, x, y, z);

Looking at GuiHandler and everything else - it should work.

[DuckCreeper]

  On 2/4/2016 at 10:22 PM, Ernio said:

Well, basically Draco is right (Was that a pure guess Draco?).

 

You need to override onBlockActivated method in your Block class and call player.openGui(id, Mod.instance, x, y, z);

Looking at GuiHandler and everything else - it should work.

 

		player.openGui(1, SteelMain.instance, world, x, y, z);

 

I tried this but its says that

SteelMain.instance

"Instance" should be changed to int but i dont know what to do lol

[Ernio]

My mistake for giving bad example, your bad for not looking at method.

openGui(instance, id, world, x, y, z);

[DuckCreeper]

LOL I've been playing around and I figured that out after i posted it. Now I'm testing a thing. Didnt work. I have to go. Ill play around more with it

[Draco18s]

  On 2/4/2016 at 10:22 PM, Ernio said:

Well, basically Draco is right (Was that a pure guess Draco?).

 

Not a pure guess. I guessed, looked at his block (labeled container) and it wasn't there.

[lynchiem]

Hello DuckCreeper,

 

I am fairly new to modding, so feel free to beheckle me if my observation is born of ignorance, but I noticed in your GUI handler, you have overridden the drawGuiContainerBackgroundLayer, but you are not doing anything in the body. Which, if I understand things correctly, would mean that the UI would be effectively invisible.

 

If it is any help at all, I have recently been learning in this area too, and my source repository has code that does not much more than enable a block inventory / gui... so it may be of some help as cross-reference in resolving your issues.

 

(Though, as stated, I am very new, so there maybe many glaring errors and bad practices in play in my code.)

 

https://github.com/lynchiem/ClockworkAdventure/blob/master/src/main/java/net/clockworkrocketry/clockworkadventure/gui/GUIPortableChest.java

 

I also noticed that when you register your guiHandler, you are passing in "this" where all the examples I have seen pass in "instance"... though I am not sure this will make any difference, as assuming the instantiation is done without voodoo they are probably equivalent.

 

Sorry I could not be of more help, and best of luck resolving your issue.

 

 

 

 

[DuckCreeper]

Still cant figure it out.....

[lynchiem]

If you are still having trouble and can post your updated code (is it in github or similar where it can easily be browsed by chance?), I would be happy to help you debug.

[DuckCreeper]

https://github.com/DuckCreeper/SteelCraftFiles

 

There ya go

[lynchiem]

Sorry, I haven't had a huge amount of time today, but had a quick look over now and my first thoughts are:

 

Your proxy and guihandler are separate. When I implemented mine, based on examples I had seen, my proxy is also the guihandler.

 

I am not experienced enough with modding to know if this would cause a problem, but figured I would mention it, because perhaps there is a conflict happening between server code and client code.

 

If you look at my code (linked in a post above) you will note that my CommonProxy, used by the server, returns null for the getClientGuiElement, but my ClientProxy overrides that behaviour.

 

I will have more time tomorrow after work, so if you haven't resolved by then, I will take a more thorough look at your code.

[DuckCreeper]

So I've been playing around. Now I get this in my console and my game crashes. I mean now i know its trying to make one. But I dont know what to do.

 

Invalid attempt to create a GUI during mod construction. Use an EventHandler instead
at net.minecraftforge.fml.common.network.NetworkRegistry.registerGuiHandler(NetworkRegistry.java:219)

 

Here is my main class

 @EventHandler
 public void load(FMLInitializationEvent event)
    {
    	NetworkRegistry.INSTANCE.registerGuiHandler(instance, proxy);
            }

 

 

NOTE

 

I changed the part that says

   

NetworkRegistry.INSTANCE.registerGuiHandler(instance, proxy);

 

to

   

NetworkRegistry.INSTANCE.registerGuiHandler(this, proxy);

 

and now my game loads.... and when i right click it... nothing pops up but my hand moves... just like when you open a chest or something. So basically my GUI is not showing ingame now...

 

What do i do!?!?!!?

[Draco18s]

  On 2/7/2016 at 1:34 PM, lynchiem said:

Your proxy and guihandler are separate. When I implemented mine, based on examples I had seen, my proxy is also the guihandler.

 

Ew.

There is no reason to combine them.

[DuckCreeper]

So in the onBlockActivated Method. I'm using

player.openGui(SteelMain.instance, 1, world, pos.getX(), pos.getY(), pos.getZ());

But When looking at the openGui its self. The second parameter is the GuiID for the gui. Here is what its set to in my code. :

    @Override
    public String getGuiID()
    {
        return Reference.SteelCraftBlocks.BASIC_REBAR_MACHINE.getRegistryName() + ":Basic";
    }

 

But what do I do? I'm a bit confused on this. An explanation would be helpful

[DuckCreeper]

So I thought it would be helpful if i posted another update .... Playing around more I finally understand that. But still the GUI wont open. I get that right arm punching animation like when you open a furnace etc. But nothing happens when i do so.... rip

[Draco18s]

  On 2/7/2016 at 9:04 PM, DuckCreeper said:

So in the onBlockActivated Method. I'm using

player.openGui(SteelMain.instance, 1, world, pos.getX(), pos.getY(), pos.getZ());

But When looking at the openGui its self. The second parameter is the GuiID for the gui. Here is what its set to in my code. :

    @Override
    public String getGuiID()
    {
        return Reference.SteelCraftBlocks.BASIC_REBAR_MACHINE.getRegistryName() + ":Basic";
    }

 

But what do I do? I'm a bit confused on this. An explanation would be helpful

 

The gui id is the integer 1 in the opengui call.that other method is badly named as does not do what it is called, nor have you shown what you use it for.

[DuckCreeper]

  On 2/7/2016 at 10:04 PM, Draco18s said:

  Quote

So in the onBlockActivated Method. I'm using

player.openGui(SteelMain.instance, 1, world, pos.getX(), pos.getY(), pos.getZ());

But When looking at the openGui its self. The second parameter is the GuiID for the gui. Here is what its set to in my code. :

    @Override
    public String getGuiID()
    {
        return Reference.SteelCraftBlocks.BASIC_REBAR_MACHINE.getRegistryName() + ":Basic";
    }

 

But what do I do? I'm a bit confused on this. An explanation would be helpful

 

The gui id is the integer 1 in the opengui call.that other method is badly named as does not do what it is called, nor have you shown what you use it for.

 

Draco I have figured that out now. I now need help on figureing out why the GUI wont show ingame... Tell me if code is needed.

[Rexozz]

Try replacing:

 

@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new ContainerSCTileEntity(player.inventory, (SCTileEntity) tile_entity, x, y);
	}
	return null;
}

@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	if (tile_entity instanceof SCTileEntity)
	{
		return new GuiSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
	}

	return null;
}

 

with:

 

@Override
public Object getServerGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	switch(id){
                     case YOURCONTAINERANDGUIID:
                             return new ContainerSCTileEntity(player.inventory, (SCTileEntity) tile_entity, x, y);
                             break;
                }
	return null;
}

@Override
public Object getClientGuiElement(int id, EntityPlayer player, World world, int x, int y, int z)
{
	TileEntity tile_entity = world.getTileEntity(new BlockPos(x, y, z));
	switch(id){
                     case YOURCONTAINERANDGUIID:
                             return new GuiSCTileEntity(player.inventory, (SCTileEntity) tile_entity);
                             break;
                }

	return null;
}

 

PS: sorry for my bad english

[DuckCreeper]

What do you mean by YOURCONTAINERANDGUIID? So ContainerSCTileEntity0: My gui id is 0....

[Rexozz]

Yeah. YOURCONTAINERANDGUIID is the number you used when opening the gui

 

player.openGui(modInstance, ID....);

 

additional info: gui id's should be unique for your mod, don't define them multiple

 

Create a field in your gui handler class:

 

public static final YOUR_GUIID=0;

public static final ANOTHER_GUIID=1;

 

and then access this ids instead of typing a number, so you can't get confused if you need them

 

EDIT: please place the onBlockActivated in your block class and not in your tileentity (maybe you already done that, i've seen it in your sources you linked)

[lynchiem]

  On 2/7/2016 at 5:31 PM, Draco18s said:

  Quote

Your proxy and guihandler are separate. When I implemented mine, based on examples I had seen, my proxy is also the guihandler.

 

Ew.

There is no reason to combine them.

 

Thank you.

 

As I have mentioned, I am very new to modding, and was simply sharing thoughts based on the difference I could identify between my working code and the problematic code.

 

Combined is probably an overstatement, given the GUI related functions are the only ones I am currently handling over riding in the "proxy".

 

I am still struggling to get my head around how the proxy system works. I need to explicitly tell the mod which class to use as the client and server proxies.

 

@SidedProxy(clientSide = "net.clockworkrocketry.clockworkadventure.network.ClientProxy", serverSide = "net.clockworkrocketry.clockworkadventure.network.CommonProxy")

 

If I get rid of this, my GUI stops working.

 

And my proxy is nothing more than in implementation of IGuiHandler at this stage, containing pretty much only the getClientGuiElement and getServerGuiElement functions.

 

So I thought it was worth exploring as a possible breakage point, as I thought maybe the getClientGuiElement and getServerGuiElement functions needed to be "wired" up to the server/client workflow.

 

But obviously this is incorrect.

[Draco18s]

Take a look at my code sometime.

I don't so things the best, objectively. But the class layout is more standard.

 

https://github.com/Draco18s/Artifacts/tree/master/main/java/com/draco18s/artifacts/