Skip to content

Commit b838781

Browse files
authored
Merge pull request #354 from FHIR/content-type-binding-extensible
Make SQLQuery contentType binding extensible
2 parents 5ee784f + cf12503 commit b838781

2 files changed

Lines changed: 9 additions & 3 deletions

File tree

input/fsh/profiles/library-profiles.fsh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,10 @@ versioning.
2626
* content 1..* MS
2727
* content.contentType 1..1 MS
2828
* content.contentType ^short = "application/sql or application/sql;dialect=..."
29-
* content.contentType from AllSQLContentTypeCodes (required)
29+
* content.contentType from http://hl7.org/fhir/ValueSet/mimetypes (required)
30+
* content.contentType ^binding.additional[+].purpose = #extensible
31+
* content.contentType ^binding.additional[=].valueSet = Canonical(AllSQLContentTypeCodes)
32+
* content.contentType ^binding.additional[=].documentation = "SQLQuery content types, including dialect-specific variants. Authors SHOULD use a code from this value set when one applies; codes outside this value set MAY be used for SQL dialects not yet enumerated, subject to the sql-must-be-sql-expressions invariant."
3033
* content.extension contains sql-text named sqlText 0..1 MS
3134
* content.extension[sqlText] ^short = "Plain-text SQL for readability"
3235
* content.data 1..1 MS

input/pagecontent/StructureDefinition-SQLQuery-intro.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,11 @@ same logical result set.
108108

109109
### Conformance
110110

111-
**Terminology:** `contentType` SHALL come from
112-
[All SQL Content Type Codes](ValueSet-AllSQLContentTypeCodes.html).
111+
**Terminology:** `contentType` SHOULD come from
112+
[All SQL Content Type Codes](ValueSet-AllSQLContentTypeCodes.html). The binding
113+
is extensible: when one of these codes covers the dialect, that code SHALL be
114+
used; otherwise, an alternative code MAY be supplied (subject to the constraint
115+
that every `contentType` starts with `application/sql`).
113116

114117
**Constraints:**
115118

0 commit comments

Comments
 (0)