From bf6feab9cac0bbf54607ba3401a5116440abcf85 Mon Sep 17 00:00:00 2001
From: Santo Cariotti <santo@dcariotti.me>
Date: Mon, 17 Oct 2022 22:58:40 +0200
Subject: Fix: edit warning for resolved_by already defined as number

---
 src/warning/models.rs | 2 +-
 src/warning/routes.rs | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/warning/models.rs b/src/warning/models.rs
index 9e79e6c..421ff38 100644
--- a/src/warning/models.rs
+++ b/src/warning/models.rs
@@ -59,7 +59,7 @@ pub struct WarningCreate {
 #[derive(Deserialize)]
 pub struct WarningEdit {
     pub admin_note: String,
-    pub resolved_by: Option<bool>,
+    pub resolved_by: Option<i32>,
 }
 
 /// Payload used for warning filtering
diff --git a/src/warning/routes.rs b/src/warning/routes.rs
index e57b8a8..bb08963 100644
--- a/src/warning/routes.rs
+++ b/src/warning/routes.rs
@@ -101,7 +101,7 @@ async fn edit_warning(
         return Err(AppError::Unauthorized);
     }
 
-    if payload.resolved_by.is_none() || payload.resolved_by.unwrap() {
+    if payload.resolved_by.is_none() || payload.resolved_by.unwrap() > 0 {
         warning.edit(Some(user.id), payload).await?;
     } else {
         warning.edit(None, payload).await?;
-- 
cgit v1.2.3-71-g8e6c