Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#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: trunk
Keywords: Cc:


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 Changed 10 years ago by mleslie

Status: newassigned

Seems to work as expected in LineToCurve?. Fixed on trunk at r4232.

Porting back to 1.3 is still outstanding.

comment:2 Changed 10 years ago by mcayland

Thanks for this Mark. I assume this is okay to patch into 1.4 branch aswell?



comment:3 Changed 10 years ago by pramsey

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 Changed 10 years ago by robe

Resolution: fixed
Status: assignedclosed

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 Changed 10 years ago by mleslie

To port this back to 1.3 I would need to port ticket 212 back as well. I no longer intend to do that.

Note: See TracTickets for help on using tickets.