Skip to content

[BUGFIX] Chart Editor - Don't add default difficulty to song metadata#4216

Merged
EliteMasterEric merged 1 commit intoFunkinCrew:developfrom
Lasercar:chart-editor-don't-add-default-difficulty-to-song-metadata
Apr 29, 2025
Merged

[BUGFIX] Chart Editor - Don't add default difficulty to song metadata#4216
EliteMasterEric merged 1 commit intoFunkinCrew:developfrom
Lasercar:chart-editor-don't-add-default-difficulty-to-song-metadata

Conversation

@Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Feb 23, 2025

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

Fixes #3111

Briefly describe the issue(s) fixed.

The chart editor would always end up adding the default difficulty to the metadata if it wasn't already there.

Now, not only does it set the selected difficulty when a song is loaded to the first available difficulty, it also only adds a difficulty to the metadata if the one that's selected when the song is saved doesn't exist in it.

Include any relevant screenshots or videos.

It'll now only add the current difficulty to the metadata if it doesn't exist in it when saved.
@Lasercar Lasercar changed the base branch from main to develop February 23, 2025 11:56
@github-actions github-actions bot added status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. pr: haxe PR modifies game code. and removed status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. labels Feb 23, 2025
@AbnormalPoof AbnormalPoof added type: minor bug Involves a minor bug or issue. status: pending triage Awaiting review. topic: chart editor Related to the operation of the Chart Editor. size: medium A medium pull request with 100 or fewer changes. labels Feb 23, 2025
@Kade-github
Copy link
Member

looks good, accepted into v0.6.4.

thanks!

@AbnormalPoof AbnormalPoof 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 Apr 28, 2025
@AbnormalPoof AbnormalPoof added this to the 0.6.4 milestone Apr 28, 2025
@EliteMasterEric EliteMasterEric merged commit 8074d57 into FunkinCrew:develop Apr 29, 2025
9 checks passed
@Lasercar Lasercar deleted the chart-editor-don't-add-default-difficulty-to-song-metadata branch May 4, 2025 17:47
@EliteMasterEric EliteMasterEric added size: medium A medium pull request with 100 or fewer changes. and removed size: medium A medium pull request with 100 or fewer changes. labels May 14, 2025
@Lasercar Lasercar changed the title Chart editor - Don't add default difficulty to song metadata [BUGFIX] Chart Editor - Don't add default difficulty to song metadata Jun 19, 2025
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: medium A medium pull request with 100 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. topic: chart editor Related to the operation of the Chart Editor. type: minor bug Involves a minor bug or issue.

Development

Successfully merging this pull request may close these issues.

Bug Report: When you make a song with custom difficulties, every time you save the .fnfc it adds one of the default difficulties back to the file.

4 participants