diff options
author | Frédéric Guillot <fred@miniflux.net> | 2018-09-23 21:02:26 -0700 |
---|---|---|
committer | Frédéric Guillot <fred@miniflux.net> | 2018-09-23 21:02:26 -0700 |
commit | 9d08139f4363d3503398002bc82cb3746e3438cf (patch) | |
tree | b09c1efb14445624e4a2771bf8cbcf3a9828ecf9 /api/category.go | |
parent | 844680e57328034c20a2d2b84bd315b55fee9e23 (diff) |
Improve request package and add more unit tests
Diffstat (limited to 'api/category.go')
-rw-r--r-- | api/category.go | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/api/category.go b/api/category.go index e74aa3b..b8699e2 100644 --- a/api/category.go +++ b/api/category.go @@ -43,11 +43,7 @@ func (c *Controller) CreateCategory(w http.ResponseWriter, r *http.Request) { // UpdateCategory is the API handler to update a category. func (c *Controller) UpdateCategory(w http.ResponseWriter, r *http.Request) { - categoryID, err := request.IntParam(r, "categoryID") - if err != nil { - json.BadRequest(w, err) - return - } + categoryID := request.RouteInt64Param(r, "categoryID") category, err := decodeCategoryPayload(r.Body) if err != nil { @@ -85,11 +81,7 @@ func (c *Controller) GetCategories(w http.ResponseWriter, r *http.Request) { // RemoveCategory is the API handler to remove a category. func (c *Controller) RemoveCategory(w http.ResponseWriter, r *http.Request) { userID := request.UserID(r) - categoryID, err := request.IntParam(r, "categoryID") - if err != nil { - json.BadRequest(w, err) - return - } + categoryID := request.RouteInt64Param(r, "categoryID") if !c.store.CategoryExists(userID, categoryID) { json.NotFound(w, errors.New("Category not found")) |