From 97576a41c48b8415890c33b7d6eddcc838c033e5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E2=98=99=E2=97=A6=20The=20Tablet=20=E2=9D=80=20GamerGirla?= =?UTF-8?q?ndCo=20=E2=97=A6=E2=9D=A7?= Date: Sat, 16 Aug 2025 15:55:44 -0400 Subject: [PATCH] export `GetUnitPerms` from `routers/web/org` package for reuse in future group route handlers --- routers/web/org/teams.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/routers/web/org/teams.go b/routers/web/org/teams.go index 3e64f800b0..74b3f79554 100644 --- a/routers/web/org/teams.go +++ b/routers/web/org/teams.go @@ -295,7 +295,7 @@ func NewTeam(ctx *context.Context) { // - Org owner need to click the permission for each unit, but can't just set a common "write" permission for all units. // // Ideally, "team.authorize=write" should mean the team has write access to all units including newly (future) added ones. -func getUnitPerms(forms url.Values, teamPermission perm.AccessMode) map[unit_model.Type]perm.AccessMode { +func GetUnitPerms(forms url.Values, teamPermission perm.AccessMode) map[unit_model.Type]perm.AccessMode { unitPerms := make(map[unit_model.Type]perm.AccessMode) for _, ut := range unit_model.AllRepoUnitTypes { // Default access mode is none @@ -326,7 +326,7 @@ func NewTeamPost(ctx *context.Context) { form := web.GetForm(ctx).(*forms.CreateTeamForm) includesAllRepositories := form.RepoAccess == "all" teamPermission := perm.ParseAccessMode(form.Permission, perm.AccessModeNone, perm.AccessModeAdmin) - unitPerms := getUnitPerms(ctx.Req.Form, teamPermission) + unitPerms := GetUnitPerms(ctx.Req.Form, teamPermission) t := &org_model.Team{ OrgID: ctx.Org.Organization.ID, @@ -492,7 +492,7 @@ func EditTeamPost(ctx *context.Context) { form := web.GetForm(ctx).(*forms.CreateTeamForm) t := ctx.Org.Team teamPermission := perm.ParseAccessMode(form.Permission, perm.AccessModeNone, perm.AccessModeAdmin) - unitPerms := getUnitPerms(ctx.Req.Form, teamPermission) + unitPerms := GetUnitPerms(ctx.Req.Form, teamPermission) isAuthChanged := false isIncludeAllChanged := false includesAllRepositories := form.RepoAccess == "all"