Ticket #1503 (closed defect: fixed)

Opened 16 months ago

Last modified 16 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 16 months ago by strk

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

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