Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed #13670 -- Comparisons with the spatial adapter won't blow up in…

… some corner cases. Thanks, milosu for the bug report and jpaulett for the patch.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@16757 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit 0516ac3d28a7ac8b756ada0e6f89473c7a6e0474 1 parent e061b03
Justin Bronn jbronn authored
2  django/contrib/gis/db/backends/adapter.py
View
@@ -8,6 +8,8 @@ def __init__(self, geom):
self.srid = geom.srid
def __eq__(self, other):
+ if not isinstance(other, WKTAdapter):
+ return False
return self.wkt == other.wkt and self.srid == other.srid
def __str__(self):
2  django/contrib/gis/db/backends/postgis/adapter.py
View
@@ -21,6 +21,8 @@ def __conform__(self, proto):
raise Exception('Error implementing psycopg2 protocol. Is psycopg2 installed?')
def __eq__(self, other):
+ if not isinstance(other, PostGISAdapter):
+ return False
return (self.ewkb == other.ewkb) and (self.srid == other.srid)
def __str__(self):
12 django/contrib/gis/tests/geoapp/test_regress.py
View
@@ -2,7 +2,7 @@
from django.contrib.gis.tests.utils import no_mysql, no_spatialite
from django.contrib.gis.shortcuts import render_to_kmz
from django.test import TestCase
-from models import City, PennsylvaniaCity
+from models import City, PennsylvaniaCity, State
class GeoRegressionTests(TestCase):
@@ -43,3 +43,13 @@ def test04_unicode_date(self):
mansfield = PennsylvaniaCity.objects.create(name='Mansfield', county='Tioga', point='POINT(-77.071445 41.823881)',
founded=founded)
self.assertEqual(founded, PennsylvaniaCity.objects.dates('founded', 'day')[0])
+
+ def test05_empty_count(self):
+ "Testing that PostGISAdapter.__eq__ does check empty strings, see #13670"
+ # contrived example, but need a geo lookup paired with an id__in lookup
+ pueblo = City.objects.get(name='Pueblo')
+ state = State.objects.filter(poly__contains=pueblo.point)
+ cities_within_state = City.objects.filter(id__in=state)
+
+ # .count() should not throw TypeError in __eq__
+ self.assertEqual(cities_within_state.count(), 1)
Please sign in to comment.
Something went wrong with that request. Please try again.