kavin/LibreTube
1
0
Fork 0
mirror of https://github.com/libre-tube/LibreTube.git synced 2025-04-28 07:50:31 +05:30
Code Issues Projects Releases Wiki Activity

fix: error toasts when switching between audio/video playback

Browse Source
This commit is contained in:
Bnyro 2024-11-19 14:07:55 +01:00
parent 4441a2bf4e
commit b1a3a80ab3
3 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/src/main/java/com/github/libretube/services/AbstractPlayerService.kt
View File

@ -98,7 +98,7 @@ abstract class AbstractPlayerService : MediaLibraryService(), MediaLibrarySessio
onServiceCreated(args)
notificationProvider?.intentActivity = getIntentActivity()
startPlayback()
if (::videoId.isInitialized) startPlayback()
}
}
STOP_SERVICE_ACTION -> {
@ -330,8 +330,6 @@ abstract class AbstractPlayerService : MediaLibraryService(), MediaLibrarySessio
}
}
fun isVideoIdInitialized() = this::videoId.isInitialized
/**
* Stop the service when app is removed from the task manager.
*/

2
app/src/main/java/com/github/libretube/services/OfflinePlayerService.kt
View File

@ -79,8 +79,6 @@ open class OfflinePlayerService : AbstractPlayerService() {
* Attempt to start an audio player with the given download items
*/
override suspend fun startPlayback() {
if (!isVideoIdInitialized()) return
val downloadWithItems = withContext(Dispatchers.IO) {
Database.downloadDao().findById(videoId)
}!!

4
app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
View File

@ -566,7 +566,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
}
binding.playImageView.setOnClickListener {
playerController.togglePlayPauseState()
if (::playerController.isInitialized) playerController.togglePlayPauseState()
}
activity?.supportFragmentManager
@ -628,7 +628,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
binding.relPlayerBackground.setOnClickListener {
// pause the current player
playerController.pause()
if (::playerController.isInitialized) playerController.pause()
// start the background mode
playOnBackground()