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

Composite search parameters #525

Closed
jackliums opened this issue Jan 2, 2018 · 5 comments · Fixed by #2459
Closed

Composite search parameters #525

jackliums opened this issue Jan 2, 2018 · 5 comments · Fixed by #2459
Assignees

Comments

@jackliums
Copy link

Hi there,

Is it possible to add the underlying value type of Composite Search Parameters to ModelInfo class?

For example, for Observation resource, it would be nice to know that code-value-concept is a composite search parameter with underlying value type of CodeableConcept and code-value-date is a composite search parameter with underlying value type of Date?

The type information could be very helpful when we need to parse the underlying value programmatically.

Also, the FHIRPath property for all of the composite search parameters are empty. Are these going to be supported?

Thanks,

@brianpos
Copy link
Collaborator

brianpos commented Jan 2, 2018

That's still on my servers back log.
Would need to have a different modelinfo definition for the composite parameters too.
The datatype should be accessible from the model processing.
Have a look at the code in the fhirpathtester for that.

@marcovisserFurore
Copy link
Member

Those composite Search Parameters are in ModelInfo.SearchParameters and they have the type SearchParamType.Composite. The definition of those parameters are only not defined. For that we should alter the code generator.

@cknaap
Copy link
Member

cknaap commented Feb 22, 2023

That would be handy for our FS unit testing as well.

@marcovisserFurore
Copy link
Member

Internal communication: should be discussed first with @cknaap .

@marcovisserFurore marcovisserFurore self-assigned this May 10, 2023
@ewoutkramer
Copy link
Member

Gino just told me he has a fix for this as well ;-)

mmsmits added a commit that referenced this issue May 23, 2023
…parameters

#525: Generated composite SearchParameterDefinition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants