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

[BUGFIX] Actually check if the Playable Character is unlocked #3748

Merged

Conversation

AbnormalPoof
Copy link
Collaborator

@AbnormalPoof AbnormalPoof commented Oct 21, 2024

Does this PR close any issues? If so, link them below.

Fixes #3747

Briefly describe the issue(s) fixed.

CharSelectSubState never actually checked to see if the playable character was unlocked before pushing them to nonLocks (to play the unlock animation). The only thing it did was check to see if the playable character ID was in Save.instance.charactersSeen, if it wasn't, it would be pushed to nonLocks and still play the unlock animation despite never actually unlocking them.

This, in turn, basically unlocked Pico right from the get-go if you entered the Character Select screen early, whether it be through a modded Playable Character, or the Debug Menu.

Include any relevant screenshots or videos.

screenshot-2024-10-21-10-26-47
Now, Pico remains locked (since WeekEnd 1 was never beaten), while Madoka (a modded playable character) is unlocked.

@github-actions github-actions bot added pr: haxe PR modifies game code. size: small A small pull request with 10 or fewer changes. labels Oct 21, 2024
@EliteMasterEric EliteMasterEric added the status: pending triage Awaiting review. label Jan 17, 2025
@Hundrec Hundrec added the type: minor bug Involves a minor bug or issue. label Jan 22, 2025
@AbnormalPoof AbnormalPoof force-pushed the bugfix/pico-autounlock branch from d81d105 to b80a6ee Compare January 26, 2025 16:00
@EliteMasterEric EliteMasterEric added this to the 0.6.0 (Pit Stop 2) milestone Feb 18, 2025
@EliteMasterEric EliteMasterEric added status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: pending triage Awaiting review. labels Feb 18, 2025
@EliteMasterEric EliteMasterEric merged commit 7058126 into FunkinCrew:develop Feb 18, 2025
2 checks passed
@AbnormalPoof AbnormalPoof deleted the bugfix/pico-autounlock branch February 18, 2025 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: small A small pull request with 10 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants