Consistently return nil users if creating a new user failed
This commit is contained in:
parent
36b9a9b2d5
commit
934e94a5f2
|
@ -47,7 +47,7 @@ func CreateUser(user *User) (newUser *User, err error) {
|
|||
// Hash the password
|
||||
user.Password, err = hashPassword(user.Password)
|
||||
if err != nil {
|
||||
return &User{}, err
|
||||
return nil, err
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -64,7 +64,7 @@ func CreateUser(user *User) (newUser *User, err error) {
|
|||
// Insert it
|
||||
_, err = x.Insert(user)
|
||||
if err != nil {
|
||||
return &User{}, err
|
||||
return nil, err
|
||||
}
|
||||
|
||||
// Update the metrics
|
||||
|
@ -73,7 +73,7 @@ func CreateUser(user *User) (newUser *User, err error) {
|
|||
// Get the full new User
|
||||
newUserOut, err := GetUserByID(user.ID)
|
||||
if err != nil {
|
||||
return &User{}, err
|
||||
return nil, err
|
||||
}
|
||||
|
||||
err = sendConfirmEmail(user)
|
||||
|
|
Loading…
Reference in New Issue