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

Regression - the game She Awaits locks you in a wall when using the bed due to a move route being incorrectly processed in the continuous build #3129

Closed
Mimigris opened this issue Oct 24, 2023 · 2 comments · Fixed by #3130

Comments

@Mimigris
Copy link

Name of the game:

She Awaits (version 0.02 or version 0.025c).

Player platform:

Windows, 64 bits continuous build of the Player.

Describe the issue in detail and how to reproduce it:

After starting a new game, use the bed in your room. Normally, the game should move you on it with a move route that disables collision, moves you right and reenables it, and then teleports you to another map.
image
In the continuous build however, the move route is processed again on the new map, and since a wall is present on your right, the game locks you in a wall, making the game unplayable.
This issue was not present in the version 0.8.0 of the Player nor in a continuous build that I had from the 22th of October, so the regression must have likely occurred with the pull request #3120 .

@ell1e
Copy link
Contributor

ell1e commented Oct 24, 2023

I'm downloading the game right now, I'll test and debug it immediately. My deepest apologies for possibly being responsible for this regression. (Whoever has access to changing this, feel free to assign me for this ticket. I should hopefully have some info in a few hours what's going on.)

@ell1e
Copy link
Contributor

ell1e commented Oct 24, 2023

I can reproduce the bug and I can confirm 7221769 works and a33e699 is broken, making my change clearly the responsible one. I'm looking at it in the debugger right now.

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

Successfully merging a pull request may close this issue.

2 participants