vikunja
/
vikunja
6
43
Fork 71
Code Issues 150 Pull Requests 15 Packages Releases 32 Activity

Fix checking for changing username or user email
continuous-integration/drone/pr Build is passing Details

This commit is contained in:
kolaente 2020-08-13 17:02:11 +02:00
parent 31c61d7feb
commit 93c9b9c291
Signed by: konrad
GPG Key ID: F40E70337AB24C9B
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/user/user.go
View File

@ -330,7 +330,7 @@ func UpdateUser(user *User) (updatedUser *User, err error) {
if err != nil && !IsErrUserDoesNotExist(err) {
return nil, err
}
if uu.ID != user.ID {
if uu.ID != 0 && uu.ID != user.ID {
return nil, &ErrUsernameExists{Username: user.Username, UserID: uu.ID}
}
}
@ -343,7 +343,7 @@ func UpdateUser(user *User) (updatedUser *User, err error) {
if err != nil && !IsErrUserDoesNotExist(err) {
return nil, err
}
if uu.ID != user.ID {
if uu.ID != 0 && uu.ID != user.ID {
return nil, &ErrUserEmailExists{Email: user.Email, UserID: uu.ID}
}
}