Remove errors.go

This commit refactors the codebase. The functionality of the error
handling is slightly reduced but the benefit are around 80 lines of code
which could be removed.

1 files changed, 17 insertions, 16 deletions

diff --git a/handler.go b/handler.go
index 1c922fe..fc3d653 100644
--- a/handler.go
+++ b/handler.go
@@ -33,7 +33,7 @@ func indexGet(w http.ResponseWriter, r *http.Request, db *Database, templateRoot
 	log.Printf("Query: %s", cmd)
 	rows, err := db.Backend.Query(cmd)
 	if err != nil {
-		Err(w, 1)
+		http.Error(w, "Failed to load recipes from database.", 500)
 		return
 	}
 	defer rows.Close()
@@ -50,7 +50,7 @@ func indexGet(w http.ResponseWriter, r *http.Request, db *Database, templateRoot
 		var element Element
 		err := rows.Scan(&element.Id, &element.Title)
 		if err != nil {
-			Err(w, 2)
+			http.Error(w, "Could not parse recipe from database request.", 500)
 			return
 		} else {
 			elements = append(elements, element)
@@ -69,7 +69,8 @@ func recipe(db *Database, templateRoot string) func(http.ResponseWriter, *http.R
 		// get id from URL parameters
 		ids := r.URL.Query()["id"]
 		if len(ids) != 1 {
-			Err(w, 3, len(ids))
+			msg := fmt.Sprintf("Exactly 1 'id' URL parameter expected but %d provided.", len(ids))
+			http.Error(w, msg, 400)
 			return
 		}
 		idStr := ids[0]
@@ -77,7 +78,7 @@ func recipe(db *Database, templateRoot string) func(http.ResponseWriter, *http.R
 		// validate id
 		idRegex := regexp.MustCompile(VALID_ID_REGEX)
 		if !(idRegex.MatchString(idStr)) {
-			Err(w, 4, idStr, VALID_ID_REGEX)
+			http.Error(w, "Bad 'id' URL parameter.", 400)
 			return
 		}
 
@@ -88,7 +89,7 @@ func recipe(db *Database, templateRoot string) func(http.ResponseWriter, *http.R
 			log.Printf("Query: %s", cmd)
 			rows, err := db.Backend.Query(cmd)
 			if err != nil {
-				Err(w, 5, err)
+				http.Error(w, "Database returned error: " + err.Error(), 500)
 				return
 			}
 			defer rows.Close()
@@ -109,7 +110,7 @@ func recipe(db *Database, templateRoot string) func(http.ResponseWriter, *http.R
 				element.Id = idStr
 				err := rows.Scan(&element.Title, &element.UpstreamUrl, &element.DescriptionMarkdown)
 				if err != nil {
-					Err(w, 2)
+					http.Error(w, "Could not parse recipe from database request.", 500)
 					return
 				} else {
 					elements = append(elements, element)
@@ -118,7 +119,7 @@ func recipe(db *Database, templateRoot string) func(http.ResponseWriter, *http.R
 
 			// check result
 			if len(elements) != 1 {
-				Err(w, 6, len(elements))
+				http.Error(w, "Expected exactly 1 recipe from database.", 500)
 				return
 			}
 
@@ -150,7 +151,7 @@ func recipe_edit(db *Database, templateRoot string) func(http.ResponseWriter, *h
 		// get id from URL parameters
 		ids := r.URL.Query()["id"]
 		if len(ids) != 1 {
-			Err(w, 3, len(ids))
+			http.Error(w, "Exactly 1 'id' URL parameter expected.", 400)
 			return
 		}
 		idStr := ids[0]
@@ -158,7 +159,7 @@ func recipe_edit(db *Database, templateRoot string) func(http.ResponseWriter, *h
 		// validate id
 		idRegex := regexp.MustCompile(VALID_ID_REGEX)
 		if !(idRegex.MatchString(idStr)) {
-			Err(w, 4, idStr, VALID_ID_REGEX)
+			http.Error(w, "Bad 'id' URL parameter.", 400)
 			return
 		}
 
@@ -169,7 +170,7 @@ func recipe_edit(db *Database, templateRoot string) func(http.ResponseWriter, *h
 			log.Printf("Query: %s", cmd)
 			rows, err := db.Backend.Query(cmd)
 			if err != nil {
-				Err(w, 5, err)
+				http.Error(w, "Got error from database: " + err.Error(), 500)
 				return
 			}
 			defer rows.Close()
@@ -190,7 +191,7 @@ func recipe_edit(db *Database, templateRoot string) func(http.ResponseWriter, *h
 				element.Id = idStr
 				err := rows.Scan(&element.Title, &element.UpstreamUrl, &element.DescriptionMarkdown)
 				if err != nil {
-					Err(w, 2)
+					http.Error(w, "Could not parse recipe from database request.", 500)
 					return
 				} else {
 					elements = append(elements, element)
@@ -199,7 +200,7 @@ func recipe_edit(db *Database, templateRoot string) func(http.ResponseWriter, *h
 
 			// check result
 			if len(elements) != 1 {
-				Err(w, 6, len(elements))
+				http.Error(w, "Did not get exactly one recipe from database.", 500)
 				return
 			}
 
@@ -248,7 +249,7 @@ func image(storageRoot string) func(http.ResponseWriter, *http.Request) {
 		// get ID
 		ids := r.URL.Query()["id"]
 		if len(ids) != 1 {
-			Err(w, 3, len(ids))
+			http.Error(w, "Expected exactly one 'id' URL parameter.", 400)
 			return
 		}
 		idStr := ids[0]
@@ -256,7 +257,7 @@ func image(storageRoot string) func(http.ResponseWriter, *http.Request) {
 		// validate ID
 		idRegex := regexp.MustCompile(VALID_ID_REGEX)
 		if !idRegex.MatchString(idStr) {
-			Err(w, 4, idStr, VALID_ID_REGEX)
+			http.Error(w, "Bad 'id' URL parameter.", 400)
 			return
 		}
 
@@ -286,12 +287,12 @@ func add_recipes(db *Database, storageRoot string, staticRoot string) func(http.
 			log.Println(cmd)
 			res,err := db.Backend.Exec(cmd)
 			if err != nil {
-				Err(w, 9, err)
+				http.Error(w, "Could not add recipe.", 500)
 				return
 			}
 			id,err := res.LastInsertId()
 			if err != nil {
-				Err(w, 11, err)
+				http.Error(w, "Expected exactly one recipe URL.", 400)
 				return
 			} else {
 				log.Println("Added custom recipe.")