Skip to content

Commit 4f44e72

Browse files
committed
rewrite by using existing sub func
1 parent 2494db4 commit 4f44e72

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

models/org.go

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -301,15 +301,13 @@ type OrgUser struct {
301301
}
302302

303303
func isOrganizationOwner(e Engine, orgID, uid int64) (bool, error) {
304-
ownerTeam := &Team{
305-
OrgID: orgID,
306-
Name: ownerTeamName,
307-
}
308-
if has, err := e.Get(ownerTeam); err != nil {
304+
ownerTeam, err := getTeam(e, orgID, ownerTeamName)
305+
if err != nil {
306+
if err == ErrTeamNotExist {
307+
log.Error("Organization does not have owner team: %d", orgID)
308+
return false, nil
309+
}
309310
return false, err
310-
} else if !has {
311-
log.Error("Organization does not have owner team: %d", orgID)
312-
return false, nil
313311
}
314312
return isTeamMember(e, orgID, ownerTeam.ID, uid)
315313
}

0 commit comments

Comments
 (0)