Do not autocreate directory for new user/orgs (#4828) (#4849)

release/v1.15
SagePtr 2018-09-07 02:40:58 +02:00 committed by techknowlogick
parent 3c6cc56143
commit 303d7f7e9c
2 changed files with 6 additions and 7 deletions

View File

@ -182,10 +182,6 @@ func CreateOrganization(org, owner *User) (err error) {
return fmt.Errorf("insert team-user relation: %v", err) return fmt.Errorf("insert team-user relation: %v", err)
} }
if err = os.MkdirAll(UserPath(org.Name), os.ModePerm); err != nil {
return fmt.Errorf("create directory: %v", err)
}
return sess.Commit() return sess.Commit()
} }

View File

@ -798,8 +798,6 @@ func CreateUser(u *User) (err error) {
if _, err = sess.Insert(u); err != nil { if _, err = sess.Insert(u); err != nil {
return err return err
} else if err = os.MkdirAll(UserPath(u.Name), os.ModePerm); err != nil {
return err
} }
return sess.Commit() return sess.Commit()
@ -898,7 +896,12 @@ func ChangeUserName(u *User, newUserName string) (err error) {
return fmt.Errorf("Delete repository wiki local copy: %v", err) return fmt.Errorf("Delete repository wiki local copy: %v", err)
} }
return os.Rename(UserPath(u.Name), UserPath(newUserName)) // Do not fail if directory does not exist
if err = os.Rename(UserPath(u.Name), UserPath(newUserName)); err != nil && !os.IsNotExist(err) {
return fmt.Errorf("Rename user directory: %v", err)
}
return nil
} }
// checkDupEmail checks whether there are the same email with the user // checkDupEmail checks whether there are the same email with the user