Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed #16594 -- Added wkt 3D support for GEOS geometries

This requires GEOS >= 3.3.0 to function properly. On previous
versions, the Z dimension will simply not appear in the wkt.
Disabled OpenLayers editing for 3D geometries (unsupported).
  • Loading branch information...
commit 5e80571bf92d93c177bea9e5ee7d89153ecadbad 1 parent 8a92139
@claudep claudep authored
View
5 django/contrib/gis/admin/options.py
@@ -51,9 +51,10 @@ def media(self):
def formfield_for_dbfield(self, db_field, **kwargs):
"""
Overloaded from ModelAdmin so that an OpenLayersWidget is used
- for viewing/editing GeometryFields.
+ for viewing/editing 2D GeometryFields (OpenLayers 2 does not support
+ 3D editing).
"""
- if isinstance(db_field, models.GeometryField):
+ if isinstance(db_field, models.GeometryField) and db_field.dim < 3:
request = kwargs.pop('request', None)
# Setting the widget with the newly defined widget.
kwargs['widget'] = self.get_map_widget(db_field)
View
2  django/contrib/gis/geos/geometry.py
@@ -382,7 +382,7 @@ def ewkt(self):
@property
def wkt(self):
"Returns the WKT (Well-Known Text) representation of this Geometry."
- return wkt_w().write(self).decode()
+ return wkt_w(self.hasz and 3 or 2).write(self).decode()
@property
def hex(self):
View
26 django/contrib/gis/geos/prototypes/io.py
@@ -45,6 +45,18 @@ class WKBWriter_st(Structure): pass
wkt_writer_write.restype = geos_char_p
wkt_writer_write.errcheck = check_string
+try:
+ wkt_writer_get_outdim = GEOSFunc('GEOSWKTWriter_getOutputDimension')
+ wkt_writer_get_outdim.argtypes = [WKT_WRITE_PTR]
+ wkt_writer_get_outdim.restype = c_int
+ wkt_writer_set_outdim = GEOSFunc('GEOSWKTWriter_setOutputDimension')
+ wkt_writer_set_outdim.argtypes = [WKT_WRITE_PTR, c_int]
+except AttributeError:
+ # GEOSWKTWriter_get/setOutputDimension has been introduced in GEOS 3.3.0
+ # Always return 2 if not available
+ wkt_writer_get_outdim = lambda ptr: 2
+ wkt_writer_set_outdim = lambda ptr, dim: None
+
### WKBReader routines ###
wkb_reader_create = GEOSFunc('GEOSWKBReader_create')
wkb_reader_create.restype = WKB_READ_PTR
@@ -151,6 +163,17 @@ def write(self, geom):
"Returns the WKT representation of the given geometry."
return wkt_writer_write(self.ptr, geom.ptr)
+ @property
+ def outdim(self):
+ return wkt_writer_get_outdim(self.ptr)
+
+ @outdim.setter
+ def outdim(self, new_dim):
+ if not new_dim in (2, 3):
+ raise ValueError('WKT output dimension must be 2 or 3')
+ wkt_writer_set_outdim(self.ptr, new_dim)
+
+
class WKBWriter(IOBase):
_constructor = wkb_writer_create
_destructor = wkb_writer_destroy
@@ -217,9 +240,10 @@ def wkt_r():
thread_context.wkt_r = _WKTReader()
return thread_context.wkt_r
-def wkt_w():
+def wkt_w(dim=2):
if not thread_context.wkt_w:
thread_context.wkt_w = WKTWriter()
+ thread_context.wkt_w.outdim = dim
return thread_context.wkt_w
def wkb_r():
View
3  django/contrib/gis/geos/tests/test_geos.py
@@ -80,7 +80,8 @@ def test_wkt(self):
"Testing WKT output."
for g in self.geometries.wkt_out:
geom = fromstr(g.wkt)
- self.assertEqual(g.ewkt, geom.wkt)
+ if geom.hasz and geos_version_info()['version'] >= '3.3.0':
+ self.assertEqual(g.ewkt, geom.wkt)
def test_hex(self):
"Testing HEX output."
View
4 django/contrib/gis/tests/data/geometries.json
@@ -20,6 +20,7 @@
],
"wkt_out": [
{"wkt": "POINT (110 130)", "ewkt": "POINT (110.0000000000000000 130.0000000000000000)", "kml": "<Point><coordinates>110.0,130.0,0</coordinates></Point>", "gml": "<gml:Point><gml:coordinates>110,130</gml:coordinates></gml:Point>"},
+ {"wkt": "POINT (110 130 85)", "ewkt": "POINT Z (110.0000000000000000 130.0000000000000000 85.0000000000000000)", "kml": "<Point><coordinates>110.0,130.0,85.0</coordinates></Point>", "gml": "<gml:Point><gml:coordinates>110,130,85</gml:coordinates></gml:Point>"},
{"wkt": "LINESTRING (40 40,50 130,130 130)", "ewkt": "LINESTRING (40.0000000000000000 40.0000000000000000, 50.0000000000000000 130.0000000000000000, 130.0000000000000000 130.0000000000000000)", "kml": "<LineString><coordinates>40.0,40.0,0 50.0,130.0,0 130.0,130.0,0</coordinates></LineString>", "gml": "<gml:LineString><gml:coordinates>40,40 50,130 130,130</gml:coordinates></gml:LineString>"},
{"wkt": "POLYGON ((150 150,410 150,280 20,20 20,150 150),(170 120,330 120,260 50,100 50,170 120))", "ewkt": "POLYGON ((150.0000000000000000 150.0000000000000000, 410.0000000000000000 150.0000000000000000, 280.0000000000000000 20.0000000000000000, 20.0000000000000000 20.0000000000000000, 150.0000000000000000 150.0000000000000000), (170.0000000000000000 120.0000000000000000, 330.0000000000000000 120.0000000000000000, 260.0000000000000000 50.0000000000000000, 100.0000000000000000 50.0000000000000000, 170.0000000000000000 120.0000000000000000))", "kml": "<Polygon><outerBoundaryIs><LinearRing><coordinates>150.0,150.0,0 410.0,150.0,0 280.0,20.0,0 20.0,20.0,0 150.0,150.0,0</coordinates></LinearRing></outerBoundaryIs><innerBoundaryIs><LinearRing><coordinates>170.0,120.0,0 330.0,120.0,0 260.0,50.0,0 100.0,50.0,0 170.0,120.0,0</coordinates></LinearRing></innerBoundaryIs></Polygon>", "gml": "<gml:Polygon><gml:outerBoundaryIs><gml:LinearRing><gml:coordinates>150,150 410,150 280,20 20,20 150,150</gml:coordinates></gml:LinearRing></gml:outerBoundaryIs><gml:innerBoundaryIs><gml:LinearRing><gml:coordinates>170,120 330,120 260,50 100,50 170,120</gml:coordinates></gml:LinearRing></gml:innerBoundaryIs></gml:Polygon>"},
{"wkt": "MULTIPOINT (10 80,110 170,110 120)", "ewkt": "MULTIPOINT (10.0000000000000000 80.0000000000000000, 110.0000000000000000 170.0000000000000000, 110.0000000000000000 120.0000000000000000)", "kml": "<MultiGeometry><Point><coordinates>10.0,80.0,0</coordinates></Point><Point><coordinates>110.0,170.0,0</coordinates></Point><Point><coordinates>110.0,120.0,0</coordinates></Point></MultiGeometry>", "gml": "<gml:MultiPoint><gml:pointMember><gml:Point><gml:coordinates>10,80</gml:coordinates></gml:Point></gml:pointMember><gml:pointMember><gml:Point><gml:coordinates>110,170</gml:coordinates></gml:Point></gml:pointMember><gml:pointMember><gml:Point><gml:coordinates>110,120</gml:coordinates></gml:Point></gml:pointMember></gml:MultiPoint>"},
@@ -29,6 +30,7 @@
],
"hex_wkt": [
{"wkt": "POINT(0 1)", "hex": "01010000000000000000000000000000000000F03F"},
+ {"wkt": "POINT(0 1 5)", "hex": "01010000800000000000000000000000000000F03F0000000000001440"},
{"wkt": "LINESTRING(0 1, 2 3, 4 5)", "hex": "0102000000030000000000000000000000000000000000F03F0000000000000040000000000000084000000000000010400000000000001440"},
{"wkt": "POLYGON((0 0, 10 0, 10 10, 0 10, 0 0))", "hex": "010300000001000000050000000000000000000000000000000000000000000000000024400000000000000000000000000000244000000000000024400000000000000000000000000000244000000000000000000000000000000000"},
{"wkt": "MULTIPOINT(0 0, 10 0, 10 10, 0 10, 0 0)", "hex": "010400000005000000010100000000000000000000000000000000000000010100000000000000000024400000000000000000010100000000000000000024400000000000002440010100000000000000000000000000000000002440010100000000000000000000000000000000000000"},
@@ -118,4 +120,4 @@
{"wkt": "POLYGON ((-5 0,-5 10,5 10,5 5,10 5,10 -5,0 -5,0 0,-5 0))"},
{"wkt": "POLYGON ((2 0, 2 15, 18 15, 18 0, 2 0))"}
]
-}
+}
Please sign in to comment.
Something went wrong with that request. Please try again.