Merge pull request #1178 from darrell-k/include-track-artists

return all roles not undef when all roles requested

Slim/Schema.pm

@@ -2347,7 +2347,7 @@ sub artistOnlyRoles {
 
 	# And if the user has asked for ALL, give them it.
 	if ($roles{'ALL'}) {
-		return undef;
+		return [ Slim::Schema::Contributor->contributorRoleIds ];
 	}
 
 	# Loop through each pref to see if the user wants to show that contributor role.
@@ -2359,13 +2359,8 @@ sub artistOnlyRoles {
 		}
 	}
 
-	# If we're using all roles, don't bother with the constraint.
-	if (scalar keys %roles != Slim::Schema::Contributor->totalContributorRoles) {
-
-		return [ sort map { Slim::Schema::Contributor->typeToRole($_) } keys %roles ];
-	}
+	return [ sort map { Slim::Schema::Contributor->typeToRole($_) } keys %roles ];
 
-	return undef;
 }
 
 sub registerRatingImplementation {