From 759f00d38c8c1787872dacc2e195e572f36c471f Mon Sep 17 00:00:00 2001
From: phoenix <kundeng94@gmail.com>
Date: Sat, 22 Mar 2025 19:40:30 -0400
Subject: [PATCH] Migrating over icarus-model coverart

---
 src/managers/commit_manager.rs | 28 ++++++++++++++++++++--------
 src/models/mod.rs              |  2 +-
 src/syncers/upload.rs          | 10 +++++++---
 3 files changed, 28 insertions(+), 12 deletions(-)

diff --git a/src/managers/commit_manager.rs b/src/managers/commit_manager.rs
index d7c99b5..facedb5 100644
--- a/src/managers/commit_manager.rs
+++ b/src/managers/commit_manager.rs
@@ -229,10 +229,12 @@ impl CommitManager {
         match Runtime::new().unwrap().block_on(result_fut) {
             Ok(o) => {
                 println!("Success");
-                let filename = String::from("audio") + icarus_models::constants::DEFAULTMUSICEXTENSION;
+                let filename =
+                    String::from("audio") + icarus_models::constants::DEFAULTMUSICEXTENSION;
                 let data = o.as_bytes();
                 let mut file = std::fs::File::create(filename).expect("Failed to save");
-                file.write_all(&data).expect("Failed to save downloaded song");
+                file.write_all(&data)
+                    .expect("Failed to save downloaded song");
             }
             Err(er) => {
                 println!("Error {:?}", er);
@@ -365,7 +367,12 @@ impl CommitManager {
             panic!("Error");
         }
 
-        let mut cover_art = models::song::CoverArt::default();
+        let mut cover_art = icarus_models::coverart::CoverArt {
+            id: 0,
+            title: String::new(),
+            path: String::new(),
+            data: Vec::new(),
+        };
         let mut song = icarus_models::song::Song::default();
         let mut filenames = Vec::new();
         let mut fp = String::new();
@@ -398,7 +405,7 @@ impl CommitManager {
             _ => {}
         }
 
-        cover_art.path = Some(cover_path.clone());
+        cover_art.path = cover_path.clone();
 
         let album = self.retrieve_metadata(&meta_path);
         let trck = i32::from_str(track_id).unwrap();
@@ -410,7 +417,7 @@ impl CommitManager {
         // s.album = album.title.clone();
         s.data = s.to_data().unwrap();
 
-        cover_art.data = Some(cover_art.to_data().unwrap());
+        cover_art.data = cover_art.to_data().unwrap();
 
         let mut up = syncers::upload::Upload::default();
         let host = self.ica_action.retrieve_flag_value(&String::from("-h"));
@@ -464,7 +471,12 @@ impl CommitManager {
             panic!("Directory does not exist");
         }
 
-        let mut cover_art = models::song::CoverArt::default();
+        let mut cover_art = icarus_models::coverart::CoverArt {
+            id: 0,
+            title: String::new(),
+            path: String::new(),
+            data: Vec::new(),
+        };
         let mut songs: Vec<icarus_models::song::Song> = Vec::new();
         let mut filenames: Vec<String> = Vec::new();
         let mut metadatapath: String = String::new();
@@ -486,7 +498,7 @@ impl CommitManager {
                     let directory_part = sourcepath.clone();
                     let fname = self.o_to_string(&file_name);
                     let fullpath = directory_part + "/" + &fname.unwrap();
-                    cover_art.path = Some(fullpath);
+                    cover_art.path = fullpath;
                 }
                 En::MetadataFile => {
                     let directory_part = sourcepath.clone();
@@ -525,7 +537,7 @@ impl CommitManager {
         let host = self.ica_action.retrieve_flag_value(&String::from("-h"));
         up.set_api(&host);
 
-        cover_art.data = Some(cover_art.to_data().unwrap());
+        cover_art.data = cover_art.to_data().unwrap();
 
         println!("");
 
diff --git a/src/models/mod.rs b/src/models/mod.rs
index 719772a..c9c4e79 100644
--- a/src/models/mod.rs
+++ b/src/models/mod.rs
@@ -1,5 +1,5 @@
 pub mod api;
 pub mod flags;
 pub mod icarus_action;
-pub mod song;
+// pub mod song;
 pub mod upload_form;
diff --git a/src/syncers/upload.rs b/src/syncers/upload.rs
index 5905656..f15e463 100644
--- a/src/syncers/upload.rs
+++ b/src/syncers/upload.rs
@@ -48,7 +48,7 @@ impl Upload {
         &mut self,
         token: &icarus_models::token::AccessToken,
         song: &icarus_models::song::Song,
-        cover: &models::song::CoverArt,
+        cover: &icarus_models::coverart::CoverArt,
         album: &icarus_models::album::collection::Album,
     ) -> Result<reqwest::Response, std::io::Error> {
         self.api.endpoint = String::from("song/data/upload/with/data");
@@ -132,9 +132,13 @@ impl Upload {
             .part("file", file.file_name(song_filename));
     }
 
-    fn init_form(&self, song: &Song, cover: &models::song::CoverArt) -> reqwest::multipart::Form {
+    fn init_form(
+        &self,
+        song: &Song,
+        cover: &icarus_models::coverart::CoverArt,
+    ) -> reqwest::multipart::Form {
         let songpath = song.songpath.clone();
-        let coverpath = cover.path.clone().unwrap();
+        let coverpath = cover.path.clone();
         let song_detail = song.to_metadata_json().unwrap();
 
         println!("\n{}\n", song_detail);