The logic for Album Artists seems to be a bit mind-bending! I can manipulate the results of the import that happens in CUE.pm, but I don't really know what happens downstream. I don't know, for example, why there are two fields that seem to store the same information most of the time (ARTIST and ALBUMARTIST). Are the per-track ARTIST tags later used to determine whether the album is a compilation perhaps? For the case of Phil Spector's compilation, there are numerous different artists (in the TRACKARTIST fields), but I think it would be sensible to have his name in the ALBUMARTIST field for each track. But what should go in the ARTIST field in each track? The existing logic in CUE.pm makes ARTIST match ALBUMARTIST if there is one, or TRACKARTIST if not. Why? How will the ARTIST fields be used downstream?
In my second example above, even though there was a different performer for each track, the album was not identified as a Compilation. I've fixed that by using a rule that says if ANY subsequent PERFORMER tag is not identical to the first PERFORMER tag in the file (which would be the top level one in the case where such a tag was set, or the first track performer if not), then the Compilation flag is set.* Then, after the first read through, if the compilation flag has been set, I go through and delete all the ARTIST fields, on the basis that I already have a TRACKARTIST for each track, and I may or may not have an ALBUMARTIST depending whether one was set by a top level PERFORMER tag. There's then existing code that copies the ALBUMARTIST fields back into the ARTIST field, so this makes me think that its primary use is as a duplicate of ALBUMARTIST. Why do we need a duplicate? So my new approach makes sense to me, and fixes the issue of different track-only artists populating the Album Artists menu. I hasn't fixed the issue of the specific ALBUMARTIST fields not appearing in the Album Artists menu though. All very confusing, but at least I'm getting some exposure to perl (and its damned semi-colons!). ------------------------------------------------------------------------ chill's Profile: http://forums.slimdevices.com/member.php?userid=10839 View this thread: http://forums.slimdevices.com/showthread.php?t=113834 _______________________________________________ ripping mailing list [email protected] http://lists.slimdevices.com/mailman/listinfo/ripping
