Only show play option when mime type is audio or video

3 files changed, 6 insertions, 2 deletions

diff --git a/app/src/main/java/email/schaal/ocreader/ItemPagerActivity.kt b/app/src/main/java/email/schaal/ocreader/ItemPagerActivity.kt
index d56b1b36..949e4c8b 100644
--- a/app/src/main/java/email/schaal/ocreader/ItemPagerActivity.kt
+++ b/app/src/main/java/email/schaal/ocreader/ItemPagerActivity.kt
@@ -130,7 +130,7 @@ class ItemPagerActivity : AppCompatActivity() {
     }
 
     override fun onPrepareOptionsMenu(menu: Menu): Boolean {
-        menu.findItem(R.id.action_play_enclosure_media).isVisible = item.enclosureLink != null
+        menu.findItem(R.id.action_play_enclosure_media).isVisible = item.isPlayable()
         updateMenuItem(menu.findItem(R.id.menu_mark_read), !item.unread, R.drawable.ic_check_box, R.drawable.ic_check_box_outline_blank)
         updateMenuItem(menu.findItem(R.id.menu_mark_starred), item.starred, R.drawable.ic_star, R.drawable.ic_star_outline)
         return super.onPrepareOptionsMenu(menu)
diff --git a/app/src/main/java/email/schaal/ocreader/database/model/Item.kt b/app/src/main/java/email/schaal/ocreader/database/model/Item.kt
index d12a173d..e5842712 100644
--- a/app/src/main/java/email/schaal/ocreader/database/model/Item.kt
+++ b/app/src/main/java/email/schaal/ocreader/database/model/Item.kt
@@ -116,6 +116,10 @@ open class Item(
         RealmObject.deleteFromRealm(this)
     }
 
+    fun isPlayable(): Boolean {
+        return enclosureMime?.run { startsWith("video/") || startsWith("audio/") } ?: false
+    }
+
     fun play(context: Context) {
         if(enclosureLink != null) {
             val playIntent = Intent(Intent.ACTION_VIEW)
diff --git a/app/src/main/java/email/schaal/ocreader/view/ItemViewHolder.kt b/app/src/main/java/email/schaal/ocreader/view/ItemViewHolder.kt
index 2d75846b..cc4318df 100644
--- a/app/src/main/java/email/schaal/ocreader/view/ItemViewHolder.kt
+++ b/app/src/main/java/email/schaal/ocreader/view/ItemViewHolder.kt
@@ -59,7 +59,7 @@ class ItemViewHolder(private val binding: ListItemBinding, private val clickList
             clickListener.onItemLongClick(item, position)
             true
         }
-        if (item.enclosureLink != null) {
+        if (item.isPlayable()) {
             binding.play.visibility = View.VISIBLE
             binding.play.setOnClickListener { item.play(itemView.context) }
         } else {