Comforts (Fabric/Forge/Quilt)

Comforts (Fabric/Forge/Quilt)

125M Downloads

Not allowing to respawn player in 1.12 with sponge.

Raycoms opened this issue ยท 7 comments

commented

Newest spongeforge version with newest comfort version with best forge for that spongeforge
https://cdn.discordapp.com/attachments/372264384690388992/381436589601062914/image.png

commented

I've successfully replicated the bug. I'll see what I can do, this is my first time working with SpongeForge so it might take a while.

commented

@TheIllusiveC4 If you are going to be debugging in a SpongeForge environment and use IntelliJ I highly recommend installing https://github.com/minecraft-dev/MinecraftDev It provides debugging support for mixins that would otherwise be painful. We absolutely appreciate any help that mod authors are willing to put in.

commented

@ryantheleach Thank you for informing me of that, I was unaware.

commented

If you are using windows as well, you may need to use the nightly release as a pretty major bug was fixed in build 69. if when you install it you get a version before 69, you can switch to the nightly releases in the settings.

image

commented

I'm pushing out a new update on CurseForge, version 1.0.0.2, that should address this issue. I added a simple null check in the appropriate method that should stop this problem from happening.

commented

@TheIllusiveC4 Is the forge event for setSpawn non nullable or is null allowed? If it's non-nullable I'll edit the ticket on SpongeCommon to fix it

commented

@ryantheleach Null is allowed as far as I'm aware, it's just that Forge never passes a null argument for the event so I never encountered this problem in a solely Forge environment.