-
Notifications
You must be signed in to change notification settings - Fork 492
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
Matched resources should not also be _included #2448
Conversation
03056c7
to
d4bd7bf
Compare
3ba627c
to
8446bcd
Compare
8446bcd
to
37562b8
Compare
37562b8
to
e2ff59e
Compare
[InlineData("_include")] | ||
[InlineData("_revinclude")] | ||
[Theory] | ||
public async Task GivenAnIncludeSearchExpressionWithLocationLinkedToItself_WhenSearched_ThenCorrectBundleShouldBeReturned(string includeType) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would a test showing that this also works for cyclical references have value? IE: resource 1 references resource 2 and resource 2 references resource 1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, I think this can only be done with SQL (:iterate)
Description
Adds a filter so that if a resource is in the "matched" set of results, it should not also be there as an "include".
Limitation: this is only for the current page, it doesn't guarantee uniqueness across pages.
Related issues
Addresses #2037.
Testing
Adds E2E tests
FHIR Team Checklist
Semver Change (docs)
Patch (bug)