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

WFS requests from mapstore by default return a filter <ogc:And>undefined</ogc:And> #10695

Closed
1 task done
offtherailz opened this issue Nov 27, 2024 · 0 comments · Fixed by #10696 or #10704
Closed
1 task done

WFS requests from mapstore by default return a filter <ogc:And>undefined</ogc:And> #10695

offtherailz opened this issue Nov 27, 2024 · 0 comments · Fixed by #10696 or #10704
Assignees
@offtherailz @ElenaGallo
Labels
Accepted Internal
Milestone
2024.02.02

Comments

@offtherailz
Copy link
Member

Description

How to reproduce

Expected Result
There should not a filter

Current Result
The filter contains <ogc:Filter><ogc:And>undefined</ogc:And></ogc:Filter> . Undefined string is tolerated by GeoServer, but not valid
Image

  • Not browser related
Browser info (use this site: https://www.whatsmybrowser.org/ for non expert users)
Browser Affected Version
Internet Explorer
Edge
Chrome
Firefox
Safari

Other useful information
@offtherailz offtherailz self-assigned this Nov 27, 2024
offtherailz added a commit to offtherailz/MapStore2 that referenced this issue Nov 27, 2024
@offtherailz
Fix geosolutions-it#10695 Handle empty values for and/or/not/nor filters
38fbc1c
@offtherailz offtherailz mentioned this issue Nov 27, 2024
Merged
12 tasks
@offtherailz offtherailz added this to the 2024.02.02 milestone Nov 27, 2024
@offtherailz offtherailz removed the bug label Nov 27, 2024
@offtherailz offtherailz mentioned this issue Nov 27, 2024
Closed
1 task
@tdipisa tdipisa added Internal BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch labels Nov 27, 2024
tdipisa pushed a commit that referenced this issue Nov 27, 2024
@offtherailz @MV88
Fix #10695 Handle empty values for and/or/not/nor filters (#10696)
e0fbce0 
* Fix #10695 Handle empty values for and/or/not/nor filters

* Update web/client/utils/ogc/Filter/operators.js

Co-authored-by: Matteo V. <matteo.velludini@geosolutionsgroup.com>

* Apply suggestions from code review

---------

Co-authored-by: Matteo V. <matteo.velludini@geosolutionsgroup.com>
@ElenaGallo ElenaGallo self-assigned this Dec 2, 2024
offtherailz added a commit to offtherailz/MapStore2 that referenced this issue Dec 2, 2024
@offtherailz @MV88
Fix geosolutions-it#10695 Handle empty values for and/or/not/nor filt…
623cbe0 
…ers (geosolutions-it#10696)

* Fix geosolutions-it#10695 Handle empty values for and/or/not/nor filters

* Update web/client/utils/ogc/Filter/operators.js

Co-authored-by: Matteo V. <matteo.velludini@geosolutionsgroup.com>

* Apply suggestions from code review

---------

Co-authored-by: Matteo V. <matteo.velludini@geosolutionsgroup.com>
@offtherailz offtherailz mentioned this issue Dec 2, 2024
Merged
tdipisa pushed a commit that referenced this issue Dec 2, 2024
@offtherailz @MV88
Fix #10695 Handle empty values for and/or/not/nor filters (#10696) (#…
624099a 
…10704)

* Fix #10695 Handle empty values for and/or/not/nor filters

* Update web/client/utils/ogc/Filter/operators.js



* Apply suggestions from code review

---------

Co-authored-by: Matteo V. <matteo.velludini@geosolutionsgroup.com>
@tdipisa tdipisa linked a pull request Dec 2, 2024 that will close this issue
[Backport 2024.02.xx] Fix #10695 Handle empty values for and/or/not/nor filters (#10696) #10704
Merged
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@offtherailz offtherailz

@ElenaGallo ElenaGallo

Labels
Accepted Internal
Projects
None yet
Milestone
2024.02.02
3 participants
@offtherailz @tdipisa @ElenaGallo