From 9ec7e3a01f6ca34ab95bd6fec3f078c6c673af6c Mon Sep 17 00:00:00 2001 From: Relwi Date: Sat, 5 Oct 2024 09:10:43 +0200 Subject: [PATCH] refactor: click on video playlist won't load if autoplay is off --- .../com/github/libretube/ui/fragments/PlayerFragment.kt | 6 +++--- 1 file changed, 3 insertions(+), 3 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 7291c18df7..c33a657fa1 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 @@ -970,7 +970,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { (videoStream?.height ?: 0) > (videoStream?.width ?: 0) PlayingQueue.setOnQueueTapListener { streamItem -> - streamItem.url?.toID()?.let { playNextVideo(it, true) } + streamItem.url?.toID()?.let { playNextVideo(it) } } // hide the button to skip SponsorBlock segments manually @@ -1042,13 +1042,13 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions { /** * Can be used for autoplay and manually skipping to the next video. */ - private fun playNextVideo(nextId: String? = null, isOnQueueClick: Boolean = false) { + private fun playNextVideo(nextId: String? = null) { if (nextId == null && PlayingQueue.repeatMode == Player.REPEAT_MODE_ONE) { viewModel.player.seekTo(0) return } - if (!PlayerHelper.isAutoPlayEnabled(playlistId != null) && !isOnQueueClick) return + if (!PlayerHelper.isAutoPlayEnabled(playlistId != null) && nextId == null) return // save the current watch position before starting the next video saveWatchPosition()