@dra2020/baseclient 1.0.108 → 1.0.110

package/dist/baseclient.js

@@ -2655,12 +2655,19 @@ class GeoMultiCollection {
                 e.topo = geoCollectionToTopoNonNull(e.col);
             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) {
+        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) { return this._col(this.entries[tag]); }
     mapOf(tag) { return this._map(this.entries[tag]); }
     topoOf(tag) { return this._topo(this.entries[tag]); }
@@ -2735,9 +2742,9 @@ 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) => { return { topology: this._topo(e), filterout: this.hidden }; });
-                this.all.topo = Poly.topoSplice(topoarray);
+                let topoarray = Object.values(this.entries).filter((e) => this._length(e) > 0)
+                    .map((e) => { 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;
@@ -2789,14 +2796,7 @@ class GeoMultiCollection {
     }
     get length() {
         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;
     }
     // Use forEach in preference to iteration using this function