fix: split on & when deriving artist folder name
"Arriez & King Dif" was used as-is for the folder; now splits on , ; & so only the first artist is used: "Arriez". Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
1 parent
11a31f85aa
commit
0823c1ca86
2 files changed
+2
-2
No files matched your search
@@ -49,7 +49,7 @@ function buildVariantFilter(title) {
|
|||||||
|
|
||||||
async function downloadTrack({ jobId, artist, albumArtist, title, album, year, trackNum, totalTracks, discNum, discTotal, compilation, genre, artworkUrl, explicit, outputDir, durationMs, sourceUrl }) {
|
async function downloadTrack({ jobId, artist, albumArtist, title, album, year, trackNum, totalTracks, discNum, discTotal, compilation, genre, artworkUrl, explicit, outputDir, durationMs, sourceUrl }) {
|
||||||
const mainArtist = albumArtist || artist;
|
const mainArtist = albumArtist || artist;
|
||||||
const safeArtist = sanitize(mainArtist.split(',')[0].trim());
|
const safeArtist = sanitize(mainArtist.split(/[,;&]/)[0].trim());
|
||||||
const safeAlbum = sanitize(album || 'Singles');
|
const safeAlbum = sanitize(album || 'Singles');
|
||||||
const dir = path.join(outputDir || CONFIG.musicPath, safeArtist, safeAlbum);
|
const dir = path.join(outputDir || CONFIG.musicPath, safeArtist, safeAlbum);
|
||||||
fs.mkdirSync(dir, { recursive: true });
|
fs.mkdirSync(dir, { recursive: true });
|
||||||
|
|||||||
@@ -718,7 +718,7 @@ app.post('/api/edit-metadata', async (req, res) => {
|
|||||||
const oldAlbumDir = path.dirname(currentFile);
|
const oldAlbumDir = path.dirname(currentFile);
|
||||||
const oldArtistDir = path.dirname(oldAlbumDir);
|
const oldArtistDir = path.dirname(oldAlbumDir);
|
||||||
const newAlbumName = album ? sanitize(album) : path.basename(oldAlbumDir);
|
const newAlbumName = album ? sanitize(album) : path.basename(oldAlbumDir);
|
||||||
const newArtistName = normalAlbumArtist ? sanitize(normalAlbumArtist.split(/[,;]/)[0].trim()) : path.basename(oldArtistDir);
|
const newArtistName = normalAlbumArtist ? sanitize(normalAlbumArtist.split(/[,;&]/)[0].trim()) : path.basename(oldArtistDir);
|
||||||
const newDir = path.join(CONFIG.musicPath, newArtistName, newAlbumName);
|
const newDir = path.join(CONFIG.musicPath, newArtistName, newAlbumName);
|
||||||
|
|
||||||
if (newDir !== oldAlbumDir) {
|
if (newDir !== oldAlbumDir) {
|
||||||
|
|||||||
Reference in new issue
Block a user