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

Fix: NPE in CorpseLocator #2268

Merged
merged 1 commit into from
Jul 30, 2024
Merged

Fix: NPE in CorpseLocator #2268

merged 1 commit into from
Jul 30, 2024

Conversation

hannibal002
Copy link
Owner

@hannibal002 hannibal002 commented Jul 28, 2024

What

Fixed an NullPointerException when using EntityArmorStand.getCurrentArmor
Reported here: https://discord.com/channels/997079228510117908/1265062036857946182

Stack Trace
SkyHanni 0.26.Beta.18: Caught an NullPointerException in CorpseLocator at SecondPassedEvent: getCurrentArmor(...) must not be null
 
Caused by java.lang.NullPointerException: getCurrentArmor(...) must not be null
    at SH.features.mining.glacitemineshaft.CorpseLocator.findCorpse(CorpseLocator.kt:51)
    at SH.features.mining.glacitemineshaft.CorpseLocator.onSecondPassed(CorpseLocator.kt:93)
    at SH.data.FixedRateTimerManager$1$1.run(FixedRateTimerManager.kt:17)
    at MC.util.Util.func_181617_a(Util.java:19)
    at MC.client.Minecraft.func_71411_J(Minecraft.java:1014)
    at MC.client.Minecraft.handler$bpc000$run(Minecraft.java:3250)
    at MC.client.Minecraft.func_99999_d(Minecraft.java)
    at MC.client.main.Main.main(SourceFile:124)

Changelog Fixes

  • Fixed a rare error when detecting corpses in the Mineshaft. - hannibal2

@hannibal002 hannibal002 added the Bug Fix Bug fixes label Jul 28, 2024
@hannibal002 hannibal002 added this to the Version 0.26 milestone Jul 28, 2024
@hannibal002 hannibal002 merged commit bacc6a3 into beta Jul 30, 2024
9 checks passed
@hannibal002 hannibal002 deleted the NPE-in-corpse-locator branch July 30, 2024 08:29
@github-actions github-actions bot removed the Bug Fix Bug fixes label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant