ICL-1580 修复编辑要素时feature不添加geometry无法更新成功 review by luox

Author: chenxianhuii

src/mapboxgl/core/Util.js

@@ -45,7 +45,7 @@ export const Util = {
         if (geoJSON && geoJSON.type) {
             var format = new GeoJSONFormat();
             var result = format.read(geoJSON, "FeatureCollection");
-            return result[0].geometry;
+            return result && result[0].geometry;
         }
     },
 

src/maplibregl/core/Util.js

@@ -45,7 +45,7 @@ export const Util = {
         if (geoJSON && geoJSON.type) {
             var format = new GeoJSONFormat();
             var result = format.read(geoJSON, "FeatureCollection");
-            return result[0].geometry;
+            return result && result[0].geometry;
         }
     },
 

src/openlayers/core/Util.js

@@ -72,7 +72,7 @@
        return null;
      }
      const result = new GeoJSONFormat().read(geoJSON, 'FeatureCollection');
-     return result[0].geometry;
+     return result && result[0].geometry;
    },
 
    /**

test/mapboxgl/core/UtilSpec.js

@@ -0,0 +1,65 @@
+import { Util } from '../../../src/mapboxgl/core/Util';
+
+describe('Util', () => {
+  it('toSuperMapGeometry', () => {
+      var geoJSON = {
+          type: 'FeatureCollection',
+          features: [
+              {
+                  type: 'Feature',
+                  properties: {
+                      attributes: {
+                          SmArea: '1.6060069623493825E15',
+                          SmGeoPosition: '65536',
+                          SmID: '1',
+                          SmPerimeter: '1.6030006674231339E8'
+                      },
+                      id: 1,
+                      layerName: 'World@China',
+                      searchValues: '',
+                      type: 'REGION'
+                  },
+                  geometry: {
+                      type: 'MultiPolygon',
+                      coordinates: [
+                          [
+                              [
+                                  [-2, 258],
+                                  [258, 258],
+                                  [-2, 258],
+                                  [-2, 258]
+                              ]
+                          ]
+                      ]
+                  }
+              }
+          ]
+      };
+      var result = Util.toSuperMapGeometry(geoJSON);
+      expect(result).not.toBeNull();
+      
+      var geoJSON2 = {
+        type: 'FeatureCollection',
+        features: [
+            {
+                type: 'Feature',
+                properties: {
+                    attributes: {
+                        SmArea: '1.6060069623493825E15',
+                        SmGeoPosition: '65536',
+                        SmID: '1',
+                        SmPerimeter: '1.6030006674231339E8'
+                    },
+                    id: 1,
+                    layerName: 'World@China',
+                    searchValues: '',
+                    type: 'REGION'
+                },
+                geometry: null
+            }
+        ]
+      };
+      var result2 = Util.toSuperMapGeometry(geoJSON2);
+      expect(result2).toBeNull();
+  });
+});

test/maplibregl/core/UtilSpec.js

@@ -0,0 +1,65 @@
+import { Util } from '../../../src/maplibregl/core/Util';
+
+describe('Util', () => {
+  it('toSuperMapGeometry', () => {
+      var geoJSON = {
+          type: 'FeatureCollection',
+          features: [
+              {
+                  type: 'Feature',
+                  properties: {
+                      attributes: {
+                          SmArea: '1.6060069623493825E15',
+                          SmGeoPosition: '65536',
+                          SmID: '1',
+                          SmPerimeter: '1.6030006674231339E8'
+                      },
+                      id: 1,
+                      layerName: 'World@China',
+                      searchValues: '',
+                      type: 'REGION'
+                  },
+                  geometry: {
+                      type: 'MultiPolygon',
+                      coordinates: [
+                          [
+                              [
+                                  [-2, 258],
+                                  [258, 258],
+                                  [-2, 258],
+                                  [-2, 258]
+                              ]
+                          ]
+                      ]
+                  }
+              }
+          ]
+      };
+      var result = Util.toSuperMapGeometry(geoJSON);
+      expect(result).not.toBeNull();
+      
+      var geoJSON2 = {
+        type: 'FeatureCollection',
+        features: [
+            {
+                type: 'Feature',
+                properties: {
+                    attributes: {
+                        SmArea: '1.6060069623493825E15',
+                        SmGeoPosition: '65536',
+                        SmID: '1',
+                        SmPerimeter: '1.6030006674231339E8'
+                    },
+                    id: 1,
+                    layerName: 'World@China',
+                    searchValues: '',
+                    type: 'REGION'
+                },
+                geometry: null
+            }
+        ]
+      };
+      var result2 = Util.toSuperMapGeometry(geoJSON2);
+      expect(result2).toBeNull();
+  });
+});

test/openlayers/core/UtilSpec.js

@@ -121,6 +121,29 @@ describe('openlayers_Util', () => {
         expect(point instanceof Point).toBeTruthy();
         expect(point.x).toBe(-2);
         expect(point.y).toBe(258);
+        var geoJSON2 = {
+          type: 'FeatureCollection',
+          features: [
+              {
+                  type: 'Feature',
+                  properties: {
+                      attributes: {
+                          SmArea: '1.6060069623493825E15',
+                          SmGeoPosition: '65536',
+                          SmID: '1',
+                          SmPerimeter: '1.6030006674231339E8'
+                      },
+                      id: 1,
+                      layerName: 'World@China',
+                      searchValues: '',
+                      type: 'REGION'
+                  },
+                  geometry: null
+              }
+          ]
+        };
+        var result2 = Util.toSuperMapGeometry(geoJSON2);
+        expect(result2).toBeNull();
     });
 
     it('resolutionToScale', () => {