[postgis-devel] [PostGIS] #965: ST_AddIsoNode allows adding non-isolated nodes (nodes with containing_face=null)

[PostGIS]

#965: ST_AddIsoNode allows adding non-isolated nodes (nodes with
containing_face=null)
----------------------+-----------------------------------------------------
 Reporter:  strk      |       Owner:  strk         
     Type:  defect    |      Status:  new          
 Priority:  medium    |   Milestone:  PostGIS 2.0.0
Component:  topology  |     Version:  trunk        
 Keywords:            |  
----------------------+-----------------------------------------------------

Comment(by strk):

 You could avoid the double check by implementing the semantic Scarponcini
 suggested when 'null' is passed as the containing_face, and always passing
 'null', but then again you wouldn't know (at the client side) which face
 your node was in, which would speed up next call (if you're adding
 multiple nodes you know all being within the same face.

 I suggest the next step here would be providing a comprehensive regression
 test for ST_AddIsoNode. Some of them could be extrapolated from the
 current test/regress/sqlmm.sql file (which is expecting the current
 behavior, of course...)

-- 
Ticket URL: <http://trac.osgeo.org/postgis/ticket/965#comment:6>
PostGIS <http://trac.osgeo.org/postgis/>
The PostGIS Trac is used for bug, enhancement & task tracking, a user and developer wiki, and a view into the subversion code repository of PostGIS project.