The 'Hide Various Artists from appearing in the artist list for
compilations' case can be fixed with the following patch:


Code:
--------------------
    
  Index: Control/Queries.pm
  ===================================================================
  --- Control/Queries.pm        (Revision 31569)
  +++ Control/Queries.pm        (Arbeitskopie)
  @@ -4831,7 +4831,7 @@
        my $songData  = _songData(
                $request,
                $track,
  -             $current ?'AalKNJ' : 'alKNJ',                   # tags needed 
for our entities
  +             $current ?'ACalKNJ' : 'alKNJ',                  # tags needed 
for our entities
        );
        
        $request->addResultLoop($loop, $count, 'trackType', $track->remote ? 
'radio' : 'local');
  @@ -4846,7 +4846,11 @@
        # (even getting it just for the current track is pretty expensive)
        if ($current) {
                my (%artists, @artists);
  -             foreach ('albumartist', 'trackartist', 'artist') {
  +             my @artistlist = ( 'trackartist', 'artist' );
  +             if (not $songData->{compilation}) {
  +                     unshift(@artistlist, 'albumartist' );
  +             }
  +             foreach (@artistlist) {
                        foreach my $a ( $songData->{"arrayRef_$_"} ? 
@{$songData->{"arrayRef_$_"}} : $songData->{$_} ) {
                                if ( $a && !$artists{$a} ) {
                                        push @artists, $a;
  
--------------------


Michael, what do you think of that patch? Off-hand, I see no use case
for displaying the album artist for compilations...

The patch is against 7.5 trunk, BTW.


-- 
maniac103
------------------------------------------------------------------------
maniac103's Profile: http://forums.slimdevices.com/member.php?userid=42191
View this thread: http://forums.slimdevices.com/showthread.php?t=75308

_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/beta

Reply via email to