Jump to content
Search the Forums BEFORE making a new post.
Currently supported: All versions

[1.12.1] Multiple recipe questions: Recipe Book tabs, unlocking recipes, and replacing vanilla recipes

Aarilight

By Aarilight
in Modder Support

 Share

Recommended Posts

Aarilight Stone Miner

Aarilight

Posted
Posted (edited)

I have five questions:

 

  1. From what I've gathered, unlocking recipes (to add them to the recipe book) occurs from advancements that trigger when you have the items. Can I use them to unlock recipes that I've added via registering in the code?
  2. Are all recipes by default locked? If I'm adding all of the recipes from ForgeRegistries.RECIPES.register, and therefore have a reference to the recipes and their ResourceLocation, is there a way I unlock all of them by default?
  3. To get a recipe to appear in a specific tab of the recipe book, I read that you have to have them in the matching creative tab. Since my items all appear in a custom creative tab, does that mean they will only show in the compass (search/all) tab?
  4. I plan to convert most of my recipes to JSON, but there are a few complicated recipes that take any number of certain items and set NBT data on the resulting items. What's the best practice on recipes like this? Do they even work with the recipe book?
  5. [Solved] How can I replace vanilla recipes? In another topic, this was posted:

But doing what it says, when setRegistryName is called, I get an error java.lang.IllegalStateException: Attempted to set registry name with existing registry name! New: minecraft:bone_meal_from_bone

What am I doing wrong here?

Edited by Aarilight
link to solution for the last question
Aarilight Stone Miner

Aarilight

Posted
  • Author
Posted (edited)

Fixed the last issue, I had assumed that Attempted to set registry name with existing registry name meant I couldn't set two items to the same registry name, but it actually means that the item already has a registry name, and it can't be changed. I don't have an answer for my other four questions yet tho. I updated the initial post marking this last question as solved.

 

However, setting the registryName to replace a vanilla registryName causes this warning: Dangerous alternative prefix `minecraft` for name `bone_meal_from_bone`, expected `souls` invalid registry invocation/invalid name?

 

Is there a way to suppress this warning?

Edited by Aarilight
Draco18s Reality Controller

Draco18s

Posted
Posted
1 hour ago, Aarilight said:

Is there a way to suppress this warning?

No. You have to live with it.

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.

Draco18s Reality Controller

Draco18s

Posted
Posted

1) Don't know, probably

2) Yes, but that just means it isn't shown in the recipe book unless game settings are changed to make unknown recipes uncraftable

3) Don't know

4) They should be convertible, even if you have to write custom conditions for them. I haven't really messed with it at all

5) Relevant, but if you remove / replace recipes you need to provide a new advancement to unlock them and the original advancement cannot be killed and will throw an error when it is deserialized. This error cannot be prevented, as it happens during the deserialization process.

  • Like 1

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.

Aarilight Stone Miner

Aarilight

Posted
  • Author
Posted (edited)

Not sure if I misunderstood your answer for 5), but by replacing the minecraft:bone_meal_from_bone recipe, my new, replaced recipe unlocked when I had bones in my inventory (via original advancement). So the only real issue with replacing recipes is that it logs warnings about them having the minecraft: prefix even though it's intentional. Besides this warning everything works as I intended.

 

I would need a new advancement to unlock it if the recipe was changing, but the recipe is the same, it's just the output that's different now. I guess I should have mentioned that.

Edited by Aarilight
Draco18s Reality Controller

Draco18s

Posted
Posted

Must only be removed recipes then (e.g. I removed 3 wheat for bread and replaced it with 3 flour for bread, wheat requiring to be milled into flour).

Still a warning worth giving.

21 minutes ago, Aarilight said:

it logs warnings about them having the minecraft: prefix even though it's intentional. Besides this warning everything works as I intended.

Pretty much. Any object given a domain other than the domain being processed will log that warning. The substitution system has always done this.

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.

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.

×
 Share
Go to topic listing

