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.

Sign up for GitHub

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

Jump to bottom

Preserve the SRID of allowed area when retrieving it through AccessInfo #161

Closed
taba90 opened this issue Dec 14, 2020 · 2 comments
Closed

Preserve the SRID of allowed area when retrieving it through AccessInfo #161

taba90 opened this issue Dec 14, 2020 · 2 comments

Comments

@taba90
Copy link
Contributor

taba90 commented Dec 14, 2020

When the AccessInfo object is built from a Rule, the SRID of the Geometry is not included in the WKT representation of allowedArea.
@taba90 taba90 changed the title Preserver the SRID of allowed area when retrieving it through AccessInfo Preserve the SRID of allowed area when retrieving it through AccessInfo Dec 14, 2020
taba90 added a commit to taba90/geofence that referenced this issue Dec 14, 2020
@taba90
geoserver#161 Preserve the SRID of allowed area when retrieving it th…
cb75c6e 
…rough AccessInfo
taba90 added a commit to taba90/geofence that referenced this issue Dec 21, 2020
@taba90
geoserver#161 Preserve the SRID of allowed area when retrieving it th…
157fad2 
…rough AccessInfo
etj pushed a commit that referenced this issue Jan 22, 2021
@taba90
#161 handle SRID in allowed area
d15d4ad 
* #161 Preserve the SRID of allowed area when retrieving it through AccessInfo

* review feedback

* removed unused imports

* manage SRID in REST service
@etj
Copy link
Member

etj commented Feb 4, 2021

Merged in 3.4.x: #162

@etj
Copy link
Member

etj commented Feb 10, 2021

Adding a note that has been discussed by voice: simply storing the SRID in the geometries in not enough: GeoFence performs some operations (union, intersect) with the internal geometries, so, if geometries have different SR, they need to be reprojected.

@etj etj closed this as completed Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@etj @taba90