Merge pull request 'todos' (#5) from todos into master

Reviewed-on: http://raspberrypi.fritz.box:7920/fx/modlist/pulls/5

1 files changed, 14 insertions, 3 deletions

diff --git a/src/commands/list.rs b/src/commands/list.rs
index 13176f4..4aa4306 100644
--- a/src/commands/list.rs
+++ b/src/commands/list.rs
@@ -2,9 +2,9 @@ use crate::{
     config::Cfg,
     db::{
         config_change_current_list, config_get_current_list, lists_get, lists_insert, lists_remove,
-        lists_version,
+        lists_version, lists_get_all_ids,
     },
-    error::MLE,
+    error::{MLE, MLError, ErrorType},
     update, Modloader,
 };
 
@@ -32,7 +32,9 @@ pub fn list_add(
 }
 
 pub fn list_change(config: Cfg, id: String) -> MLE<()> {
-    //TODO check if list exists
+    if lists_get_all_ids(config.clone())?.into_iter().find(|l| l == &id).is_none() {
+        return Err(MLError::new(ErrorType::ArgumentError, "List not found"));
+    };
     println!("Change default list to: {}", id);
     config_change_current_list(config, id)
 }
@@ -68,3 +70,12 @@ pub async fn list_version(
     let list = lists_get(config.clone(), id)?;
     update(config, vec![list], true, download, delete).await
 }
+
+pub fn list_list(config: Cfg) -> MLE<()> {
+    let lists = lists_get_all_ids(config.clone())?;
+    for list in lists {
+        let l = lists_get(config.clone(), list)?;
+        println!("{}: | {} | {}", l.id, l.mc_version, l.modloader)
+    }
+    Ok(())
+}