wojtek/musichoard
1
0
Fork 0
Code Issues 17 Pull Requests 1 Actions Packages Projects 1 Releases 4 Wiki Activity

Fetch loses albums with CannotHaveMbid as mb_ref #259

Merged
wojtek merged 3 commits from 255---fetch-loses-albums-with-cannothavembid-as-mb_ref into main 2025-01-05 14:55:36 +01:00
Conversation 0 Commits 3 Files Changed 1 +5 -1
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 6919f0214a - Show all commits

6
src/tui/app/machine/match_state.rs
View File

@ -212,6 +212,10 @@ impl AppMachine<MatchState> {
mh.set_artist_mb_ref(&matches.matching.id, tuple.mb_ref) mh.set_artist_mb_ref(&matches.matching.id, tuple.mb_ref)
} }
fn filter_mb_ref(left: &AlbumMbRef, right: &AlbumMbRef) -> bool {
(left == right) && !matches!(left, AlbumMbRef::CannotHaveMbid)
}
fn select_album( fn select_album(
inner: &mut AppInner, inner: &mut AppInner,
matches: &AlbumMatches, matches: &AlbumMatches,
@ -225,7 +229,7 @@ impl AppMachine<MatchState> {
let iter = artist.albums.iter(); let iter = artist.albums.iter();
let filtered = iter let filtered = iter
.filter(|album| album.meta.id != matches.matching) .filter(|album| album.meta.id != matches.matching)
.filter(|album| album.meta.id.mb_ref == tuple.mb_ref); .filter(|album| Self::filter_mb_ref(&album.meta.id.mb_ref, &tuple.mb_ref));
clashing.extend(filtered.map(|a| &a.meta.id).cloned()) clashing.extend(filtered.map(|a| &a.meta.id).cloned())
} }