fix codacy

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>

Author: alperozturk96

app/src/main/java/com/nextcloud/client/player/ui/PlayerActivity.kt

@@ -13,6 +13,7 @@ import android.content.Intent
 import android.content.res.Configuration
 import android.graphics.Rect
 import android.media.AudioManager
+import android.os.Build
 import android.os.Bundle
 import android.util.Rational
 import android.view.View
@@ -119,10 +120,9 @@ class PlayerActivity :
         playerView.onStart()
     }
 
-    private fun Intent.getPlaybackFileType(): PlaybackFileType {
-        return getSerializableArgument(PLAYBACK_FILE_TYPE, PlaybackFileType::class.java)
+    private fun Intent.getPlaybackFileType(): PlaybackFileType =
+        getSerializableArgument(PLAYBACK_FILE_TYPE, PlaybackFileType::class.java)
             ?: throw IllegalStateException("Playback file type was not defined")
-    }
 
     override fun onStart() {
         super.onStart()
@@ -175,11 +175,14 @@ class PlayerActivity :
 
     private fun createPictureInPictureParams(): PictureInPictureParams = PictureInPictureParams.Builder().let {
         it.setAspectRatio(pipAspectRatio)
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+            it.setAutoEnterEnabled(true)
+        }
         getSourceRectHint().let(it::setSourceRectHint)
         it.build()
     }
 
-    private fun getSourceRectHint(): Rect? {
+    private fun getSourceRectHint(): Rect {
         val containerRect = Rect()
         playerView.getGlobalVisibleRect(containerRect)
         val sourceHeightHint = (containerRect.width() / pipAspectRatio.toFloat()).toInt()

app/src/main/java/com/nextcloud/client/player/ui/control/PlayerControlView.kt

@@ -190,26 +190,35 @@ class PlayerControlView @JvmOverloads constructor(
     private fun renderRepeatButton(repeatSingle: Boolean) {
         binding.ivRepeat.iconTint = ContextCompat.getColorStateList(
             binding.root.context,
-            if (repeatSingle) R.color.player_accent_color
-            else R.color.player_default_icon_color
+            if (repeatSingle) {
+                R.color.player_accent_color
+            } else {
+                R.color.player_default_icon_color
+            }
         )
         binding.ivRepeat.tag = if (repeatSingle) TAG_CLICK_COMMAND_DO_NOT_REPEAT else TAG_CLICK_COMMAND_REPEAT
     }
 
     private fun renderShuffleButton(shuffle: Boolean) {
         binding.ivRandom.iconTint = ContextCompat.getColorStateList(
             binding.root.context,
-            if (shuffle) R.color.player_accent_color
-            else R.color.player_default_icon_color
+            if (shuffle) {
+                R.color.player_accent_color
+            } else {
+                R.color.player_default_icon_color
+            }
         )
         binding.ivRandom.tag = if (shuffle) TAG_CLICK_COMMAND_DO_NOT_SHUFFLE else TAG_CLICK_COMMAND_SHUFFLE
     }
 
     private fun renderPlayPauseButton(isPlaying: Boolean) {
         binding.ivPlayPause.icon = AppCompatResources.getDrawable(
             binding.root.context,
-            if (isPlaying) R.drawable.player_ic_pause
-            else R.drawable.player_ic_play
+            if (isPlaying) {
+                R.drawable.player_ic_pause
+            } else {
+                R.drawable.player_ic_play
+            }
         )
         binding.ivPlayPause.tag = if (isPlaying) TAG_CLICK_COMMAND_PAUSE else TAG_CLICK_COMMAND_PLAY
     }