FireFly Media Server › Firefly Media Server Forums › Firefly Media Server › Feature Requests › Not to advertise empty playlists
- This topic has 4 replies, 3 voices, and was last updated 18 years ago by
rpedde.
-
AuthorPosts
-
04/11/2005 at 5:25 PM #120
eddieb
ParticipantPlease, do not advertise empty playlists …
14/11/2005 at 12:37 AM #3774CCRDude
ParticipantNo answer to this yet? 😉
My comment: when adding playlist folders, this may get a bit different since playlist folders may be empty playlists just containing all entries from sub-playlists. So I’ld delay this feature request until playlist folders are available.
After that… I’ld make it an option only, because showing empty playlists can be useful at least for debugging (if the playlist vanishes, is this due to a bug or due to the fact that it’s empty? 😉 ).
14/11/2005 at 1:21 AM #3775rpedde
ParticipantMy comment:
Playlist folders are in fact empty playlists with a new dmap flag.
Also, determining if a playlist is empty before serving it is expensive. It means that you have to have the correct playlist count when iTunes enumerates the containerlist. I guess sizes could be updated before a containerlist is pulled.
I dunno. I’m kind of iffy on this. Doing it for static playlists wouldn’t be too bad, but I think doing it for smart playlists would be expensive.
This is a “maybe”, and might not be possible in a manner that is efficient.
14/11/2005 at 2:21 AM #3776eddieb
Participantfor debugging it might be usefull, but bug should not exists …. B)
iTunes does not advertize empty playlists, so why should mt-daapd do ?
Since the number of songs in a playlist is counted in sql, it should not be that hard to not advertize empty playlist …
Eddie
14/11/2005 at 4:37 AM #3777rpedde
ParticipantRe iTunes does it that way:
iTunes doesn’t use a sql backend. It only recalculates smart playlists whenever songs are added, so it can keep a running count of the number of songs in each playlist. I can’t do that.
As I said, we’ll see. Depends on if I can do it efficiently or not.
— Ron
-
AuthorPosts
- The forum ‘Feature Requests’ is closed to new topics and replies.