Docker/NovaSonic
0
0 Fork 0
Code Issues 15 Pull Requests Actions Packages Projects 1 Releases 1 Wiki Activity

Revert "fix(backend): keep source artist name as-is, no iTunes override"

Browse Source 
This reverts commit 1616d98fa6.
This commit is contained in:
Honey committed 2026-06-21 16:44:26 +02:00
1 parent 1616d98fa6
commit 9aa1947568
1 file changed
+8 -7
Download Patch File Download Diff File Expand all files Collapse all files
backend/novasonic-server.js
+8 -7
View File
@@ -600,8 +600,8 @@ app.post('/api/download', async (req, res) => {
if (!meta.trackNum && genreMeta.trackNum) { meta.trackNum = genreMeta.trackNum; } if (!meta.trackNum && genreMeta.trackNum) { meta.trackNum = genreMeta.trackNum; }
} }
} }
// Keep source (Spotify) artist name as-is — no iTunes override const finalArtist = meta.itunesArtistName || artist;
const result = await downloadTrack({ jobId, artist, rawArtist: artist, title, ...meta, sourceUrl: sourceUrl || sp.sourceUrl || undefined }); const result = await downloadTrack({ jobId, artist: finalArtist, rawArtist: artist, title, ...meta, sourceUrl: sourceUrl || sp.sourceUrl || undefined });
if (result?.skipped) { if (result?.skipped) {
updateJob(jobId, { status: 'done', progress: 100, message: 'Bereits vorhanden – übersprungen', skipped: true }); updateJob(jobId, { status: 'done', progress: 100, message: 'Bereits vorhanden – übersprungen', skipped: true });
} else if (result?.notFound) { } else if (result?.notFound) {
@@ -688,13 +688,14 @@ app.post('/api/download', async (req, res) => {
log(jobId, `📋 iTunes: "${trackArtist} – ${trackTitle}"`); log(jobId, `📋 iTunes: "${trackArtist} – ${trackTitle}"`);
itunes = await lookupItunesTrackMeta(searchArtist, itunesTitle); itunes = await lookupItunesTrackMeta(searchArtist, itunesTitle);
} }
// Keep source (Spotify) artist name as-is — no iTunes override. // Prefer iTunes-resolved name (from server lookup or frontend enrichment) for sort tags / folders.
// iTunes collectionArtistName is used as albumArtist for consistent album grouping. // trackArtist stays as rawArtist for main tags and YTM search.
const resolvedArtist = itunes.itunesArtistName || track.itunesArtistName || trackArtist;
return downloadTrack({ return downloadTrack({
jobId, jobId,
artist: trackArtist, artist: resolvedArtist, // iTunes English name → used for sort tags + folder names
rawArtist: trackArtist, rawArtist: trackArtist, // original Spotify/Japanese name → used for main tag + YTM search
albumArtist: itunes.albumArtist || trackArtist, albumArtist: resolvedArtist,
title: trackTitle, title: trackTitle,
album: itunes.album || track.collectionName || track.album || '', album: itunes.album || track.collectionName || track.album || '',
year: itunes.year || (track.releaseDate ? new Date(track.releaseDate).getFullYear() : undefined) || track.year, year: itunes.year || (track.releaseDate ? new Date(track.releaseDate).getFullYear() : undefined) || track.year,