Merge master

Logic/MetaTagging.ts

@@ -7,10 +7,9 @@ import CountryCoder from "latlon2country"
 import {UIEventSource} from "./UIEventSource";
 
 class SimpleMetaTagger {
-    private _f: (feature: any, index: number) => void;
     public readonly keys: string[];
     public readonly doc: string;
-
+    private _f: (feature: any, index: number) => void;
 
     constructor(keys: string[], doc: string, f: ((feature: any, index: number) => void)) {
         this.keys = keys;
@@ -90,7 +89,7 @@ export default class MetaTagging {
                 if (tags.opening_hours === undefined || tags._country === undefined) {
                     return;
                 }
-                try{
+                try {
 
                     const oh = new opening_hours(tags["opening_hours"], {
                         lat: tags._lat,
@@ -121,8 +120,8 @@ export default class MetaTagging {
                         }
                     }
                     updateTags();
-                }catch(e){
-                    console.error(e);
+                } catch (e) {
+                    console.error("Error while parsing opening hours of ", tags.id, e);
                     tags["_isOpen"] = "parse_error";
                 }
 
@@ -251,16 +250,16 @@ export default class MetaTagging {
                 carWidthUsed +
                 cyclistWidthUsed +
                 parallelParkingCount * carWidthUsed;
-            properties["_width:needed:no_pedestrians"] =Utils.Round(targetWidthIgnoringPedestrians);
+            properties["_width:needed:no_pedestrians"] = Utils.Round(targetWidthIgnoringPedestrians);
 
             const pedestriansNeed = Math.max(0, pedestrianFlowNeeded) * pedestrianWidth;
-            const targetWidth = targetWidthIgnoringPedestrians + pedestriansNeed ;
+            const targetWidth = targetWidthIgnoringPedestrians + pedestriansNeed;
             properties["_width:needed"] = Utils.Round(targetWidth);
             properties["_width:needed:pedestrians"] = Utils.Round(pedestriansNeed)
 
 
-            properties["_width:difference"] = Utils.Round(targetWidth - width );
-            properties["_width:difference:no_pedestrians"] = Utils.Round(targetWidthIgnoringPedestrians - width) ;
+            properties["_width:difference"] = Utils.Round(targetWidth - width);
+            properties["_width:difference:no_pedestrians"] = Utils.Round(targetWidthIgnoringPedestrians - width);
 
         }
     );

assets/layers/cycling_themed_object/cycling_themed_objects.json

@@ -7,7 +7,15 @@
     "de": "Mit Fahrrad zusammenhängendes Objekt"
   },
   "minzoom": 13,
-  "overpassTags": {"or": ["theme~cycling|bicycle", "sport=cycling"]},
+  "overpassTags": {
+    "or": [
+      "theme~cycling|bicycle",
+      "sport=cycling",
+      "association~cycling|bicycle",
+      "ngo~cycling|bicycle",
+      "club~bicycle|cycling"
+    ]
+  },
   "title": {
     "render": {
       "en": "Bike related object",
@@ -18,7 +26,7 @@
     "mappings": [
       {
         "if": "name~*",
-        "then":"<i>{name}</i>"
+        "then": "<i>{name}</i>"
       },
       {
         "if": "leisure=track",
@@ -30,32 +38,13 @@
     ]
   },
   "description": {},
-  "tagRenderings": [ "images",
-
-    {
-      "render": "<a href='{website}' target='_blank'>{website}</a>",
-      "freeform": {
-        "key": "website"
-      }
-    },
-    {
-      "render": "<a href='tel:{phone}'>{phone}</a>",
-      "freeform": {
-        "key": "phone"
-      }
-    },
-    {
-      "render": "<a href='mailto:{email}' target='_blank'>{email}</a>",
-      "freeform": {
-        "key": "email"
-      }
-    },
-    {
-      "render": "{opening_hours_table(opening_hours)}",
-      "freeform": {
-        "key": "opening_hours"
-      }
-    }
+  "tagRenderings": [
+    "images",
+    "description",
+    "website",
+    "email",
+    "phone",
+    "opening_hours"
   ],
   "hideUnderlayingFeaturesMinPercentage": 0,
   "icon": {