From 4beb77fb112307433addc36c42de07347ed7c0d0 Mon Sep 17 00:00:00 2001
From: Bnyro <bnyro@tutanota.com>
Date: Sun, 22 Jan 2023 19:35:36 +0100
Subject: [PATCH] Cleanup with some `when` expressions

---
 .../libretube/ui/fragments/PlayerFragment.kt  | 17 +++++------
 .../libretube/ui/views/CustomExoPlayerView.kt | 29 +++++++------------
 2 files changed, 18 insertions(+), 28 deletions(-)

diff --git a/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
index a038814e7..1359a0033 100644
--- a/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
+++ b/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
@@ -1115,16 +1115,13 @@ class PlayerFragment : BaseFragment(), OnlinePlayerOptions {
     }
 
     private fun updatePlayPauseButton() {
-        if (exoPlayer.isPlaying) {
-            // video is playing
-            binding.playImageView.setImageResource(R.drawable.ic_pause)
-        } else if (exoPlayer.playbackState == Player.STATE_ENDED) {
-            // video has finished
-            binding.playImageView.setImageResource(R.drawable.ic_restart)
-        } else {
-            // player in any other state
-            binding.playImageView.setImageResource(R.drawable.ic_play)
-        }
+        binding.playImageView.setImageResource(
+            when {
+                exoPlayer.isPlaying -> R.drawable.ic_pause
+                exoPlayer.playbackState == Player.STATE_ENDED -> R.drawable.ic_restart
+                else -> R.drawable.ic_play
+            }
+        )
     }
 
     private fun initializeRelatedVideos(relatedStreams: List<StreamItem>?) {
diff --git a/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt b/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt
index 568b5ef5d..33a4f2502 100644
--- a/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt
+++ b/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt
@@ -152,16 +152,12 @@ internal class CustomExoPlayerView(
         }
 
         binding.playPauseBTN.setOnClickListener {
-            if (player?.isPlaying == false) {
-                // start or go on playing
-                if (player?.playbackState == Player.STATE_ENDED) {
-                    // restart video if finished
+            when {
+                player?.isPlaying == false && player?.playbackState == Player.STATE_ENDED -> {
                     player?.seekTo(0)
                 }
-                player?.play()
-            } else {
-                // pause the video
-                player?.pause()
+                player?.isPlaying == false -> player?.play()
+                else -> player?.pause()
             }
         }
 
@@ -198,16 +194,13 @@ internal class CustomExoPlayerView(
     }
 
     private fun updatePlayPauseButton() {
-        if (player?.isPlaying == true) {
-            // video is playing
-            binding.playPauseBTN.setImageResource(R.drawable.ic_pause)
-        } else if (player?.playbackState == Player.STATE_ENDED) {
-            // video has finished
-            binding.playPauseBTN.setImageResource(R.drawable.ic_restart)
-        } else {
-            // player in any other state
-            binding.playPauseBTN.setImageResource(R.drawable.ic_play)
-        }
+        binding.playPauseBTN.setImageResource(
+            when {
+                player?.isPlaying == true -> R.drawable.ic_pause
+                player?.playbackState == Player.STATE_ENDED -> R.drawable.ic_restart
+                else -> R.drawable.ic_play
+            }
+        )
     }
 
     private fun cancelHideControllerTask() {