remove allow module_name_repetition and fix itHEADmain

1 files changed, 92 insertions, 87 deletions

diff --git a/src/commands/list.rs b/src/commands/list.rs
index 47c1dc6..148bd16 100644
--- a/src/commands/list.rs
+++ b/src/commands/list.rs
@@ -1,4 +1,3 @@
-#![allow(clippy::module_name_repetitions)]
 use indicatif::{ProgressBar, ProgressStyle};
 
 use crate::{
@@ -19,101 +18,107 @@ pub struct List {
     pub download_folder: String,
 }
 
-/// # Errors
-pub fn get_current_list(config: &Cfg) -> MLE<List> {
-    let id = config_get_current_list(config)?;
-    lists_get(config, &id)
-}
+impl List {
+    /// # Errors
+    pub fn get_current_list(config: &Cfg) -> MLE<List> {
+        let id = config_get_current_list(config)?;
+        lists_get(config, &id)
+    }
 
-/// # Errors
-pub fn list_add(
-    config: &Cfg,
-    id: &str,
-    mc_version: &str,
-    modloader: &Modloader,
-    directory: &str,
-) -> MLE<()> {
-    let p = ProgressBar::new_spinner();
-    p.set_style(
-        ProgressStyle::with_template(STYLE_OPERATION)
-            .map_err(|_| MLErr::new(EType::LibIndicatif, "template error"))?,
-    );
-    p.set_message(format!("Create {id}"));
-    lists_insert(config, id, mc_version, modloader, directory)?;
-    p.finish_with_message(format!("Created {id}"));
-    Ok(())
-}
+    /// # Errors
+    pub fn add(
+        config: &Cfg,
+        id: &str,
+        mc_version: &str,
+        modloader: &Modloader,
+        directory: &str,
+    ) -> MLE<()> {
+        let p = ProgressBar::new_spinner();
+        p.set_style(
+            ProgressStyle::with_template(STYLE_OPERATION).map_err(|_| {
+                MLErr::new(EType::LibIndicatif, "template error")
+            })?,
+        );
+        p.set_message(format!("Create {id}"));
+        lists_insert(config, id, mc_version, modloader, directory)?;
+        p.finish_with_message(format!("Created {id}"));
+        Ok(())
+    }
 
-/// # Errors
-pub fn list_change(config: &Cfg, id: &str) -> MLE<()> {
-    let p = ProgressBar::new_spinner();
-    p.set_style(
-        ProgressStyle::with_template(STYLE_OPERATION)
-            .map_err(|_| MLErr::new(EType::LibIndicatif, "template error"))?,
-    );
-    p.set_message(format!("Change default list to {id}"));
+    /// # Errors
+    pub fn change(config: &Cfg, id: &str) -> MLE<()> {
+        let p = ProgressBar::new_spinner();
+        p.set_style(
+            ProgressStyle::with_template(STYLE_OPERATION).map_err(|_| {
+                MLErr::new(EType::LibIndicatif, "template error")
+            })?,
+        );
+        p.set_message(format!("Change default list to {id}"));
 
-    if !lists_get_all_ids(config)?.into_iter().any(|l| l == id) {
-        return Err(MLErr::new(EType::ArgumentError, "List not found"));
-    };
-    config_change_current_list(config, id)?;
+        if !lists_get_all_ids(config)?.into_iter().any(|l| l == id) {
+            return Err(MLErr::new(EType::ArgumentError, "List not found"));
+        };
+        config_change_current_list(config, id)?;
 
-    p.finish_with_message(format!("Changed default list to {id}"));
-    Ok(())
-}
+        p.finish_with_message(format!("Changed default list to {id}"));
+        Ok(())
+    }
 
-/// # Errors
-pub fn list_remove(config: &Cfg, id: &str) -> MLE<()> {
-    let p = ProgressBar::new_spinner();
-    p.set_style(
-        ProgressStyle::with_template(STYLE_OPERATION)
-            .map_err(|_| MLErr::new(EType::LibIndicatif, "template error"))?,
-    );
-    p.set_message(format!("Remove {id}"));
-    lists_remove(config, id)?;
-    p.finish_with_message(format!("Removed {id}"));
-    Ok(())
-}
+    /// # Errors
+    pub fn remove(config: &Cfg, id: &str) -> MLE<()> {
+        let p = ProgressBar::new_spinner();
+        p.set_style(
+            ProgressStyle::with_template(STYLE_OPERATION).map_err(|_| {
+                MLErr::new(EType::LibIndicatif, "template error")
+            })?,
+        );
+        p.set_message(format!("Remove {id}"));
+        lists_remove(config, id)?;
+        p.finish_with_message(format!("Removed {id}"));
+        Ok(())
+    }
 
-///Changing the current lists version and updating it
-///
-/// #Arguments
-///
-/// * `config` - The current config
-/// * `args` - All args, to extract the new version
-/// # Errors
-pub async fn list_version(
-    config: &Cfg,
-    id: &str,
-    mc_version: String,
-    download: bool,
-    delete: bool,
-) -> MLE<()> {
-    let p = ProgressBar::new_spinner();
-    p.set_style(
-        ProgressStyle::with_template(STYLE_OPERATION)
-            .map_err(|_| MLErr::new(EType::LibIndicatif, "template error"))?,
-    );
-    p.set_message(format!(
-        "Change version for list {id} to minecraft version: {mc_version}"
-    ));
+    ///Changing the current lists version and updating it
+    ///
+    /// #Arguments
+    ///
+    /// * `config` - The current config
+    /// * `args` - All args, to extract the new version
+    /// # Errors
+    pub async fn version(
+        config: &Cfg,
+        id: &str,
+        mc_version: String,
+        download: bool,
+        delete: bool,
+    ) -> MLE<()> {
+        let p = ProgressBar::new_spinner();
+        p.set_style(
+            ProgressStyle::with_template(STYLE_OPERATION).map_err(|_| {
+                MLErr::new(EType::LibIndicatif, "template error")
+            })?,
+        );
+        p.set_message(format!(
+            "Change version for list {id} to minecraft version: {mc_version}"
+        ));
 
-    lists_version(config, id, &mc_version)?;
+        lists_version(config, id, &mc_version)?;
 
-    p.finish_with_message(format!(
-        "Changed version for list {id} to minecraft version: {mc_version}"
-    ));
+        p.finish_with_message(format!(
+            "Changed version for list {id} to minecraft version: {mc_version}"
+        ));
 
-    let list = lists_get(config, id)?;
-    update(config, vec![list], true, download, delete).await
-}
+        let list = lists_get(config, id)?;
+        update(config, vec![list], true, download, delete).await
+    }
 
-/// # Errors
-pub fn list_lists(config: &Cfg) -> MLE<()> {
-    let lists = lists_get_all_ids(config)?;
-    for list in lists {
-        let l = lists_get(config, &list)?;
-        println!("{}: | {} | {}", l.id, l.mc_version, l.modloader);
+    /// # Errors
+    pub fn list(config: &Cfg) -> MLE<()> {
+        let lists = lists_get_all_ids(config)?;
+        for list in lists {
+            let l = lists_get(config, &list)?;
+            println!("{}: | {} | {}", l.id, l.mc_version, l.modloader);
+        }
+        Ok(())
     }
-    Ok(())
 }