Ticket #1503 (closed defect: fixed)

Opened 4 months ago

Last modified 4 months ago

ST_AddIsoNode fails detecting containing face

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: topology Version: trunk
Keywords: Cc:

Description

When pasing a non-null value as containing_face to ST_addIsoNode the function checks if the node is _really_ inside the mentioned face. When passing a null, the function figures out by itself. When passing 0, no check is performed !

Change History

Changed 4 months ago by strk

Actually, it seems worst than that. The good value fails too! May be an invalid ticket, inspecting further...

Changed 4 months ago by strk

  • status changed from new to closed
  • resolution set to fixed
  • summary changed from ST_AddIsoNode doesn't check face containment if 0 is given to ST_AddIsoNode fails detecting containing face

I do confirm the bug, but couldn't find the time to produce a testcase for it. It doesn't seem an easy case, possibly affected by a bug in ST_Intersect or something like that.

Commit r8941 simplifies the code and fixes the bug for me.

The existing testcase (test/regress/st_addisonode.sql) was hard to understand so I rewrote it with r8942

Note: See TracTickets for help on using tickets.