Skip to content

Commit e1db950

Browse files
committed
Impruve ChangeUserName
1 parent f6926ae commit e1db950

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

models/user.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -913,19 +913,19 @@ func ChangeUserName(u *User, newUserName string) (err error) {
913913
return err
914914
}
915915

916-
isExist, err := IsUserExist(0, newUserName)
917-
if err != nil {
918-
return err
919-
} else if isExist {
920-
return ErrUserAlreadyExist{newUserName}
921-
}
922-
923916
sess := x.NewSession()
924917
defer sess.Close()
925918
if err = sess.Begin(); err != nil {
926919
return err
927920
}
928921

922+
isExist, err := isUserExist(sess, 0, newUserName)
923+
if err != nil {
924+
return err
925+
} else if isExist {
926+
return ErrUserAlreadyExist{newUserName}
927+
}
928+
929929
if _, err = sess.Exec("UPDATE `repository` SET owner_name=? WHERE owner_name=?", newUserName, u.Name); err != nil {
930930
return fmt.Errorf("Change repo owner name: %v", err)
931931
}

0 commit comments

Comments
 (0)