Jump to content

[deleted]


Guest

Recommended Posts

Show your code. 

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

I dont see in your code anywhere where you are actually calling the method RegisterRenders which would hook the modellocation. Are you versed in Java? If not I would highly suggest at least looking into some java crashcourses and not blindly following tutorials. This is supported by the fact that your log doesnt even have any errors for the item model in the first place, as it has no item model to reference to. You are following a 1.11 tutorial in the 1.12 environment that uses a new system for registries. 

 

If you want a tutorial on how to registry items in 1.12 (albeit still messy), check this channel: https://www.youtube.com/channel/UCUAawSqNFBEj-bxguJyJL9g/videos

 

Again if your not familiar with Java please learn it properly instead of blindly following tutorials, otherwise you will not know what your code is doing and why it is doing it and in the end you are just wasting your own time since you woulddnt understand it.

Edited by daruskiy
Link to comment
Share on other sites

This tutorial is not the best, on a couple of notes:

  1. There is not, and never has been, a "CommonProxy". Per definition, a Minecraft-proxy can only be either Client-side only, or Server-Side, the very antonym of "common". "CommonProxy" is a misnomer and based on faulty logic.
  2. Never call ForgeRegistries#<REGISTRY>#register. Use the RegistryEvent's for this. ForgeRegistries is, and I quote the documentation, "...a central place to access registries directly... queries and iterations can use this"
15 hours ago, erose524 said:

and now the item isn't even in the game

Likely due to #2.

 

The code for setting the models, is never actually called. Just because you have the method doesn't mean it's gonna get used. Use the ModelRegistryEvent, and for each of your items with IHasModel, call IHasModel::registerModel.

  • Like 1

Also previously known as eAndPi.

"Pi, is there a station coming up where we can board your train of thought?" -Kronnn

Published Mods: Underworld

Handy links: Vic_'s Forge events Own WIP Tutorials.

Link to comment
Share on other sites

@Subscribe
    public static void onItemRegister(RegistryEvent.Register<Item> event) {
        event.getRegistry().registerAll(ItemInit.ITEMS.toArray(new Item[0]));
    }

 

replace @Subscribe w/ @SubscribeEvent 

 

reference:

@SubscribeEvent
public void registerBlocks(RegistryEvent.Register<Block> event) {
    event.getRegistry().registerAll(block1, block2, ...);
}

Retrieved from: http://mcforge.readthedocs.io/en/latest/concepts/registries/

Edited by daruskiy
  • Like 1
Link to comment
Share on other sites

  • Guest changed the title to [deleted]

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.