Brass recipe does not work when inventory tweaks is loaded
Trinsdar opened this issue ยท 8 comments
Description of the Bug
If inventory tweaks is loaded brass cannot be crafted from 3 copper ingots and one zinc ingot
To Reproduce
- install inventory tweaks
- get three copper ingots and one zinc ingot
- try to craft brass
Expected behavior
brass should craft from 3 copper ingots and one zinc ingot
Logs & Environment
latest.log: https://paste.dimdev.org/qamoluhafi.md
Additional context
Railcraft 12.0.0-beta2
forge14.23.5.2779
inventory tweaks-1.63
Edit: tested with latest alpha version of inventroy tweaks on curseforge and still happens
My best guesses:
- Inventory tweaks coremod tweaked container code and broke recipe identification;
- inventory tweaks registers oredict entry
dustZinc
anddustCopper
, making railcraft assume that there is alternative logical ways to craft brass
Needs digging.
So, as far as my side goes:
- The coremod's arbitrary-ish edits are limited solely to adding new methods that are used for internal things. The only stuff it does that actively edits existing things is the small tweak to prevent registering sort commands while typing, and a few replacements of internal things to hook them up the the added methods.
- I don't register new ore entries anywhere. The use of the ore dictionary is solely limited to asking for the list of items registered with that entry (and monitoring for new registrations).
Looking at Forge, I do concur it's probably 2 (the 'get entries in the dictionary from a name' method ends up adding that name to the dictionary for... some reason), but while I can sidestep it by adding a similar initial check, I do think that railcraft needs to fix this as well: assuming existence of a key in the dictionary means anything in particular is obviously incorrect.
I'm making it avoid unnecessarily adding the entries just because that's a good thing to do in general anyway. (That code's been there for quite a while so I probably either didn't notice or the non-cluttering version didn't exist at the time)
What is this code even doing? Why doesn't it just register the recipes?
Looking at the history, this looks like a bad merge by @Generalcamo.
Removed here:
4a6527f
Then readded here:
0201daf