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

Fixes the use of storage CRS code from list of supported CRS #90

Merged
merged 1 commit into from
Apr 5, 2023

Conversation

stempler
Copy link
Member

...instead of the CRS name.
If no match can be found for the storage CRS codes in the list of supported CRS, the first of the storage CRS codes is used.

This is done to if possible fulfill the requirement in the specification of OGC API Features Part 2, that the storage CRS shall be one of the identifiers of the supported CRS.

Since we can't determine for the storage CRS which code was initially used to specify it, we use the list of supported query CRS for deegree OAF as a way to determine the desired code, if possible. So with these changes to fulfill the requirement of the OGC spec you need to specify the desired code for the storage CRS as part of the supported query CRS.

Solves #89

...instead of the CRS name.
If no match can be found for the storage CRS codes in the list of
supported CRS, the first of the storage CRS codes is used.

This is done to if possible fulfill the requirement in the
specification of OGC API Features Part 2, that the storage CRS shall be
one of the identifiers of the supported CRS.
@tfr42 tfr42 added enhancement Improvement of existing feature or function bug Something isn't working and removed enhancement Improvement of existing feature or function labels Apr 5, 2023
@tfr42 tfr42 changed the title fix: use storage CRS code from list of supported CRS Fixes the use of storage CRS code from list of supported CRS Apr 5, 2023
@copierrj copierrj merged commit a42d248 into deegree:main Apr 5, 2023
@tfr42 tfr42 added this to the 1.3.0 milestone Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants