Comment 21 for bug 588858

Revision history for this message
reacocard (reacocard) wrote :

> Regarding the track ID, actually it should not be the URI at all.
> I actually suggest using '/org/exaile/trackid/' + str(hash(track)).

Actually, since Track objects in Exaile are unique per-uri, that's effectively the same as just using the uri outright.

The proper way to do this would be to take advantage of playlists in trunk using MetadataLists and storing a unique id in "mpris_track_id" or something for each track. I don't think the MetadataList api is exposed on Playlist itself yet but that's easy enough to fix. This would require a fair bit of extra code though, so for now I'm in favor of just not supporting the TrackList section of the MPRIS api.

I don't have any objection to supporting a minimal MPRIS2 in mainline alongside our current dbus api - it's a pretty small amount of extra code and opens a lot of possibilities for integration with other apps. Extra MPRIS2 features like TrackList can either be added later when we figure out a good way to do them, or can be added onto the core via plugins.