Merge pull request #111 from opentripplanner/zero-length-leg-geometry-fix

Fix a bug that occurred when rendering geometry with 0 length

Author: landonreed

Diff for: lib/components/map/base-map.js

@@ -275,6 +275,7 @@ class BaseMap extends Component {
         center={center}
         // onClick={this._onLeftClick}
         zoom={config.map.initZoom || 13}
+        maxZoom={config.map.maxZoom}
         onOverlayAdd={this._onOverlayAdd}
         onOverlayRemove={this._onOverlayRemove}
         onViewportChanged={this._onViewportChanged}

Diff for: lib/util/itinerary.js

@@ -228,11 +228,21 @@ export function getItineraryBounds (itinerary) {
   return latLngBounds(coords)
 }
 
+/**
+ * Return a leaflet LatLngBounds object that encloses the given leg's geometry.
+ */
 export function getLegBounds (leg) {
-  return latLngBounds(polyline
+  const coords = polyline
     .toGeoJSON(leg.legGeometry.points)
     .coordinates.map(c => [c[1], c[0]])
-  )
+
+  // in certain cases, there might be zero-length coordinates in the leg
+  // geometry. In these cases, build us an array of coordinates using the from
+  // and to data of the leg.
+  if (coords.length === 0) {
+    coords.push([leg.from.lat, leg.from.lon], [leg.to.lat, leg.to.lon])
+  }
+  return latLngBounds(coords)
 }
 
 export function routeComparator (a, b) {