FHIR Chat · build error (immunization) · committers

Stream: committers

Topic: build error (immunization)


view this post on Zulip Craig Newman (Feb 12 2018 at 14:22):

I'm trying to commit some changes to the immunization resources. While my local copy builds successfully, I'm getting a thumbs down status when I sync it up. I've looked through the build log (https://travis-ci.org/hl7-fhir/fhir-svn/builds/340498405) and I think the issue may be:
[java] ERROR @ Immunization: Resource Immunization in compartment practitioner: parameter practitioner was not found (practitioner) (src = ResourceValidator)

But I don't know what this means. Anyone have an idea of what is happening here? Thanks.

view this post on Zulip Lloyd McKenzie (Feb 12 2018 at 15:04):

It means that Immunization doesn't have a search parameter named "practitioner" but it's listed as part of the practitioner compartment. Did you remove/rename a search criteria?

view this post on Zulip Craig Newman (Feb 12 2018 at 15:06):

I renamed the practitioner backbone element to be performer to match the Event pattern and to accommodate the fact it will now be able to reference a Practitioner or Organization.

What is a "practitioner compartment"?

view this post on Zulip Grahame Grieve (Feb 12 2018 at 19:26):

see http://build.fhir.org/compartmentdefinition.html

view this post on Zulip Craig Newman (Feb 12 2018 at 20:14):

I looked at that, but it wasn't obvious to me how that definition related to Immunization being in the practitioner compartment. Is this an error that stems from something in the Immunization resource or the CompartmentDefinition resource

view this post on Zulip Grahame Grieve (Feb 12 2018 at 20:37):

the issue is in compartments.xml, which is an excel spreadsheet that registers resources into the compartments - it identifies the search parameters that are used to determine whether a resource is in the compartment. You renamed the search parameter name, but didn't update the name in compartments.xml. I just fixed it

view this post on Zulip Craig Newman (Feb 12 2018 at 20:55):

Thank you. That makes sense.


Last updated: Apr 12 2022 at 19:14 UTC