Eric Coissac
5701fbf465
Fix bug where tracks were duplicated in queue position 0 during playlist refreshes by comparing items with URI or didl_id Fix race condition in internal queue playback where transient STOPPED states caused unwanted auto-advance - Updated sync_queue in interne.rs to use didl_id as fallback for URI comparison - Added items_match function in openhome.rs for robust item comparison - Modified lcs_flags in openhome.rs to use items_match for LCS algorithm - Added has_played_since_track_start flag in musicrenderer.rs to prevent auto-advance on transient STOPPED states - Updated play_* methods in musicrenderer.rs to reset the has_played flag before starting playback - Added diagnostic logs in sync_queue for tracking item matching issues
2.5 KiB
Bug : Duplication de piste en position 0 lors de la lecture
Statut : Terminé
Crate : pmocontrol
Description initiale du bug
Contexte
Lecture d'une playlist liée (bindée) à une queue OpenHome ou interne.
Comportement observé
- Sélection d'une playlist → la queue se charge correctement
- Lecture démarre à la piste 1 → OK
- Fin de la piste 1 → passage à la piste 2
- BUG : Après un certain délai (~60s), la piste 2 est dupliquée en position 0
- La lecture continue depuis cette nouvelle position 0
Symptôme clé
Toute piste en cours de lecture finit par être dupliquée en première position de la queue.
Analyse et cause racine
Mécanisme du bug
La fonction sync_queue (appelée lors des refreshes périodiques de playlist toutes les 60 secondes) comparait les items uniquement par leur URI.
Si le MediaServer retournait une URI légèrement différente pour le même morceau (tokens de session, encodage différent, etc.), l'item courant n'était pas reconnu dans la nouvelle playlist et était préservé en position 0, créant une duplication.
Flux problématique
1. Playlist attachée → lecture piste N
2. Refresh périodique (60s) → sync_queue()
3. Comparaison URI courante vs URIs playlist
4. URI non trouvée → piste préservée en position 0
5. Résultat : duplication
Solution implémentée
Principe
Extension de la logique de comparaison pour utiliser l'URI OU le didl_id comme critère d'identification. Le didl_id est l'identifiant DIDL-Lite stable assigné par le MediaServer, indépendant de l'URI de streaming.
Fichiers modifiés
| Fichier | Modifications |
|---|---|
pmocontrol/src/queue/interne.rs |
Comparaison par didl_id en fallback + logs diagnostic |
pmocontrol/src/queue/openhome.rs |
Fonction items_match + modification sync_queue et lcs_flags |
Code clé
// pmocontrol/src/queue/openhome.rs
fn items_match(a: &PlaybackItem, b: &PlaybackItem) -> bool {
a.uri == b.uri || a.didl_id == b.didl_id
}
// pmocontrol/src/queue/interne.rs
let new_idx = items.iter().position(|item| item.uri == current_uri)
.or_else(|| items.iter().position(|item| item.didl_id == current_didl_id));
Diagnostic
Pour activer les logs :
RUST_LOG=pmocontrol::queue=debug
Messages de trace :
sync_queue: current item found in new playlist→ Comportement normalsync_queue: current item NOT found in new playlist→ Cas problématique (ne devrait plus apparaître)