refactor to be able to write not only default

Author: Sk7Str1p3

src/config.rs

@@ -34,24 +34,23 @@ impl ConfigOptions {
         }
     }
 
-    pub fn write_default<P>(path: P) -> Result<()>
+    pub fn write<P>(self, path: P) -> Result<()>
     // I believe user would like to generate config with CLI flags
     // I mean to write disabled_fields with --disabled-fields flag
     where
-        // Nah this feels really bad but rust-analyser told me to add ' + std::fmt::Display'
         P: AsRef<Path>,
     {
         // I dont think this can panic so i simply unwrapped it
-        let defaults = toml::to_string(&Self::default()).unwrap();
+        let contents = toml::to_string(&self).unwrap();
         match fs::create_dir_all(&path.as_ref().parent().unwrap_or(Path::new("/"))) {
-            Ok(_) => match fs::write(&path, &defaults) {
+            Ok(_) => match fs::write(&path, &contents) {
                 Ok(_) => {
                     let path = path.as_ref().display();
-                    println!("Default config file created at: {path}")
+                    println!("Config config file created at: {path}")
                 }
                 Err(e) => {
                     let path = path.as_ref().display();
-                    eprintln!("Failed to write default config file at {path}: {e}")
+                    eprintln!("Failed to write config file at {path}: {e}")
                 }
             },
             Err(e) => {

src/main.rs

@@ -8,7 +8,6 @@ use onefetch::config::ConfigOptions;
 use onefetch::info::build_info;
 use onefetch::ui::printer::Printer;
 use std::io;
-use std::process::exit;
 
 fn main() -> Result<()> {
     setup_panic!();
@@ -33,12 +32,22 @@ fn main() -> Result<()> {
     }
 
     if cli_options.config.generate_config {
-        // what the actual FUCK is happening here?
-        // why default path is EMPTY?
-        return ConfigOptions::write_default(&cli_options.config.config_path.unwrap_or_default());
+        return ConfigOptions::default().write(
+            &cli_options.config.config_path.unwrap_or(
+                dirs::config_dir()
+                    .expect("Could not find config dir!")
+                    .join("onefetch/config.toml"),
+            ),
+        );
     }
 
-    let config_options = ConfigOptions::read(&cli_options.config.config_path.clone().unwrap_or_default());
+    let config_options = ConfigOptions::read(
+        &cli_options.config.config_path.as_ref().unwrap_or(
+            &dirs::config_dir()
+                .expect("Could not find config dir!")
+                .join("onefetch/config.toml"),
+        ),
+    );
 
     let info = build_info(&cli_options, &config_options)?;