Not to advertise empty playlists

[eddieb]

Please, do not advertise empty playlists …

[CCRDude]

No 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? 😉 ).

[rpedde]

My 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.

[eddieb]

for 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

[rpedde]

Re 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

[Author]

Posts