anas-elgarhy
/
cmus-notify
1
0
Fork 0
Code Issues 1 Pull Requests 1 Packages Projects Releases 1 Wiki Activity

Refactor TrackCover set_notification_image to use String for external path

This commit is contained in:
Anas Elgarhy 2023-02-23 13:51:06 +02:00
parent cdc5085506
commit 8c4a2a5501
No known key found for this signature in database
GPG Key ID: 0501802A1D496528
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/lib.rs
View File

@ -121,7 +121,7 @@ pub enum TrackCover {
Embedded(image::DynamicImage),
/// The cover is an external file.
/// The `String` contains the absolute path of the external file.
External(image::DynamicImage),
External(String),
/// The track does not have a cover.
None,
}
@ -130,17 +130,20 @@ impl TrackCover {
pub fn set_notification_image(&self, notification: &mut notify_rust::Notification) {
use TrackCover::*;
match self {
Embedded(cover) | External(cover) => {
Embedded(cover) => {
#[cfg(feature = "debug")]
debug!("Setting the cover as the notification image.");
let Ok(image) = notify_rust::Image::try_from(cover.clone()) else { return; };
notification.image_data(image);
}
External(path) => {
#[cfg(feature = "debug")]
debug!("Setting the cover as the notification image.");
notification.image_path(path);
}
None => {
#[cfg(feature = "debug")]
debug!("The track does not have a cover.");
// reset the notification image
notification.image_path("");
}
}
}
@ -188,11 +191,6 @@ pub fn track_cover(
) {
#[cfg(feature = "debug")]
info!("Found the external cover \"{cover}\".");
let Ok(cover) = image::open(cover) else {
#[cfg(feature = "debug")]
info!("Could not open the external cover.");
return TrackCover::None;
};
return TrackCover::External(cover);
}
}