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

Revive tableflip fixes #3882

Merged
merged 5 commits into from
Jul 15, 2023
Merged

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Jul 13, 2023

About the pull request

This PR revives #3740 and applies my suggested changes to that PR preventing T and cross shaped table flips.

Explain why it's good for the game

Fixes a game ending exploit, and also gives a bit more sanity to table flipping.

Testing Photographs and Procedure

Screenshots & Videos

Old testing:
This cannot be flipped now:
image
But this can from the checkmark locations:
image

Changelog

🆑 Drathek, Fira
fix: Fixed an issue with table flips that could make some tables incorrectly unflippable, and cause infinite loops. It is no longer possible to flip tables that t-shape or cross, or spans more than 5 tiles away from you.
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Jul 13, 2023
@Drulikar Drulikar added the Priority: High Oh god label Jul 13, 2023
@harryob harryob added this pull request to the merge queue Jul 15, 2023
Merged via the queue into cmss13-devs:master with commit 7f5a984 Jul 15, 2023
30 checks passed
cm13-github added a commit that referenced this pull request Jul 15, 2023
github-actions bot added a commit that referenced this pull request Jul 15, 2023
@Drulikar Drulikar deleted the Revive_Tableflip_Fixes branch August 6, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more Priority: High Oh god
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants