-
Notifications
You must be signed in to change notification settings - Fork 9
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
absolute reference vs urn #318
Comments
Indeed Philippe, the file by RZ Tienen is correct. This is also a valid way to refer to a resource in a bundle. If you check the examples on the testserver, you will find many examples like this (e.g. for test 1B, the Vivalia IFAC test with number 2855). So you will need to support this, I am afraid! |
Hello, I believe testcase 1A from Regionaal Ziekenhuis Tienen vzw, with ID 1841 is an example to test if this is a problem. |
@bdc-ehealth I have a similar question after our call on tuesday. "performer": [ { "fullUrl": "https://www.saintluc.be/fhir/Organization/fL6a.vFxYqeg6BaBT4fTD7Q4", |
no this one is not valid, because it does not follow the bundle resolution rules: https://www.hl7.org/fhir/R4/bundle.html#:~:text=Except%20for%20transactions,used%20(urn%3Auuid%3A...). |
Hi @bdc-ehealth ,
This type of resolution is included in the example Bundle included in the FHIR R4 specs.
|
if you take a look at this section (https://www.hl7.org/fhir/R4/bundle.html#:~:text=Except%20for%20transactions,used%20(urn%3Auuid%3A...): "Where a resource is not assigned a persistent identity that can be used in the Bundle, a UUID should be used (urn:uuid:...)", it is clear that if an URL of a type that is different from 'urn:uuid:' is used then the resource should be persisted at that location ('persistent identity'). If you use "Patient/23" in the fullUrl field, you have to take the following steps:
It will be very difficult to come up with URLs where both conditions hold. Therefore they say you are supposed to use (urn:uuid:...) |
@bdc-ehealth
So can you clarify what kind of reference we can use in eHealthBox bundles?
This should also be clearly stated in the Belgian FHIR profiles (here it says that any way allowed by international FHIR is fine). |
the current policy for approval is that everything the validator_cli.jar accepts is accepted + the relative URLs that were accepted at the moment of the creation of the standard. From phase 2 on, only what is accepted by validator_cli.jar will be accepted. We align with what is allowed in the international standard (and for phase 1, we are even more lenient). Since your opinion and mine differ regarding the international standard, there seems to be room for interpretation. To be on the safe side, I would use "urn:uuid:...." , because that is certainly accepted by validator_cli.jar. If you, however, use another solution that is accepted by validator_cli.jar, that is perfectly OK for me. You can find validator_cli.jar here: https://github.com/hapifhir/org.hl7.fhir.core/releases . As you can see, it is a moving target and not always backwards compatible, but that is what we have to live with if we want to use HL7 FHIR (it has already caused us trouble in the past: #278 ) |
We are receiving test protocol from labo tienenen to be integrated in HEALHT one (They are testing live with some practice)
and we have some process issues.
They are using Urn while we have use the absolute reference in our implementation
For example, we receive a link to a practitioner
"generalPractitioner": [ { "reference": "urn:uuid:4c4c2e0d-722e-5961-a91d-c81eebd0afe4" } ]
and the resource of the practitioner is:
{
"fullUrl": "urn:uuid:4c4c2e0d-722e-5961-a91d-c81eebd0afe4",
"resource": {
"resourceType": "Practitioner",
"id": "35512",
"meta": {
"profile": [ "https://www.ehealth.fgov.be/standards/fhir/core/StructureDefinition/be-practitioner" ]
},
Normally the ID of the resource correspond to the URL (At least in all sample on the test server...)
What is the recommended approach for the lab?
Should we support both?
Many thanks
Philippe Quertemont
The text was updated successfully, but these errors were encountered: