Extension StructureDefinition incorrectly declares kind = primitive-type in no-basis-middlename

[helgaarm]

Vi har oppdaget det som ser ut til å være en feil i StructureDefinitionen for no-basis-middlename:

URL: http://hl7.no/fhir/StructureDefinition/no-basis-middlename

Profilen er definert som en utvidelse (type = Extension) med:

"type": "Extension",
"baseDefinition": "http://hl7.org/fhir/StructureDefinition/Extension",
"derivation": "constraint"

men samtidig er følgende angitt: "kind": "primitive-type"

Ved validering (FHIR R4 / Fhirly Validator) får vi feilmeldingen om kind: "primitive-type" that should have been a complex-type

Ettersom Extension er en kompleks datatype i FHIR R4, forventer validatoren at kind er satt til: "kind": "complex-type"

Tidligere var ikke validatoren så nøye på dette og det gir ikke så jeg veit utslag i den gamle java validatoren.

[helgaarm]

Hvis ok kan jeg fikse det selv.

Endringsbeskrivelse – no-basis-middlename

Det er identifisert en valideringsfeil i profilen no-basis-middlename, hvor StructureDefinition.kind er satt til primitive-type. Siden profilen er en begrensning av FHIRs Extension-type, skal verdien være complex-type i henhold til FHIR-spesifikasjonen.

Følgende endringer foreslås:

1. Korrigering av StructureDefinition.kind

Fra:

"kind": "primitive-type"

Til:

"kind": "complex-type"

Denne endringen korrigerer en ugyldig definisjon og sikrer samsvar med FHIR-spesifikasjonen. Endringen påvirker ikke hvordan instanser representeres eller valideres, og vurderes derfor som en feilretting uten funksjonell påvirkning for eksisterende implementasjoner.

2. Presisering av at profilen er en enkel utvidelse

Følgende element legges til i differentialen:

{
"id": "Extension.extension",
"path": "Extension.extension",
"max": "0"
}

Profilen er definert som en enkel FHIR-utvidelse hvor mellomnavn representeres direkte ved hjelp av Extension.valueString. Bruk av underutvidelser (Extension.extension) er ikke en del av den tiltenkte informasjonsmodellen.

Ved å sette Extension.extension.max = 0 tydeliggjøres det at profilen kun skal benyttes som en enkel utvidelse, og at komplekse/nestede utvidelser ikke er tillatt. Dette gjør profilens hensikt mer eksplisitt og bidrar til mer konsistent validering på tvers av FHIR-verktøy og implementasjoner.

Vurdering av bakoverkompatibilitet

Endringen av kind fra primitive-type til complex-type vurderes som en ren feilretting og er ikke forventet å påvirke eksisterende implementasjoner.

Tillegget av Extension.extension.max = 0 innebærer en innstramming av profilen og kan medføre at instanser som benytter underutvidelser ikke lenger validerer. Slike instanser anses imidlertid ikke å være i samsvar med profilens opprinnelige hensikt, da mellomnavn skal representeres ved hjelp av valueString.

Samlet vurderes endringene som nødvendige for å bringe profilen i samsvar med FHIR-spesifikasjonen og for å tydeliggjøre den tiltenkte bruken av profilen.

[helgaarm]

@thomiz @rockphotog hva er beste fremgangs måten her - skal I lage egen branch for denne fiksen?

[rockphotog]

@helgaarm Ja, lag en branch med forslaget ditt - så tar vi QA på merge. Regner også med at dette kan være en bug som kun fører til versjon x.x.n+1.

[helgaarm]

Endret kun det som var nødvendig:

Korrigering av StructureDefinition.kind
Fra: "kind": "primitive-type"
Til: "kind": "complex-type"