diff options
author | Joe Chen <jc@unknwon.io> | 2022-11-25 22:40:20 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-11-25 22:40:20 +0800 |
commit | 644a3a9d78a691af8f0f5ab04d5d44207b5d03dc (patch) | |
tree | 5afd3817f440db0425e691221798889e6e2f3ba8 /internal/db/user.go | |
parent | 93736cbc12b51bfdeef16b0934fbd5fa888f89de (diff) |
refactor(db): migrate `ChangeUsername` off `user.go` (#7261)
Diffstat (limited to 'internal/db/user.go')
-rw-r--r-- | internal/db/user.go | 35 |
1 files changed, 0 insertions, 35 deletions
diff --git a/internal/db/user.go b/internal/db/user.go index 922a340a..5d75bfca 100644 --- a/internal/db/user.go +++ b/internal/db/user.go @@ -12,7 +12,6 @@ import ( "strings" "time" - "github.com/unknwon/com" log "unknwon.dev/clog/v2" "xorm.io/xorm" @@ -57,40 +56,6 @@ func (u *User) getOrganizationCount(e Engine) (int64, error) { return e.Where("uid=?", u.ID).Count(new(OrgUser)) } -// ChangeUserName changes all corresponding setting from old user name to new one. -func ChangeUserName(u *User, newUserName string) (err error) { - if err = isUsernameAllowed(newUserName); err != nil { - return err - } - - if Users.IsUsernameUsed(context.TODO(), newUserName) { - return ErrUserAlreadyExist{args: errutil.Args{"name": newUserName}} - } - - if err = ChangeUsernameInPullRequests(u.Name, newUserName); err != nil { - return fmt.Errorf("ChangeUsernameInPullRequests: %v", err) - } - - // Delete all local copies of repositories and wikis the user owns. - if err = x.Where("owner_id=?", u.ID).Iterate(new(Repository), func(idx int, bean interface{}) error { - repo := bean.(*Repository) - deleteRepoLocalCopy(repo) - // TODO: By the same reasoning, shouldn't we also sync access to the local wiki path? - RemoveAllWithNotice("Delete repository wiki local copy", repo.LocalWikiPath()) - return nil - }); err != nil { - return fmt.Errorf("delete repository and wiki local copy: %v", err) - } - - // Rename or create user base directory - baseDir := repoutil.UserPath(u.Name) - newBaseDir := repoutil.UserPath(newUserName) - if com.IsExist(baseDir) { - return os.Rename(baseDir, newBaseDir) - } - return os.MkdirAll(newBaseDir, os.ModePerm) -} - func updateUser(e Engine, u *User) error { // Organization does not need email if !u.IsOrganization() { |