@dra2020/baseclient 1.0.4 → 1.0.5

package/dist/baseclient.js

@@ -7949,24 +7949,14 @@ function ringsCancel(outerPoly, innerRing) {
 function correctGeometry(f) {
     if (f && f.geometry && f.geometry.type === 'MultiPolygon' && f.geometry.coordinates) {
         let multiPoly = f.geometry.coordinates;
-        /* Comment this out right now - might have been do to not rewinding merge output
-        // topojson will under certain circumstances return a MultiPolygon with the first Polygon containing holes
-        // that are precisely filled by a subsequent polygon. We make a secondary union pass to try to correct for this.
-        // If we have a really degenerate multipolygon (test above some number of polygons) omit this expensive pass
-        // since cleanup is unlikely.
-        if (multiPoly.length > 1 && multiPoly.length < 50)
-        {
-          let result = Q.unionPolys(multiPoly);
-          if (Util.depthof(result) == 4) result = [ result ];
-          multiPoly = result;
-        }
-        */
         // Convert degenerate MultiPolygon to Polygon
         if (multiPoly.length == 1) {
             f.geometry.type = 'Polygon';
             f.geometry.coordinates = multiPoly[0];
         }
     }
+    // TopoJSON does not guarantee proper winding order which messes up later processing. Fix it.
+    P.featureRewind(f);
     return f;
 }
 function topoContiguity(topo) {
@@ -8166,27 +8156,7 @@ function topoMerge(topo, geoids) {
         return null;
     let objects = [];
     geoids.forEach((geoid) => objects.push(topo.objects[geoid]));
-    let f = correctGeometry({ type: 'Feature', properties: {}, geometry: TopoClient.merge(topo, objects) });
-    P.featureRewind(f);
-    /* Comment out for now - may be due to merge output needing to be rewound
-    // If I get a bad output from topoMerge, just do more expensive poly union. This can happen if input polygons
-    // are a little funky (in particular, if they double back along the same edge.
-    if (selfIntersectFast(f))
-    {
-      //console.log('topoMerge: patching selfIntersect');
-      let polys: any[] = [];
-      geoids.forEach((geoid) => polys.push(topoToFeature(topo, geoid).geometry.coordinates));
-      let result = Q.unionPolys(polys);
-      let depth = Util.depthof(result);
-      if (depth === 5 && result.length === 1)
-      {
-        depth = 4;
-        result = result[0];
-      }
-      f = { type: 'feature', properties: {}, geometry: { type: depth == 4 ? 'Polygon' : 'MultiPolygon', coordinates: result } };
-    }
-    */
-    return f;
+    return correctGeometry({ type: 'Feature', properties: {}, geometry: TopoClient.merge(topo, objects) });
 }
 exports.topoMerge = topoMerge;
 function topoMergeFeatures(topo, features) {