Jump to content

[1.10.2] Object Holder not working


Meldexun

Recommended Posts

6 minutes ago, Meldexun said:

Does the ObjectHolder annotation only work for classes as of 1.12.2?

No.

 

3 minutes ago, RaphGamingz said:

do you put your mod_id into the object holder? im not sure

You should post your code. And yes your modid needs to be present.

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

1 minute ago, Animefan8888 said:

You should post your code.

Sure. Here it is:

@ObjectHolder(BetterDiving.MOD_ID)
public class ModItems {

	public static final Item TITANIUM = null;
	public static final Item COPPER_ORE = null;
	public static final Item SILVER_ORE = null;
	public static final Item LEAD = null;
	public static final Item GOLD = null;

	@EventBusSubscriber
	public static class ItemRegistrationHandler {

		public static final List<Item> ITEMS = new ArrayList<Item>();

		@SubscribeEvent
		public static void registerItems(RegistryEvent.Register<Item> event) {
			final Item[] items = {
					setItemName(new Item(), "titanium"),
					setItemName(new Item(), "copper_ore"),
					setItemName(new Item(), "silver_ore"),
					setItemName(new Item(), "lead"),
					setItemName(new Item(), "gold")
			};

			IForgeRegistry<Item> registry = event.getRegistry();

			for (Item item : items) {
				registry.register(item);
				ITEMS.add(item);
			}
		}

		private static Item setItemName(Item item, String name) {
			return setItemNameAndTab(item, name, BetterDiving.TAB_BETTER_DIVING);
		}

		private static Item setItemNameAndTab(Item item, String name, @Nullable CreativeTabs tab) {
			return item.setUnlocalizedName(name).setRegistryName(BetterDiving.MOD_ID, name).setCreativeTab(tab);
		}
	}

}

 

Link to comment
Share on other sites

25 minutes ago, Meldexun said:

After some testing it seems like i have to annotate every field with @ObjectHolder.

Also looking at Choonster's TestMod3 i saw that he also annotated every field.

Not true. Yes, you can do that, but it is not required.

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/HarderOres.java#L174

Just that the name-matching has to be exact (only ignoring case).

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

1 hour ago, Meldexun said:

Probably naming the fields with lowercase letters is the key.

https://mcforge.readthedocs.io/en/latest/concepts/registries/#injecting-registry-values-into-fields

    public static final Item ENDER_PEARL = null;    // Note that the actual name is "minecraft:ender_pearl", not "minecraft:ENDER_PEARL".
                                                    // However, since constructing a ResourceLocation lowerc

 

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

On 10/12/2019 at 3:05 AM, Draco18s said:

https://mcforge.readthedocs.io/en/latest/concepts/registries/#injecting-registry-values-into-fields


    public static final Item ENDER_PEARL = null;    // Note that the actual name is "minecraft:ender_pearl", not "minecraft:ENDER_PEARL".
                                                    // However, since constructing a ResourceLocation lowerc

 

Well that's 1.12.2 not 1.10.2. Also when lowercasing the names of the fields it works as well as annotating the fields but leaving them uppercase doesn't work.

Edited by Meldexun
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.



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.