izebra_projects/musicfetch
1
0
Fork 0
Code Issues Pull Requests Activity

Merge feat/unknown-album: default blank album to 'Unknown Album'

Browse Source
This commit is contained in:
Zeb Hering
2026-06-11 21:53:11 -07:00
parent 530b5b0406 8b881c14bf
commit 7951c436dd
2 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
musicfetch
View File

@@ -618,6 +618,9 @@ def yt_download(url_or_query: str, target_folder: Optional[str], quality: str, d
cmd += ["--parse-metadata", f"{hit.title}:%(title)s"] cmd += ["--parse-metadata", f"{hit.title}:%(title)s"]
if hit.year: if hit.year:
cmd += ["--parse-metadata", f"{hit.year}:%(release_year)s"] cmd += ["--parse-metadata", f"{hit.year}:%(release_year)s"]
# Always embed an album: the resolved/native album if present, else a
# placeholder so players (e.g. Plexamp) don't choke on a blank album.
cmd += ["--parse-metadata", "%(album|Unknown Album)s:%(meta_album)s"]
cmd.append(url_or_query) cmd.append(url_or_query)
dest = outtmpl or target_folder dest = outtmpl or target_folder

8
tests/test_playlist.py
View File

@@ -102,3 +102,11 @@ def test_yt_download_returns_true_on_zero_exit(monkeypatch):
def test_yt_download_dry_run_returns_true(): def test_yt_download_dry_run_returns_true():
assert mf.yt_download("u", "/tmp/x", "best", True) is True assert mf.yt_download("u", "/tmp/x", "best", True) is True
def test_yt_download_always_sets_album_default(monkeypatch):
captured = {}
monkeypatch.setattr(mf.os, "makedirs", lambda *a, **k: None)
monkeypatch.setattr(mf.subprocess, "run", lambda cmd, **k: captured.update(cmd=cmd) or _CP(""))
mf.yt_download("u", "/tmp/x", "best", False)
assert "%(album|Unknown Album)s:%(meta_album)s" in captured["cmd"]