fix: Playlist retrieval if list has podcasts (#1691)

* fix: Playlist retrieval if list has podcasts

Fixes #1689

* fix: Gracefully handle unknown playable types

Author: hrkfdn

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## Unreleased
+
+### Fixed
+
+- Playlist retrieval crashing when list contains podcast episodes
+
 ## [1.3.1]
 
 ### Fixed

Cargo.lock

@@ -144,6 +144,7 @@ impl From<&PlayableItem> for Playable {
         match item {
             PlayableItem::Episode(episode) => Self::Episode(episode.into()),
             PlayableItem::Track(track) => Self::Track(track.into()),
+            PlayableItem::Unknown(value) => panic!("Unknown playable item {value:?}"),
         }
     }
 }

src/model/playable.rs

@@ -431,6 +431,16 @@ impl WebApi {
                         items: page
                             .items
                             .iter()
+                            .filter(|pt| {
+                                if let Some(t) = pt.track.as_ref()
+                                    && !t.is_unknown()
+                                {
+                                    true
+                                } else {
+                                    error!("Could not process item {pt:?}, ignoring");
+                                    false
+                                }
+                            })
                             .enumerate()
                             .flat_map(|(index, pt)| {
                                 pt.track.as_ref().map(|t| {