Rewrite DeleteSubscriber Handler

api/main.go

@@ -1257,34 +1257,24 @@ func (app *App) DeleteSubscriber(w http.ResponseWriter, r *http.Request) {
  return
  }
 
- // Extract the subscriber ID from the URL path
- vars := mux.Vars(r)
- subscriberID, err := strconv.Atoi(vars["id"])
+ subscriberID, err := strconv.Atoi(mux.Vars(r)["id"])
  if err != nil {
  http.Error(w, "Invalid subscriber ID", http.StatusBadRequest)
  return
  }
 
- // Query to delete the subscriber
- deleteQuery := `
- DELETE FROM subscribers
- WHERE id = ?
- `
-
- // Execute the query to delete the subscriber
- result, err := app.DB.Exec(deleteQuery, subscriberID)
+ result, err := app.DB.Exec(`DELETE FROM subscribers WHERE id = ?`, subscriberID)
  if err != nil {
- app.Logger.Printf("Failed to delete subscriber: %v", err)
- http.Error(w, fmt.Sprintf("Failed to delete subscriber: %v", err), http.StatusInternalServerError)
+ http.Error(w, "Failed to delete subscriber", http.StatusInternalServerError)
  return
  }
 
- // Check if any row was actually deleted
  rowsAffected, _ := result.RowsAffected()
  if rowsAffected == 0 {
  http.Error(w, "Subscriber not found", http.StatusNotFound)
  return
  }
 
- fmt.Fprintf(w, "Subscriber deleted successfully")
+ fmt.Fprintln(w, "Subscriber deleted successfully")
 }
+

api/main_test.go

@@ -3444,3 +3444,4 @@ func TestUpdateSubscriber_FailedToRetrieveAffectedRows(t *testing.T) {
 
 
 
+