spotizerrphoenix/deezspot-spotizerr-phoenix

Support multi-value artist tags via null separator #5

Open

dashsnuff wants to merge 3 commits from dashsnuff/deezspot-spotizerr-phoenix:feat/null-separator-multi-value-tags into main

dashsnuff commented

2026-02-06 00:47:17 +00:00

First-time contributor

When artistSeparator is set to \u0000 (null byte) in the config, artist metadata fields are now written as proper multi-value tag entries instead of a single joined string.

MP3 (ID3): Passes a list to mutagen's TPE1/TPE2/TCOM text param, which writes null-separated multi-value strings in the ID3
frame
FLAC/OGG/Opus (Vorbis): Assigns a list to tag keys, creating multiple fields per key
M4A (MP4): Assigns a list to atom keys, creating multiple string values

Also updated the hardcoded "; " separators in taggers.py for author, composer, lyricist, and album_artist from Deezer contributors, these now respect the configured separator.

When artistSeparator is set to \u0000 (null byte) in the config, artist metadata fields are now written as proper multi-value tag entries instead of a single joined string. - MP3 (ID3): Passes a list to mutagen's TPE1/TPE2/TCOM text param, which writes null-separated multi-value strings in the ID3 frame - FLAC/OGG/Opus (Vorbis): Assigns a list to tag keys, creating multiple fields per key - M4A (MP4): Assigns a list to atom keys, creating multiple string values Also updated the hardcoded "; " separators in taggers.py for author, composer, lyricist, and album_artist from Deezer contributors, these now respect the configured separator.

dashsnuff force-pushed feat/null-separator-multi-value-tags from ad23f13aa8 to 609f164ab4

2026-02-09 20:12:26 +00:00

Compare

dashsnuff force-pushed feat/null-separator-multi-value-tags from 609f164ab4 to 161381f53b

2026-02-09 20:17:06 +00:00

Compare

spotizerrphoenix commented

2026-05-22 17:21:54 +00:00

Owner

thanks for the pr! will check on this soon

❤️ 1

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u feat/null-separator-multi-value-tags:dashsnuff-feat/null-separator-multi-value-tags

git switch dashsnuff-feat/null-separator-multi-value-tags

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff dashsnuff-feat/null-separator-multi-value-tags

git switch dashsnuff-feat/null-separator-multi-value-tags

git rebase main

git switch main

git merge --ff-only dashsnuff-feat/null-separator-multi-value-tags

git switch dashsnuff-feat/null-separator-multi-value-tags

git rebase main

git switch main

git merge --no-ff dashsnuff-feat/null-separator-multi-value-tags

git switch main

git merge --squash dashsnuff-feat/null-separator-multi-value-tags