Announcements



  • Recently Browsing

    • No registered users viewing this page.

  • Posts

    • Orichalcum
      Getting Holder<Enchantment> from RegistryKey in GLM

      By Orichalcum · Posted

      I know that this may be a basic question, but I am very new to modding. I am trying to have it so that I can create modified Vanilla loot tables that use a custom enchantment as a condition (i.e. enchantment present = item). However, I am having trouble trying to implement this; the LootItemRandomChanceWithEnchantedBonusCondition constructor needs a Holder<Enchantment> and I am unable to use the getOrThrow() method on the custom enchantment declared in my mod's enchantments class. Here is what I have so far in the GLM:   protected void start(HolderLookup.Provider registries) { HolderLookup.RegistryLookup<Enchantment> registrylookup = registries.lookupOrThrow(Registries.ENCHANTMENT); LootItemRandomChanceWithEnchantedBonusCondition lootItemRandomChanceWithEnchantedBonusCondition = new LootItemRandomChanceWithEnchantedBonusCondition(0.0f, LevelBasedValue.perLevel(0.07f), registrylookup.getOrThrow(*enchantment here*)); this.add("nebu_from_deepslate", new AddItemModifier(new LootItemCondition[]{ LootItemBlockStatePropertyCondition.hasBlockStateProperties(Blocks.DEEPSLATE).build(), LootItemRandomChanceCondition.randomChance(0.25f).build(), lootItemRandomChanceWithEnchantedBonusCondition }, OrichalcumItems.NEBU.get())); }   Inserting Enchantments.[vanilla enchantment here] actually works but trying to declare an enchantment from my custom enchantments class as [mod enchantment class].[custom enchantment] does not work even though they are both a ResourceKey and are registered in Registries.ENCHANTMENT. Basically, how would I go about making it so that a custom enchantment declared as a ResourceKey<Enchantment> of value ResourceKey.create(Registries.ENCHANTMENT, ResourceLocation.fromNamespaceAndPath([modid], [name])), declared in a seperate enchantments class, can be used in the LootItemRandomChanceWithEnchantedBonusCondition constructor as a Holder? I can't use getOrThrow() because there is no level or block entity/entity in the start() method and it is running as datagen. It's driving me nuts.
    • NezumiMaus5
      cant find out what causing these errors (crash log below) Mixin config bbb.mixins.json does not specify "minVersion" property

      By NezumiMaus5 · Posted

      https://paste.ee/p/lw0XkwYJ
    • socrates3992
      Help resolving the root cause for Exception: Item id not set

      By socrates3992 · Posted

      Hi here is an update. I was able to fix the code so my mod does not crash Minecraft. Please understand that I am new to modding but I honestly am having a hard time understanding how anyone can get this to work without having extensive programming and debugging experience as well as searching across the Internet, multiple gen AI bots (claude, grok, openai), and examining source code hidden in the gradle directory and in various github repositories. I guess I am wrong because clearly there are thousands of mods so maybe I am just a newbie. Ok, rant over, here is a step by step summary so others can save the 3 days it took me to figure this out.   1. First, I am using forge 54.1.0 and Minecraft 1.21.4 2. I am creating a mod to add a shotgun to Minecraft 3. After creating the mod and compiling it, I installed the .jar file to the proper directory in Minecraft and used 1.21.4-forge-54.1.0 4. The mod immediately crashed with the error: Caused by: java.lang.NullPointerException: Item id not set 5. Using the stack trace, I determined that the Exception was being thrown from the net.minecraft.world.item.Item.Properties class 6. It seems that there are no javadocs for this class, so I used IntelliJ which was able to provide a decompiled version of the class, which I then examined to see the source of the error. Side question: Are there javadocs? 7. This method, specifically, was the culprit: protected String effectiveDescriptionId() {      return this.descriptionId.get(Objects.requireNonNull(this.id, "Item id not set"));  } 8. Now my quest was to determine how to set this.id. Looking at the same source file, I determined there was another method:  public Item.Properties setId(ResourceKey<Item> pId) {             this.id = pId;             return this;   } 9. So now, I need to figure out how to call setId(). This required working backwards a bit. Starting from the constructor, I stubbed out the variable p which is of type Item.Properties public static final RegistryObject<Item> SHOTGUN = ITEMS.register("shotgun", () -> new ShotgunItem(p)); Rather than putting this all on one line, I split it up for readability like this: private static final Item.Properties p = new Item.Properties().useItemDescriptionPrefix().setId(rk); Here is was the missing function, setId(), which takes a type of ResourceKey<Item>. My next problem is that due to the apparent lack of documentation (I tried searching the docs on this site) I could not determine the full import path to ResourceKey. I did some random searching on the Internet and stumbled across a Github repository which gave two clues: import net.minecraft.resources.ResourceKey; import net.minecraft.resources.ResourceLocation; Then I created the rk variable like this: private static ResourceKey<Item> rk = ResourceKey.create(Registries.ITEM, ResourceLocation.parse("modid:shotgunmod")); And now putting it all together in order: private static ResourceKey<Item> rk = ResourceKey.create(Registries.ITEM, ResourceLocation.parse("modid:shotgunmod")); private static final Item.Properties p = new Item.Properties().useItemDescriptionPrefix().setId(rk); public static final RegistryObject<Item> SHOTGUN = ITEMS.register("shotgun", () -> new ShotgunItem(p)); This compiled and the mod no longer crashes. I still have more to do on it, but hopefully this will save someone hours. I welcome any feedback and if I missed some obvious modding resource or tutorial that has this information. If not, I might suggest we add it somewhere for people trying to write a mod that doesn't crash. Thank you !!!  
    • socrates3992
      Help resolving the root cause for Exception: Item id not set

      By socrates3992 · Posted

      I will keep adding to this thread with more information in case anyone can help, or at the very least I can keep my troubleshooting organized. I decided to downgrade to 54.1.0 in the hopes that this would fix the issue but it didn't. At least now I am on a "recommended" version. The crash report did confirm my earlier post that the Exception is coming from effectiveDescriptionId(). I'll continue to see if I can find a way to set the ID manually.   Caused by: java.lang.NullPointerException: Item id not set         at java.base/java.util.Objects.requireNonNull(Objects.java:259) ~[?:?]         at TRANSFORMER/[email protected]/net.minecraft.world.item.Item$Properties.effectiveDescriptionId(Item.java:465) ~[forge-1.21.4-54.1.0-client.jar!/:?]         at TRANSFORMER/[email protected]/net.minecraft.world.item.Item.<init>(Item.java:111) ~[forge-1.21.4-54.1.0-client.jar!/:?]         at TRANSFORMER/[email protected]/com.example.shotgunmod.ShotgunItem.<init>(ShotgunItem.java:19) ~[shotgunmod-1.0.0.jar!/:1.0.0]         at TRANSFORMER/[email protected]/com.example.shotgunmod.ModItems.lambda$static$0(ModItems.java:15) ~[shotgunmod-1.0.0.jar!/:1.0.0]         at TRANSFORMER/[email protected]/net.minecraftforge.registries.DeferredRegister$EventDispatcher.lambda$handleEvent      
    • JackelM95
      Minecraft sever wont start com.google.gson issue!

      By JackelM95 · Posted

      It just randomly stop working after a rebooted my dedicated server PLEASE HELP!   com.google.gson   Failed to start the minecraft server com.google.gson.JsonSyntaxException: Expected a com.google.gson.JsonObject but was com.google.gson.JsonPrimitive; at path $  

  • Topics

×
×
  • Create New...

Important Information

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

  I accept