Jump to content

Items Models/Textures not correctly matched


ARuther

Recommended Posts

Hey, I'm trying to teach myself Forge, and I've been doing my best to follow Shadowfacts' tutorial  and change what I need to to work in 1.12, but I'm running into some issues. I've followed the tutorial along to adding in Corn, but my items are either being given the wrong texture (eg. Copper Ingot with Corn texture), or no texture at all. This affects both Items and ItemBlocks, though Blocks always have the correct texture once placed in the environment. If I don't assign a renderer to my Items, then my ItemBlocks work just fine, so I think the problem lies there.

Here are the files I think are relevant, I can post whatever I need to if asked, though, or I can make a git. I'd also appreciate any advice on other tutorials worth looking into, especially those that have been updated for 1.12.

 

@Mod.EventBusSubscriber
public class ModItems {

	public static List<Item> itemsToReg;
		
	public static ItemBase ingotCopper;
	public static ItemCornSeed cornSeed;
	public static ItemBase corn;
	
	public static void init() {
	}
	
	@SubscribeEvent
	public static void registerItems(RegistryEvent.Register<Item> event) {
		
		itemsToReg = new ArrayList<Item>();
		ingotCopper = register(new ItemBase("ingot_copper").setCreativeTab(CreativeTabs.MATERIALS));
		cornSeed = register(new ItemCornSeed());
		corn = register(new ItemBase("corn").setCreativeTab(CreativeTabs.FOOD));
		
	    event.getRegistry().registerAll(itemsToReg.toArray(new Item[0]));
	}

	private static <T extends Item> T register(T item) {

		itemsToReg.add(item);
		
		if (item instanceof ItemModelProvider) {
			((ItemModelProvider)item).registerItemModel(item);
		}

		return item;
	}

}
public class ItemBase extends Item implements ItemModelProvider {

	protected String name;

	public ItemBase(String name) {
		this.name = name;
		setUnlocalizedName(name);
		setRegistryName(name);
	}
	
	@Override
	public void registerItemModel(Item item) {
		TestMod.proxy.registerItemRenderer(item, 0, name);
	}

	@Override
	public ItemBase setCreativeTab(CreativeTabs tab) {
		super.setCreativeTab(tab);
		return this;
	}

}
public class ItemCornSeed extends ItemSeeds implements ItemModelProvider {

	public ItemCornSeed() {
		super(ModBlocks.cropCorn, Blocks.FARMLAND);
		setUnlocalizedName("corn_seed");
		setRegistryName("corn_seed");
	}

	@Override
	public void registerItemModel(Item item) {
		TestMod.proxy.registerItemRenderer(item, 0, "corn_seed");
	}
}
public interface ItemModelProvider {

	void registerItemModel(Item item);

}
@Mod.EventBusSubscriber
public class ModBlocks {

	public static List<Block> blocksToReg;
	public static List<Item> itemBlocksToReg;
	
	public static BlockOre oreCopper;
	public static BlockCropCorn cropCorn;
	
	public static void init() {
	}
	
	@SubscribeEvent
	public static void registerItems(RegistryEvent.Register<Item> event) {
	    event.getRegistry().registerAll(itemBlocksToReg.toArray(new Item[0]));
	}
	
	@SubscribeEvent
	public static void registerBlocks(RegistryEvent.Register<Block> event) {

		blocksToReg = new ArrayList<Block>();
		itemBlocksToReg = new ArrayList<Item>();
		oreCopper = register(new BlockOre("ore_copper").setCreativeTab(CreativeTabs.MISC));
		cropCorn = register(new BlockCropCorn(), null);
		
	    event.getRegistry().registerAll(blocksToReg.toArray(new Block[0]));
	    
	}

	private static <T extends Block> T register(T block, ItemBlock itemBlock) {

		blocksToReg.add(block);
		if (itemBlock != null) {
			itemBlocksToReg.add(itemBlock);
		}
		
		if (block instanceof ItemModelProvider) {
			((ItemModelProvider)block).registerItemModel(itemBlock);
		}

		return block;
	}

	private static <T extends Block> T register(T block) {
		ItemBlock itemBlock = new ItemBlock(block);
		itemBlock.setRegistryName(block.getRegistryName());
		return register(block, itemBlock);
	}

}


 

Link to comment
Share on other sites

The error lies within your registerItemRenderer method of your client proxy (probably), which you did not include.

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

That is a very good point, my bad. I believe this is written exactly as in the tutorial, but perhaps things changed in 1.12

 

public class ClientProxy extends CommonProxy {
	
	@Override
	public void registerItemRenderer(Item item, int meta, String id) {
		ModelLoader.setCustomModelResourceLocation(item, meta, new ModelResourceLocation(Variables.MOD_ID + ":" + id, "inventory"));
	}

}

 

 

