-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
OGRVRTLayer throws CE_Failure with description ignoring it #11518
Comments
rouault
added a commit
to rouault/gdal
that referenced
this issue
Dec 22, 2024
…tSpatialFilter() Fixes OSGeo#11518
I tested the PR and the results are as expected. Thanks for the incredible quick fix. |
yes |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What is the bug?
I'm using Mapserver (v8.2.2) and a VRT file with a SrcRegion defined. And when performing a WMS getfeatureinfo request the request fails. To my opinion this is due to the type of error thrown in
ogrvrtlayer.cpp
.Case:
I'm doing a WMS request that results in an error thrown by the OGR VRT layer:
CPLError(CE_Failure, CPLE_AppDefined,"Spatial filter should be polygon when a SrcRegion is defined. Ignoring it");
. However thepoSrcRegion
is correctly set and execution continues.This exception is caught by mapserver with
if (CPLGetLastErrorType() == CE_Failure)
that results in failure.Expected result:
OGR VRT layer to throw a CE_Warning like this:
CPLError(CE_Warning, CPLE_AppDefined,"Spatial filter should be polygon when a SrcRegion is defined. Ignoring it");
Steps to reproduce the issue
Dataset:
BAG geopackage: https://service.pdok.nl/lv/bag/atom/bag.xml
VRT file:
No idea how to reproduce this with ogr2ogr since I cannot find out how the
m_poFilterGeom
is set that seems to be something different then a polygon.Versions and provenance
GDAL 3.6, 3.9.3, 3.10.0 self-built from binaries.
Additional context
No response
The text was updated successfully, but these errors were encountered: