Add config option to show album participations under artists in Subsonic clients

conf/configuration.go

@@ -17,64 +17,61 @@ import (
 )
 
 type configOptions struct {
-	ConfigFile              string
+	ConfigFile                   string
-	Address                 string
+	Address                      string
-	Port                    int
+	Port                         int
-	MusicFolder             string
+	MusicFolder                  string
-	DataFolder              string
+	DataFolder                   string
-	DbPath                  string
+	DbPath                       string
-	LogLevel                string
+	LogLevel                     string
-	ScanInterval            time.Duration
+	ScanInterval                 time.Duration
-	ScanSchedule            string
+	ScanSchedule                 string
-	SessionTimeout          time.Duration
+	SessionTimeout               time.Duration
-	BaseURL                 string
+	BaseURL                      string
-	UILoginBackgroundURL    string
+	UILoginBackgroundURL         string
-	EnableTranscodingConfig bool
+	EnableTranscodingConfig      bool
-	EnableDownloads         bool
+	EnableDownloads              bool
-	EnableExternalServices  bool
+	EnableExternalServices       bool
-	EnableMediaFileCoverArt bool
+	EnableMediaFileCoverArt      bool
-	TranscodingCacheSize    string
+	TranscodingCacheSize         string
-	ImageCacheSize          string
+	ImageCacheSize               string
-	AutoImportPlaylists     bool
+	AutoImportPlaylists          bool
-	PlaylistsPath           string
+	PlaylistsPath                string
-	AutoTranscodeDownload   bool
+	AutoTranscodeDownload        bool
-
+	DefaultDownsamplingFormat    string
-	SearchFullString       bool
+	SearchFullString             bool
-	RecentlyAddedByModTime bool
+	RecentlyAddedByModTime       bool
-	IgnoredArticles        string
+	IgnoredArticles              string
-	IndexGroups            string
+	IndexGroups                  string
-	ProbeCommand           string
+	SubsonicArtistParticipations bool
-	CoverArtPriority       string
+	ProbeCommand                 string
-	CoverJpegQuality       int
+	CoverArtPriority             string
-	UIWelcomeMessage       string
+	CoverJpegQuality             int
-	EnableGravatar         bool
+	UIWelcomeMessage             string
-	EnableFavourites       bool
+	EnableGravatar               bool
-	EnableStarRating       bool
+	EnableFavourites             bool
-	EnableUserEditing      bool
+	EnableStarRating             bool
-	DefaultTheme           string
+	EnableUserEditing            bool
-	DefaultLanguage        string
+	DefaultTheme                 string
-	DefaultUIVolume        int
+	DefaultLanguage              string
-	EnableCoverAnimation   bool
+	DefaultUIVolume              int
-	GATrackingID           string
+	EnableReplayGain             bool
-	EnableLogRedacting     bool
+	EnableCoverAnimation         bool
-	AuthRequestLimit       int
+	GATrackingID                 string
-	AuthWindowLength       time.Duration
+	EnableLogRedacting           bool
-	PasswordEncryptionKey  string
+	AuthRequestLimit             int
-	ReverseProxyUserHeader string
+	AuthWindowLength             time.Duration
-	ReverseProxyWhitelist  string
+	PasswordEncryptionKey        string
-	Prometheus             prometheusOptions
+	ReverseProxyUserHeader       string
-	EnableReplayGain       bool
+	ReverseProxyWhitelist        string
-
+	Prometheus                   prometheusOptions
-	Scanner scannerOptions
+	Scanner                      scannerOptions
 
 	Agents       string
 	LastFM       lastfmOptions
 	Spotify      spotifyOptions
 	ListenBrainz listenBrainzOptions
 
-	//test downsampling
-	DefaultDownsamplingFormat string
-
 	// DevFlags. These are used to enable/disable debugging and incomplete features
 	DevLogSourceLine                 bool
 	DevLogLevels                     map[string]string
@@ -236,12 +233,12 @@ func init() {
 	viper.SetDefault("enableexternalservices", true)
 	viper.SetDefault("enableMediaFileCoverArt", true)
 	viper.SetDefault("autotranscodedownload", false)
-
+	viper.SetDefault("defaultdownsamplingformat", consts.DefaultDownsamplingFormat)
-	// Config options only valid for file/env configuration
 	viper.SetDefault("searchfullstring", false)
 	viper.SetDefault("recentlyaddedbymodtime", false)
 	viper.SetDefault("ignoredarticles", "The El La Los Las Le Les Os As O A")
 	viper.SetDefault("indexgroups", "A B C D E F G H I J K L M N O P Q R S T U V W X-Z(XYZ) [Unknown]([)")
+	viper.SetDefault("subsonicartistparticipations", false)
 	viper.SetDefault("probecommand", "ffmpeg %s -f ffmetadata")
 	viper.SetDefault("coverartpriority", "cover.*, folder.*, front.*, embedded, external")
 	viper.SetDefault("coverjpegquality", 75)
@@ -253,6 +250,7 @@ func init() {
 	viper.SetDefault("defaulttheme", "Dark")
 	viper.SetDefault("defaultlanguage", "")
 	viper.SetDefault("defaultuivolume", consts.DefaultUIVolume)
+	viper.SetDefault("enablereplaygain", false)
 	viper.SetDefault("enablecoveranimation", true)
 	viper.SetDefault("gatrackingid", "")
 	viper.SetDefault("enablelogredacting", true)
@@ -266,8 +264,6 @@ func init() {
 	viper.SetDefault("prometheus.enabled", false)
 	viper.SetDefault("prometheus.metricspath", "/metrics")
 
-	viper.SetDefault("enablereplaygain", false)
-
 	viper.SetDefault("scanner.extractor", consts.DefaultScannerExtractor)
 	viper.SetDefault("scanner.genreseparators", ";/,")
 
@@ -281,8 +277,6 @@ func init() {
 	viper.SetDefault("listenbrainz.enabled", true)
 	viper.SetDefault("listenbrainz.baseurl", "https://api.listenbrainz.org/1/")
 
-	viper.SetDefault("defaultdownsamplingformat", consts.DefaultDownsamplingFormat)
-
 	// DevFlags. These are used to enable/disable debugging and incomplete features
 	viper.SetDefault("devlogsourceline", false)
 	viper.SetDefault("devautocreateadminpassword", "")

server/subsonic/browsing.go

@@ -141,15 +141,12 @@ func (api *Router) GetArtist(r *http.Request) (*responses.Subsonic, error) {
 		return nil, err
 	}
 
-	albums, err := api.ds.Album(ctx).GetAllWithoutGenres(filter.AlbumsByArtistID(id))
-	if err != nil {
-		log.Error(ctx, "Error retrieving albums by artist", "id", id, "name", artist.Name, err)
-		return nil, err
-	}
-
 	response := newResponse()
-	response.ArtistWithAlbumsID3 = api.buildArtist(r, artist, albums)
+	response.ArtistWithAlbumsID3, err = api.buildArtist(r, artist)
-	return response, nil
+	if err != nil {
+		log.Error(ctx, "Error retrieving albums by artist", "id", artist.ID, "name", artist.Name, err)
+	}
+	return response, err
 }
 
 func (api *Router) GetAlbum(r *http.Request) (*responses.Subsonic, error) {
@@ -370,11 +367,18 @@ func (api *Router) buildArtistDirectory(ctx context.Context, artist *model.Artis
 	return dir, nil
 }
 
-func (api *Router) buildArtist(r *http.Request, artist *model.Artist, albums model.Albums) *responses.ArtistWithAlbumsID3 {
+func (api *Router) buildArtist(r *http.Request, artist *model.Artist) (*responses.ArtistWithAlbumsID3, error) {
+	ctx := r.Context()
 	a := &responses.ArtistWithAlbumsID3{}
 	a.ArtistID3 = toArtistID3(r, *artist)
+
+	albums, err := api.ds.Album(ctx).GetAllWithoutGenres(filter.AlbumsByArtistID(artist.ID))
+	if err != nil {
+		return nil, err
+	}
+
 	a.Album = childrenFromAlbums(r.Context(), albums)
-	return a
+	return a, nil
 }
 
 func (api *Router) buildAlbumDirectory(ctx context.Context, album *model.Album) (*responses.Directory, error) {

server/subsonic/filter/filters.go

@@ -1,9 +1,11 @@
 package filter
 
 import (
+	"fmt"
 	"time"
 
 	"github.com/Masterminds/squirrel"
+	"github.com/navidrome/navidrome/conf"
 	"github.com/navidrome/navidrome/model"
 )
 
@@ -49,9 +51,15 @@ func AlbumsByGenre(genre string) Options {
 }
 
 func AlbumsByArtistID(artistId string) Options {
+	var filters squirrel.Sqlizer
+	if conf.Server.SubsonicArtistParticipations {
+		filters = squirrel.Like{"all_artist_ids": fmt.Sprintf("%%%s%%", artistId)}
+	} else {
+		filters = squirrel.Eq{"album_artist_id": artistId}
+	}
 	return Options{
 		Sort:    "max_year",
-		Filters: squirrel.Eq{"album_artist_id": artistId},
+		Filters: filters,
 	}
 }