Description
CR fixes the problem in
Activity
Thomas Beale November 2, 2024 at 4:07 PM
I’m not sure that changing ATTESTATION.items
to DV_OPENEHR_URI
makes sense - what’s the logical explanation? It would imply that things other than EHR entries are being attested, rather than just committed. This would seem to change the meaning of ‘attestation’.
Sebastian Iancu September 15, 2024 at 11:39 AM
2024-09-15: renamed ticket and rephrased task description according to
Thomas Beale June 15, 2020 at 3:11 PM
and to work out IANA registration approach.
Thomas Beale May 27, 2020 at 7:18 AM
Right - that’s exactly the potential problem I was thinking of. If the dev groups of Better / DIPS / Code24 / Ocean / EhrBase … could give us an impact assessment and/or preference, we’ll know which way to go.
Matija Polajnar May 27, 2020 at 5:40 AM
I vote for the second approach. There is only one problem I foresee: if there are any compositions in any of the various JSON (canonical or simplified) formats lying around (or hard-coded into applications as data placeholders), the second approach might need additional logic to automatically determine the correct subtype to use if we will change any fields from type DV_EHR_URI to its new parent DV_OPENEHR_URI. Otherwise deserialisation will fail for old documents where the type attribute was not mandatory because DV_EHR_URI had no subtypes.
There are several issues in regarding use of DV_EHR_URI in common package types. For details see analysis document at
The following changes or additions are necessary:
add a new
DV_OPENEHR_URI
typethis should be ‘in-between’ placed inheritance of
DV_EHR_URI
andDV_URI
specify it to be for any scheme space used in openEHR, including ‘ehr’, ‘demographic (or 'registry’), ‘tp’, etc.
add
as_urn()
functionChange
LINK.target
toDV_OPENEHR_URI
Change
ATTESTATION.items
toList<DV_OPENEHR_URI>
The invariant condition of
DV_EHR_URI.value
should still require the string to start with “ehr:
“ as alwaysIn documentation clearly point out that
DV_OPENEHR_URI
andDV_EHR_URI
are only meant for system internal useChange to include also schemas introduced by
DV_OPENEHR_URI
not only theehr:/