FHIR Chat · Bad Search parameter reference · committers

Stream: committers

Topic: Bad Search parameter reference


view this post on Zulip Grahame Grieve (Apr 05 2019 at 04:05):

hmm, I just found a problem....in US-Core, there's a reference to the search parameter http://hl7.org/fhir/SearchParameter/Patient-family but this doesn't actually exist

view this post on Zulip Grahame Grieve (Apr 05 2019 at 04:06):

is it reasonable, in the validator, to create an error if there's a canonical reference to http://hl7.org/fhir/xxx that isn't resolved?

view this post on Zulip Lloyd McKenzie (Apr 05 2019 at 04:11):

At minimum a warning, I think.

view this post on Zulip Grahame Grieve (Apr 05 2019 at 06:15):

error for sure after playing running a couple of IGs.... wow

view this post on Zulip Eric Haas (Apr 05 2019 at 22:02):

@GG Why are the SPs not published in the fhir core specification. As you discovered I screwed up the SP urls in the US Core Cap statements partly because they are buried deep in package definitions and not easily discover-able like other urls. Would be nice to have all the urls using the clipboard copy like I use for profile.

view this post on Zulip Eric Haas (Apr 05 2019 at 22:03):

Of course I could have missed them but I don't think so...

view this post on Zulip Eric Haas (Apr 05 2019 at 22:15):

for example http://hl7.org/fhir/SearchParameter/Patient-family should be http://hl7.org/fhir/SearchParameter/individual-family, not super obvious or following any pattern.

Right now I have to find that by looping through the definitions and looking for code == family and base == Patient.

I think a a link to the SP definition right from here would be handy

view this post on Zulip Eric Haas (Apr 05 2019 at 22:19):

back to your point it would be super helpful if it resulted in a validation error.

view this post on Zulip Eric Haas (Apr 05 2019 at 23:05):

GF#20697 for fix to US Core

view this post on Zulip Grahame Grieve (Apr 06 2019 at 21:36):

it will be a validation error in the future. The complexity is due to sharing parameters across resources


Last updated: Apr 12 2022 at 19:14 UTC