Opened 13 years ago

Closed 13 years ago

#971 closed defect (duplicate)

ST_AddIsoNode - containing_face value should be zero instead is NULL

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

Description

Hi,

starting from an empty topology. Adding an Isolated Node with the ST_AddIsoNode , the "cointaining_face" field of Node table should be set to idface '0' (the UniverseFace) instead of NULL as now.

This code is a test:

select topology.DropTopology('topo_test2');
select topology.CreateTopology('topo_test2',3003);
select topology.ST_AddIsoNode('topo_test2',NULL,ST_GeomFromEWKT('SRID=3003;POINT(1 4)'));

select containing_face from topo_test2.node;

The actual result is (null)

it should be 0

Attachments (1)

patch_topology.zip (2.3 KB ) - added by aperi2007 13 years ago.
Patch for ST_AddIsoNode, ST_AddIsoEdge, ST_GetFaceGeometry

Download all attachments as: .zip

Change History (3)

by aperi2007, 13 years ago

Attachment: patch_topology.zip added

Patch for ST_AddIsoNode, ST_AddIsoEdge, ST_GetFaceGeometry

comment:1 by aperi2007, 13 years ago

Added patch for ST_AddIsoNode, ST_AddIsoEdge, ST_GetFaceGeometry to resolve tickets #971, #972 , #973, #976 http://trac.osgeo.org/postgis/attachment/ticket/971/patch_topology.zip

comment:2 by strk, 13 years ago

Resolution: duplicate
Status: newclosed

Please use #965 for modifications to ST_AddIsoNode. It isn't yet clear what the behavior should be and we're waiting for feedback from the ISO people on that.

Feel free to file other tickets for ST_AddIsoEdge and ST_GetFaceGeometry, or add your patches to existing tickets.

Separate, smaller, focused patches are easier to deal with.

Thanks.

Note: See TracTickets for help on using tickets.