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 , 13 years ago
comment:2 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Summary: | ST_AddIsoNode doesn't check face containment if 0 is given → 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
Actually, it seems worst than that. The good value fails too! May be an invalid ticket, inspecting further...