And here's CommonProxy for good measure

 

public class CommonProxy {
	

	@Mod.EventHandler
	public void preInit(FMLPreInitializationEvent event) {
	}

	@Mod.EventHandler
	public void init(FMLInitializationEvent event) {
		
	}

	@Mod.EventHandler
	public void postInit(FMLPostInitializationEvent event) {

	}
	
	public void registerItemRenderer(Item item, int meta, String id) {
		
	}
	
}

 

Link to comment
Share on other sites

Oh.

I found your problem.

		ingotCopper = register(new ItemBase("ingot_copper").setCreativeTab(CreativeTabs.MATERIALS));
		cornSeed = register(new ItemCornSeed());
		corn = register(new ItemBase("corn").setCreativeTab(CreativeTabs.FOOD));

Note what class you instantiate here.

You're creating two ItemBase objects and completely bypass your ItemCornSeed class.

 

That said:

new ModelResourceLocation(Variables.MOD_ID + ":" + id

You should use item.getRegistryName() here instead. Similarly with TestMod.proxy.registerItemRenderer(item, 0, "corn_seed") you only need to pass the item and metadata (and you don't need to override this method at all!).  Additionally, the use of "corn_seed" here instead of "corn" means that you're telling the game to look in one place, when it should be looking in another (another reason to use getRegistryName()).

	public ItemBase(String name) {
		this.name = name;
		setUnlocalizedName(name);
		setRegistryName(name);
	}

You should use setUnlocalizedName(getRegistryName()) instead, to avoid name conflicts with other mods (lang files are not mod unique).

Nitpick:

ItemModelProvider 

You should name the interface IItemModelProvider. The initial "I" indicates that it is an interface, rather than a class and make for cleaner, more readable code.

 

Awesome sauce the power of generics time:

private static <T extends Item> T register(T item)

If you change this signature to <T extends Item & IItemModelProvider> you won't need the instanceof check. ;) Presumably all your items would implement it, wouldn't they?

Edited by Draco18s

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

 

Quote

Note what class you instantiate here.

You're creating two ItemBase objects and completely bypass your ItemCornSeed class.

I'm not sure I follow, I'm assigning those three instantiations to three different objects. How do you mean I'm "bypassing" ItemCornSeed? How would I rewrite this to fix what problem you see?

 

Most of the other things you mentioned are just me following the tutorial. I know about I-prefixes for interfaces, Java is not at all new to me. I can make sense of the rest of your changes, but I'm just trying to follow the tutorial as best as I can until I'm finished with it. Afterwards I'll fix any "best practices" problems. But I appreciate the input.


As far as "corn" vs "corn_seed", those actually are two separate objects, so I don't actually think I'm telling it to look in the wrong place, though I do agree with not passing an explicit name and just using getRegistryName.
 

 

Edited by ARuther
Link to comment
Share on other sites

1 hour ago, ARuther said:

I'm not sure I follow, I'm assigning those three instantiations to three different objects. How do you mean I'm "bypassing" ItemCornSeed? How would I rewrite this to fix what problem you see?

*Squints at*

Oh, you're right. My mistake.

 

Anyway, I can't see the problem then.

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

29 minutes ago, Animefan8888 said:

Try registering your Models in the ModelRegistryEvent.

This fixed things! Thanks so much! Here's what ModItems looked like when it finished in case someone else runs into the same problem.

 

@Mod.EventBusSubscriber
public class ModItems {

	public static List<Item> itemsToReg;
		
	public static ItemBase ingotCopper;
	public static ItemCornSeed cornSeed;
	public static ItemBase corn;
	
	public static void init() {
		itemsToReg = new ArrayList<Item>();
		ingotCopper = register(new ItemBase("ingot_copper").setCreativeTab(CreativeTabs.MATERIALS));
		cornSeed = register(new ItemCornSeed());
		corn = register(new ItemBase("corn").setCreativeTab(CreativeTabs.FOOD));
	}
	
	@SubscribeEvent
	public static void registerItems(RegistryEvent.Register<Item> event) {

		for (Item item : itemsToReg.toArray(new Item[0])) {
			    event.getRegistry().register(item);
		}
	}

	@SubscribeEvent
	public static void registerItemModels(ModelRegistryEvent event) {
		for (Item item : itemsToReg.toArray(new Item[0])) {
			if (item instanceof ItemModelProvider) {
				((ItemModelProvider)item).registerItemModel(item);
			}
		}
	}
	
	private static <T extends Item> T register(T item) {
		itemsToReg.add(item);
		return item;
	}

}

 

Link to comment
Share on other sites

1 minute ago, ARuther said:

This fixed things! Thanks so much! Here's what ModItems looked like when it finished in case someone else runs into the same problem.

No problem. And thanks.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

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.