fix: add return after failure req
This commit is contained in:
@@ -25,7 +25,7 @@ func NewPostHandlers(repo repositories.PostRepository) *PostHandlers {
|
||||
// @Tags posts
|
||||
// @Accept json
|
||||
// @Produce json
|
||||
// @Success 200 {object} []storage.PostReq
|
||||
// @Success 200 {object} models.SuccessResponse{data=[]storage.PostReq}
|
||||
// @Failure 404 {object} models.ErrorResponse "No Post found"
|
||||
// @Failure 500 {object} models.ErrorResponse "Internal server error"
|
||||
// @Failure 400 {object} models.ErrorResponse "Invalid ID format"
|
||||
@@ -36,9 +36,11 @@ func (h *PostHandlers) GetPosts(c *gin.Context) {
|
||||
if err != nil {
|
||||
h.logger.Error("error request: " + err.Error())
|
||||
models.Error(c, 500, "Internal server error", err.Error())
|
||||
return
|
||||
}
|
||||
if result == nil {
|
||||
models.Error(c, 404, "No Post found", "")
|
||||
return
|
||||
}
|
||||
h.logger.Info("200 OK GET /posts")
|
||||
models.Success(c, result)
|
||||
@@ -62,12 +64,14 @@ func (h *PostHandlers) GetPost(c *gin.Context) {
|
||||
if err != nil {
|
||||
h.logger.Error("error request: " + err.Error())
|
||||
models.Error(c, 500, "Internal server error", err.Error())
|
||||
return
|
||||
}
|
||||
id_p := c.Param("id")
|
||||
id, err := strconv.Atoi(id_p)
|
||||
if err != nil {
|
||||
h.logger.Error("error request: " + err.Error())
|
||||
models.Error(c, 400, "Invalid ID format", err.Error())
|
||||
return
|
||||
}
|
||||
if id > last_id {
|
||||
models.Error(c, 404, "Post not found", "")
|
||||
@@ -77,6 +81,7 @@ func (h *PostHandlers) GetPost(c *gin.Context) {
|
||||
if err != nil {
|
||||
h.logger.Error("error request: " + err.Error())
|
||||
models.Error(c, 500, "Internal server error", err.Error())
|
||||
return
|
||||
}
|
||||
h.logger.Info("200 OK GET /posts/" + id_p)
|
||||
models.Success(c, result)
|
||||
@@ -133,6 +138,7 @@ func (h *PostHandlers) UpdatePost(c *gin.Context) {
|
||||
if err != nil {
|
||||
h.logger.Error("error request: " + err.Error())
|
||||
models.Error(c, 500, "Internal server error", err.Error())
|
||||
return
|
||||
}
|
||||
var req storage.Post
|
||||
if err := c.ShouldBindJSON(&req); err != nil {
|
||||
@@ -167,6 +173,7 @@ func (h *PostHandlers) DeletePost(c *gin.Context) {
|
||||
if err != nil {
|
||||
h.logger.Error("error request: " + err.Error())
|
||||
models.Error(c, 400, "Invalid ID format", err.Error())
|
||||
return
|
||||
}
|
||||
exsist := h.repo.IsExist(c.Request.Context(), id)
|
||||
if !exsist {
|
||||
|
||||
Reference in New Issue
Block a user