From f537984bbf2c7e4145b2fd8506e3c2df9ec3b9e2 Mon Sep 17 00:00:00 2001
From: Deluan <deluan@deluan.com>
Date: Tue, 7 Apr 2020 11:55:45 -0400
Subject: [PATCH] Use `trackId` instead of simply `id`, as it seems to conflict
 with internal id generated by the player. fixes #153

---
 ui/src/audioplayer/Player.js | 10 +++++-----
 ui/src/audioplayer/queue.js  |  3 ++-
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/ui/src/audioplayer/Player.js b/ui/src/audioplayer/Player.js
index 4544699d1..5b16b3c10 100644
--- a/ui/src/audioplayer/Player.js
+++ b/ui/src/audioplayer/Player.js
@@ -58,17 +58,17 @@ const Player = () => {
     if (isNaN(info.duration) || progress < 90) {
       return
     }
-    const item = queue.queue.find((item) => item.id === info.id)
+    const item = queue.queue.find((item) => item.trackId === info.trackId)
     if (item && !item.scrobbled) {
-      dispatch(scrobbled(info.id))
-      subsonic.scrobble(info.id, true)
+      dispatch(scrobbled(info.trackId))
+      subsonic.scrobble(info.trackId, true)
     }
   }
 
   const OnAudioPlay = (info) => {
     if (info.duration) {
-      subsonic.scrobble(info.id, false)
-      dataProvider.getOne('keepalive', { id: info.id })
+      subsonic.scrobble(info.trackId, false)
+      dataProvider.getOne('keepalive', { id: info.trackId })
     }
   }
 
diff --git a/ui/src/audioplayer/queue.js b/ui/src/audioplayer/queue.js
index dece0326f..d7129ae93 100644
--- a/ui/src/audioplayer/queue.js
+++ b/ui/src/audioplayer/queue.js
@@ -9,6 +9,7 @@ const PLAYER_PLAY_ALBUM = 'PLAYER_PLAY_ALBUM'
 
 const mapToAudioLists = (item) => ({
   id: item.id,
+  trackId: item.id,
   name: item.title,
   singer: item.artist,
   cover: subsonic.url('getCoverArt', item.id, { size: 300 }),
@@ -60,7 +61,7 @@ const playQueueReducer = (
       const newQueue = previousState.queue.map((item) => {
         return {
           ...item,
-          scrobbled: item.scrobbled || item.id === data
+          scrobbled: item.scrobbled || item.trackId === data
         }
       })
       return { queue: newQueue, clear: false }