Update grammar to support documented ODIN keys.

Raise CR

Analysis

Execution

Raise CR

Analysis

Execution

Description

Currently the grammar allows ODIN keys to be any primitve value, which would include non-sorting types like Booleans and also intervals, which don't make sense as keys. They should be limited to Strings, Integers, Dates, Times and Date_times.

Activity

Show:

Thomas Beale October 19, 2017 at 2:36 PM

Changes made to grammar (Github updates not visible in activity log):

diff of odin.g4: https://github.com/openEHR/adl-antlr/commit/c4a6d48709726fbf7d4c6001c3dfdd73949742e0#diff-5449d4036c9efec5131a26c60747cba1
current version: https://github.com/openEHR/adl-antlr/blob/master/src/main/antlr/odin.g4#L32
in published spec: http://www.openehr.org/releases/BASE/latest/docs/odin/odin.html#_syntax_specification (search for key_id)

Done

Details
Reporter
Thomas Beale
Raised By
Claude Nanjo
Original estimate
1d
Components

Created April 26, 2017 at 4:15 PM

Updated November 10, 2018 at 7:01 PM

Resolved August 29, 2018 at 7:14 AM

Update grammar to support documented ODIN keys.

Description

Activity

Thomas Beale October 19, 2017 at 2:36 PM

DetailsReporterThomas BealeThomas BealeRaised ByClaude NanjoOriginal estimate1dComponents

Details

Reporter

Raised By

Original estimate

Components

Details
Reporter
Thomas Beale
Raised By
Claude Nanjo
Original estimate
1d
Components