joshpatra/allstarr
1
0
Fork 0
mirror of https://github.com/SoPat712/allstarr.git synced 2026-02-09 15:45:10 -05:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

fix: enable deduplication for cache mode
Some checks failed
Docker Build & Push / build-and-test (push) Has been cancelled
Details
Docker Build & Push / docker (push) Has been cancelled
Details

Browse Source 
- Cache mode now registers downloaded songs in mappings
- Prevents duplicate downloads in cache mode
- Fixes wasted API calls and duplicate cache files
- Both cache and download modes benefit from deduplication
This commit is contained in:
Josh Patra
2026-01-30 14:11:47 -05:00
parent 2297455923
commit bc4e5d9442
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
allstarr/Services/Common/BaseDownloadService.cs
View File

@@ -298,11 +298,8 @@ public abstract class BaseDownloadService : IDownloadService
song.LocalPath = localPath;
// Register BEFORE releasing lock to prevent race conditions
if (!isCache)
{
await LocalLibraryService.RegisterDownloadedSongAsync(song, localPath);
}
// Register BEFORE releasing lock to prevent race conditions (both cache and download modes)
await LocalLibraryService.RegisterDownloadedSongAsync(song, localPath);
// Check if this track belongs to a playlist and update M3U
if (PlaylistSyncService != null)
@@ -322,7 +319,7 @@ public abstract class BaseDownloadService : IDownloadService
}
}
// Trigger library scan and album download AFTER releasing lock
// Trigger library scan and album download AFTER releasing lock (download mode only)
if (!isCache)
{
// Trigger a Subsonic library rescan (with debounce)