fix(jellyfin): Use correct id for tracking first-time seen media

FoxxMD · Oct 14, 2024 · bb13210 · bb13210
1 parent e251da9
commit bb13210
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/backend/sources/JellyfinApiSource.ts b/src/backend/sources/JellyfinApiSource.ts
@@ -312,7 +312,6 @@ export default class JellyfinApiSource extends MemorySource {
             DeviceId,
             DeviceName,
             Client,
-            Id,
             LastActivityDate,
             PlayState: {
                 PositionTicks,
@@ -323,11 +322,6 @@ export default class JellyfinApiSource extends MemorySource {
         const msDeviceId = combinePartsToString([shortDeviceId(DeviceId), DeviceName, Client]);
         const playerPosition = PositionTicks !== undefined ? ticksToSeconds(PositionTicks) : undefined; // dayjs.duration(PositionTicks / 1000, 'ms').asSeconds() : undefined;
 
-        if(this.config.options.logPayload && !this.mediaIdsSeen.includes(Id)) {
-            this.logger.debug(`First time seeing media ${Id} on ${msDeviceId} (play position ${playerPosition}) => ${JSON.stringify(NowPlayingItem)}`);
-            this.mediaIdsSeen.push(Id);
-        }
-
         let play: PlayObject | undefined;
         if(NowPlayingItem !== undefined) {
             const sessionPlay = JellyfinApiSource.formatPlayObj(NowPlayingItem);
@@ -342,6 +336,11 @@ export default class JellyfinApiSource extends MemorySource {
                     trackProgressPosition: playerPosition
                 }
             }
+
+            if(this.config.options.logPayload && !this.mediaIdsSeen.includes(NowPlayingItem.Id)) {
+                this.logger.debug(`First time seeing media ${NowPlayingItem.Id} on ${msDeviceId} (play position ${playerPosition}) => ${JSON.stringify(NowPlayingItem)}`);
+                this.mediaIdsSeen.push(NowPlayingItem.Id);
+            }
         }
 
         let reportedStatus = REPORTED_PLAYER_STATUSES.stopped;