summaryrefslogtreecommitdiffstats
path: root/routers/repo
diff options
context:
space:
mode:
author6543 <6543@obermui.de>2021-02-25 08:49:27 +0100
committerGitHub <noreply@github.com>2021-02-25 15:49:27 +0800
commitedef62e69ec2a54a870530a81b20a3cc7ca71dbd (patch)
treef45731baaafd9bd0e93281dc8539c0275109ccc0 /routers/repo
parentcdff144f76a70d61a6795ffe88d691c6e2b11f47 (diff)
downloadgitea-edef62e69ec2a54a870530a81b20a3cc7ca71dbd.tar.gz
gitea-edef62e69ec2a54a870530a81b20a3cc7ca71dbd.zip
Backport: Repo Transfer permission checks (#14792) (#14794)
* Backport: Repo Transfer permission checks (#14792) * update tests
Diffstat (limited to 'routers/repo')
-rw-r--r--routers/repo/setting.go5
1 files changed, 4 insertions, 1 deletions
diff --git a/routers/repo/setting.go b/routers/repo/setting.go
index e4f8adc38f..e722cd8c28 100644
--- a/routers/repo/setting.go
+++ b/routers/repo/setting.go
@@ -475,9 +475,12 @@ func SettingsPost(ctx *context.Context, form auth.RepoSettingForm) {
ctx.Repo.GitRepo.Close()
ctx.Repo.GitRepo = nil
}
- if err = repo_service.TransferOwnership(ctx.User, newOwner, repo, nil); err != nil {
+ if err = repo_service.StartRepositoryTransfer(ctx.User, newOwner, repo, nil); err != nil {
if models.IsErrRepoAlreadyExist(err) {
ctx.RenderWithErr(ctx.Tr("repo.settings.new_owner_has_same_repo"), tplSettingsOptions, nil)
+ } else if models.IsErrCancelled(err) {
+ // this err msg is not translated, since it was introduced in a backport
+ ctx.RenderWithErr("user has no right to create repo for new owner", tplSettingsOptions, nil)
} else {
ctx.ServerError("TransferOwnership", err)
}