Skip to content

Remove Dead Playlist Hashes#1452

Open
Dansla116 wants to merge 2 commits into
YARC-Official:devfrom
Dansla116:dead-playlist-hashes
Open

Remove Dead Playlist Hashes#1452
Dansla116 wants to merge 2 commits into
YARC-Official:devfrom
Dansla116:dead-playlist-hashes

Conversation

@Dansla116

@Dansla116 Dansla116 commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

When deleting or modifying a song on disc (and therefore, removing/changing song hash), that inaccessible, dead hash was still on the playlist file.
For normal users, this resulted in empty spaces in the playlist manager in game. You could press left/right to re-order your playlist, and it would sometimes look like nothing is happening because you're moving songs around the dead hashes that no longer exist.
For power users/developers, the playlist in game will say one number, and the playlist file on disc will include all the dead hashes and show a higher number.
Made a change so when rescanning your library, it removes the dead hashes from existing playlists. (see feedback item below)

note: This change was originally part of #1413. I've split it out as they were only very loosely related.

@wyrdough

Copy link
Copy Markdown
Contributor

I'm going to hold off on this until after 0.15 since I have no good ideas yet...

@Dansla116 Dansla116 force-pushed the dead-playlist-hashes branch from 3d27fd7 to 1181faa Compare June 28, 2026 16:44
@Dansla116

Dansla116 commented Jun 28, 2026

Copy link
Copy Markdown
Contributor Author

Feedback Item:
There was some concern about players who use removable storage. When rescanning, they would lose songs in their playlists, and not get them back when plugging their storage back in.
I'm open to ideas to work around this. Perhaps lock this behind a toggle?

Made a change so when manually editing a playlist, it removes the dead hashes from only that playlist. Editing can include re-ordering (either with left/right, or using the "Sort by: ..." from the popup menu), adding or removing a song from the playlist.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants