From ebd673601d4ff7e135c1dde95d78061bf879fbcd Mon Sep 17 00:00:00 2001 From: Kai Elwert Date: Mon, 30 Aug 2010 12:49:35 +0200 Subject: [PATCH] Set an empty album_artist for songs part of a compilation if not set Songs in a compilation must all have the same album_artist and album (due to songalbumid); if a song in a compilation doesn't have an album_artist, set it to an empty string instead of defaulting to the value of artist. This is less likely to break the compilation. --- src/filescanner.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/filescanner.c b/src/filescanner.c index 5c6513d8..a7287cee 100644 --- a/src/filescanner.c +++ b/src/filescanner.c @@ -191,7 +191,12 @@ fixup_tags(struct media_file_info *mfi) /* If we don't have an album_artist, set it to artist */ if (!mfi->album_artist) - mfi->album_artist = strdup(mfi->artist); + { + if (mfi->compilation) + mfi->album_artist = strdup(""); + else + mfi->album_artist = strdup(mfi->artist); + } }