Lyof429 / EndsPhantasm

The vanilla friendly End update we were all waiting for!
https://www.curseforge.com/minecraft/mc-mods/phantasm
5 stars 2 forks source link

AoF7 players reporting empty value for "elytra_boost_advancement" does *not* re-enable vanilla elytra boost behavior for all players #15

Open SubordinalBlue opened 3 months ago

SubordinalBlue commented 3 months ago

In AoF7, we've pushed out an update to set "elytra_boost_advancement" to "".

Now getting reports this is not working as intended.

In the second link, from AOE discord, the player reports that their server console prints the following when trying to rocket boost elytra:

[Server thread/ERROR]: Failed to handle packet net.minecraft.class_2886@4cab1a4b, suppressing error
java.lang.NullPointerException: null
Linguardium commented 3 months ago

seems to be caused by https://github.com/Lyof429/EndsPhantasm/blob/37cb43ab1c6f4d4ec58bba648a2bcd9d01e72a95/src/main/java/net/lyof/phantasm/mixin/FireworkRocketItemMixin.java#L19

getAdvancementLoader().get(new Identifier("")) will never find an Advancement. at best case freetheend is null after this getProgress(freetheend) will throw a NPE from the map lookup this.progress.get(advancement) due to the null key

Lyof429 commented 3 months ago

This is an issue indeed, sorry for it I'm changing it rn so that if the advancement can't be found, it considers boosting as allowed no matter what The fixed version will be released in a few days, i have a few others things to modify In the meantime, setting this value to an early game advancement like "minecraft:story/root" should allow elytras from the ground up without causing crashes

SubordinalBlue commented 3 months ago

Just wanna say, thank you! A reply with details and a work-around for the immediate time being: 👍

Lyof429 commented 3 months ago

You're welcome, thank you for reporting it!

SubordinalBlue commented 2 months ago

<small, polite bump>

Lyof429 commented 2 months ago

Just published the fixed version! https://modrinth.com/mod/ends-phantasm Sorry for the wait, let me know if you find any other issue ^^