Upgrade Recipes & Anvil Enchanted books
Mohron opened this issue ยท 6 comments
I use the Json recipe format where I have to specify the nbt data.
I don't think there is any way of handling these OB extended nbt books.
I'm guessing the repair data is an over-generic handling issue on their part then? Because Using an anvil on anything else definitely add a repair cost prop. I didn't try combining the books in a normal anvil after I found the issue was the book's NBT tag.
I've not taken apart the tags yet, but a vanilla anvil does seem to go generate 2 nbt tags when combining books, where a cheated book only has one.
I'll see if there is any other way of handling the books in the recipes. I've yet to find anyone else using the json technique with enchanted books and the built-in processing. It looks like it only handles exact NBT matching which is a shame.
However because it probably breaks using the vanilla anvil, I'll mark as a bug and see if I can find a solution.
Need to update the recipes to use registry enchantment name rather than id, as these are allocated in a registry - Choonster via Forge forums
After asking for some pointers on the Forge forums I think I have something working.
So far I've got it working with both "pure" enchanted books and anvil produced books.
More testing required, but looking hopeful.
I've ran some testing and it looks like this fix will allow both anvil and enchantment table books to match. As long as only one enchant is on the book at it is the correct one it should work in the recipe.
The only catch is that if you setup AE2 autocrafting with the recipes for the non-anvil ones, then it wont use the anvil ones. You can still manually craft however, it is just autocrafting.