>>>>> kdf  <[EMAIL PROTECTED]> writes:

>> i don't think it has ANYTHING to do with mapping.  TPE2 is SC7 internal
>> tag BAND period.  and no current option affects that.  it currents only
>> affects how things are denoted when displayed.

> correct.  The pref, 'useBandAsAlbumArtist' is only used in one place in
> the code.  When the $album->artists are requested, setting this option
> means you get the BAND in the album listing "ThisAlbum by BAND (20xx)". 
> It has no other internal use aside from being in the list of contributors.

> If you want BAND to have a different purpose, such as the ALBUMARTIST you
> could add it to the %tagMapping in Formats::MP3.

> I don't personally use BAND tags, but based on that pref's use in the
> code, it works as advertised.

Why would you infer the intended semantics from the current code?  How
do you know someone else that didn't understand what the option was
supposed to do hasn't changed the code implementing it?  Or that
several major rounds of changes to the scanning and db schema haven't
invalidated assumptions that the original implementation of this
preference was relying on?  Or that the original implementation was
complete?

It is *very* hard for me to believe with an internal name as clear as
'useBandAsAlbumArtist' that the preference was not intended to cause
any BAND(TPE2) tags to be treated as though they were ALBUMARTIST
tags, for exactly the reason MrSinatra wants.

JimC, if the resolution is to create a new preference to resolve
MrSinatra's request, please change the internal name of
'useBandAsAlbumArtist' to more clearly reflect its semantics and
intended use, in addition to clarifying the description on the
preferences page.

thanks,
Greg
_______________________________________________
discuss mailing list
discuss@lists.slimdevices.com
http://lists.slimdevices.com/lists/listinfo/discuss

Reply via email to