Commit 44faf108 authored by Max Kellermann's avatar Max Kellermann

SongFilter: search for album artist falls back to the artist tag

Implement Mantis ticket 0003646.
parent e354c5c2
...@@ -3,6 +3,7 @@ ver 0.18 (2012/??/??) ...@@ -3,6 +3,7 @@ ver 0.18 (2012/??/??)
- allow tilde paths for socket - allow tilde paths for socket
* protocol: * protocol:
- new command "toggleoutput" - new command "toggleoutput"
- search for album artist falls back to the artist tag
* innput: * innput:
- curl: enable https - curl: enable https
- soup: plugin removed - soup: plugin removed
......
...@@ -94,15 +94,27 @@ SongFilter::Item::Match(const Tag &_tag) const ...@@ -94,15 +94,27 @@ SongFilter::Item::Match(const Tag &_tag) const
return true; return true;
} }
/** If the search critieron was not visited during the sweep if (tag < TAG_NUM_OF_ITEM_TYPES && !visited_types[tag]) {
* through the song's tag, it means this field is absent from /* If the search critieron was not visited during the
* the tag or empty. Thus, if the searched string is also sweep through the song's tag, it means this field
* empty (first char is a \0), then it's a match as well and is absent from the tag or empty. Thus, if the
* we should return true. searched string is also empty (first char is a \0),
*/ then it's a match as well and we should return
if (*value == 0 && tag < TAG_NUM_OF_ITEM_TYPES && true. */
!visited_types[tag]) if (*value == 0)
return true; return true;
if (tag == TAG_ALBUM_ARTIST && visited_types[TAG_ARTIST]) {
/* if we're looking for "album artist", but
only "artist" exists, use that */
for (unsigned i = 0; i < _tag.num_items; i++) {
const TagItem &item = *_tag.items[i];
if (item.type == TAG_ARTIST &&
StringMatch(item.value))
return true;
}
}
}
return false; return false;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment