From cb5cf3a8cabdc2471194f5f4b66895f4ae5ea005 Mon Sep 17 00:00:00 2001 From: phoenix Date: Sat, 11 Oct 2025 18:56:58 -0400 Subject: [PATCH] Build changes --- src/main.rs | 72 ++++++++++++++++--------------------------------- src/the_rest.rs | 51 ----------------------------------- 2 files changed, 23 insertions(+), 100 deletions(-) diff --git a/src/main.rs b/src/main.rs index 7ac8a7c..90301de 100644 --- a/src/main.rs +++ b/src/main.rs @@ -5,8 +5,6 @@ pub mod the_rest; pub mod update_queued_song; pub mod util; -use std::io::Write; - pub const SECONDS_TO_SLEEP: u64 = 5; #[tokio::main] @@ -64,24 +62,22 @@ async fn main() -> Result<(), Box> { // TODO: Do something with the result later match some_work(&app, &song_queue_id, &user_id).await { Ok(( - _song, - _coverart, - (song_queue_id, song_queue_path), - (coverart_queue_id, coverart_queue_path), + song, + coverart, + (song_queue_id, _song_queue_path), + (coverart_queue_id, _coverart_queue_path), )) => { match wipe_data_from_queues(&app, &song_queue_id, &coverart_queue_id) .await { - Ok(_) => { - match cleanup(&song_queue_path, &coverart_queue_path).await { - Ok(_) => { - println!("Successful cleanup"); - } - Err(err) => { - eprintln!("Error: {err:?}"); - } + Ok(_) => match cleanup(&song, &coverart).await { + Ok(_) => { + println!("Successful cleanup"); } - } + Err(err) => { + eprintln!("Error: {err:?}"); + } + }, Err(err) => { eprintln!("Error: {err:?}"); } @@ -206,17 +202,17 @@ async fn wipe_data_from_queues( } async fn cleanup( - song_queue_path: &String, - coverart_queue_path: &String, + song: &icarus_models::song::Song, + coverart: &icarus_models::coverart::CoverArt, ) -> Result<(), std::io::Error> { - match the_rest::cleanup::clean_song_queue(song_queue_path) { + match song.remove_from_filesystem() { Ok(_) => {} Err(err) => { eprintln!("Error: Problem cleaning up SongQueue files {err:?}"); } } - match the_rest::cleanup::clean_coverart_queue(coverart_queue_path) { + match coverart.remove_from_filesystem() { Ok(_) => Ok(()), Err(err) => Err(err), } @@ -353,21 +349,13 @@ async fn prep_song( ..Default::default() }; let songpath = match song.song_path() { - Ok(songpath) => { - songpath - } - Err(_err) => { - String::new() - } + Ok(songpath) => songpath, + Err(_err) => String::new(), }; let song_queue_path = match song.save_to_filesystem() { - Ok(_) => { - std::path::Path::new(&songpath) - } - Err(_err) => { - std::path::Path::new("") - } + Ok(_) => std::path::Path::new(&songpath), + Err(_err) => std::path::Path::new(""), }; println!("Saved at: {song_queue_path:?}"); @@ -445,7 +433,10 @@ async fn prep_song( // TODO: Consider having something like this in icarus_models pub async fn generate_song_queue_dir_and_filename() -> (String, String) { let mut song = icarus_models::song::Song::default(); - song.filename = icarus_models::song::generate_filename(icarus_models::types::MusicTypes::FlacExtension, true); + song.filename = icarus_models::song::generate_filename( + icarus_models::types::MusicTypes::FlacExtension, + true, + ); song.directory = icarus_envy::environment::get_root_directory().await.value; @@ -481,23 +472,6 @@ pub async fn generate_coverart_queue_dir_and_filename() -> (String, String) { (directory, filename) } -// TODO: Check to see if this is available in icarus_models -async fn save_file_to_fs( - directory: &String, - filename: &String, - data: &[u8], -) -> std::path::PathBuf { - // TODO: Add function to save bytes to a file in icarus_models - // repo - let dir = std::path::Path::new(directory); - let save_path = dir.join(filename); - - let mut file = std::fs::File::create(&save_path).unwrap(); - file.write_all(data).unwrap(); - - save_path -} - pub async fn apply_metadata( song_queue_path: &String, coverart_queue_path: &String, diff --git a/src/the_rest.rs b/src/the_rest.rs index 08aeb4b..5d66e39 100644 --- a/src/the_rest.rs +++ b/src/the_rest.rs @@ -126,54 +126,3 @@ pub mod wipe_data { } } } - -pub mod cleanup { - pub fn clean_song_queue(song_queue_path: &String) -> Result<(), std::io::Error> { - let file_path = std::path::Path::new(song_queue_path); - if file_path.exists() { - match std::fs::remove_file(file_path) { - Ok(_) => { - if check_file_existence(song_queue_path) { - Err(std::io::Error::other(String::from( - "SongQueue file exists after a deletion", - ))) - } else { - Ok(()) - } - } - Err(err) => Err(std::io::Error::other(err.to_string())), - } - } else { - Err(std::io::Error::other(String::from( - "SongQueue file path does not exists", - ))) - } - } - - pub fn clean_coverart_queue(coverart_queue_path: &String) -> Result<(), std::io::Error> { - let coverart_file_path = std::path::Path::new(coverart_queue_path); - if coverart_file_path.exists() { - match std::fs::remove_file(coverart_file_path) { - Ok(_) => { - if !check_file_existence(coverart_queue_path) { - Ok(()) - } else { - Err(std::io::Error::other(String::from( - "CoverArt file stil exists", - ))) - } - } - Err(err) => Err(std::io::Error::other(err.to_string())), - } - } else { - Err(std::io::Error::other(String::from( - "CoverArt file does not exists", - ))) - } - } - - fn check_file_existence(file_path: &String) -> bool { - let path = std::path::Path::new(file_path); - path.exists() - } -}