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
Crash on startup Forge 31.1.95+ #1602
Comments
yes indeed, confirming this crash in my live server test also. |
I am having this same issue, Does anyone know how to fix it? |
So First Off What Version are you playing? |
Can also confirm the bug. Repro confirmed: Forge 31.1.95+ inclusive & BOP 10.0.0.360 |
ive rolled back to forge 31.1.93 and its working fine |
the crash report log that you send on your main comment is the biomesoplenty was not load_registries from this issue there are few ways to fix this
if one of this 3 aren't working for you try waiting for the dev to update them |
First of all referencing the log I gave as if I am one of the people asking for the fix instead of just being willing to wait for the fix is silly as I also clearly pointed out which line/what the issue is. The only place I use Biomes O' Plenty is in Mekanism's Data generators for the compat recipes we add. If I really wanted to get it working right now instead of not minding if it takes a few days I would spend the 5-10 minutes to clone the repo fix the bug and submit a PR request building and playing with it while I wait. I mainly made this issue to inform the devs that the crash exists rather than them not necessarily knowing until they try to update forge. Secondly unless the release build does not contain the music discs that BOP adds it will also crash. |
I'm a total noob and have no idea how to code or what to do. Would you guys suggest just waiting for an update to get BoP to work? |
Crashes also with BoP 345 (BoP release for 1.15) and Forge 1.15.2 - 31.2.0 (now the recommended Forge release). As reported above, does not crash on Forge 31.1.93 and BoP 345. |
There is a fix here (worked on my machine locally in a singleplayer environment at least), but it only works with Forge 31.2.0 as it uses methods that were added in the patch referenced by the OP. If someone with privileges makes a 31.2.0 and above branch I can submit the PR. https://github.com/AtlaStar/BiomesOPlenty/tree/Forge-31.2.0-Fixes EDIT: I suppose it works with Forge 31.1.95 considering that was the version where the change to the MusicDiscItem was made. |
Bug Report
How can the crash be reproduced?
Start Biomes O' Plenty with forge 31.1.95. The issue has to do with this commit or more specifically this line. SoundEvents are not registered before items so it tries to call
.delegate
on thenull
SoundEvent that Biomes O' Plenty is passing to it. I recommend using the new constructor that takes a supplier so can properly work.Crash Report and Logs
Latest.log: https://gist.github.com/pupnewfster/597ea6f522613c8796506f7eb4a3ec50 (with a couple extra mods from me testing something but does not have anything to do with this crash). I am also not including the crash report as an invalid one is produced due to MinecraftForge/MinecraftForge#6374
Mod Version
10.0.0.360
Forge: 31.2.0
The text was updated successfully, but these errors were encountered: