Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support ECL 1.6 #6

Merged
merged 13 commits into from
Oct 8, 2021
Merged

Support ECL 1.6 #6

merged 13 commits into from
Oct 8, 2021

Conversation

apeteri
Copy link
Member

@apeteri apeteri commented Oct 8, 2021

The pull request updates the grammar definition and related files to make it compatible with version 1.6 of the specification. See this SNOMED International Confluence page🌍 for differences.

apeteri added 12 commits October 7, 2021 17:52
- Set domain on filter constraint
- Remove QL-style domain qualifier from filters
- Add definition status filter
- Allow sub-expressions in filters (where previously only single concept
references or whitespace-separated ID lists were accepted)
- Accept 0 and 1 in component status filter
If regular expression parsing fails, the original String passed along in
ValueConverterWithValueException still gets set on the model.
- Validate acceptability tokens ("prefer" and "accept")
- Validate definition status tokens ("primitive" and "defined")
- Allow empty string in effective time filters
- Check consistency of filter constraint and contained filter domains
...of acceptability tokens to concept identifiers
- Component filters should consult the container FilterConstraint to
determine the domain
- The default domain should be "description" for ECL 1.6
- Change QL-style filter domains to single-letter filter constraint
domains
- Add test cases for numeric component status values
- Add test cases for definition status filter
- Add test case for empty effective time literal
Copy link
Member

@cmark cmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@cmark cmark added the enhancement New feature or request label Oct 8, 2021
@apeteri apeteri merged commit 6c86d17 into main Oct 8, 2021
@apeteri apeteri deleted the upgrade/ecl-1.6 branch October 8, 2021 14:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants