Use primary type in the album sort key #243

Merged
wojtek merged 2 commits from 231---differentiate-release-groups-with-same-title-but-different-type-clash into main 2025-01-02 23:24:02 +01:00
2 changed files with 20 additions and 10 deletions
Showing only changes of commit e0f3ae2fd3 - Show all commits

View File

@ -101,14 +101,14 @@ impl From<(u32, u8, u8)> for AlbumDate {
pub struct AlbumSeq(pub u8);
/// Based on [MusicBrainz types](https://musicbrainz.org/doc/Release_Group/Type).
#[derive(Clone, Copy, Debug, PartialEq, Eq)]
#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord)]
pub enum AlbumPrimaryType {
/// Album
Album,
/// Single
Single,
/// EP
Ep,
/// Album
Album,
/// Broadcast
Broadcast,
/// Other
@ -217,8 +217,13 @@ impl AlbumMeta {
self
}
pub fn get_sort_key(&self) -> (&AlbumDate, &AlbumSeq, &AlbumId) {
(&self.date, &self.seq, &self.id)
pub fn get_sort_key(&self) -> (&AlbumDate, &AlbumSeq, &str, &Option<AlbumPrimaryType>) {
(
&self.date,
&self.seq,
&self.id.title,
&self.info.primary_type,
)
}
pub fn set_mb_ref(&mut self, mb_ref: AlbumMbRef) {

View File

@ -1,7 +1,7 @@
use std::cmp;
use musichoard::collection::{
album::{Album, AlbumDate, AlbumId, AlbumSeq},
album::{Album, AlbumDate, AlbumPrimaryType, AlbumSeq},
track::Track,
};
@ -165,7 +165,7 @@ impl AlbumSelection {
}
pub struct KeySelectAlbum {
key: (AlbumDate, AlbumSeq, AlbumId),
key: (AlbumDate, AlbumSeq, String, Option<AlbumPrimaryType>),
track: Option<KeySelectTrack>,
}
@ -175,14 +175,19 @@ impl KeySelectAlbum {
let album = &albums[index];
let key = album.meta.get_sort_key();
KeySelectAlbum {
key: (key.0.to_owned(), key.1.to_owned(), key.2.to_owned()),
key: (
key.0.to_owned(),
key.1.to_owned(),
key.2.to_owned(),
key.3.to_owned(),
),
track: KeySelectTrack::get(&album.tracks, &selection.track),
}
})
}
pub fn get_sort_key(&self) -> (&AlbumDate, &AlbumSeq, &AlbumId) {
(&self.key.0, &self.key.1, &self.key.2)
pub fn get_sort_key(&self) -> (&AlbumDate, &AlbumSeq, &str, &Option<AlbumPrimaryType>) {
(&self.key.0, &self.key.1, &self.key.2, &self.key.3)
}
}