thermokarst/bactdb
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Users: only use modl hooks for times

Browse source 
Fixes #23 (ensures modl hooks are used everywhere)
This commit is contained in:
Matthew Dillon 2015-11-13 09:06:28 -07:00
parent 5bae0238de
commit 548f65eeb7
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
api/users.go
View file

@ -124,9 +124,6 @@ func (u UserService) Update(id int64, e *types.Entity, dummy string, claims *typ
func (u UserService) Create(e *types.Entity, dummy string, claims *types.Claims) *types.AppError { func (u UserService) Create(e *types.Entity, dummy string, claims *types.Claims) *types.AppError {
user := (*e).(*payloads.User).User user := (*e).(*payloads.User).User
ct := helpers.CurrentTime()
user.CreatedAt = ct
user.UpdatedAt = ct
hash, err := bcrypt.GenerateFromPassword([]byte(user.Password), 12) hash, err := bcrypt.GenerateFromPassword([]byte(user.Password), 12)
if err != nil { if err != nil {
return newJSONError(err, http.StatusInternalServerError) return newJSONError(err, http.StatusInternalServerError)
@ -156,7 +153,7 @@ func (u UserService) Create(e *types.Entity, dummy string, claims *types.Claims)
return newJSONError(err, http.StatusInternalServerError) return newJSONError(err, http.StatusInternalServerError)
} }
// TODO: fix this // TODO: fix this
_, err = models.DBH.Exec(q, user.ID, nonce, claims.Ref, ct) _, err = models.DBH.Exec(q, user.ID, nonce, claims.Ref, helpers.CurrentTime())
if err != nil { if err != nil {
return newJSONError(err, http.StatusInternalServerError) return newJSONError(err, http.StatusInternalServerError)
} }
@ -208,7 +205,6 @@ func HandleUserVerify(w http.ResponseWriter, r *http.Request) *types.AppError {
return newJSONError(err, http.StatusInternalServerError) return newJSONError(err, http.StatusInternalServerError)
} }
user.UpdatedAt = helpers.CurrentTime()
user.Verified = true user.Verified = true
count, err := models.DBH.Update(&user) count, err := models.DBH.Update(&user)