[Solved][1.8.9] ModelBakery.addVariantName is deprecated.

[OrangeVillager61]

In 1.8.9 ModelBakery.addVariantName is deprecated. What are the alternatives to this and why did it became deprecated?

 

Thanks.

 

OrangeVillager61

[AntariaN]

Try to use ModelBakery.registerItemVariants(Item item, ModelResourceLocation location) instead of addVariantName

 

You can see the example here.

[OrangeVillager61]

The texture only works when placed, it doesn't work in the inventory.

ModelResourceLocation emptyModel1 = new ModelResourceLocation("advanced_brine_filter" + "_empty", "inventory");
ModelResourceLocation full1Model1 = new ModelResourceLocation("advanced_brine_filter" + "_full", "inventory");

ModelBakery.registerItemVariants(Item.getItemFromBlock(MoDropsBlocks.advanced_brine_filter), emptyModel1, full1Model1);

 

I get the following errors.

Caused by: java.lang.RuntimeException: Encountered an exception when loading model definition of model minecraft:blockstates/advanced_brine_filter_empty.json
at net.minecraft.client.resources.model.ModelBakery.getModelBlockDefinition(ModelBakery.java:165) ~[ModelBakery.class:?]
at net.minecraftforge.client.model.ModelLoader.getModelBlockDefinition(ModelLoader.java:211) ~[ModelLoader.class:?]
... 22 more
Caused by: java.io.FileNotFoundException: minecraft:blockstates/advanced_brine_filter_empty.json
at net.minecraft.client.resources.FallbackResourceManager.getAllResources(FallbackResourceManager.java:93) ~[FallbackResourceManager.class:?]
at net.minecraft.client.resources.SimpleReloadableResourceManager.getAllResources(SimpleReloadableResourceManager.java:78) ~[simpleReloadableResourceManager.class:?]
at net.minecraft.client.resources.model.ModelBakery.getModelBlockDefinition(ModelBakery.java:143) ~[ModelBakery.class:?]
at net.minecraftforge.client.model.ModelLoader.getModelBlockDefinition(ModelLoader.java:211) ~[ModelLoader.class:?]
... 22 more
[16:37:39] [Client thread/ERROR] [FML]: Model definition for location minecraft:advanced_brine_filter_full#inventory not found
[16:37:39] [Client thread/ERROR] [FML]: Model definition for location minecraft:advanced_brine_filter_empty#inventory not found

[Choonster]

You didn't set the domain of your

ModelResourceLocation

s, so it defaulted to

"minecraft"

.

[OrangeVillager61]

You didn't set the domain of your

ModelResourceLocation

s, so it defaulted to

"minecraft"

.

 

*facepalm*

 

Thank you! Last question why was ModelBakery.addVariantName deprecated in the first place?

[Choonster]

Thank you! Last question why was ModelBakery.addVariantName deprecated in the first place?

 

It was deprecated in this commit to allow items to use models specified in blockstates files.