From 21ea77eadc08c6617fa1fd461cd0405d361070ad Mon Sep 17 00:00:00 2001 From: Eric Coissac Date: Sat, 4 Apr 2026 12:09:42 +0200 Subject: [PATCH] :art: Simplify album art URL transformation logic Refactor queue item cover image handling to directly access `album_art_uri` from the top-level struct instead of nested metadata, and update return type to use fully qualified path `crate::openapi::QueueSnapshot` for clarity. --- pmocontrol/src/pmoserver_ext.rs | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/pmocontrol/src/pmoserver_ext.rs b/pmocontrol/src/pmoserver_ext.rs index f0d68e0e..4b741077 100644 --- a/pmocontrol/src/pmoserver_ext.rs +++ b/pmocontrol/src/pmoserver_ext.rs @@ -230,7 +230,7 @@ async fn get_renderer_queue( State(state): State, Path(renderer_id): Path, headers: HeaderMap, -) -> Result, (StatusCode, Json)> { +) -> Result, (StatusCode, Json)> { let rid = DeviceId(renderer_id.clone()); let mut snapshot = state .control_point @@ -243,11 +243,9 @@ async fn get_renderer_queue( // Transform cover URLs in all queue items for item in &mut snapshot.queue.items { - if let Some(ref mut metadata) = item.metadata { - if let Some(ref album_art) = metadata.album_art_uri { - if let Some(transformed) = transform_cover_url(Some(album_art), &base_url).await { - metadata.album_art_uri = Some(transformed); - } + if let Some(ref album_art) = item.album_art_uri { + if let Some(transformed) = transform_cover_url(Some(album_art), &base_url).await { + item.album_art_uri = Some(transformed); } } }