From 9f6ab949921e1581730d0040ab21a277bb4f7cbc Mon Sep 17 00:00:00 2001
From: Andrew Phillips <andrew.phillips2@canada.ca>
Date: Wed, 13 Aug 2025 11:09:09 -0300
Subject: [PATCH] fix: remove duplicate code in handle_delete_item and fix path
 parameter syntax

Co-authored-by: aider (openai/andrew/openrouter/qwen/qwen3-coder) <aider@aider.chat>
---
 src/modes/server/api/item.rs | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/src/modes/server/api/item.rs b/src/modes/server/api/item.rs
index 3f4e3f7..fe88ae2 100644
--- a/src/modes/server/api/item.rs
+++ b/src/modes/server/api/item.rs
@@ -212,29 +212,6 @@ pub async fn handle_delete_item(
     } else {
         Err(StatusCode::NOT_FOUND)
     }
-        let mut conn = state.db.lock().await;
-        
-        if let Some(item) = db::get_item(&mut *conn, id).map_err(|e| {
-            warn!("Failed to get item {} for deletion: {}", id, e);
-            StatusCode::INTERNAL_SERVER_ERROR
-        })? {
-            db::delete_item(&mut *conn, item).map_err(|e| {
-                warn!("Failed to delete item {}: {}", id, e);
-                StatusCode::INTERNAL_SERVER_ERROR
-            })?;
-            
-            let response = ApiResponse::<()> {
-                success: true,
-                data: None,
-                error: None,
-            };
-            Ok(Json(response))
-        } else {
-            Err(StatusCode::NOT_FOUND)
-        }
-    } else {
-        Err(StatusCode::BAD_REQUEST)
-    }
 }
 
 #[utoipa::path(