Fix owners cannot create organization repos bug (go-gitea#20841)

lunny · Gusted · Sysoev, Vladimir · commit 555b8288a397 · 2022-08-28T20:23:05.000+03:00
* Fix owners cannot create organization repos bug

* Fix api

* Update routers/api/v1/org/team.go

Co-authored-by: Gusted &lt;williamzijl7@hotmail.com&gt;

Co-authored-by: Gusted &lt;williamzijl7@hotmail.com&gt;
diff --git a/routers/api/v1/org/team.go b/routers/api/v1/org/team.go
@@ -262,7 +262,7 @@ func EditTeam(ctx *context.APIContext) {
 	}
 
 	if form.CanCreateOrgRepo != nil {
-		team.CanCreateOrgRepo = *form.CanCreateOrgRepo
+		team.CanCreateOrgRepo = team.IsOwnerTeam() || *form.CanCreateOrgRepo
 	}
 
 	if len(form.Name) > 0 {
diff --git a/routers/web/org/teams.go b/routers/web/org/teams.go
@@ -416,7 +416,11 @@ func EditTeamPost(ctx *context.Context) {
 			isIncludeAllChanged = true
 			t.IncludesAllRepositories = includesAllRepositories
 		}
+		t.CanCreateOrgRepo = form.CanCreateOrgRepo
+	} else {
+		t.CanCreateOrgRepo = true
 	}
+
 	t.Description = form.Description
 	if t.AccessMode < perm.AccessModeAdmin {
 		units := make([]organization.TeamUnit, 0, len(unitPerms))
@@ -433,7 +437,6 @@ func EditTeamPost(ctx *context.Context) {
 			return
 		}
 	}
-	t.CanCreateOrgRepo = form.CanCreateOrgRepo
 
 	if ctx.HasError() {
 		ctx.HTML(http.StatusOK, tplTeamNew)

Original file line number	Diff line number	Diff line change
`@@ -262,7 +262,7 @@ func EditTeam(ctx *context.APIContext) {`
`262`	`262`	`}`
`263`	`263`
`264`	`264`	`if form.CanCreateOrgRepo != nil {`
`265`		`- team.CanCreateOrgRepo = *form.CanCreateOrgRepo`
	`265`	`+ team.CanCreateOrgRepo = team.IsOwnerTeam() \|\| *form.CanCreateOrgRepo`
`266`	`266`	`}`
`267`	`267`
`268`	`268`	`if len(form.Name) > 0 {`
Original file line number	Diff line number	Diff line change
`@@ -416,7 +416,11 @@ func EditTeamPost(ctx *context.Context) {`
`416`	`416`	`isIncludeAllChanged = true`
`417`	`417`	`t.IncludesAllRepositories = includesAllRepositories`
`418`	`418`	`}`
	`419`	`+ t.CanCreateOrgRepo = form.CanCreateOrgRepo`
	`420`	`+ } else {`
	`421`	`+ t.CanCreateOrgRepo = true`
`419`	`422`	`}`
	`423`	`+`
`420`	`424`	`t.Description = form.Description`
`421`	`425`	`if t.AccessMode < perm.AccessModeAdmin {`
`422`	`426`	`units := make([]organization.TeamUnit, 0, len(unitPerms))`
`@@ -433,7 +437,6 @@ func EditTeamPost(ctx *context.Context) {`
`433`	`437`	`return`
`434`	`438`	`}`
`435`	`439`	`}`
`436`		`- t.CanCreateOrgRepo = form.CanCreateOrgRepo`
`437`	`440`
`438`	`441`	`if ctx.HasError() {`
`439`	`442`	`ctx.HTML(http.StatusOK, tplTeamNew)`