Minetweaker support doesn't work on Drying Basin?
Claycorp opened this issue ยท 19 comments
mods.integrateddynamics.DryingBasin.removeRecipesWithOutput(<minecraft:leather>); mods.integrateddynamics.DryingBasin.removeRecipesWithOutput(<minecraft:hardened_clay>); mods.integrateddynamics.DryingBasin.removeRecipesWithOutput(<minecraft:dirt:1>);
All of these do not remove the recipe from the drying basin.
Here is the MineTweaker log https://gist.github.com/Claycorp/24b6790c0e2abace64cda78b0ac56abb
ID: 0.7.2
CT: 3.0.25
If there is something I am missing the Wiki is missing it also as that's where i got the info from.
This is a bug with the JEI integration. JEI shows the recipe but it doesn't exist.
@Claycorp Can you report this to the Minetweaker repo then? Sounds like something that should be fixed on their end.
@Claycorp We add the Minetweaker support, but conflicts with JEI sounds like something that should be fixed on their end.
I highly doubt its a bug with minetweaker. Its either you missed something adding in the support or whoever adds the JEI info is messing it up. When I found issues with Better Beginnings they had to fix the recipes duplicating, thus I would assume that you aren't telling JEI to update.
Sounds similar to jaredlll08/ModTweaker#437, I'll lurk at that issue to see how that'll be resolved.
That isn't the issue at all..... The JEI recipes are never updated. They aren't removed or added. None of your minetweaker scripts update JEI.
@Claycorp Note the comment on IC2 and Tinkers support: jaredlll08/ModTweaker#437 (comment)
There is a difference, Those mods are supported by MT itself and ModTweaker respectively. Not their own custom MT support.
I did some quick comparing to other peoples code and you don't have any JEI updates in yours anywhere.
The MineTweaker API
Underworld This was a totally random mod I found based off this issue
So... Again.... This isn't an issue with MT/CT/ModTweaker or any other tweaking/addon like I said before.....
As mentioned before, I planned to wait until jaredlll08/ModTweaker#437 is resolved, to see how that is fixed, as I will most likely have to do something similar.
But It looks like you already found some similar fixes, so I can start looking into that.
The latest dev build fixes the issue, had to rewrite a couple of things. Thanks for persisting :-)
It wasn't fixed FYI.
I finally figured out the exact issue.
You have your own recipe stuff in the XML and Minetweaker it will still show the recipes that are in the XML even if you remove them via Minetweaker thus creating the recipes that you can see but don't work. So you need to do some magic to sync up whats removed in Minetweaker to the XML side.
@Claycorp Please send me your script so I can try to reproduce it.
I then added this
and they went away.
Files were renamed because Github dosn't allow .zs or .xml Just rename and its all good.
Now that I know how to get rid of them no need to rush. It was getting annoying when people constantly ask "Why doesn't this recipe work?? Its in JEI thus it must work!"
Remove the XML file. I added the XML and they went away, but if you remove it they should show up again even though the recipes don't exist.