Uploaded image for project: 'FHIR Specification Feedback'
  1. FHIR Specification Feedback
  2. FHIR-43780

Relax prohibition against string-based membership testing

    XMLWordPrintableJSON

Details

    • Icon: Change Request Change Request
    • Resolution: Persuasive
    • Icon: Medium Medium
    • Using CQL With FHIR (FHIR)
    • 1.0.0-ballot
    • Clinical Decision Support
    • STU
    • Using CQL
    • 2.4.4
    • Hide

      Agreed, given the precedents in CQL and FHIR for allowing binding to and comparison of strings for value set membership, relax this to a SHOULD, and recommend that tooling issue warnings when this type of comparison is performed.

      Show
      Agreed, given the precedents in CQL and FHIR for allowing binding to and comparison of strings for value set membership, relax this to a SHOULD, and recommend that tooling issue warnings when this type of comparison is performed.
    • Chris Moesel / Juliet Rubini : 23-0-0
    • Enhancement
    • Compatible, substantive

    Description

      The IG prohibits string-based membership testing, but there may be cases where it is appropriate. For example, Address.state is represented as a string in FHIR and is often bound to value sets in regional IGs. Would we want to prohibit CQL such as Patient.address in NewEnglandStatesVS? If we do prohibit that, there is no alternative except to represent the ValueSet as a literal List<String> instead – which is not ideal.

      The argument that it should be avoided because it does not consider the code system also does not apply to any FHIR element that is a code type (like most status elements), since they don't carry a system either.

      For this reason, I think the prohibition should be relaxed to SHOULD NOT (rather than MUST NOT). I think the key point is that if an element is represented as a CodeableConcept or Coding in FHIR, membership testing must be applied to the full element rather than the primitive code part – although I'm not sure what the best way to say that is.

       

       

      Attachments

        Activity

          People

            bryantaustin13 Bryant Austin
            cmoesel Chris Moesel
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: