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

chenxianhuii · chenxianhuii · commit 58277ce30f87 · 2025-05-21T17:52:18.000+08:00
diff --git a/src/mapboxgl/core/Util.js b/src/mapboxgl/core/Util.js
@@ -42,7 +42,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;
         }
     },
 
diff --git a/src/maplibregl/core/Util.js b/src/maplibregl/core/Util.js
@@ -42,7 +42,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;
         }
     },
 
diff --git a/src/openlayers/core/Util.js b/src/openlayers/core/Util.js
@@ -76,7 +76,7 @@
        return null;
      }
      const result = new GeoJSONFormat().read(geoJSON, 'FeatureCollection');
-     return result[0].geometry;
+     return result && result[0].geometry;
    },
 
    /**
diff --git a/test/mapboxgl/core/UtilSpec.js b/test/mapboxgl/core/UtilSpec.js
@@ -39,4 +39,65 @@ describe('Util', () => {
     expect(Util.isMatchAdministrativeName('北京', '北京市')).toBeTruthy();
     expect(Util.isMatchAdministrativeName('北京', null)).toBeFalsy();
   });
+  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();
+  });
 });
diff --git a/test/maplibregl/core/UtilSpec.js b/test/maplibregl/core/UtilSpec.js
@@ -39,4 +39,65 @@ describe('Util', () => {
     expect(Util.isMatchAdministrativeName('北京', '北京市')).toBeTruthy();
     expect(Util.isMatchAdministrativeName('北京', null)).toBeFalsy();
   });
+  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();
+  });
 });
diff --git a/test/openlayers/core/UtilSpec.js b/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', () => {

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ export const Util = {`
`42`	`42`	`if (geoJSON && geoJSON.type) {`
`43`	`43`	`var format = new GeoJSONFormat();`
`44`	`44`	`var result = format.read(geoJSON, "FeatureCollection");`
`45`		`- return result[0].geometry;`
	`45`	`+ return result && result[0].geometry;`
`46`	`46`	`}`
`47`	`47`	`},`
`48`	`48`
Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@`
`76`	`76`	`return null;`
`77`	`77`	`}`
`78`	`78`	`const result = new GeoJSONFormat().read(geoJSON, 'FeatureCollection');`
`79`		`- return result[0].geometry;`
	`79`	`+ return result && result[0].geometry;`
`80`	`80`	`},`
`81`	`81`
`82`	`82`	`/**`