From 0455da359b9fe36bd6808b7e506fe24e48257bea Mon Sep 17 00:00:00 2001
From: xengineering <me@xengineering.eu>
Date: Mon, 22 Apr 2024 21:50:26 +0200
Subject: model: Enforce recipe titles

If a recipe has no title it is hard to reference in the front end.
Especially the /recipes page makes problems in that case since it is
impossible to click on that recipes and thus also to remove it.
---
 model/recipe.go | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/model/recipe.go b/model/recipe.go
index c4f0d47..d8c0f2f 100644
--- a/model/recipe.go
+++ b/model/recipe.go
@@ -23,11 +23,24 @@ func (r Recipe) String() string {
 	return string(b)
 }
 
+func (r *Recipe) Validate() error {
+	if r.Title == "" {
+		return fmt.Errorf("Recipes must have a title")
+	}
+
+	return nil
+}
+
 func (r *Recipe) Create(tx *sql.Tx) error {
 	if r.Id != "" {
 		return fmt.Errorf("Cannot create recipe if ID is given")
 	}
 
+	err := r.Validate()
+	if err != nil {
+		return err
+	}
+
 	cmd := `
 INSERT INTO recipes
     (title, portions, url, notes, created, last_changed)
@@ -138,10 +151,15 @@ WHERE
 		}
 	}
 
-	return nil
+	return r.Validate()
 }
 
 func (r *Recipe) Update(tx *sql.Tx) error {
+	err := r.Validate()
+	if err != nil {
+		return err
+	}
+
 	oldSteps, err := r.getStepIds(tx)
 	if err != nil {
 		return err
-- 
cgit v1.2.3-70-g09d2