Null check deleteRoles

[ZITADOPER-1]
This commit is contained in:
Haim Kortovich
2024-05-20 11:17:31 -05:00
parent 0026f20c04
commit 33071d2b8b

View File

@@ -207,14 +207,17 @@ func (wr *wrappedProjectReconciler) reconcileRoles(ctx context.Context, ztdClien
} }
} }
for _, dRole := range deleteRoles { if deleteRoles != nil {
if _, err = ztdClient.RemoveProjectRole(middleware.SetOrgID(ctx, org.Status.OrgId), &pb.RemoveProjectRoleRequest{ for _, dRole := range deleteRoles {
ProjectId: wr.project.Status.ProjectId, if _, err = ztdClient.RemoveProjectRole(middleware.SetOrgID(ctx, org.Status.OrgId), &pb.RemoveProjectRoleRequest{
RoleKey: dRole.Key, ProjectId: wr.project.Status.ProjectId,
}); err != nil { RoleKey: dRole.Key,
return fmt.Errorf("Error removing project role: %v", err) }); err != nil {
return fmt.Errorf("Error removing project role: %v", err)
}
} }
} }
if len(roles) > 0 { if len(roles) > 0 {
_, err = ztdClient.BulkAddProjectRoles(middleware.SetOrgID(ctx, org.Status.OrgId), &pb.BulkAddProjectRolesRequest{ _, err = ztdClient.BulkAddProjectRoles(middleware.SetOrgID(ctx, org.Status.OrgId), &pb.BulkAddProjectRolesRequest{
ProjectId: wr.project.Status.ProjectId, ProjectId: wr.project.Status.ProjectId,