antebudimir/feishin
1
0
Fork 0
mirror of https://github.com/antebudimir/feishin.git synced 2025-12-31 18:13:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

bugfix: remove duplicate scrobbling when there is only one item in the queue (#1222)

Browse source
This commit is contained in:
Kendall Garner 2025-10-25 20:25:07 +00:00 • committed by GitHub
parent ff0ca09644
commit 58ccd0cfd0
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/renderer/features/player/hooks/use-scrobble.ts
View file

@ -268,6 +268,7 @@ export const useScrobble = () => {
}
} else {
const isLastTrackInQueue = usePlayerStore.getState().actions.checkIsLastTrack();
const isFisrtTrackInQueue = usePlayerStore.getState().actions.checkIsFirstTrack();
const previousTimeSec = previous[1];
// If not already scrobbled, send a 'submission' scrobble if conditions are met
@ -275,8 +276,12 @@ export const useScrobble = () => {
scrobbleAtDurationMs: (scrobbleSettings?.scrobbleAtDuration ?? 0) * 1000,
scrobbleAtPercentage: scrobbleSettings?.scrobbleAtPercentage,
// If scrobbling the last song in the queue, use the previous time instead of the current time since otherwise time value will be 0
// Note that if the queue has one item (both first and last), use the elapsed time, as this will otherwise result
// in duplicate scrobbles in tandem with handleScrobbleFromSongChange
songCompletedDurationMs:
(isLastTrackInQueue ? previousTimeSec : currentTimeSec) * 1000,
(isLastTrackInQueue && !isFisrtTrackInQueue
? previousTimeSec
: currentTimeSec) * 1000,
songDurationMs: currentSong.duration,
});