diff options
Diffstat (limited to 'item/validators.go')
-rw-r--r-- | item/validators.go | 58 |
1 files changed, 3 insertions, 55 deletions
diff --git a/item/validators.go b/item/validators.go index b808ae4..63c254f 100644 --- a/item/validators.go +++ b/item/validators.go @@ -32,11 +32,11 @@ func (b *Brand) validate() error { return errors.ErrEmptyBrandName } - // make sure GSTIN is unique + // make sure brand name is unique var count int64 err := db.Model(&Brand{}). Select("name"). - Where("name = ? and user_id = ?", b.Name, b.UserID). + Where("name = ?", b.Name). Count(&count). Error @@ -75,7 +75,7 @@ func (i *SavedItem) validate() error { var count int64 err = db.Model(&SavedItem{}). Select("name, brand_id"). - Where("brand_id = ? and name = ? and user_id = ?", i.BrandID, i.Name, i.UserID). + Where("brand_id = ? and name = ?", i.BrandID, i.Name). Count(&count). Error @@ -89,55 +89,3 @@ func (i *SavedItem) validate() error { return nil } - -func checkBrandOwnership(brandId, userId uint) error { - var brand Brand - err := db. - Select("id", "user_id"). - Where("id = ?", brandId). - Find(&brand). - Error - - // TODO: handle potential errors - if err != nil { - return err - } - - // brand doesn't exist - if brand.ID == 0 { - return errors.ErrNotFound - } - - // user doesn't own this brand - if brand.UserID != userId { - return errors.ErrForbidden - } - - return nil -} - -func checkItemOwnership(itemId, userId uint) error { - var item SavedItem - err := db. - Select("id", "user_id"). - Where("id = ?", itemId). - Find(&item). - Error - - // TODO: handle potential errors - if err != nil { - return err - } - - // item doesn't exist - if item.ID == 0 { - return errors.ErrNotFound - } - - // user doesn't own this item - if item.UserID != userId { - return errors.ErrForbidden - } - - return nil -} |