model: Move timestamp updates to controller

The model package should handle the object relational mapping (ORM).
This requires implementing the four CRUD methods:

- create
- read
- update
- delete

On create and update the model package used to modify the timestamps
like `last_changed`. This was detected by the unit tests which tested
that the data is not changed in a create / read cycle or is updated
correctly in an update / read cycle.

This raised the question if it is a good idea that the model package is
"smart" and updates timestamps. To keep the model package and the
included unit tests simple the new design enforces that the complete
data - including metadata - is always exactly the same after using any
CRUD methods.

The functionality of updating the timestamp is moved to the HTTP handler
inside the controller package. This also matches the definition of the
controller package as the part of the code which is alone responsible to
actually change the data.

This commit finally fixes the unit test suite.

2 files changed, 4 insertions, 5 deletions

diff --git a/controller/recipe.go b/controller/recipe.go
index 818a077..6acb062 100644
--- a/controller/recipe.go
+++ b/controller/recipe.go
@@ -13,6 +13,8 @@ import (
 func RecipeCreate(w http.ResponseWriter, r *http.Request) {
 	recipe := model.Recipe{}
 	recipe.Title = "recipe without title"
+	recipe.Touch()
+	recipe.Created = recipe.LastChanged
 
 	err := recipe.Create()
 	if err != nil {
@@ -42,6 +44,8 @@ func RecipeUpdate(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
+	recipe.Touch()
+
 	err = recipe.Update()
 	if err != nil {
 		http.Error(w, err.Error(), http.StatusInternalServerError)
diff --git a/model/recipe.go b/model/recipe.go
index b9830e6..db3af70 100644
--- a/model/recipe.go
+++ b/model/recipe.go
@@ -29,9 +29,6 @@ func (r *Recipe) Touch() {
 }
 
 func (r *Recipe) Create() error {
-	r.Touch()
-	r.Created = r.LastChanged
-
 	query := `INSERT INTO recipes
     (title, portions, url, notes, created, last_changed)
 VALUES
@@ -88,8 +85,6 @@ WHERE id = ?`
 }
 
 func (r *Recipe) Update() error {
-	r.Touch()
-
 	query := `UPDATE
     recipes
 SET