Jim,

Turns out I had one track on a non-compilation album which was missing
the ARTISTSORT.  So I think it's working OK for me, but I did add
myself to the bug you referenced.

FWIW, the following patch might fix your problem, I couldn't really
test it though as I no longer have a problem myself..

thanks,
greg


Index: Contributor.pm
===================================================================
--- Contributor.pm      (revision 6939)
+++ Contributor.pm      (working copy)
@@ -72,15 +72,16 @@
        # contributorTrack entry.
        #
        # Split both the regular and the normalized tags
-       my @artistList   = Slim::Music::Info::splitTag($artist);
-       my @sortedList   = Slim::Music::Info::splitTag($artistSort);
+       my @artistList = Slim::Music::Info::splitTag($artist);
+       my @sortedList = Slim::Music::Info::splitTag($artistSort);
 
        for (my $i = 0; $i < scalar @artistList; $i++) {
 
-               # The search columnn is the canonical text that we match 
against in a search.
+               # The search column is the canonical text that we match against 
in a search.
                my $name   = $artistList[$i];
                my $search = Slim::Utils::Text::ignoreCaseArticles($name);
                my $sort   = 
Slim::Utils::Text::ignoreCaseArticles(($sortedList[$i] || $name));
+               my $sort_p = defined($args->{'sortBy'}) && 
defined($sortedList[$i]);
 
                my ($contributorObj) = 
Slim::DataStores::DBI::Contributor->search({
                        'namesearch' => $search,
@@ -94,6 +95,10 @@
                                'namesort'       => $sort,
                                'musicbrainz_id' => $brainzID,
                        });
+               } elsif ($sort_p) {
+                   # bug 3069 - if we have an explicit sort value, update the 
contributor
+                   $contributorObj->namesort($sort);
+                       $contributorObj->update;
                }
 
                push @contributors, $contributorObj;
_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/beta

Reply via email to