Skip to content

Commit

Permalink
fix: don't nest query
Browse files Browse the repository at this point in the history
This causes a nested invocation, which throws an EF exception.
  • Loading branch information
koenmetsu authored and CumpsD committed Dec 27, 2019
1 parent 90f55cf commit a28f77b
Showing 1 changed file with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -254,19 +254,22 @@ private bool ShouldPersonBeShownOnVlaamseOverheidSites(PersonDocument personDocu
{
var organisationIds =
personDocument.Capacities
.Select(capacity => capacity.OrganisationId);
.Select(capacity => capacity.OrganisationId)
.ToList();

var organisationsInPersonCapacitiesShownOnVlaamseOverheidSites =
context.ShowOnVlaamseOverheidSitesPerOrganisationList
.AsNoTracking()
.Where(organisation => organisationIds.Contains(organisation.Id))
.Where(organisation => organisation.ShowOnVlaamseOverheidSites)
.Select(organisation => organisation.Id);
.Select(organisation => organisation.Id)
.ToList();

var capacitiesWithOrganisationShownOnVlaamseOverheidSites =
personDocument.Capacities
.Where(capacity => organisationsInPersonCapacitiesShownOnVlaamseOverheidSites.Contains(capacity.OrganisationId))
.Select(capacity => capacity.PersonCapacityId);
.Select(capacity => capacity.PersonCapacityId)
.ToList();

return context.IsActivePerOrganisationCapacityList
.AsNoTracking()
Expand Down

0 comments on commit a28f77b

Please sign in to comment.