(SOLVED) [1.11] Changing Texture/Model file on right click?

[Rohzek]

Is there an easy way to change the texture of an item in say, an overridden onItemRightClick, by changing which layer, or which model file is being loaded?

[Animefan8888]

Is there an easy way to change the texture of an item in say, an overridden onItemRightClick, by changing which layer, or which model file is being loaded?

It depends is it limited? Or can it be anything?

[Rohzek]

It depends is it limited? Or can it be anything?

 

Just the easiest way. Preferably if it doesn't involve just removing the item, and adding in another one.

[Animefan8888]

It depends is it limited? Or can it be anything?

 

Just the easiest way. Preferably if it doesn't involve just removing the item, and adding in another one.

Modifing the metadata of the stack, and assigning the different metadata a different model when registered using ModelLoader.setCustomModelResourceLocation(...)

[Rohzek]

Modifing the metadata of the stack, and assigning the different metadata a different model when registered using ModelLoader.setCustomModelResourceLocation(...)

 

Hmm. That sounded easy until I managed to confuse myself. I was originally using

 

 

public static void registerRenders() 
{
registerRender(testItem);
}

public static void registerRender(Item item)
{
Minecraft.getMinecraft().getRenderItem().getItemModelMesher().register(item, 0, new ModelResourceLocation(item.getRegistryName(), "inventory"));
}

 

 

and it could find my original json file just fine, and it showed in game. I tried changing that to take a string to seperate them, making it:

 

 

 

public static void registerRenders() 
{
registerRender(testItem, 0, "Stage0");
registerRender(testItem, 1, "Stage1");
registerRender(testItem, 2, "Stage2");
}

public static void registerRender(Item item, int meta, String addon)
{
Minecraft.getMinecraft().getRenderItem().getItemModelMesher().register(item, meta, new ModelResourceLocation(item.getRegistryName()+addon, "inventory"));
}

 

 

And that didn't complain about not being able to find my new json files, but it was broken texture in game.

 

After switching that to

 

ModelLoader.setCustomModelResourceLocation(item, meta, new ModelResourceLocation(item.getRegistryName() + addon, "inventory"));

Now it also can't find my json files.

 

After remembering to register my event, same problem as above. It doesn't complain that it can't find the json files... ...it just doesn't load the textures in game.

[Draco18s]

You shouldn't be using the ModelMesher anyway, that method has been outdated for a while.

 

Make sure you're calling ModelLoader.setCustomModelResourceLocation during the ModelRegistryEvent (preferred 1.11 way, although I think during PreInit still works).

[Rohzek]

I seem to be failing to properly register my ModelRegistryEvent, as it seems it doesn't complain because that method never runs?

 

I've tried registering it with the forge event bus in both my Main, and ClientProxy and neither works...

 

...This is probably all wrong?

 

MinecraftForge.EVENT_BUS.register(new LoadModelEvent());

 

LoadModelEvent:

...

@SubscribeEvent
public void onModelRegistry(ModelRegistryEvent event)
{
     testItems.registerRenders();
}

 

TestItems:

...

public static void registerRenders() 
{
registerRender(testItem, 0, "Stage0");
registerRender(testItem, 1, "Stage1");
registerRender(testItem, 2, "Stage2");
}

public static void registerRender(Item item, int meta, String addon)
{	
ModelLoader.setCustomModelResourceLocation(item, meta, new ModelResourceLocation(item.getRegistryName() + addon, "inventory"));

LogHelper.debug("I should be loading the json file for: " + item.getRegistryName() + addon + " and setting metadata: " + meta);
}

[Choonster]

ModelRegistryEvent

is fired before preInit, so it needs to be handled by a static

@SubscribeEvent

method in a class annotated with

@Mod.EventBusSubscriber

.

 

You should also be registering all of your

IForgeRegistryEntry

implementations (e.g.

Item

,

Block

,

SoundEvent

) in response to

RegistryEvent.Register<T>

, where

T

is the registry's type.

 

You can read more about events here.

[Draco18s]

MinecraftForge.EVENT_BUS.register(new LoadModelEvent());

 

When are you calling this?

[Rohzek]

When are you calling this?

 

I was calling it during the preinit of my main class

public static void PreInit(FMLPreInitializationEvent PreEvent)
{
     MinecraftForge.EVENT_BUS.register(new LoadModelEvent());
}

 

I also tried (after reading through Choonster's link) making it static, and making that

 

public static void PreInit(FMLPreInitializationEvent PreEvent)
{
     MinecraftForge.EVENT_BUS.register(LoadModelEvent.class);
}

 

Which also didn't work. I then tried removing that, and using:

 

@Mod.EventBusSubscriber
public class LoadModelEvent 
{
     ...
}

 

Which runs, and gets a nullpointerexception because the items aren't loaded yet...

Attempting to load the items by registering them in the event too instead of normal fixes that, but then they get initialized as being from

minecraft instead of being from my MODID

 

Exception loading model for variant minecraft:testitemstage0#inventory for item "minecraft:testitem", normal location exception: 

 

Sorry, I am 100% thoroughly lost and confused.

[Choonster]

Events that extend

FMLEvent

(e.g.

FMLPreInitializationEvent

) still need to be handled by instance

@Mod.EventHandler

methods in your

@Mod

class.

 

You should instantiate your

Item

s in a static initialiser and register them in response to

RegistryEvent.Register<Item>

. The same applies to all other

IForgeRegistryEntry

implementations.

 

You can see how I do this here and in the other classes in the

init

package.

[Rohzek]

You can see how I do this here and in the other classes in the

init

package.

 

Registering my item this way, worked perfectly. I now have textures in the game, and can cycle between the different textures no problem.

 

Thank you all, so much.