diff --git a/routes/api/v1/author_delete.go b/routes/api/v1/author_delete.go index ee88c43..f40f8e1 100644 --- a/routes/api/v1/author_delete.go +++ b/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."}) } diff --git a/routes/api/v1/authors_add_update.go b/routes/api/v1/authors_add_update.go index 4ddc2ee..3cbfec1 100644 --- a/routes/api/v1/authors_add_update.go +++ b/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"}) } diff --git a/routes/api/v1/book_delete.go b/routes/api/v1/book_delete.go index 6901899..14eb8f9 100644 --- a/routes/api/v1/book_delete.go +++ b/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."}) } diff --git a/routes/api/v1/books_add_update.go b/routes/api/v1/books_add_update.go index abe18bb..115afe2 100644 --- a/routes/api/v1/books_add_update.go +++ b/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) { diff --git a/routes/api/v1/items_delete.go b/routes/api/v1/items_delete.go index 31948ba..e105364 100644 --- a/routes/api/v1/items_delete.go +++ b/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."}) } diff --git a/routes/api/v1/publishers_delete.go b/routes/api/v1/publishers_delete.go index 2c26ff4..7f3c011 100644 --- a/routes/api/v1/publishers_delete.go +++ b/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."}) } diff --git a/routes/api/v1/user_delete.go b/routes/api/v1/user_delete.go index 0abd496..88313ad 100644 --- a/routes/api/v1/user_delete.go +++ b/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."}) }