Fix: actually load geojson layers; formatting

Logic/FeatureSource/Sources/GeoJsonSource.ts

@@ -67,7 +67,7 @@ export default class GeoJsonSource implements FeatureSource {
                 this.LoadJSONFrom(url, eventsource, layer)
                     .then((_) => console.log("Loaded geojson " + url))
                     .catch((err) => console.error("Could not load ", url, "due to", err))
-                return true
+                return true // data is loaded, we can safely unregister
             })
         } else {
             this.LoadJSONFrom(url, eventsource, layer)
@@ -77,13 +77,20 @@ export default class GeoJsonSource implements FeatureSource {
         this.features = eventsource
     }
 
+    /**
+     * Init the download, write into the specified event source for the given layer.
+     * Note this method caches the requested geojson for five minutes
+     */
     private async LoadJSONFrom(
         url: string,
         eventSource: UIEventSource<Feature[]>,
-        layer: LayerConfig
+        layer: LayerConfig,
-    ): Promise<void> {
+        options?: {
+            maxCacheAgeSec?: number | 300
+        }
+    ): Promise<Feature[]> {
         const self = this
-        let json = await Utils.downloadJsonCached(url, 60 * 60)
+        let json = await Utils.downloadJsonCached(url, (options?.maxCacheAgeSec ?? 300) * 1000)
 
         if (json.features === undefined || json.features === null) {
             json.features = []
@@ -134,5 +141,6 @@ export default class GeoJsonSource implements FeatureSource {
         }
 
         eventSource.setData(newFeatures)
+        return newFeatures
     }
 }

Logic/FeatureSource/Sources/LayoutSource.ts

@@ -63,8 +63,6 @@ export default class LayoutSource extends FeatureSourceMerger {
             LayoutSource.setupGeojsonSource(l, mapProperties, isDisplayed(l.id))
         )
 
-        const expiryInSeconds = Math.min(...(layers?.map((l) => l.maxAgeOfCache) ?? []))
-
         super(overpassSource, osmApiSource, ...geojsonSources, ...fromCache)
 
         const self = this
@@ -84,7 +82,7 @@ export default class LayoutSource extends FeatureSourceMerger {
     ): FeatureSource {
         const source = layer.source
         isActive = mapProperties.zoom.map(
-            (z) => (isActive?.data ?? true) && z >= layer.maxzoom,
+            (z) => (isActive?.data ?? true) && z >= layer.minzoom,
             [isActive]
         )
         if (source.geojsonZoomLevel === undefined) {