From 0de2cb26b80fe618b2d1489a954ebf0ce4f2bfff Mon Sep 17 00:00:00 2001
From: Eugene Burmakin <iamfrey@gmail.com>
Date: Wed, 7 Jan 2026 00:18:26 +0100
Subject: [PATCH] Add fallbacks when retrieving full route features to handle
 cases where source data access methods vary.

---
 .../maps/maplibre/event_handlers.js           | 26 +++++++++++++++----
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/app/javascript/controllers/maps/maplibre/event_handlers.js b/app/javascript/controllers/maps/maplibre/event_handlers.js
index 5cb91de8..4c450822 100644
--- a/app/javascript/controllers/maps/maplibre/event_handlers.js
+++ b/app/javascript/controllers/maps/maplibre/event_handlers.js
@@ -142,8 +142,8 @@ export class EventHandlers {
     if (!routesLayer) return
 
     // Get the full feature from source (not the clipped tile version)
-    const fullFeature = this._getFullRouteFeature(clickedFeature.properties)
-    if (!fullFeature) return
+    // Fallback to clipped feature if full feature not found
+    const fullFeature = this._getFullRouteFeature(clickedFeature.properties) || clickedFeature
 
     // If a route is selected and we're hovering over a different route, show both
     if (this.selectedRouteFeature) {
@@ -200,7 +200,23 @@ export class EventHandlers {
     if (!source) return null
 
     // Get the source data (GeoJSON FeatureCollection)
-    const sourceData = source._data || routesLayer.data
+    // Try multiple ways to access the data
+    let sourceData = null
+
+    // Method 1: Internal _data property (most common)
+    if (source._data) {
+      sourceData = source._data
+    }
+    // Method 2: Serialize and deserialize (fallback)
+    else if (source.serialize) {
+      const serialized = source.serialize()
+      sourceData = serialized.data
+    }
+    // Method 3: Use cached data from layer
+    else if (routesLayer.data) {
+      sourceData = routesLayer.data
+    }
+
     if (!sourceData || !sourceData.features) return null
 
     // Find the matching feature by properties
@@ -290,8 +306,8 @@ export class EventHandlers {
     const properties = clickedFeature.properties
 
     // Get the full feature from source (not the clipped tile version)
-    const fullFeature = this._getFullRouteFeature(properties)
-    if (!fullFeature) return
+    // Fallback to clipped feature if full feature not found
+    const fullFeature = this._getFullRouteFeature(properties) || clickedFeature
 
     // Store selected route (use full feature)
     this.selectedRouteFeature = fullFeature