Zepalesque / The-Aether-Redux

An addon to the well-known Minecraft mod, the Aether, adding a variety of content additions, including biomes, accessories, and more!
23 stars 11 forks source link

[Bug]: Game crashes when disconnecting from server. ("Registry minecraft:item: Object did not get ID it asked for. Name: aether_redux:valkyrie_ring") #109

Closed Gbergz closed 4 months ago

Gbergz commented 6 months ago

Minecraft Version

1.20.1

Mod Version

2.0.12b

Forge/Neoforge Version

47.2.17

What happened?

When a player disconnects from a server they crash.

What was the expected result?

No crash.

Steps to Reproduce

Not entirely sure since most players reporting this have it happen when just leaving the server of theirs.

While myself I can't reproduce it, so it's a weird issue.

Log Output (if applicable)

https://gist.githubusercontent.com/Gbergz/80f3b34147d70f044d4e80fdf57e4de8/raw/bc84e004a6082cceb234f2ef781e7195d62910cf/gistfile1.txt

Crash Report (if applicable)

Crashlog from logs: https://gist.githubusercontent.com/Gbergz/80f3b34147d70f044d4e80fdf57e4de8/raw/bc84e004a6082cceb234f2ef781e7195d62910cf/gistfile1.txt

Generated crash report: https://gist.githubusercontent.com/Gbergz/61baefcbe8d5fb29f0f0c7f0cc29dd08/raw/195fedf48925f685bf45fcced068f17f1f34020f/gistfile1.txt

Is this a compatibility issue with another mod?

It very well could be, reporting here to make sure.

Additional Context/Information

The only reason I am reporting it here is because every time this gets reported a specific thing in the logs sticks out.

This: Registry minecraft:item: Object did not get ID it asked for. Name: aether_redux:valkyrie_ring Expected: 31653 Got: 31648

The Valkyrie Ring. And if I read correctly for your 2.0 version, this Ring was supposed to be removed? Maybe thats why this happens. Not sure to be honest.

Anyway, thanks in advance. - Gbergz

Make sure to confirm these as well before submitting the report:

Zepalesque commented 6 months ago

hmm, that's really strange was the world created in a pre-2.0 version? That could potentially be the issue, as while the valkyrie ring got removed in 2.0 I readded it a bit more recently, but yeah, not fully sure why this is happening

Gbergz commented 6 months ago

Yes the world was created pre-2.0 I am pretty sure. Sorry for late reply.

Zepalesque commented 6 months ago

hmm, was it loaded into a version after 2.0 before using the newer one (2.0.12b)? because I wonder if it could potentially be caused by the fact that the Valkyrie ring didn’t exist for a few versions, though I’m not sure why that’d cause crashing mind you but yeah that seems like a potential cause maybe, I’ll have to look into this at some point

Gbergz commented 6 months ago

Followup on the question if the world was created pre 2.0, I was mistaken, it wasn't. I made the world we're using on our servers with version 2.0.7d of Aether Redux.

Zepalesque commented 6 months ago

oh, strange :thinking: is the log included the full log, or just part of it? wondering because it seems shorter than logs usually are if not, it'd be helpful if I could have that as the crash seems to mention checking the log for more info

Gbergz commented 6 months ago

Its a part of the logs, there's no other errors in the log than that part.

Zepalesque commented 6 months ago

well yeah but it could still be useful, it mentions this: One of more entry values did not copy to the correct id. Check log for details! but I'm not seeing it in the partial log, so having the full log would be helpful as it may not be an error in the log necessarily

Gbergz commented 6 months ago

Of course, but currently I do not have the full log.

The "Check log for details" is that part I am copying from and pasting to the link above.

Gonna have to try and reproduce the issue again and get you full logs then.

Zepalesque commented 6 months ago

ah yeah that’s be helpful

Gbergz commented 6 months ago

Here you go https://gist.githubusercontent.com/Gbergz/1e2579ecc90c6912aac12ac705ef3481/raw/abc8e40c51d6d60d89d7caa2cf35bd26482a6b33/gistfile1.txt

File if needed: latest.log

Zepalesque commented 6 months ago

hmm, do you have the crash report itself? the log unfortunately doesn't seem to have much more information lol

Gbergz commented 6 months ago

The crash report has the same logs as on the bottom in the logs, kind of.

Gbergz commented 6 months ago

And I did post the crash report in the initial report. 😅 Here it is again https://gist.githubusercontent.com/Gbergz/61baefcbe8d5fb29f0f0c7f0cc29dd08/raw/195fedf48925f685bf45fcced068f17f1f34020f/gistfile1.txt

Zepalesque commented 6 months ago

hmm, I can’t seem to figure out this issue, it doesn’t make any sense why it’d be crashing items getting different ids than they expect doesn’t usually cause crashes

Gbergz commented 6 months ago

Items getting different ids than they expect doesn’t usually cause crashes

I know right, super weird issue.

Zepalesque commented 5 months ago

hmm, I have a potential idea to fix this, which involves renaming the item I was going to do that in v2.1, but I could probably do it in 2.0.17 instead

Gbergz commented 5 months ago

👀 hopefully that fixes it :)

Zepalesque commented 4 months ago

should hopefully be fixed now, took a bit longer than I meant for it to lol

Gbergz commented 4 months ago

Awesome, thanks!