Modified http status code on error

routes/api/v1/author_delete.go

@@ -35,7 +35,7 @@ func AuthorDelete(c echo.Context) error {
 
 	if err != nil {
 		if models.IsErrIDCannotBeZero(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0"})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0"})
 		}
 		return c.JSON(http.StatusInternalServerError, models.Message{"Could not delete author."})
 	}

routes/api/v1/authors_add_update.go

@@ -63,7 +63,7 @@ func AuthorAddOrUpdate(c echo.Context) error {
 
 	if err != nil {
 		if models.IsErrAuthorCannotBeEmpty(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0"})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0"})
 		}
 		return c.JSON(http.StatusInternalServerError, models.Message{"Error"})
 	}

routes/api/v1/book_delete.go

@@ -35,7 +35,7 @@ func BookDelete(c echo.Context) error {
 
 	if err != nil {
 		if models.IsErrIDCannotBeZero(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0"})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0"})
 		}
 		return c.JSON(http.StatusInternalServerError, models.Message{"Could not delete book."})
 	}

routes/api/v1/books_add_update.go

@@ -7,7 +7,6 @@ import (
 	"net/http"
 	"strconv"
 	"strings"
-	"fmt"
 )
 
 // BookAddOrUpdate is the handler to add a book
@@ -63,9 +62,8 @@ func BookAddOrUpdate(c echo.Context) error {
 	newBook, err := models.AddOrUpdateBook(*datBook)
 
 	if err != nil {
-		fmt.Println(err)
 		if models.IsErrAuthorCannotBeEmpty(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0."})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0."})
 		}
 
 		if models.IsErrBookTitleCannotBeEmpty(err) {

routes/api/v1/items_delete.go

@@ -35,7 +35,7 @@ func ItemDelete(c echo.Context) error {
 
 	if err != nil {
 		if models.IsErrIDCannotBeZero(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0"})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0"})
 		}
 		return c.JSON(http.StatusInternalServerError, models.Message{"Could not delete item."})
 	}

routes/api/v1/publishers_delete.go

@@ -35,7 +35,7 @@ func PublisherDelete(c echo.Context) error {
 
 	if err != nil {
 		if models.IsErrIDCannotBeZero(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0"})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0"})
 		}
 		return c.JSON(http.StatusInternalServerError, models.Message{"Could not delete publisher."})
 	}

routes/api/v1/user_delete.go

@@ -40,7 +40,7 @@ func UserDelete(c echo.Context) error {
 
 	if err != nil {
 		if models.IsErrIDCannotBeZero(err) {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Id cannot be 0"})
+			return c.JSON(http.StatusBadRequest, models.Message{"Id cannot be 0"})
 		}
 		return c.JSON(http.StatusInternalServerError, models.Message{"Could not delete user."})
 	}