Jump to content

Why Isn't my texture working?


darkenmoonz

Recommended Posts

Hello! I haven't modded since 1.6.4 and I understand a lot has changed since then, I'm following some basic tutorials to get myself acquainted but I've hit a roadblock. I can't understand how to debug this, my custom item isn't showing any texture (besides the purple/black cube) and I've checked every place I thought the issue would be. I'm probably overlooking something simple, thanks in advance for any help. Below is my code from my item related files, reply if I need to add one for it to make sense.

 

 

ModItems.java

Spoiler

package com.darkenmoonz.designedexpansionmod.init;

import java.util.ArrayList;
import java.util.List;

import com.darkenmoonz.designedexpansionmod.items.ItemBase;

import net.minecraft.item.Item;

public class ModItems
{

	public static final List<Item> ITEMS = new ArrayList<Item>();
	
	public static final Item RUBY = new ItemBase("ruby");
}

 

RegistryHandler.java

Spoiler

package com.darkenmoonz.designedexpansionmod.util.handlers;

import com.darkenmoonz.designedexpansionmod.init.ModItems;
import com.darkenmoonz.designedexpansionmod.util.IHasModel;

import net.minecraft.item.Item;
import net.minecraftforge.client.event.ModelRegistryEvent;
import net.minecraftforge.event.RegistryEvent;
import net.minecraftforge.fml.common.Mod.EventBusSubscriber;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

@EventBusSubscriber
public class RegistryHandler {

	@SubscribeEvent
	public static void onItemRegister(RegistryEvent.Register<Item> event)
	{
		event.getRegistry().registerAll(ModItems.ITEMS.toArray(new Item[0]));
	}
	
	@SubscribeEvent
	public static void onModelRegister(ModelRegistryEvent event)
	{
		for(Item item : ModItems.ITEMS) {
			if(item instanceof IHasModel) {
				((IHasModel)item).registerModels();
			}
		}
	}
}

 

ItemBase.java

Spoiler

package com.darkenmoonz.designedexpansionmod.items;

import com.darkenmoonz.designedexpansionmod.Main;
import com.darkenmoonz.designedexpansionmod.init.ModItems;
import com.darkenmoonz.designedexpansionmod.util.IHasModel;

import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.item.Item;

public class ItemBase extends Item implements IHasModel {

	public ItemBase(String name)
	{
		setUnlocalizedName(name);
		setRegistryName(name);
		setCreativeTab(CreativeTabs.MISC);
		
		ModItems.ITEMS.add(this);
	}
	
	@Override
	public void registerModels() {
		Main.proxy.registerItemRenderer(this, 0, "inventory");
	}

}

 

ruby.json

Spoiler

{
   "parent": "item/generated",
   "textures": {
       "layer0": "dem:items/ruby"
   }
}

 

 

Link to comment
Share on other sites

Also problematic code #7.

 

Additionally, IHasModel is stupid. ALL items need models. The IHasModel interface does fuckall but make you write the same line of code over and over again when you could instead do this:

	@SubscribeEvent
	public static void onModelRegister(ModelRegistryEvent event)
	{
		for(Item item : ModItems.ITEMS) {
			Main.proxy.registerItemRenderer(item, 0, "inventory");
		}
	}

And I'll also point out that the ModelRegistryEvent is client side only and so this event should already be in your Client Proxy, which means you can do this:

	@SubscribeEvent
	public static void onModelRegister(ModelRegistryEvent event)
	{
		for(Item item : ModItems.ITEMS) {
			ModelLoader.setCustomModelResourceLocation(item, 0, new ModelResourceLocation(item.getRegistryName(), "inventory")
		}
	}

 

Hey @diesieben07 can we add the stupidness of IHasModel to the list of Code Style issues? I've probably said "this is dumb" like 14 times now.

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

37 minutes ago, diesieben07 said:

Sure. Ideas for wording?

Hmm.

 

"The IHasModel is not necessary. All items require models that must be registered and the registration can be done without any non-public information from the Item instance. The same goes for blocks: not all blocks have items but the ones that do, follow the same rule. IHasModel was created and perpetuated through a series of uninformed tutorials and has become an example of Cargo Cult Programming."

 

May be worth noting that ModelRegistryEvent is client-side-only as a Problematic Code separately.

"The ModelRegistryEvent is in the Forge client namespace. This means that it may not be present on the dedicated server. As such, it should only be used inside a Client Proxy class to avoid issues."

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

19 minutes ago, diesieben07 said:

Put this way, I think this would fit better in the code style section.

Fair enough.

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

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.