Merge pull request #6694 from Bnyro/master

fix: hide download buttons when current video is live

app/src/main/java/com/github/libretube/api/obj/StreamItem.kt

@@ -24,7 +24,7 @@ data class StreamItem(
     val shortDescription: String? = null,
     val isShort: Boolean = false
 ) : Parcelable {
-    val isLive get() = (duration != null) && (duration <= 0L)
+    val isLive get() = (duration == null) || (duration <= 0L)
 
     fun toLocalPlaylistItem(playlistId: String): LocalPlaylistItem {
         return LocalPlaylistItem(

app/src/main/java/com/github/libretube/db/obj/WatchHistoryItem.kt

@@ -21,6 +21,8 @@ data class WatchHistoryItem(
     @ColumnInfo val duration: Long? = null,
     @ColumnInfo val isShort: Boolean = false
 ) {
+    val isLive get() = (duration == null) || (duration <= 0L)
+
     fun toStreamItem() = StreamItem(
         url = videoId,
         type = StreamItem.TYPE_STREAM,

app/src/main/java/com/github/libretube/ui/adapters/WatchHistoryAdapter.kt

@@ -92,7 +92,10 @@ class WatchHistoryAdapter(
                 true
             }
 
-            if (video.duration != null) watchProgress.setWatchProgressLength(video.videoId, video.duration)
+            if (video.duration != null) watchProgress.setWatchProgressLength(
+                video.videoId,
+                video.duration
+            )
         }
     }
 }

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

@@ -1136,6 +1136,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
                 streams.uploaderSubscriberCount.formatShort()
             )
             player.isLive = streams.livestream
+            relPlayerDownload.isVisible = !streams.livestream
         }
         playerBinding.exoTitle.text = streams.title
 

app/src/main/java/com/github/libretube/ui/sheets/VideoOptionsBottomSheet.kt

@@ -52,6 +52,7 @@ class VideoOptionsBottomSheet : BaseBottomSheet() {
         }
 
         optionsList += listOf(R.string.addToPlaylist, R.string.download, R.string.share)
+        if (streamItem.isLive) optionsList.remove(R.string.download)
 
         setSimpleItems(optionsList.map { getString(it) }) { which ->
             when (optionsList[which]) {