From ac89c35edc361159b653e97c1e0e13516c3cb826 Mon Sep 17 00:00:00 2001
From: mpostma <postma.marin@protonmail.com>
Date: Tue, 23 Feb 2021 19:46:18 +0100
Subject: [PATCH] add settings routes errors

---
 src/routes/settings/mod.rs | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/src/routes/settings/mod.rs b/src/routes/settings/mod.rs
index 56c1d34a0..00bc4220e 100644
--- a/src/routes/settings/mod.rs
+++ b/src/routes/settings/mod.rs
@@ -1,5 +1,4 @@
 use actix_web::{web, HttpResponse, delete, get, post};
-use log::error;
 
 use crate::Data;
 use crate::error::ResponseError;
@@ -33,8 +32,7 @@ macro_rules! make_setting_route {
                         Ok(HttpResponse::Ok().body(json))
                     }
                     Err(e) => {
-                        log::error!("{}", e);
-                        unimplemented!();
+                        Ok(HttpResponse::BadRequest().body(serde_json::json!({ "error": e.to_string() })))
                     }
                 }
             }
@@ -56,8 +54,7 @@ macro_rules! make_setting_route {
                         Ok(HttpResponse::Ok().body(json))
                     }
                     Err(e) => {
-                        log::error!("{}", e);
-                        unimplemented!();
+                        Ok(HttpResponse::BadRequest().body(serde_json::json!({ "error": e.to_string() })))
                     }
                 }
             }
@@ -74,8 +71,7 @@ macro_rules! make_setting_route {
                         Ok(HttpResponse::Ok().body(json))
                     }
                     Err(e) => {
-                        log::error!("{}", e);
-                        unimplemented!();
+                        Ok(HttpResponse::BadRequest().body(serde_json::json!({ "error": e.to_string() })))
                     }
                 }
             }
@@ -147,8 +143,7 @@ async fn update_all(
             Ok(HttpResponse::Ok().body(json))
         }
         Err(e) => {
-            error!("{}", e);
-            unimplemented!();
+            Ok(HttpResponse::BadRequest().body(serde_json::json!({ "error": e.to_string() })))
         }
     }
 }
@@ -164,8 +159,7 @@ async fn get_all(
             Ok(HttpResponse::Ok().body(json))
         }
         Err(e) => {
-            error!("{}", e);
-            unimplemented!();
+            Ok(HttpResponse::BadRequest().body(serde_json::json!({ "error": e.to_string() })))
         }
     }
 }
@@ -182,8 +176,7 @@ async fn delete_all(
             Ok(HttpResponse::Ok().body(json))
         }
         Err(e) => {
-            error!("{}", e);
-            unimplemented!();
+            Ok(HttpResponse::BadRequest().body(serde_json::json!({ "error": e.to_string() })))
         }
     }
 }