Fix validation and data access layer
Signed-off-by: Émile Ré <emile@getprobo.com>
This commit is contained in:
@@ -127,6 +127,7 @@ func (r *UpdateCookieCategoryRequest) Validate() error {
|
|||||||
v.Check(r.CookieCategoryID, "cookie_category_id", validator.Required(), validator.GID(coredata.CookieCategoryEntityType))
|
v.Check(r.CookieCategoryID, "cookie_category_id", validator.Required(), validator.GID(coredata.CookieCategoryEntityType))
|
||||||
v.Check(r.Name, "name", validator.SafeTextNoNewLine(255))
|
v.Check(r.Name, "name", validator.SafeTextNoNewLine(255))
|
||||||
v.Check(r.Description, "description", validator.SafeText(1000))
|
v.Check(r.Description, "description", validator.SafeText(1000))
|
||||||
|
v.Check(r.Rank, "rank", validator.Min(0))
|
||||||
|
|
||||||
return v.Error()
|
return v.Error()
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -338,11 +338,15 @@ WHERE
|
|||||||
}
|
}
|
||||||
maps.Copy(args, scope.SQLArguments())
|
maps.Copy(args, scope.SQLArguments())
|
||||||
|
|
||||||
_, err := tx.Exec(ctx, q, args)
|
result, err := tx.Exec(ctx, q, args)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return fmt.Errorf("cannot update cookie banner: %w", err)
|
return fmt.Errorf("cannot update cookie banner: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if result.RowsAffected() == 0 {
|
||||||
|
return ErrResourceNotFound
|
||||||
|
}
|
||||||
|
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user