Ticket #213 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

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:

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

Changed 4 years ago by mleslie

  • status changed from new to assigned

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

Porting back to 1.3 is still outstanding.

Changed 4 years ago by mcayland

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

ATB,

Mark.

Changed 4 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.

Changed 4 years ago by robe

  • status changed from assigned to closed
  • resolution set to fixed

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.

Changed 4 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.