yetangitu/librespot
1
0
Fork 0
mirror of https://github.com/librespot-org/librespot.git synced 2025-10-03 17:59:24 +02:00
Code Issues Releases Wiki Activity

feat(mpris): Check track_id when setting position

Browse source
This commit is contained in:
Paul Fariello 2025-09-30 08:11:26 +02:00
parent b0e0393b87
commit 4c37f2b8f4
1 changed files with 12 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/mpris_event_handler.rs
View file

@ -1,7 +1,7 @@
use std::{collections::HashMap, process, sync::Arc, time::Instant};
use librespot_connect::Spirc;
use log::{debug, warn};
use log::{debug, info, warn};
use thiserror::Error;
use time::format_description::well_known::Iso8601;
use tokio::sync::mpsc;
@ -723,12 +723,21 @@ impl MprisPlayerService {
// * `position`: Track position in microseconds. This must be between 0 and `track_length`.
async fn set_position(&self, track_id: zbus::zvariant::ObjectPath<'_>, position: TimeInUs) {
debug!("org.mpris.MediaPlayer2.Player::SetPosition({track_id:?}, {position:?})");
// FIXME: handle track_id
if position < 0 {
return;
}
if let Some(spirc) = &self.spirc {
let _ = spirc.set_position_ms((position / 1000) as u32);
let current_track_id = self.metadata.mpris.track_id.as_ref().and_then(|track_id| {
track_id
.to_id()
.ok()
.map(|id| format!("/org/librespot/track/{id}"))
});
if current_track_id.as_deref() == Some(track_id.as_str()) {
let _ = spirc.set_position_ms((position / 1000) as u32);
} else {
info!("SetPosition on wrong trackId, ignoring as stale");
}
}
}