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

Introduce polygon query #5125

Merged
merged 3 commits into from Sep 26, 2019
Merged

Conversation

adube
Copy link
Contributor

@adube adube commented Sep 18, 2019

GSGMF 1047 - phase 4

This patch is the last step towards implementing a new way to query the map by drawing polygons, as it is included in the patch.

The query component now includes a drawfeature interaction that uses polygon type. A button is also added in the query panel tool.

In the Querent service, the polygon is given as an option (like a coordinate or box could be given) and transformed into a Spatial Filter that is added to the WFS request sent.

In the filter/RuleHelper, a method is introduced to "join filters", i.e. given two filters (both of which can be null), wrap them in an AND filter, if necessary.

@sbrunner sbrunner added this to the 2.5 milestone Sep 18, 2019
@adube adube force-pushed the gsgmf-1047-4-query-polygon branch 5 times, most recently from 8cd24a3 to 73db4c6 Compare September 20, 2019 13:51
@adube adube changed the title [WIP] Introduce polygon query Introduce polygon query Sep 25, 2019
@adube adube requested a review from fredj September 25, 2019 12:16
@adube
Copy link
Contributor Author

adube commented Sep 25, 2019

@fredj Ready for review.

@adube
Copy link
Contributor Author

adube commented Sep 25, 2019

@fredj Travis is happy!

@fredj
Copy link
Member

fredj commented Sep 26, 2019

@fredj Travis is happy!

so do I ! please merge

@adube adube merged commit e05262c into camptocamp:master Sep 26, 2019
@adube adube deleted the gsgmf-1047-4-query-polygon branch September 26, 2019 12:44
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.

None yet

3 participants