Jump to content

Gui Help


RoyalReject

Recommended Posts

Quote

public static CommonProxy proxy;

Common proxy makes no sense, proxies exist to separate sided-only code. If your code is common then it goes anywhere else but your proxy.

 

Quote

serverSide = Reference.COMMON_PROXY_CLASS

This makes even less sense. A server proxy either contains noop implementation for client only stuff or references to server-only classes that would crash the client in a similar way how model registration crashes the server. A common proxy can't be your server proxy.

 

Quote

public static final Block collectorMK4 = new CollectorBase(4, cTab);

Don't use static initializers ever. Instantinate your stuff directly in the appropriate registry event.

 

Quote

GameRegistry.registerTileEntity(CollectorMK4Tile.class, Reference.MOD_ID + ":collector_mk1");

Don't use this outdated method of registering your TEs. Use the overload that takes a ResourceLocation instead.

 

Why are your blocks and items declared in two separate places(ObjectHandler and ModBlocks/ModItems)? This makes no sense because they are different objects/instances of the same class where only one is registered.

 

Quote

implements IHasModel

IHasModel is stupid. All Items need models, no exceptions and nothing about model registration requires access to private/protected things. Register your models in the ModelRegistryEvent directly.

 

You are registering your items/blocks twice. Once in the RegistryHandler, once in the ObjectHandler. You also have your Item/Block classes copied in two packages, the projecteplus one and the gameObjs one. Please structure your mod/repository correctly.

 

38 minutes ago, RoyalReject said:

why the gui will not open up

Maybe I am blind here but I can't see any class in your repository that would implement IGuiHandler. The GUI/Container pair won't magically open itself just becaue you've invoked EntityPlayer#openGui. You need a registered IGuiHandler implementation.

  • Thanks 1
Link to comment
Share on other sites

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.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.