gregklanderman;291832 Wrote: > >>>>> kdf <slim-mail (AT) deane-freeman (DOT) com> 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
We're working on the best way to handle all of this. I think I remember seeing your name on the check-in of code for at least one of the bugs as I was researching this issue, so I know you're aware of what's going on internally to the scanner code. I'm not sure what prompted the useBandAsAlbumArtist variable to be added, or how it was intended to be used, but the whole issue of normalizing tags is currently being reviewed (there's a LOT to it) and how we will provide additional "mapping" flexibility has yet to be determined. What I do know is that we have to find a way that doesn't add too much complexity, or break the exisiting tag usage, to provide that functionality. In addition, we're looking at how "various artists" are consolidated, and how the Compilation tag is/is not used in that process. -=> Jim -- JimC "well, she wasn't all of that, but she sure was some of that." -- BKlaas' college buddy ------------------------------------------------------------------------ JimC's Profile: http://forums.slimdevices.com/member.php?userid=9428 View this thread: http://forums.slimdevices.com/showthread.php?t=46093 _______________________________________________ discuss mailing list [email protected] http://lists.slimdevices.com/lists/listinfo/discuss
