This goes right back to my long-standing feeling that album artist(s)
roles need to be assigned for _every_ album at scan time. Without them,
the queries and the logic involved just to generate an album's "by" line
are very inefficient. A page requiring the display of dozens of albums
can require hundreds of queries instead of the single query that should
be necessary.
Well, it wouldn't solve this problem here: as we can have multiple album
artists, we'd still have to look them up in a separate table, adding the
same overhead.
FWIW: I think I have some working code to get the full list of
contributors. It does indeed come with a performance penalty. Whether
this penalty hurts or not, I can't say. I'd like to make it optional
anyway, as after a scan we should know whether the user has multiple
contributors for those roles or not. And some users probably don't want
to have potentially long lists of contributors eg. in the web UI's album
list, preferring the slightly incomplete list we have today.
Tomorrow will be national holiday over here - a long weekend for myself.
But I'm trying to still get something out today. Expect some changes to
give you the full album artists list when browsing albums. Thanks for
testing :-).
Do you have a list of places where this incomplete list of contributors
would be shown?
I'm not aware of any place where it is -not- shown for albums. Have you
found one?
When you drill down into an album (to the track list), you would get the
full list, wouldn't you? I think this limitation is mostly about album
lists.
The "list albums by band" option otoh is being used when displaying
albums (in some places only, as mentioned above).
It _is_ being used? That's not my understanding. Is it only used with
Mp3/ID3v2 tagging?
Yes, it's being used in some rare cases, like probably in MIP mixes, or
in the ip3k display. But I'm ready to pull it. Actually, I already did,
but didn't push the change yet. We'll see if anybody will miss it.
--
--
Michael
_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/beta
