[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 aperi2007):

 I agree with this solution.
 Because reading the specs, other funzions work like this.

 For example, the ST_GetFaceGeometry.
 From ISO specs:
 Allow to pass the 0 value but intercept it giving an
 "Error - sql/mm spatial exception - universal face has no geometry".

 Always the ST_GetFaceGeometry from ISO specs:
 with a parameter idface = NULL
 shuold intercept it giving an error:
 "Error - sql/mm spatial exception - null argument".

 So I guess is correct to think to idface=0 as the value for an
 UniversalFace, and don't use the NULL value as UniversalFace.

-- 
Ticket URL: <http://trac.osgeo.org/postgis/ticket/965#comment:1>
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.