Opened 13 years ago

Closed 13 years ago

#1503 closed defect (fixed)

ST_AddIsoNode fails detecting containing face

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: topology Version: master
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 (2)

comment:1 by strk, 13 years ago

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

comment:2 by strk, 13 years ago

Resolution: fixed
Status: newclosed
Summary: ST_AddIsoNode doesn't check face containment if 0 is givenST_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.