feat(player): set a null video options will clear the list of video i…

…tems
apivideo · Nov 9, 2023 · 26e5f8a · 26e5f8a
1 parent 0494f7f
commit 26e5f8a
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/player/src/main/java/video/api/player/ApiVideoPlayerController.kt b/player/src/main/java/video/api/player/ApiVideoPlayerController.kt
@@ -231,15 +231,17 @@ constructor(
     var videoOptions: VideoOptions?
         get() = exoplayer.currentVideoOptions
         set(value) {
-            value?.let {
-                mediaSourceFactory = ApiVideoExoPlayerMediaFactory(it) { error ->
+            if (value != null) {
+                mediaSourceFactory = ApiVideoExoPlayerMediaFactory(value) { error ->
                     listeners.forEach { listener ->
                         listener.onError(error)
                     }
                 }.apply {
                     exoplayer.setMediaSource(this)
                 }
-            } ?: throw IllegalArgumentException("VideoOptions cannot be null")
+            } else {
+                exoplayer.clearMediaItems()
+            }
         }
 
     private val exoplayerListener: AnalyticsListener = object : AnalyticsListener {