ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

The fromParts factories take geometry instances instead of coords

Browse Source 
The MultiLineString case was correct here.  The MultiPoint and MultiPolygon cases should be using the constructor instead of the `fromParts` factories.
This commit is contained in:
Tim Schaub
2013-06-26 12:23:19 -06:00
parent 16e23a33fb
commit 30d947521d
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ol/parser/kml.js
+2 -2
View File
@@ -1020,7 +1020,7 @@ ol.parser.KML.prototype.createGeometry_ = function(container,
for (i = 0, ii = container.geometry.parts.length; i < ii; i++) { for (i = 0, ii = container.geometry.parts.length; i < ii; i++) {
coordinates.push(container.geometry.parts[i].coordinates); coordinates.push(container.geometry.parts[i].coordinates);
} }
geometry = ol.geom.MultiPoint.fromParts(coordinates, opt_vertices); geometry = new ol.geom.MultiPoint(coordinates, opt_vertices);
break; break;
case ol.geom.GeometryType.MULTILINESTRING: case ol.geom.GeometryType.MULTILINESTRING:
coordinates = []; coordinates = [];
@@ -1034,7 +1034,7 @@ ol.parser.KML.prototype.createGeometry_ = function(container,
for (i = 0, ii = container.geometry.parts.length; i < ii; i++) { for (i = 0, ii = container.geometry.parts.length; i < ii; i++) {
coordinates.push(container.geometry.parts[i].coordinates); coordinates.push(container.geometry.parts[i].coordinates);
} }
geometry = ol.geom.MultiPolygon.fromParts(coordinates, opt_vertices); geometry = new ol.geom.MultiPolygon(coordinates, opt_vertices);
break; break;
case ol.geom.GeometryType.GEOMETRYCOLLECTION: case ol.geom.GeometryType.GEOMETRYCOLLECTION:
var geometries = []; var geometries = [];