Here us my code. What am I doing wrong?

 

@ObjectHolder(myfirstmod.MODID)
public class ModItems {

	public static Item redingotItem = new itemRedIngot("redingot_item");
	
	public static void init() {
		redingotItem.setRegistryName(new ResourceLocation(myfirstmod.MODID,"redingot_item"));
	}
	
	@Mod.EventBusSubscriber(modid = myfirstmod.MODID)
	public static class RegistrationHandler {

		@SubscribeEvent
		public void registerItems(RegistryEvent.Register<Item> event){
			event.getRegistry().registerAll(redingotItem);
		}
		
	}

}

 

1 minute ago, aviad1 said:

@ObjectHolder(myfirstmod.MODID)

This is not how you use this annotation.

1 minute ago, aviad1 said:

public void registerItems(RegistryEvent.Register<Item> event){

This needs to be a static method due to this

2 minutes ago, aviad1 said:

@Mod.EventBusSubscriber(modid = myfirstmod.MODID)

 

 

2 minutes ago, Animefan8888 said:

This is not how you use this annotation.

This needs to be a static method due to this

 

Can you send me the code of how it's supposed to be?

3 minutes ago, diesieben07 said:

Yes it is.

Can you send me the code I supposed to write in order to make this work?

1 minute ago, diesieben07 said:

Yes it is.

 

Oh, I didn't know that it could be used like that. Very interesting.

3 minutes ago, aviad1 said:

Can you send me the code of how it's supposed to be?

No, that is not how this forum works. Make your registerItems method a static method and then it should work.

9 minutes ago, aviad1 said:

public static Item redingotItem = new itemRedIngot("redingot_item");

Also, you shouldn't initialize IForgeRegistryEntry instances in a static initializer.

Ive figured it out.

Thanks for all who tried to help