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

Feature Info does not work according with cql_filter #1329

Closed
offtherailz opened this issue Dec 7, 2016 · 0 comments
Closed

Feature Info does not work according with cql_filter #1329

offtherailz opened this issue Dec 7, 2016 · 0 comments
Assignees
@offtherailz
Labels
bug Priority: High
Milestone
2017.01.00

Comments

@offtherailz
Copy link
Member

if you have a layer with

{params: {
   cql_filter: "a = 1"
}}

This filter is not passed to the feature info, even if not in the exclude list is empty.
@offtherailz offtherailz added this to the R52016 milestone Dec 7, 2016
@offtherailz offtherailz self-assigned this Dec 7, 2016
offtherailz added a commit to offtherailz/MapStore2 that referenced this issue Dec 7, 2016
@offtherailz
Fix geosolutions-it#1329. Use layer params in GFI
deed26a 
 - Now the getFeatureInfo gets the params from the layer.
 - The params are overridable by the params property of the feature info (the tool was using them only in the previous implementation)
 - Fixed the excludeParameter filter to add parameter properly (before it was sending to the server a json called params)
 - Add a unit test for this
offtherailz added a commit to offtherailz/MapStore2 that referenced this issue Dec 7, 2016
@offtherailz
Fix geosolutions-it#1329. Use layer params in GFI
2d37b1f 
 - Now the getFeatureInfo gets the params from the layer.
 - The params are overridable by the params property of the feature info (the tool was using them only in the previous implementation)
 - Fixed the excludeParameter filter to add parameter properly (before it was sending to the server a json called params)
 - Add a unit test for this
@ghost ghost removed the pending review label Dec 7, 2016
offtherailz added a commit that referenced this issue Dec 7, 2016
@offtherailz
Fix #1329. Use layer params in GFI (#1330)
8e8f79e 
- Now the getFeatureInfo gets the params from the layer.
 - The params are overridable by the params property of the feature info (the tool was using them only in the previous implementation)
 - Fixed the excludeParameter filter to add parameter properly (before it was sending to the server a json called params)
 - Add a unit test for this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@offtherailz offtherailz

Labels
bug Priority: High
Projects
None yet
Milestone
2017.01.00
Development

No branches or pull requests
1 participant
@offtherailz