#31162: GIS error logging when using WKT string as input to filter() query.
------------------------+------------------------------------
Reporter: Arno | Owner: nobody
Type: Bug | Status: new
Component: GIS | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------+------------------------------------
Changes (by felixxm):
* type: Uncategorized => Bug
* version: 2.2 => master
* stage: Unreviewed => Accepted
Comment:
> According to https://docs.djangoproject.com/en/2.2/ref/contrib/gis/db-
api/#geometry-lookups passing WKT strings is allowed, but there's no
mention of being able to pass a filename as parameter.
Thanks for this report. Django tries to use a `GDALRaster` if you pass
`bytes` or `str` and if conversion is not successful then uses
`GEOSGeometry`.
`GDALRaster` accepts a string representing a file path. I agree that we
should restore
[https://github.com/django/django/commit/6f44f714c92d2966dca390ebd3054e5fb0bb0c80
#diff-6fa0bf748827a59b6730835b5439ded8L69-L70 the previous check] to avoid
logging:
{{{GDAL_ERROR 4: b'LINESTRING(0 0, 1 1, 5 5): No such file or
directory'}}}
We should also protect `GDALRaster._del__()` to avoid:
{{{GDAL_ERROR 10: b"Pointer 'hObject' is NULL in 'GDALGetDescription'.}}}
Maybe:
{{{
@cached_property
def is_vsi_based(self):
return self._ptr and
self.name.startswith(VSI_FILESYSTEM_BASE_PATH)
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/31162#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/064.43ca3b8f65c9f69ac863692eb8c955ac%40djangoproject.com.
