Added more todos

.gitea/workflows/tag_release.yaml

@@ -4,7 +4,6 @@ on:
   pull_request:
     branches:
       - main
-      - next-v0.8
 
 jobs:
   release:

.gitea/workflows/workflow.yaml

@@ -7,7 +7,6 @@ on:
   pull_request:
     branches:
       - main 
-      - next-v0.8
 
 jobs:
   check:

src/coverart.rs

@@ -7,22 +7,22 @@ pub struct CoverArt {
     pub id: uuid::Uuid,
     pub title: String,
     #[serde(skip)]
-    pub directory: String,
+    pub path: String,
-    pub filename: String,
     #[serde(skip)]
     pub data: Vec<u8>,
     pub song_id: uuid::Uuid,
 }
 
 pub mod init {
-    use super::CoverArt;
+    use crate::coverart::CoverArt;
 
-    /// Initializes the CoverArt with just the directory and filename
+    pub fn init_coverart_only_path(path: String) -> CoverArt {
-    pub fn init_coverart_dir_and_filename(directory: &str, filename: &str) -> CoverArt {
         CoverArt {
-            directory: String::from(directory),
+            id: uuid::Uuid::nil(),
-            filename: String::from(filename),
+            title: String::new(),
-            ..Default::default()
+            path: path.clone(),
+            data: Vec::new(),
+            song_id: uuid::Uuid::nil(),
         }
     }
 }
@@ -30,23 +30,18 @@ pub mod init {
 impl CoverArt {
     /// Saves the coverart to the filesystem
     pub fn save_to_filesystem(&self) -> Result<(), std::io::Error> {
-        match self.get_path() {
+        match std::fs::File::create(&self.path) {
-            Ok(path) => match std::fs::File::create(&path) {
             Ok(mut file) => match file.write_all(&self.data) {
                 Ok(_) => Ok(()),
                 Err(err) => Err(err),
             },
             Err(err) => Err(err),
-            },
-            Err(err) => Err(err),
         }
     }
 
     /// Removes the coverart from the filesystem
     pub fn remove_from_filesystem(&self) -> Result<(), std::io::Error> {
-        match self.get_path() {
+        let p = std::path::Path::new(&self.path);
-            Ok(path) => {
-                let p = std::path::Path::new(&path);
         if p.exists() {
             match std::fs::remove_file(p) {
                 Ok(_) => Ok(()),
@@ -58,35 +53,6 @@ impl CoverArt {
             ))
         }
     }
-            Err(err) => Err(err),
-        }
-    }
-
-    /// Gets the path of the CoverArt
-    pub fn get_path(&self) -> Result<String, std::io::Error> {
-        if self.directory.is_empty() {
-            return Err(std::io::Error::other("Directory has not been initialized"));
-        } else if self.filename.is_empty() {
-            return Err(std::io::Error::other("Filename has not bee initialized"));
-        }
-
-        let directory = &self.directory;
-        let last_index = directory.len() - 1;
-
-        if let Some(character) = directory.chars().nth(last_index) {
-            let buffer = if character != '/' {
-                directory.clone() + "/"
-            } else {
-                directory.clone()
-            };
-
-            Ok(buffer + &self.filename.clone())
-        } else {
-            Err(std::io::Error::other(
-                "Could not access last character of directory",
-            ))
-        }
-    }
 }
 
 pub mod io {
@@ -94,8 +60,7 @@ pub mod io {
 
     /// Gets the raw data of the cover art
     pub fn to_data(coverart: &super::CoverArt) -> Result<Vec<u8>, std::io::Error> {
-        match coverart.get_path() {
+        let path: &String = &coverart.path;
-            Ok(path) => {
         let mut file = std::fs::File::open(path)?;
         let mut buffer = Vec::new();
         match file.read_to_end(&mut buffer) {
@@ -103,9 +68,6 @@ pub mod io {
             Err(err) => Err(err),
         }
     }
-            Err(err) => Err(err),
-        }
-    }
 }
 
 #[cfg(test)]

todos.txt

@@ -0,0 +1,3 @@
+// TODO: Make the functions/methods usable for checking if a token expires
+// TODO: Separate directory and filename from path in coverart
+// TODO: Change date type of song from string to date