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 -}  |