Reset Playback Engine on PlaybackInfo fetch error

This can happen at two points: * On load for the current item, which is when the PBI fetch error is raised as a player error. * On playback start of the next item, which when the PBI fetch error is raised as a player error. While this delays the issue, it is good, because it allows playback of the current item to finish.
tidal-music · Aug 21, 2024 · 88a8ca4 · 88a8ca4
1 parent 693595f
commit 88a8ca4
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/...ack-engine/src/main/kotlin/com/tidal/sdk/player/playbackengine/ExoPlayerPlaybackEngine.kt b/...ack-engine/src/main/kotlin/com/tidal/sdk/player/playbackengine/ExoPlayerPlaybackEngine.kt
@@ -802,12 +802,17 @@ internal class ExoPlayerPlaybackEngine(
         }
 
         val eventError = errorHandler.getErrorEvent(error, forwardingMediaProduct?.productType)
+        coroutineScope.launch {
+            eventSink.emit(eventError)
+        }
         playbackInfoFetchException.report(errorMessage, eventError.errorCode)
         val matchingMediaProduct = eventTime.correspondingForwardingMediaProductIfMatching
         if (matchingMediaProduct === forwardingMediaProduct ||
             matchingMediaProduct === nextForwardingMediaProduct
         ) {
-            if (matchingMediaProduct === forwardingMediaProduct) {
+            if (playbackInfoFetchException != null) {
+                reset()
+            } else if (matchingMediaProduct === forwardingMediaProduct) {
                 val positionInSeconds =
                     if (forwardingMediaProduct?.productType == ProductType.BROADCAST) {
                         extendedExoPlayer.currentPositionSinceEpochMs
@@ -822,9 +827,6 @@ internal class ExoPlayerPlaybackEngine(
                 )
             }
         }
-        coroutineScope.launch {
-            eventSink.emit(eventError)
-        }
     }
 
     override fun onVideoSizeChanged(eventTime: EventTime, videoSize: VideoSize) {