[BUGFIX] Multipoint asJSON

rldhont · rldhont · commit a860af5e67b3 · 2015-11-20T11:05:43.000+01:00
Fixes #13855 Multipoint asJSON made MultiPoint as Multilinestring with only 1 point by linestring. This bugfix adds tests.
diff --git a/src/core/geometry/qgsmultipointv2.cpp b/src/core/geometry/qgsmultipointv2.cpp
@@ -73,20 +73,23 @@ QDomElement QgsMultiPointV2::asGML3( QDomDocument& doc, int precision, const QSt
 
 QString QgsMultiPointV2::asJSON( int precision ) const
 {
-  QString json = "{\"type\": \"MultiPoint\", \"coordinates\": [";
+  QString json = "{\"type\": \"MultiPoint\", \"coordinates\": ";
+  
+  QList<QgsPointV2> pts;
   Q_FOREACH ( const QgsAbstractGeometryV2 *geom, mGeometries )
   {
     if ( dynamic_cast<const QgsPointV2*>( geom ) )
     {
       const QgsPointV2* point = static_cast<const QgsPointV2*>( geom );
-      json += QgsGeometryUtils::pointsToJSON( QList<QgsPointV2>() << *point, precision ) + ", ";
+      pts << *point;
     }
   }
-  if ( json.endsWith( ", " ) )
+  json += QgsGeometryUtils::pointsToJSON( pts, precision );
+  /*if ( json.endsWith( ", " ) )
   {
     json.chop( 2 ); // Remove last ", "
-  }
-  json += "] }";
+  }*/
+  json += " }";
   return json;
 }
 
diff --git a/tests/src/core/testqgsgeometry.cpp b/tests/src/core/testqgsgeometry.cpp
@@ -81,6 +81,8 @@ class TestQgsGeometry : public QObject
     void smoothCheck();
 
     void dataStream();
+    
+    void exportToGeoJSON();
 
   private:
     /** A helper method to do a render check to see if the geometry op is as expected */
@@ -739,6 +741,51 @@ void TestQgsGeometry::dataStream()
   QVERIFY( resultGeometry.isEmpty() );
 }
 
+void TestQgsGeometry::exportToGeoJSON()
+{
+  //Point
+  QString wkt = "Point (40 50)";
+  QScopedPointer<QgsGeometry> geom( QgsGeometry::fromWkt( wkt ) );
+  QString obtained = geom->exportToGeoJSON();
+  QString geojson = "{\"type\": \"Point\", \"coordinates\": [40, 50]}";
+  QCOMPARE( obtained, geojson );
+  
+  //MultiPoint
+  wkt = "MultiPoint (0 0, 10 0, 10 10, 20 10)";
+  geom.reset( QgsGeometry::fromWkt( wkt ) );
+  obtained = geom->exportToGeoJSON();
+  geojson = "{\"type\": \"MultiPoint\", \"coordinates\": [ [0, 0], [10, 0], [10, 10], [20, 10]] }";
+  QCOMPARE( obtained, geojson );
+
+  //Linestring
+  wkt = "LineString(0 0, 10 0, 10 10, 20 10)";
+  geom.reset( QgsGeometry::fromWkt( wkt ) );
+  obtained = geom->exportToGeoJSON();
+  geojson = "{\"type\": \"LineString\", \"coordinates\": [ [0, 0], [10, 0], [10, 10], [20, 10]]}";
+  QCOMPARE( obtained, geojson );
+  
+  //MultiLineString
+  wkt = "MultiLineString ((0 0, 10 0, 10 10, 20 10),(30 30, 40 30, 40 40, 50 40))";
+  geom.reset( QgsGeometry::fromWkt( wkt ) );
+  obtained = geom->exportToGeoJSON();
+  geojson = "{\"type\": \"MultiLineString\", \"coordinates\": [[ [0, 0], [10, 0], [10, 10], [20, 10]], [ [30, 30], [40, 30], [40, 40], [50, 40]]] }";
+  QCOMPARE( obtained, geojson );
+  
+  //Polygon
+  wkt = "Polygon ((0 0, 10 0, 10 10, 0 10, 0 0 ),(2 2, 4 2, 4 4, 2 4, 2 2))";
+  geom.reset( QgsGeometry::fromWkt( wkt ) );
+  obtained = geom->exportToGeoJSON();
+  geojson = "{\"type\": \"Polygon\", \"coordinates\": [[ [0, 0], [10, 0], [10, 10], [0, 10], [0, 0]], [ [2, 2], [4, 2], [4, 4], [2, 4], [2, 2]]] }";
+  QCOMPARE( obtained, geojson );
+
+  //MultiPolygon
+  wkt = "MultiPolygon (((0 0, 10 0, 10 10, 0 10, 0 0 )),((2 2, 4 2, 4 4, 2 4, 2 2)))";
+  geom.reset( QgsGeometry::fromWkt( wkt ) );
+  obtained = geom->exportToGeoJSON();
+  geojson = "{\"type\": \"MultiPolygon\", \"coordinates\": [[[ [0, 0], [10, 0], [10, 10], [0, 10], [0, 0]]], [[ [2, 2], [4, 2], [4, 4], [2, 4], [2, 2]]]] }";
+  QCOMPARE( obtained, geojson );
+}
+
 bool TestQgsGeometry::renderCheck( const QString& theTestName, const QString& theComment, int mismatchCount )
 {
   mReport += "<h2>" + theTestName + "</h2>\n";

Original file line number	Diff line number	Diff line change
`@@ -73,20 +73,23 @@ QDomElement QgsMultiPointV2::asGML3( QDomDocument& doc, int precision, const QSt`
`73`	`73`
`74`	`74`	`QString QgsMultiPointV2::asJSON( int precision ) const`
`75`	`75`	`{`
`76`		`- QString json = "{\"type\": \"MultiPoint\", \"coordinates\": [";`
	`76`	`+ QString json = "{\"type\": \"MultiPoint\", \"coordinates\": ";`
	`77`	`+`
	`78`	`+ QList<QgsPointV2> pts;`
`77`	`79`	`Q_FOREACH ( const QgsAbstractGeometryV2 *geom, mGeometries )`
`78`	`80`	`{`
`79`	`81`	`if ( dynamic_cast<const QgsPointV2*>( geom ) )`
`80`	`82`	`{`
`81`	`83`	`const QgsPointV2* point = static_cast<const QgsPointV2*>( geom );`
`82`		`- json += QgsGeometryUtils::pointsToJSON( QList<QgsPointV2>() << *point, precision ) + ", ";`
	`84`	`+ pts << *point;`
`83`	`85`	`}`
`84`	`86`	`}`
`85`		`- if ( json.endsWith( ", " ) )`
	`87`	`+ json += QgsGeometryUtils::pointsToJSON( pts, precision );`
	`88`	`+ /*if ( json.endsWith( ", " ) )`
`86`	`89`	`{`
`87`	`90`	`json.chop( 2 ); // Remove last ", "`
`88`		`- }`
`89`		`- json += "] }";`
	`91`	`+ }*/`
	`92`	`+ json += " }";`
`90`	`93`	`return json;`
`91`	`94`	`}`
`92`	`95`