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

Clarify language around invoking multiple groups

    XMLWordPrintableJSON

Details

    • Icon: Change Request Change Request
    • Resolution: Persuasive with Modification
    • Icon: Medium Medium
    • FHIR Core (FHIR)
    • STU3
    • FHIR Infrastructure
    • Mapping Language
    • Hide

      Change the sentence "Optional part starting with 'then'. Specifies a group or set of child rules to apply within the scope of the rule." to

      "Optional part starting with 'then'. Specifies one or more groups by name (separated by comma) or a set of child rules to apply within the scope of the rule.  Groups are executed in order."

      Show
      Change the sentence "Optional part starting with 'then'. Specifies a group or set of child rules to apply within the scope of the rule." to "Optional part starting with 'then'. Specifies one or more groups by name (separated by comma) or a set of child rules to apply within the scope of the rule.  Groups are executed in order."
    • Grahame Grieve/Vadim Peretokin: 12-0-0
    • Clarification
    • Non-substantive
    • R5

    Description

      http://build.fhir.org/mapping-language.html#rules says:

      > Dependent Rules: Optional part starting with 'then'. Specifies a group or set of child rules to apply within the scope of the rule.

      By saying "a group", it's saying you can have just 1 group after then, whereas you can have multiple.

      http://build.fhir.org/mapping-language.html#7.7.0.7.3 says:

      > [...] Alternatively, a rule can nominate another group of rules from the same or an imported mapping. Each rule or group is listed by name, and then a set of parameters are provided.

      This text does allow the possibility of multiple rules... but it's not super clear about it.

      It'd be good to see the text clarified that you can indeed run multiple groups consequitively!

      Attachments

        Activity

          People

            alexanderzautke Alexander Zautke
            vadim_peretokin Vadim Peretokin
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: