@dra2020/baseclient 1.0.63 → 1.0.66

package/dist/baseclient.js

@@ -1565,21 +1565,7 @@ function geoEnsureID(col, options) {
 exports.geoEnsureID = geoEnsureID;
 function geoNormalizeFeature(f, options) {
     options = Util.shallowAssignImmutable({}, options);
-    if (f && f.geometry && f.geometry.type === 'MultiPolygon' && f.geometry.coordinates) {
-        let multiPoly = f.geometry.coordinates;
-        // Convert degenerate MultiPolygon to Polygon
-        if (multiPoly.length == 1) {
-            f.geometry.type = 'Polygon';
-            f.geometry.coordinates = multiPoly[0];
-        }
-    }
-    else if (f && f.geometry && f.geometry.type === 'Point' && f.geometry.coordinates) {
-        while (Array.isArray(f.geometry.coordinates[0]))
-            f.geometry.coordinates = f.geometry.coordinates[0];
-    }
-    else if (options.checkRewind)
-        // Various tools do not guarantee valid GeoJSON winding rules. Verify it since internal processing
-        // assumes it is correct.
+    if (options.checkRewind)
         Poly.featureRewind(f);
     return f;
 }
@@ -6828,24 +6814,30 @@ function featureRewind(f, options) {
     // Only applies to multi-polygons with no holes
     let d = Util.depthof(f.geometry.coordinates);
     let windings = polyRingWindings(f);
-    if (options.validateHoles && d === 5 && windings.length) {
-        // If there are explicit holes, don't look for implicit ones
-        let nHoles = 0;
-        windings.forEach(w => { if (w.iRing)
-            nHoles++; });
-        if (nHoles == 0 && !misWound(windings[0])) {
-            // Looking for pattern: of (G (B*))*
-            // To convert to ((GH*))*
-            let iWinding = 0;
+    if (options.validateHoles && d === 5 && windings.length > 1) {
+        // First winding needs to be correct so we have a poly to add holes to
+        if (!misWound(windings[0])) {
+            // Flatten everything out and then rebuild hole structure based on windings
+            let nHoles = 0;
+            windings.forEach(w => { nHoles += w.iRing ? 1 : 0; });
+            if (nHoles) {
+                let c = [];
+                f.geometry.coordinates.forEach((poly) => {
+                    poly.forEach((ring) => {
+                        c.push([ring]);
+                    });
+                });
+                f.geometry.coordinates = c;
+                windings = polyRingWindings(f);
+            }
             let polys = f.geometry.coordinates;
-            let iPoly;
-            for (; iWinding < windings.length; iWinding++) {
+            let iPoly = 0;
+            for (let iWinding = 0; iWinding < windings.length; iWinding++) {
                 let good = !misWound(windings[iWinding]);
-                if (!good && iWinding == 0) // First is miswound - no good
-                    break;
                 if (good)
                     iPoly = iWinding;
                 else {
+                    // If hole, add to previous poly
                     polys[iPoly].push(polys[iWinding][0]);
                     polys[iWinding] = null;
                 }
@@ -6856,10 +6848,13 @@ function featureRewind(f, options) {
         if (f.geometry.coordinates.length == 1) {
             f.geometry.type = 'Polygon';
             f.geometry.coordinates = f.geometry.coordinates[0];
+            d = 4;
         }
     }
     // OK, now go through each ring
     let polys = f.geometry.coordinates;
+    if (d == 4)
+        polys = [polys];
     windings = polyRingWindings(f);
     windings.forEach((w) => {
         let good = !misWound(w);