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

Fix OGC API shapefile download #36

Merged
merged 1 commit into from
Jun 24, 2024
Merged

Fix OGC API shapefile download #36

merged 1 commit into from
Jun 24, 2024

Conversation

groldan
Copy link
Member

@groldan groldan commented Jun 19, 2024

Shapefile download links resulted in a 500 http error code. Reason being the GeoTools Shapefile DataStore will always set the default geometry attribute as the first one in the FeatureTypes it creates, hence simple writing of original feature to target feature failed.

Shapefile download links resulted in a 500 http error code.
Reason being the GeoTools Shapefile DataStore will always set the
default geometry attribute as the first one in the FeatureTypes it
creates, hence simple writing of original feature to target feature
failed.
@pmauduit
Copy link
Member

GeoTools Shapefile DataStore will always set the default geometry attribute as the first one in the FeatureTypes it creates

From my understanding, the ideal behaviour would have been to respect the order & definition of the original feature store, but if this a limitation of geotools and/or shapefile format, I think we can live with it.

@pmauduit pmauduit merged commit 50ba799 into main Jun 24, 2024
1 check passed
@pmauduit pmauduit deleted the GSMEL-530 branch June 24, 2024 08:00
@groldan
Copy link
Member Author

groldan commented Jun 24, 2024

it is a limitation of shapefile because geometry and attributes are in different files, and there's no sidecar file to specify the order of the attributes, nor the alias, so attribute names are truncated to 8 chars

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.

2 participants