92 lines
3.4 KiB
Markdown
92 lines
3.4 KiB
Markdown
|
# Bug: Transfert de Queue entre Renderers
|
||
|
|
|
||
|
|
**Date**: 2026-01-17
|
||
|
|
**Fichier principal**: `pmocontrol/src/control_point.rs`
|
||
|
|
**Fonction affectée**: `transfer_queue()`
|
||
|
|
|
||
|
|
## Symptôme
|
||
|
|
|
||
|
|
Le transfert de la queue de lecture d'un renderer vers un autre ne fonctionnait plus correctement, avec des comportements erratiques. La queue transférée était écrasée et le current_index perdu.
|
||
|
|
|
||
|
|
## Cause Racine
|
||
|
|
|
||
|
|
Dans `transfer_queue()` (lignes 1192-1291), lorsqu'un binding de playlist existait sur le renderer source, le code appelait `attach_queue_to_playlist()` sur la destination après avoir rempli la queue.
|
||
|
|
|
||
|
|
**Problème**: `attach_queue_to_playlist_internal()` effectue les opérations suivantes:
|
||
|
|
1. `clear_for_playlist_attach()` - efface la queue du renderer
|
||
|
|
2. `clear_queue()` - efface la queue locale
|
||
|
|
3. `refresh_attached_queue_for()` - browse le serveur et remplace la queue
|
||
|
|
|
||
|
|
Cela **écrasait complètement** les items qu'on venait de transférer avec `replace_queue()`.
|
||
|
|
|
||
|
|
### Séquence problématique (avant correction)
|
||
|
|
|
||
|
|
```
|
||
|
|
1. source_snapshot = get_renderer_queue_snapshot(source) // items + current_index
|
||
|
|
2. clear_renderer_queue(dest)
|
||
|
|
3. dest.replace_queue(source_snapshot.items, current_index) // Queue remplie ✓
|
||
|
|
4. attach_queue_to_playlist(dest, server, container) // ÉCRASE TOUT ✗
|
||
|
|
└─> clear_for_playlist_attach()
|
||
|
|
└─> clear_queue()
|
||
|
|
└─> refresh_attached_queue_for() → browse serveur → replace queue
|
||
|
|
5. play() sur destination avec mauvaise queue
|
||
|
|
```
|
||
|
|
|
||
|
|
## Correction Appliquée
|
||
|
|
|
||
|
|
Remplacement de l'appel `attach_queue_to_playlist()` par un transfert direct du binding sans déclencher de refresh:
|
||
|
|
|
||
|
|
```rust
|
||
|
|
// AVANT (problématique)
|
||
|
|
if let Some((server_id, container_id, _)) = source_binding {
|
||
|
|
self.attach_queue_to_playlist(dest_renderer_id, server_id, container_id)?;
|
||
|
|
}
|
||
|
|
|
||
|
|
// APRÈS (corrigé)
|
||
|
|
if let Some((server_id, container_id, has_seen_update)) = source_binding.clone() {
|
||
|
|
let binding = PlaylistBinding {
|
||
|
|
server_id,
|
||
|
|
container_id,
|
||
|
|
has_seen_update,
|
||
|
|
pending_refresh: false, // Pas de refresh immédiat
|
||
|
|
auto_play_on_refresh: false,
|
||
|
|
};
|
||
|
|
dest_renderer.set_playlist_binding(Some(binding));
|
||
|
|
}
|
||
|
|
```
|
||
|
|
|
||
|
|
### Séquence corrigée
|
||
|
|
|
||
|
|
```
|
||
|
|
1. source_snapshot = get_renderer_queue_snapshot(source)
|
||
|
|
2. clear_renderer_queue(dest)
|
||
|
|
3. dest.replace_queue(source_snapshot.items, current_index) // Queue remplie ✓
|
||
|
|
4. dest.set_playlist_binding(binding avec pending_refresh=false) // Binding transféré ✓
|
||
|
|
5. play() sur destination avec bonne queue ✓
|
||
|
|
```
|
||
|
|
|
||
|
|
## Événements
|
||
|
|
|
||
|
|
L'analyse a également confirmé que les émissions d'événements sont correctes:
|
||
|
|
|
||
|
|
| Méthode | Événement émis |
|
||
|
|
|---------|----------------|
|
||
|
|
| `replace_queue()` | `QueueUpdated` ✓ |
|
||
|
|
| `enqueue_items()` | `QueueUpdated` ✓ |
|
||
|
|
| `clear_queue()` | `QueueUpdated` ✓ |
|
||
|
|
| `set_playlist_binding()` | `BindingChanged` ✓ |
|
||
|
|
| `clear_playlist_binding()` | `BindingChanged` ✓ |
|
||
|
|
|
||
|
|
Le problème de lenteur UI mentionné était probablement lié au fait que la queue était écrasée puis re-remplie, causant plusieurs événements successifs et une confusion dans l'état affiché.
|
||
|
|
|
||
|
|
## Impact
|
||
|
|
|
||
|
|
- Transfert de queue fonctionnel à nouveau
|
||
|
|
- Préservation du current_index lors du transfert
|
||
|
|
- Binding de playlist correctement transféré sans perte de synchronisation
|
||
|
|
- UI réactive car un seul cycle d'événements cohérent
|
||
|
|
|
||
|
|
## Fichiers Modifiés
|
||
|
|
|
||
|
|
- `pmocontrol/src/control_point.rs`: Modification de `transfer_queue()` lignes 1223-1244
|