Skip to content

Conversation

@KoloInDaCrib
Copy link
Contributor

Associated Funkin PR

N/A

Linked Issues

image

Description

The variable tankmanGroup never gets set to null after leaving the song due to someone assuming that songs have a kill function (which they don't).
This PR changes that to using onDestroy.

Screenshots/Videos

2025-12-24.15-26-26.mp4

@github-actions github-actions bot added status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. pr: hscript PR modifies game scripts. and removed status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. labels Dec 24, 2025
@VirtuGuy
Copy link

I forgot you use light mode for Discord.

@VirtuGuy
Copy link

Also I think this fixes the issue I was having with both Stress songs where they would both take forever to exit out of.

@Hundrec Hundrec added type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression size: medium A medium pull request with 100 or fewer changes. status: needs clarification Requires more info from the author. labels Dec 30, 2025
@Hundrec
Copy link
Member

Hundrec commented Dec 30, 2025

We fixed this on the Polymod side, but would this be nice to have anyway?

@KoloInDaCrib
Copy link
Contributor Author

We fixed this on the Polymod side, but would this be nice to have anyway?

Since I have no idea how this was fixed on the Polymod side I'm going to say yes to the question, as the fix could be reverted at any point due to a bug or something, whereas this only depends on the onDestroy function getting activated with an event, which always happens without any issues.

@Hundrec Hundrec added status: pending triage Awaiting review. and removed status: needs clarification Requires more info from the author. labels Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: hscript PR modifies game scripts. size: medium A medium pull request with 100 or fewer changes. status: pending triage Awaiting review. type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants