Skip to content

[Feruchemy] BugFix: Providing a Position in canSleep in FeruchemyBronze. #224

Merged
Gerbagel merged 2 commits into
leafreynolds:developfrom
EdgarVerdi:develop
Jun 8, 2026
Merged

[Feruchemy] BugFix: Providing a Position in canSleep in FeruchemyBronze. #224
Gerbagel merged 2 commits into
leafreynolds:developfrom
EdgarVerdi:develop

Conversation

@EdgarVerdi

@EdgarVerdi EdgarVerdi commented May 31, 2026

Copy link
Copy Markdown
Contributor

Changes proposed in this pull request:

  • bug fix: provided a position in onPlayerSleepInBed of canSleep in FeruchemyBronze. This should prevent other mods from crashing when trying to access a non-null event.getPos() when using onTrySleep(PlayerSleepInBedEvent event)

Testing checklist:

  • yes I tested this in a runServer environment, and therefore won't make leaf cry
  • yes, I actually connected to that server and verified it worked and don't need to have my kneecaps removed
  • yes, I also tested the built jars manually outside of the development environment, because problems sometimes don't show up inside the dev environment.

…eruchemyBronze. This should prevent other mods from crashing when trying to access a non-null event.getPos() when using onTrySleep(PlayerSleepInBedEvent event)
@EdgarVerdi EdgarVerdi changed the title BugFix: Providing a Position in canSleep in FeruchemyBronze. [Feruchemy] BugFix: Providing a Position in canSleep in FeruchemyBronze. May 31, 2026
@EdgarVerdi

Copy link
Copy Markdown
Contributor Author

When testing with the runServer there seems to be an issue. However, it seems to also exist prior to my request. The error is related to a LocalPlayer class existing in a DEDICATED_SERVER. Hopefully I'm not missing something myself when checking this, however, this change is very small and likely didn't cause that.

@Gerbagel Gerbagel merged commit 2a3ad97 into leafreynolds:develop Jun 8, 2026
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.

2 participants