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

_elements option should not be applied to the _include option on a search. #1516

Closed
d0roppe opened this issue Sep 22, 2020 · 3 comments · Fixed by #1910
Closed

_elements option should not be applied to the _include option on a search. #1516

d0roppe opened this issue Sep 22, 2020 · 3 comments · Fixed by #1910
Assignees
Labels
bug Something isn't working search

Comments

@d0roppe
Copy link
Collaborator

d0roppe commented Sep 22, 2020

Describe the bug
A clear and concise description of what the bug is.
When issuing a search and specifying the _elements option to include specific elements only, it should not be applied to the resources returned due to the _include option. https://www.hl7.org/fhir/search.html#elements
Currently it is applied to all returned resources.

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error
    Run a query with _elements=identifier&_include=:*

Expected behavior
A clear and concise description of what you expected to happen.

Additional context
Add any other context about the problem here.

@d0roppe d0roppe added the bug Something isn't working label Sep 22, 2020
@prb112 prb112 added the search label Oct 5, 2020
@prb112
Copy link
Contributor

prb112 commented Oct 5, 2020

#1494 should be tackled at the same time.

@JohnTimm JohnTimm added this to the Sprint 20 milestone Oct 13, 2020
@lmsurpre lmsurpre modified the milestones: Sprint 20, Sprint 21 Nov 3, 2020
@JohnTimm JohnTimm added cms-interop This issue is associated with the CMS interoperability rule and removed cms-interop This issue is associated with the CMS interoperability rule labels Nov 3, 2020
@d0roppe
Copy link
Collaborator Author

d0roppe commented Nov 17, 2020

stretch goal for sprint 21

@d0roppe
Copy link
Collaborator Author

d0roppe commented Feb 12, 2021

Verified that with this fix, the _elements is not applied to the included resources.

@d0roppe d0roppe closed this as completed Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working search
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants