DMN 1.4 RTF Avatar
  1. OMG Issue

DMN14 — Wrong and Incomplete FEEL grammar rule 52

  • Key: DMN14-87
  • Status: open  
  • Source: Red Hat ( Matteo Mortari)
  • Summary:

    Grammar rule 52 is using single quote notation, and I believe this is just a typo when submitting the original change.

    However DMNv1.3 also missed to exclude grammar 52 from defining Literal terminal symbol.
    This is a concerning problem.
    This implies rule 52 as defined in DMNv1.3 is defining additional Literal terminal symbol(s) and therefore with DMNv1.3 it is no longer formally possible to name a DRGElement for example: "list of customer" or "context of business".
    I believe this is an oversight, and the proposal below addresses this problem as well.

    Proposal

    A total of 3 changes.
    With reference to DMNv1.3 dtc-19-12-06

    Chapter 10.3.1.2 Grammar rules Page 121
    replace:

    52. type =
    qualified name |
    'list' '<' type '>' |
    'context' '<' name ':' type { ',' name ':' type } '>' | 'function' '<' [ type { ', ' type } ] '>' '->' type
    ;
    

    with:

    52. type =
    qualified name |
    "list" "<" type ">" |
    "context" "<" name ":" type { "," name ":" type } ">" | "function" "<" [ type { ", " type } ] ">" "->" type
    ;
    

    Chapter 10.3.1.4 Tokens, Names, and White space Page 122
    replace:

    A literal terminal symbol in any grammar rule other than grammar rule 30. Literal terminal symbols are enclosed in double quotes in the grammar rules, e.g., “and”, “+”, “=”, or

    with:

    A literal terminal symbol in any grammar rule other than grammar rules 30, 52. Literal terminal symbols are enclosed in double quotes in the grammar rules, e.g., “and”, "function", “+”, “=”, or

    Chapter 10.3.1.4 Tokens, Names, and White space Page 122
    after bulletpoints, before "White space (except inside string) ...", insert the following phrase:

    Notice "function" is a literal terminal symbol as defined by FEEL function definition grammar rule 55.

  • Reported: DMN 1.3 — Wed, 8 Apr 2020 07:20 GMT
  • Updated: Thu, 9 Apr 2020 18:38 GMT