config doc fix, parentstation not shared

opentripplanner · Mar 21, 2024 · 5132bf2 · 5132bf2
1 parent d75df7f
commit 5132bf2
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 7 deletions.
diff --git a/doc-templates/sandbox/MapboxVectorTilesApi.md b/doc-templates/sandbox/MapboxVectorTilesApi.md
@@ -110,7 +110,7 @@ The feature must be configured in `router-config.json` as follows
         "mapper": "DigitransitRealtime",
         "maxZoom": 20,
         "minZoom": 14,
-        "cacheMaxSeconds": 600
+        "cacheMaxSeconds": 60
       },
       // This exists for backwards compatibility. At some point, we might want
       // to add a new real-time parking mapper with better translation support

diff --git a/...g/opentripplanner/ext/vectortiles/layers/stops/DigitransitRealtimeStopPropertyMapper.java b/...g/opentripplanner/ext/vectortiles/layers/stops/DigitransitRealtimeStopPropertyMapper.java
@@ -35,7 +35,13 @@ protected Collection<KeyValue> map(RegularStop stop) {
     Collection<KeyValue> sharedKeyValues = getBaseKeyValues(stop, i18NStringMapper, transitService);
     return ListUtils.combine(
       sharedKeyValues,
-      List.of(new KeyValue("closedByServiceAlert", noServiceAlert))
+      List.of(
+        new KeyValue("closedByServiceAlert", noServiceAlert),
+        new KeyValue(
+          "parentStation",
+          stop.getParentStation() != null ? stop.getParentStation().getId() : null
+        )
+      )
     );
   }
 }
diff --git a/.../java/org/opentripplanner/ext/vectortiles/layers/stops/DigitransitStopPropertyMapper.java b/.../java/org/opentripplanner/ext/vectortiles/layers/stops/DigitransitStopPropertyMapper.java
@@ -9,6 +9,7 @@
 import org.json.simple.JSONArray;
 import org.json.simple.JSONObject;
 import org.opentripplanner.apis.support.mapping.PropertyMapper;
+import org.opentripplanner.framework.collection.ListUtils;
 import org.opentripplanner.framework.i18n.I18NStringMapper;
 import org.opentripplanner.inspector.vector.KeyValue;
 import org.opentripplanner.transit.model.network.TripPattern;
@@ -34,7 +35,16 @@ protected static DigitransitStopPropertyMapper create(
 
   @Override
   protected Collection<KeyValue> map(RegularStop stop) {
-    return getBaseKeyValues(stop, i18NStringMapper, transitService);
+    Collection<KeyValue> sharedKeyValues = getBaseKeyValues(stop, i18NStringMapper, transitService);
+    return ListUtils.combine(
+      sharedKeyValues,
+      List.of(
+        new KeyValue(
+          "parentStation",
+          stop.getParentStation() != null ? stop.getParentStation().getId() : "null"
+        )
+      )
+    );
   }
 
   protected static Collection<KeyValue> getBaseKeyValues(
@@ -48,10 +58,6 @@ protected static Collection<KeyValue> getBaseKeyValues(
       new KeyValue("code", stop.getCode()),
       new KeyValue("platform", stop.getPlatformCode()),
       new KeyValue("desc", i18NStringMapper.mapToApi(stop.getDescription())),
-      new KeyValue(
-        "parentStation",
-        stop.getParentStation() != null ? stop.getParentStation().getId() : null
-      ),
       new KeyValue("type", getType(transitService, stop)),
       new KeyValue("routes", getRoutes(transitService, stop))
     );