No seek event while in pause (#268)

SRGSSR · Oct 20, 2023 · beda01c · beda01c
1 parent 003427a
commit beda01c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/...siness/src/androidTest/java/ch/srgssr/pillarbox/core/business/CommandersActTrackerTest.kt b/...siness/src/androidTest/java/ch/srgssr/pillarbox/core/business/CommandersActTrackerTest.kt
@@ -130,8 +130,6 @@ class CommandersActTrackerTest {
         val expected = listOf(
             MediaEventType.Play.toString(),
             MediaEventType.Pause.toString(),
-            MediaEventType.Seek.toString(),
-            MediaEventType.Pause.toString(),
             MediaEventType.Stop.toString()
         )
         launch(Dispatchers.Main) {

diff --git a/...in/java/ch/srgssr/pillarbox/core/business/tracker/commandersact/CommandersActStreaming.kt b/...in/java/ch/srgssr/pillarbox/core/business/tracker/commandersact/CommandersActStreaming.kt
@@ -141,7 +141,7 @@ internal class CommandersActStreaming(
     }
 
     private fun notifyPause() {
-        if (state == State.Idle) return
+        if (state != State.Playing) return
         this.state = State.Paused
         notifyEvent(MediaEventType.Pause, player.currentPosition.milliseconds)
         stopHeartBeat()
@@ -155,7 +155,7 @@ internal class CommandersActStreaming(
     }
 
     private fun notifySeek(seekStartPosition: Duration) {
-        if (state == State.Seeking || state == State.Idle) return
+        if (state != State.Playing) return
         state = State.Seeking
         notifyEvent(MediaEventType.Seek, seekStartPosition)
     }