Jump to content

[1.10.2] [SOLVED] getModelLocation called and return correct model but wrong item icon in the end


Recommended Posts

Posted (edited)

ItemMeshDefinition implementation

 

public class MESH implements ItemMeshDefinition {

    @Override
    public ModelResourceLocation getModelLocation(ItemStack itemStack) {
        if (itemStack != null
                && itemStack.hasTagCompound()
                && itemStack.getTagCompound().hasKey(TAG)) {
            return new ModelResourceLocation(ITEM.getRegistryName().toString() + "_" + ITEM.getVar(itemStack.getTagCompound().getString(TAG)));
        }
        return new ModelResourceLocation(ITEM.getRegistryName().toString());
    }
}

 

ItemRenderRegister

 

public class ItemRenderRegister {

    public static void preInit() {

        ResourceLocation[] LOC = new ResourceLocation[LENGTH];
        for (int i = 0; i < LOC.length; i++) {
            LOC[i] = new ResourceLocation(ITEM.getRegistryName().toString() + "_" + ITEM.getVar(i));
        }
        ModelBakery.registerItemVariants(ITEM, LOC);

        ModelLoader.setCustomMeshDefinition(ITEM, new MESH());

        register(ITEM);
    }

    public static void init() {
    }

    private static void register(Item item) {
        register(item, 0, item.getRegistryName().toString());
    }

    private static void register(Item item, int metadata, String file) {
        ModelLoader.setCustomModelResourceLocation(item, metadata, new ModelResourceLocation(file));
    }
}

 

I declare some subtypes in my ITEM class with NBT used to define textures.

With this code, all my ITEM have the default texture from this location :

ITEM.getRegistryName().toString()

Also I put breakpoints in :

getModelLocation

It is never called.

 

If I move

register(ITEM);

from

preInit

to

init

then

getModelLocation 

is called and the resources location are the correct ones but ITEM textures are now all null.

 

I don't have any error in my stack trace.

 

 

Edited by Islandil
Posted

Client Proxy

public class ClientProxy extends CommonProxy {

    @Override
    public void preInit(FMLPreInitializationEvent e) {
        super.preInit(e);

        ItemRenderRegister.preInit();
    }
...
}

 

New ItemRenderRegister preInit

public static void preInit() {
        register(ITEM);

        ResourceLocation[] LOC = new ResourceLocation[LENGTH];
        for (int i = 0; i < LOC.length; i++) {
            LOC[i] = new ResourceLocation(ITEM.getRegistryName().toString() + "_" + ITEM.getVar(i));
        }
        ModelBakery.registerItemVariants(ITEM, LOC);

        ModelLoader.setCustomMeshDefinition(ITEM, new MESH());
    }

 

Same result. All my item have the default texture. No call inside

getModelLocation

 

Posted (edited)

Maybe you are speaking about item register and not about item render register.

My item register is in the preInit of my CommonProxy, which is called before my ClientProxy preInit.

Is this the correct order ?

 

Edit : other way around.

Edited by Islandil
Posted
  On 2/6/2017 at 4:23 PM, Islandil said:

Maybe you are speaking about item register and not about item render register.

My item register is in the preInit of my CommonProxy, which is called after my ClientProxy preInit.

How to execute them in the correct order ?

Expand  

By not being an idiot and swapping two lines of code so they get executed in the correct order.

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.

Posted (edited)
  On 2/6/2017 at 4:25 PM, Draco18s said:

By not being an idiot and swapping two lines of code so they get executed in the correct order.

Expand  

Annnnd I am not ! Because I wrongly placed my breakpoint to test. And the calls are in the correct order.

See my edit.

Edited by Islandil
Posted

Then you're calling things in the correct order.

What's the problem now?

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.

Posted
  On 2/6/2017 at 4:12 PM, Islandil said:

Client Proxy

public class ClientProxy extends CommonProxy {

    @Override
    public void preInit(FMLPreInitializationEvent e) {
        super.preInit(e);

        ItemRenderRegister.preInit();
    }
...
}

 

New ItemRenderRegister preInit

public static void preInit() {
        register(ITEM);

        ResourceLocation[] LOC = new ResourceLocation[LENGTH];
        for (int i = 0; i < LOC.length; i++) {
            LOC[i] = new ResourceLocation(ITEM.getRegistryName().toString() + "_" + ITEM.getVar(i));
        }
        ModelBakery.registerItemVariants(ITEM, LOC);

        ModelLoader.setCustomMeshDefinition(ITEM, new MESH());
    }

 

Same result. All my item have the default texture. No call inside

getModelLocation

 

Expand  

Still the same.

Posted

Ok. My new ItemRenderRegister

public class ItemRenderRegister {

    public static void preInit() {
        ResourceLocation[] LOC = new ResourceLocation[LENGTH];
        for (int i = 0; i < LOC.length; i++) {
            LOC[i] = new ResourceLocation(ITEM.getRegistryName().toString() + "_" + ITEM.getVar(i));
        }
        ModelBakery.registerItemVariants(ITEM, LOC);

        ModelLoader.setCustomMeshDefinition(ITEM, new MESH());
    }
}

I get null model for my icons. Using a breakpoint inside my

getModelLocation

I see the correct models being called and I checked the files names.

 

So my problem is : why do I have no model for my icon in my inventory even if the right model is returned by

getModelLocation

?

Posted

Adding

"inventory"

in

getModelLocation

solved the issue.

return new ModelResourceLocation(ITEM.getRegistryName().toString() + "_" + ITEM.getVar(itemStack.getTagCompound().getString(TAG)), "inventory");

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • My name is Richie Leo, and I’m sharing this message with a heart full of gratitude and hope. Several months ago, I was a victim of a devastating online scam that cost me a staggering $873,463. I was devastated, confused, and had nearly given up on ever recovering my money — until I came across Wizard George Cyber Service. Through their exceptional cyber recovery expertise and deep investigative skills, Wizard George and his team were able to trace, track, and recover the full amount that was stolen from me. Their professionalism, speed, and transparency truly amazed me. If you’re reading this and you’ve been scammed — whether it’s crypto, investment fraud, or any kind of online theft — don’t give up. I strongly recommend reaching out to Wizard George Cyber Service. 📧 Em: wizardgeorgecyberservice(AT) g m a l L. C o M
    • Alright, here is the log file https://mclo.gs/5eCwafV
    • Please read the FAQ (https://forums.minecraftforge.net/topic/125488-rules-and-frequently-asked-questions-faq/) and post log files as described there, using a site such as https://mclo.gs/ and post the link here.  
    • I tried updating the mods in my modpack which caused incompatibilities so i have tried to revert them back to their older versions i was using before. In the logs it doesnt show me any clear incompatibilities except for tfmg & entity texture features, but when i try to remove those it still doesn't work. I have tried removing the forge-client.toml file which was a suggestion i found on  a few other posts. This is the log file i get. [inline log removed] Any help would be appreciated. Thanks in advance
    • I don't use KubeJS, never even heard of it. But after doing what "Ugdhar" suggested earlier in this post with the "config/Mekanism/generator-storage.toml", I tried going into an individual save's serverconfig folder, and just deleting everything except the parcool folder (I have that mod installed.) Then, a bit of loading and temporary freezing later, seems to have worked. Even when quitting to menu and loading back in, or also when quitting to menu, exiting to desktop, and re-launching MC, choose a save and loading it.
  • Topics

  • Who's Online (See full list)

    • There are no registered users currently online
×
×
  • Create New...

Important Information

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