# Rapport : Correction du bug URL relative de la cover ## Tâche originale Le document DIDL généré par les PMO playlists possédait une URL absolue pour le flux audio, mais une URL relative pour la cover. Les deux entités (flux audio et cover) sont stockées dans des caches `pmoaudiocache` et `pmocovers` respectivement. ## Analyse ### Architecture des URLs dans PMOMusic 1. **`pmocache::FileCache::route_for`** retourne une route **relative** (ex: `/audio/flac/abc123`, `/covers/images/def456`) 2. **Les sources (`pmoqobuz`, `pmoparadise`)** reçoivent un `base_url` à leur création et sont responsables de convertir les URLs relatives en absolues avant de retourner les résultats de Browse. 3. **`pmoplaylist::ReadHandle::to_items`** génère des items DIDL avec des URLs relatives pour l'audio ET la cover. ### Localisation du bug Le bug se trouvait dans **`pmoqobuz/src/source.rs`** dans la fonction `adapt_playlist_items_to_qobuz` (ligne ~595). Cette fonction convertissait correctement l'URL audio en URL absolue : ```rust if let Some(resource) = item.resources.first_mut() { if resource.url.starts_with('/') { resource.url = format!("{}{}", self.inner.base_url, resource.url); } } ``` **Mais ne faisait pas la même conversion pour `item.album_art`** (la cover). ### Comparaison avec pmoparadise `pmoparadise/src/source.rs` fait correctement les deux conversions (lignes 505-526 et 585-595) : ```rust // Pour l'audio if resource.url.starts_with('/') { resource.url = format!("{}{}", self.base_url, resource.url); } // Pour la cover if let Some(art) = item.album_art.as_mut() { if art.starts_with('/') { *art = format!("{}{}", self.base_url, art); } } ``` ## Correction appliquée ### Fichier modifié - `pmoqobuz/src/source.rs` ### Modification Ajout de la conversion de l'URL de la cover dans `adapt_playlist_items_to_qobuz` : ```rust // Convertir l'URL de la cover en URL absolue si elle est relative if let Some(art) = item.album_art.as_mut() { if art.starts_with('/') { *art = format!("{}{}", self.inner.base_url, art); } } ``` Cette modification a été ajoutée après la conversion de l'URL audio et avant l'assignation du `parent_id`. ## Vérification La compilation de `pmoqobuz` réussit après la correction. ## Remarques Le pattern de conversion des URLs relatives en absolues est cohérent dans le projet : chaque source qui utilise `pmoplaylist::to_items()` doit post-traiter les items pour convertir les URLs relatives (`/audio/...`, `/covers/...`) en URLs absolues en utilisant son `base_url`.