mezz / JustEnoughItems

Item and Recipe viewing mod for Minecraft
https://www.curseforge.com/minecraft/mc-mods/jei
MIT License
790 stars 305 forks source link

[1.14] Elytra Anvil Repair Recipe is Incorrect #1773

Closed oitsjustjose closed 5 years ago

oitsjustjose commented 5 years ago

In playing in modded survival 1.14 on my public Geolosys Testing server, I was curious to see if Elytra were given any crafting uses by the few mods we have. Then I noticed that the repair item was still Leather from the pre-1.13 era. Just thought I'd through it out there that that's still an issue. Here is some photographic proof if it helps:


Photos

What is shown:

javaw_muknr37CbB

My attempt at following this recipe:

javaw_t93UqamIeb

The actual recipe:

javaw_SoHPFB0IS0


Possible Source of the Problem

https://github.com/mezz/JustEnoughItems/blob/f520f423d9e518eed0a5a1067999b430d519fd70/src/main/java/mezz/jei/plugins/vanilla/anvil/AnvilRecipeMaker.java#L134

I'd be glad to make a PR with the new Phantom Membran if it makes your life easier!!

mezz commented 5 years ago

Thanks for the report! PRs are welcome. You diagnosed the problem completely, so it would be nice to give you credit for the fix.

oitsjustjose commented 5 years ago

PR made and tested, it should be good to go!