-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Incompatible with CoFHCore (for some reason) #74
Comments
Sources of that libraries are not public so I can't really do much. |
The source is visible right here: https://github.com/CoFH/CoFHCore |
Weird that they don't link it on their curse page |
Relevant method seems to be here: https://github.com/CoFH/CoFHCore/blob/de1378d4bba14500640dfb4e3d411a39322a6ec5/src/main/java/cofh/core/event/CoreCommonEvents.java#L93 |
Yeah seems like they have their own mending improvement. Event is cancelled at the end so if Forge decides that OverpoweredMending comes after Cofhcore then nothing can be done about it. The only solution possible would be to increase the priority on my side, but I don't really know what that would mean for compatibility with other mods. It's like I force to take the lead. |
I will discuss this with them. |
The important bit is probably this https://github.com/CoFH/CoFHCore/blob/8307b4bc7dda6461b620f3219f21687c286feb45/src/main/java/cofh/core/event/CoreCommonEvents.java#L116 where it stores xp in a container. |
Well, you could increase the priority just to keep the underlying compatibility until the guys at CoFH do something about it. |
You can try with that version for now. But I'm not a too big fan of setting the priority higher that way. |
Describe the bug
MultiMC Log
Video Proof
Minecraft version
1.16.5
Forge version
36.2.23
Mod version
2.2.3
Relevant log output
No response
The text was updated successfully, but these errors were encountered: