diff options
author | Vidhu Kant Sharma <vidhukant@vidhukant.com> | 2023-09-03 22:08:12 +0530 |
---|---|---|
committer | Vidhu Kant Sharma <vidhukant@vidhukant.com> | 2023-09-03 22:08:12 +0530 |
commit | 620fb2f89b5acd70af4ed075b4797f81a3f1d109 (patch) | |
tree | 191bbadfbe77fb71342853b85649b4ee5cb3c2bb /customer/validators.go | |
parent | c6bc8d0c6d1c1ab5d26b9e47e0248002d22ecf8f (diff) |
validating email addressesv0.0.3
Diffstat (limited to 'customer/validators.go')
-rw-r--r-- | customer/validators.go | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/customer/validators.go b/customer/validators.go index 2a37394..482d532 100644 --- a/customer/validators.go +++ b/customer/validators.go @@ -20,13 +20,14 @@ package customer import ( "regexp" "strings" + "net/mail" "vidhukant.com/openbills/errors" e "errors" ) // NOTE: very inefficient and really really really dumb but it works // TODO: find a better (or even a remotely good) way -func validateContactField(field, value string, userId uint) error { +func checkDuplicate(field, value string, userId uint) error { if value != "" { var count int64 err := db.Model(&Customer{}). @@ -90,9 +91,17 @@ func (c *Customer) validate() error { } } + // don't validate email if empty + if c.Email != "" { + _, err := mail.ParseAddress(c.Email) + if err != nil { + return errors.ErrInvalidEmail + } + } + var err error for _, i := range [][]string{{"phone", c.Phone}, {"email", c.Email}, {"website", c.Website}} { - err = validateContactField(i[0], i[1], c.UserID) + err = checkDuplicate(i[0], i[1], c.UserID) if err != nil { return err } |