Bug 16151 - Scanner should add ALBUMARTIST and TRACKARTIST role records to the db for VA cuesheets
: Scanner should add ALBUMARTIST and TRACKARTIST role records to the db for VA ...
Status: CLOSED WONTFIX
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.6.0
: PC All
: -- enhancement (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-04-27 10:34 UTC by Gordon Harris
Modified: 2011-05-13 10:07 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gordon Harris 2010-04-27 10:34:14 UTC
Currently the scanner doesn't seem to recognize ALBUMARTIST or TRACKARTIST tags in cuesheets.  Consequently, ROLE values of 5 or 6 are never added to the contributor_album or contributor_track tables for audio files tagged solely with cuesheet data (or referenced in external cuesheets).

More importantly, a ALBUMARTIST value is already implicit in cuesheets: the 1st PERFORMER tag, which precedes any TRACK records can be interpreted as an ALBUMARTIST value.  At least one popular audio program, Foobar2k, interprets cuesheets in exactly this way.

While failing to interpret the album-level PERFORMER tag as an ALBUMARTIST in non-various artists albums leads to no ill effects, in VA albums there are side-effects.  Some artist values which normally would show up in the all-artists query, do, in fact, get dropped, since they don't get a role==5 record.

Specific enhancement request:

1). Make the cuesheet scanner code recognize:

REM ALBUMARTIST "xxxx"

..and..

REM TRACKARTIST "xxx"

..tags.

2). For any VA cuesheet (i.e. any cuesheet with more than 1 PERFORMER value) and lacking a specific ALBUMARTIST tag: interpret the 1st PERFORMER value (before any defined TRACKs) as an ALBUMARTIST value and add the appropriate role==5 record to the contributor_album table.
Comment 1 Gordon Harris 2010-04-29 14:40:09 UTC
OK, I've changed my mind.  Please don't do this.  There would be unintended side effects.  See discussion at: http://forums.slimdevices.com/showthread.php?t=77928
Comment 2 Bradley D. Wall 2011-05-13 10:07:46 UTC
Closing per request