Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.11] JEI not working after Ore Dictionary nullability fixes #626

Closed
Choonster opened this issue Nov 17, 2016 · 4 comments
Closed

[1.11] JEI not working after Ore Dictionary nullability fixes #626

Choonster opened this issue Nov 17, 2016 · 4 comments
Labels

Comments

@Choonster
Copy link

Choonster commented Nov 17, 2016

MinecraftForge/MinecraftForge@bf090cb fixed the ItemStack nullability issues with the Ore Dictionary.

This means that IRecipe#getRecipeOutput always returns a non-null value for ore recipes and ShapelessOreRecipe#getInput returns a NonNullList instead of an ArrayList.

The latter is causing a NoSuchMethodError at runtime from ShapelessOreRecipeHandler#isRecipeValid because the return type was included in the bytecode (even though JEI never explicitly uses it as an ArrayList).

[09:44:01] [Server thread/ERROR] [FML/] []: Caught exception from Just Enough Items (jei)
java.lang.NoSuchMethodError: net.minecraftforge.oredict.ShapelessOreRecipe.getInput()Ljava/util/ArrayList;
	at mezz.jei.plugins.vanilla.crafting.ShapelessOreRecipeHandler.isRecipeValid(ShapelessOreRecipeHandler.java:43) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.plugins.vanilla.crafting.ShapelessOreRecipeHandler.isRecipeValid(ShapelessOreRecipeHandler.java:13) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.RecipeRegistry.addRecipe(RecipeRegistry.java:206) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.RecipeRegistry.addRecipe(RecipeRegistry.java:183) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.RecipeRegistry.addRecipes(RecipeRegistry.java:160) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.RecipeRegistry.<init>(RecipeRegistry.java:86) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.util.ModRegistry.createRecipeRegistry(ModRegistry.java:195) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.JeiStarter.start(JeiStarter.java:48) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.ProxyCommonClient.loadComplete(ProxyCommonClient.java:111) ~[jei_1.11-4.0.1.190.jar:?]
	at mezz.jei.JustEnoughItems.loadComplete(JustEnoughItems.java:53) ~[jei_1.11-4.0.1.190.jar:?]

Full Log

@mezz
Copy link
Owner

mezz commented Nov 18, 2016

Thanks for the heads up!
This is a good change.

@mezz
Copy link
Owner

mezz commented Nov 18, 2016

I think the bytecode change should be reversed, there's no benefit to having it as a NonNullList.
In any case I'll build a new JEI to work with this.

@mezz
Copy link
Owner

mezz commented Nov 18, 2016

Should be fixed in 4.0.1.191

@mezz mezz closed this as completed Nov 18, 2016
@Choonster
Copy link
Author

Thanks for the quick fix.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants