Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Updates in to the latest spec changes #63

Merged
merged 1 commit into from
May 22, 2024
Merged

Updates in to the latest spec changes #63

merged 1 commit into from
May 22, 2024

Conversation

tschaub
Copy link
Member

@tschaub tschaub commented May 22, 2024

This updates a number of things to deal with changes in the CQL2 spec. Most of these changes came in opengeospatial/ogcapi-features#831

Breaking changes include:

  • Previously, all "user defined" functions were in an object like this {"function": {"name": "custom"}}. Now, these are collapsed into the existing built-in "op" expressions. So {"op": "custom", args: []} is the equivalent (args are required).

  • Previously, case insensitive and accent insensitive expressions were in {"casei": "Foo"} and {"accenti": "bǎr"} type expressions. Now these are collapsed into the existing built-in "op" expressions. So {"op": "casei", "args": ["Foo"]} and {"op": "accenti", "args": ["bǎr"]}.

Another change here is that interval arrays are string literals instead of date/timestamp objects. This was either my misreading the spec previously or another change.

I've added validation for all JSON output using the latest CQL2 JSON Schema.

This should be more stable now since voting on the spec is underway.

Fixes #62.

@tschaub tschaub merged commit 820a1ae into main May 22, 2024
2 checks passed
@tschaub tschaub deleted the spec-updates branch May 22, 2024 21:54
@bjgiraudon
Copy link

bjgiraudon commented May 23, 2024

Thank you so much for that update!

That was blazingly fast, nice job @tschaub!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Type issue when using t_intersects filter
2 participants