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

Added srsName param to the spatial filter configuration #2449

Merged

Conversation

MV88
Copy link
Contributor

@MV88 MV88 commented Dec 4, 2017

Description

This pr introduce a fix on the crs used to query the features via wvs in the spatialFilter custom.

Issues

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix

What is the current behavior? (You can also link to an open issue here)
without this srsParam it was using the default one for which was not working for certain layers which had a different crs.

What is the new behavior?
Actually the srsParam name is added to the wfs requests and used by openlayers in order to reproject them correctly.

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • No

@MV88 MV88 self-assigned this Dec 4, 2017
@MV88 MV88 requested a review from offtherailz December 4, 2017 14:20
@ghost ghost added In Test labels Dec 4, 2017
@MV88 MV88 removed the In Test label Dec 4, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.083% when pulling 89f8586 on MV88:fix_2418_missing_crs_definition into 3e5a624 on geosolutions-it:master.

@offtherailz offtherailz merged commit d4d5e9f into geosolutions-it:master Dec 4, 2017
@ghost ghost added Accepted and removed pending review labels Dec 4, 2017
offtherailz added a commit that referenced this pull request Dec 4, 2017
@tdipisa tdipisa removed the Accepted label Jan 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an option to query panel to filter by WFS geometry
4 participants