[Feature Request] More Datapack support
MuteTiefling opened this issue ยท 6 comments
I was just poking around trying to change the crafting recipes for the book shelves and noticed they aren't data pack compatible. Would it be possible to add support for your crafting recipes?
All recipes from apoth/anything that I have is datapack overridable, but the default files are not jsons.
Hrm. Alright then, thanks for the prompt response. I'll take another hack at it, but so far I've been unable to override the hellshelf recipe. I'm just getting another recipe with the same ID.
Well that would be impossible - two recipes cannot share the same ID (they're stored in a Map<ResourceLocation,IRecipe<?>>) - Even if two existed at the same ID, one would kick the other out.
The code recipes don't even attempt to add themselves if a recipe is already present with the same ID https://github.com/Shadows-of-Fire/Placebo/blob/1.16/src/main/java/shadows/placebo/recipe/RecipeHelper.java#L111
Well, I've tried just a straight datapack that should result in an ID override and it's not doing anything. Your recipe remains present and registered despite mine being there from startup.
Not sure what else to do here. I have to say I've modified thousands of recipes to date and this is one of two mods that have given me any difficulty.