From c4a4a8b5f60a568abd2af614ca4a5d06855bc3a1 Mon Sep 17 00:00:00 2001
From: xengineering <me@xengineering.eu>
Date: Sun, 7 Apr 2024 10:42:03 +0200
Subject: model: Add helper function for safe CRUD

This removes the redundant setup of a database/sql.Tx in each HTTP
handler.
---
 view/recipe.go | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

(limited to 'view')

diff --git a/view/recipe.go b/view/recipe.go
index 52b7a7e..ba670a2 100644
--- a/view/recipe.go
+++ b/view/recipe.go
@@ -12,20 +12,8 @@ func RecipeRead(w http.ResponseWriter, r *http.Request) {
 	recipe := model.Recipe{}
 	recipe.Id = mux.Vars(r)[`id`]
 
-	tx, err := model.NewTx()
-	if err != nil {
-		http.Error(w, err.Error(), http.StatusInternalServerError)
-		return
-	}
-
-	err = recipe.Read(tx)
-	if err != nil {
-		model.Rollback(tx)
-		http.Error(w, err.Error(), http.StatusInternalServerError)
-		return
-	}
-
-	err = tx.Commit()
+	var obj model.Object = &recipe
+	err := model.SafeCrud(obj.Read)
 	if err != nil {
 		http.Error(w, err.Error(), http.StatusInternalServerError)
 		return
-- 
cgit v1.2.3-70-g09d2