@dra2020/baseclient 1.0.108 → 1.0.110

package/dist/geo/geo.d.ts

@@ -62,6 +62,7 @@ export declare class GeoMultiCollection {
     _col(e: GeoEntry): GeoFeatureCollection;
     _map(e: GeoEntry): GeoFeatureMap;
     _topo(e: GeoEntry): Poly.Topo;
+    _length(e: GeoEntry): number;
     colOf(tag: string): GeoFeatureCollection;
     mapOf(tag: string): GeoFeatureMap;
     topoOf(tag: string): Poly.Topo;

package/lib/geo/geo.ts

@@ -390,13 +390,21 @@ export class GeoMultiCollection
       else if (e.map)
       {
         e.col = geoMapToCollectionNonNull(e.map);
-        console.log(`toposplice: constructing topology for ${e.col.features.length} features`);
         e.topo = geoCollectionToTopoNonNull(e.col);
       }
     }
     return e.topo;
   }
 
+  _length(e: GeoEntry): number
+  {
+    if (e == null) return 0;
+    return  e.col ? e.col.features.length
+          : e.map ? Util.countKeys(e.map)
+          : e.topo ? Util.countKeys(e.topo.objects)
+          : 0;
+  }
+
   colOf(tag: string): GeoFeatureCollection { return this._col(this.entries[tag]); }
   mapOf(tag: string): GeoFeatureMap { return this._map(this.entries[tag]); }
   topoOf(tag: string): Poly.Topo { return this._topo(this.entries[tag]); }
@@ -485,10 +493,9 @@ export class GeoMultiCollection
         // Old-style, goes through map (to filter hidden) and then collection
         // this.all.topo = geoCollectionToTopoNonNull(this.allCol());
         // New style, use splice on packed topologies
-        console.log(`toposplice: splicing ${this.nEntries} topologies`);
-        let topoarray = Object.values(this.entries).map((e: GeoEntry) =>
-          { return { topology: this._topo(e), filterout: this.hidden } });
-        this.all.topo = Poly.topoSplice(topoarray);
+        let topoarray = Object.values(this.entries).filter((e: GeoEntry) => this._length(e) > 0)
+          .map((e: GeoEntry) => { return { topology: this._topo(e), filterout: this.hidden } });
+        this.all.topo = topoarray.length == 0 ? null : topoarray.length == 1 ? topoarray[0].topology : Poly.topoSplice(topoarray);
       }
     }
     return this.all.topo;
@@ -554,14 +561,7 @@ export class GeoMultiCollection
   get length(): number
   {
     let n = 0;
-    this.forEachEntry(e => {
-        if (e.col)
-          n += e.col.features.length;
-        else if (e.map)
-          n += Util.countKeys(e.map);
-        else if (e.topo)
-          n += Util.countKeys(e.topo.objects);
-      });
+    this.forEachEntry(e => n += this._length(e));
     return n;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dra2020/baseclient",
-  "version": "1.0.108",
+  "version": "1.0.110",
   "description": "Utility functions for Javascript projects.",
   "main": "dist/baseclient.js",
   "types": "./dist/all/all.d.ts",