#213 closed defect (fixed)
ST_CurveToLine doesn't support Compound Curves in Curve Polygons
Reported by: | mleslie | Owned by: | mleslie |
---|---|---|---|
Priority: | medium | Milestone: | PostGIS 1.4.0 |
Component: | postgis | Version: | master |
Keywords: | Cc: |
Description
I'm splitting this report (from Regina) out of ticket 212 into a home of it's own.
=========
SELECT ST_CurveToLine(ST_GeomFromEWKT('CURVEPOLYGON(COMPOUNDCURVE(CIRCULARSTRING(0 0,2 0, 2 1, 2 3, 4 3),(4 3, 4 5, 1 4, 0 0)))'));
gives me
ERROR: Invalid ring type found in CurvePoly?. CONTEXT: SQL function "st_curvetoline" statement 1
This is testing against my PostGIS 1.4b build on windows.
=========
It seems sensible to test ST_LineToCurve as well.
Change History (5)
comment:1 by , 15 years ago
Status: | new → assigned |
---|
comment:2 by , 15 years ago
Thanks for this Mark. I assume this is okay to patch into 1.4 branch aswell?
ATB,
Mark.
comment:3 by , 15 years ago
Patch this back into the 1.4 branch, and with Mark's fix in place on #210 I can roll out RC1 tomorrow morning my time.
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
I'm convinced this one is fixed. I don't think we should patch this back to 1.3 since like I mentioned, I don't think we ever patched 1.3 with compound curve in curve polygon. So I'm closing this out unless Mark or Mark has a compelling reason to patch back to 1.3.
comment:5 by , 15 years ago
To port this back to 1.3 I would need to port ticket 212 back as well. I no longer intend to do that.
Seems to work as expected in LineToCurve. Fixed on trunk at r4232.
Porting back to 1.3 is still outstanding.