diff options
| author | Joe Chen <jc@unknwon.io> | 2022-11-05 17:55:05 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-05 17:55:05 +0800 |
| commit | fd798b4197dc53df872d20f5d10edb8d73c32386 (patch) | |
| tree | e59c8e5881f2162e211850ede549a1f57df64515 /internal/route/repo/setting.go | |
| parent | a66c90462da24a916ee62afcb5a1f79d06ed8399 (diff) | |
refactor(db): migrate methods off `user.go` (#7228)
Diffstat (limited to 'internal/route/repo/setting.go')
| -rw-r--r-- | internal/route/repo/setting.go | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/internal/route/repo/setting.go b/internal/route/repo/setting.go index a6df14b6..bc7c62bc 100644 --- a/internal/route/repo/setting.go +++ b/internal/route/repo/setting.go @@ -225,16 +225,12 @@ func SettingsPost(c *context.Context, f form.RepoSetting) { } newOwner := c.Query("new_owner_name") - isExist, err := db.IsUserExist(0, newOwner) - if err != nil { - c.Error(err, "check if user exists") - return - } else if !isExist { + if !db.Users.IsUsernameUsed(c.Req.Context(), newOwner) { c.RenderWithErr(c.Tr("form.enterred_invalid_owner_name"), SETTINGS_OPTIONS, nil) return } - if err = db.TransferOwnership(c.User, newOwner, repo); err != nil { + if err := db.TransferOwnership(c.User, newOwner, repo); err != nil { if db.IsErrRepoAlreadyExist(err) { c.RenderWithErr(c.Tr("repo.settings.new_owner_has_same_repo"), SETTINGS_OPTIONS, nil) } else { |