arian/navidrome
1
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

Fix log message about artist with Various Artists' mbid

Browse Source
This commit is contained in:
Deluan
2021-07-19 11:49:39 -04:00
committed by Deluan Quintão
parent 00e418cb2a
commit 254e5673e1
1 changed files with 17 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
persistence/helpers.go
+9 -7
View File
@@ -65,12 +65,13 @@ func getMostFrequentMbzID(ctx context.Context, mbzIDs, entityName, name string)
if len(ids) == 0 { if len(ids) == 0 {
return "" return ""
} }
if len(ids) == 1 {
return ids[0]
}
idCounts := map[string]int{}
var topId string var topId string
var topCount int var topCount int
idCounts := map[string]int{}
if len(ids) == 1 {
topId = ids[0]
} else {
for _, id := range ids { for _, id := range ids {
c := idCounts[id] + 1 c := idCounts[id] + 1
idCounts[id] = c idCounts[id] = c
@@ -79,14 +80,15 @@ func getMostFrequentMbzID(ctx context.Context, mbzIDs, entityName, name string)
topCount = c topCount = c
} }
} }
}
if len(idCounts) > 1 && name != consts.VariousArtists { if len(idCounts) > 1 && name != consts.VariousArtists {
if topId == consts.VariousArtistsMbzId {
log.Warn(ctx, "Artist with mbid of Various Artists", "name", name, "mbid", topId)
} else {
log.Warn(ctx, "Multiple MBIDs found for "+entityName, "name", name, "mbids", idCounts) log.Warn(ctx, "Multiple MBIDs found for "+entityName, "name", name, "mbids", idCounts)
} }
if topId == consts.VariousArtistsMbzId && name != consts.VariousArtists {
log.Warn(ctx, "Artist with mbid of 'Various Artists'", "name", name, "mbid", topId)
} }
return topId return topId
} }