Only update access usernames where the username is changing #282

models/user.go

@@ -297,10 +297,12 @@ func ChangeUserName(user *User, newUserName string) (err error) {
 		}
 
 		for j := range accesses {
-			accesses[j].UserName = newUserName
+			// if the access is not the user's access (already updated above)
-			accesses[j].RepoName = newUserName + "/" + repos[i].LowerName
+			if accesses[j].UserName != user.LowerName {
-			if err = UpdateAccessWithSession(sess, &accesses[j]); err != nil {
+				accesses[j].RepoName = newUserName + "/" + repos[i].LowerName
-				return err
+				if err = UpdateAccessWithSession(sess, &accesses[j]); err != nil {
+					return err
+				}
 			}
 		}
 	}