I have the png i want to use but the log says Exception loading model for variant examplemod:my_sword#inventory 

CommonProxy.java CustomSword.java my_sword.json latest.log

My guess is, the png is in the wrong path within the project.

 

Mine are located in src/main/resources/assets/modname/textures/items/myitem.png

My model is in src/main/resources/assets/modname/models/item/myitem.json

 

For you, it looks like the json expects the file in:

src/main/resources/assets/sword/textures/items/my_sword.png

Edited March 7, 20205 yr by KindOfWay

I put the png in assets/sword/textures/items though

 

Also post ExampleMod java, to make sure the registered name is correct, it should be registering as "my_sword" to work.

ExampleMod.java

This looks like outdated code to me.

There is no reason for you to use CommonProxy to register the items.

The EventSubscriber is wrong the notation is

@EventBusSubscriber(modid = Mymod.modId, bus = EventBusSubscriber.Bus.<target>)
  
Where target is either MOD or FORGE depending which events you need

 

I register my items like this in 1.15.2

public final class ModItems {

	public static final DeferredRegister<Item> ITEMS = new DeferredRegister<>(ForgeRegistries.ITEMS, MyMod.modId);

	public static final RegistryObject<Item> sample_item = ITEMS.register("sample_item",
			() -> new Item(new Item.Properties().group(ModItemGroups.MOD_ITEM_GROUP)));
}

Then in my MyMod java

		final IEventBus modEventBus = FMLJavaModLoadingContext.get().getModEventBus();
		
		ModItems.ITEMS.register(modEventBus);

 

is it ok to use it in 1.12?

No, support for 1.12.2 was dropped.

1.12 is no longer supported on these forums due to age.

Please update to a modern version to receive support.

 

See the LTS link at the top of every page for more information.