Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AccreditationList needs to be filtered on servicecontext and not just owner #12

Open
erlendoksvoll opened this issue Nov 2, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@erlendoksvoll
Copy link
Member

Description

In order to prevent cross-context lookups

Additional Information

            while (query.HasMoreResults)
            {
                foreach (Accreditation accreditation in await query.ExecuteNextAsync<Accreditation>())
                {
                    var listOfServiceContexts = new List<string>();

                    if (!string.IsNullOrEmpty(accreditation.ServiceContext))
                        listOfServiceContexts.Add(accreditation.ServiceContext);

                    listOfServiceContexts.AddRange(accreditation.EvidenceCodes.FirstOrDefault().GetBelongsToServiceContexts());
                   
                    if (listOfServiceContexts.Contains(_requestContextService.ServiceContext.Name))
                        accreditations.Add(accreditation);
                }
            }

Tasks

No response

@erlendoksvoll erlendoksvoll added the enhancement New feature or request label Nov 2, 2022
@erlendoksvoll erlendoksvoll moved this to 📋 Backlog in DAN public backlog Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant