[postgis-tickets] [PostGIS] #2588: GeometryCollection constructor parse defect for sub geometries (POINTM, POLYGONM, ...) of GEOMETRYCOLLECTIONM

[PostGIS]

#2588: GeometryCollection constructor parse defect for sub geometries
(POINTM,POLYGONM,...) of GEOMETRYCOLLECTIONM
------------------------------------------------------+---------------------
 Reporter:  bergun                                    |       Owner:  delawen      
     Type:  defect                                    |      Status:  new          
 Priority:  medium                                    |   Milestone:  PostGIS 2.2.0
Component:  java                                      |     Version:  trunk        
 Keywords:  GEOMETRYCOLLECTIONM jdbc POINTM POLYGONM  |  
------------------------------------------------------+---------------------

Comment(by bergun):

 Our tests can be viewed from https://github.com/datanucleus/tests
 repository and jdo/geospatial project.

 PgGeometryMappingTest.java
 PgGeometryMySQLTest.java
 PgGeometryPostGISExtMappingTest.java
 PgGeometrySpatialHelperTest.java
 PgGeometrySpatialTest.java

 There are 205 tests for Mysql/Mariadb and around 230 tests for Postgresql
 db. There are 2D, 3D and M geometries tests if u have look at the test
 classes. All of the test cases I have tested to be ok after updating the
 GeometryCollection class.

 I didnt actually understand why the second signature was added to the
 below method

 public void outerWKT(StringBuffer sb, boolean putM)

 because even putM is declared as false or true inside the method there is
 check for if Geometry haveM and 2dimensional.

 if (putM && haveMeasure && dimension == 2) {

 on line 263 Geometry.class

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