Skip to content
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

[1.16.5] Client-crash after putting living armor #1853

Closed
Tristan-LeBras opened this issue Oct 6, 2021 · 7 comments · Fixed by #1867
Closed

[1.16.5] Client-crash after putting living armor #1853

Tristan-LeBras opened this issue Oct 6, 2021 · 7 comments · Fixed by #1867

Comments

@Tristan-LeBras
Copy link

Tristan-LeBras commented Oct 6, 2021

Issue Description:

Client-crash after putting the 'living armor' in Valhesia 3.4.4 on a server (linked the crash report to Val3 team and they asked me to put it here).
Cannot relog 'internal server error' .

What happens:

I just crafted the full set of 'living armor' and the moment I put it on I crashed.
After that I'm not able to connect to the server 'internal server error' but others can and they can play normally.

What you expected to happen:

Steps to reproduce:

Don't know if Valhesia 3 has an impact here (didn't test the mod alone)

  1. Put the living armor
  2. Enjoy the crash

...


Affected Versions (Do not use "latest"):

Valhesia 3.4.4 launched via Curseforge

  • BloodMagic: BloodMagic-1.16.4-3.1.5-25
  • Minecraft: 1.16.5
  • Forge: 36.2.8

Link to log + crash report :
https://gist.github.com/Tristan-LeBras/188c165e08d8e117d43edce40c22d1a3

@the-first-ideal
Copy link

Tried the downgrade, that didn't work, so as admin of the server I spammed /clear [playerName] bloodmagic:livingplate over and over as my player joined the server. It finally caught him, so now he can play.
It does seem like though that there is a potential fix from user VT-14, just needs to be implemented.

@Takeuout
Copy link

Takeuout commented Nov 6, 2021

Putting on the full set of living armor completely breaks the game... had to roll back the server.
i have seen posts about this going back as far as 2017 does this actually ever plan on getting fixed?

enigmatica 6
Minecraft Version: 1.16.5
Modpack Version: 0.5.20

[05.11 16:54:22] [Server] Caused byjava.lang.NullPointerException: Cannot invoke "wayoftime.bloodmagic.core.living.LivingStats.getLevel(net.minecraft.util.ResourceLocation)" because the return value of "wayoftime.bloodmagic.core.living.LivingStats.fromPlayer(net.minecraft.entity.player.PlayerEntity)" is null
[05.11 16:54:22] [Server] at wayoftime.bloodmagic.common.item.ItemLivingArmor.hasElytraUpgrade(ItemLivingArmor.java:224) ~[bloodmagic:1.16.4-3.1.5-{re:classloading,pl:runtimedistcleaner:A}
[05.11 16:54:22] [Server] at wayoftime.bloodmagic.common.item.ItemLivingArmor.canElytraFly(ItemLivingArmor.java:210) ~[bloodmagic:1.16.4-3.1.5-{re:classloading,pl:runtimedistcleaner:A}

@wrincewind
Copy link
Collaborator

the bug isn't from 2017 - it was introduced in version 19(?) or so, with the Elytra upgrade, and only occurs on modpacks with the Caelus API, or other APIs that interact with Elytra flight. A fix has been PRd and will be included in the next version.

@wrincewind wrincewind pinned this issue Nov 7, 2021
@Infinoid
Copy link

Infinoid commented Nov 8, 2021

enigmatica 6 Minecraft Version: 1.16.5 Modpack Version: 0.5.20

I got the same crash in the same modpack. It crashes the first time I put on the full set of living armor, and it crashes again after restarting the game and re-entering the (single-player) world.

I tried a custom build of the mod with #1856 applied, and confirmed that it fixes the crash for me.

@Darkmega18
Copy link

Darkmega18 commented Nov 24, 2021

Soooo, when can we expect this one to drop in an update? it seems like quite a big one for those budding blood magic enthusiasts and probably worth the triage hotfix update... and it happens in my own pack also. A friend got hit by it just the other night. Glad it's know and has been slapped out though.

@TehNut
Copy link
Collaborator

TehNut commented Nov 24, 2021

When Way has time to do a release. We have lives outside of this hobby.

@Darkmega18
Copy link

Darkmega18 commented Nov 25, 2021

shoulda expected that kinda response honestly, and I didn't mean to insinuate that you don't have lives, but I guess big mod takes more time to build a release than others and theres probably devvy-tester things still enabled in dev that would slow that down... :V

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants