Pass itinerary builder as option to control

Otherwise, this wasn't working anymore and just displayed `[object
Object]` instead of a useful instruction. This approach also doesn't
overwrite the original `ItineraryBuilder` class of the
`leaflet-routing-machine` anymore, which feels a bit cleaner, too.

Author: mstock

src/index.js

@@ -18,7 +18,7 @@ var mergedOptions = L.extend(leafletOptions.defaultState, parsedOptions);
 var local = localization.get(mergedOptions.language);
 
 // load only after language was chosen
-var itineraryBuilder = require('./itinerary_builder')(mergedOptions.language);
+var ItineraryBuilder = require('./itinerary_builder')(mergedOptions.language);
 
 var mapLayer = leafletOptions.layer;
 var overlay = leafletOptions.overlay;
@@ -128,8 +128,6 @@ var plan = new ReversablePlan([], {
   }
 });
 
-L.extend(L.Routing, itineraryBuilder);
-
 // add marker labels
 var controlOptions = {
   plan: plan,
@@ -146,7 +144,8 @@ var controlOptions = {
   serviceUrl: leafletOptions.services[0].path,
   useZoomParameter: options.lrm.useZoomParameter,
   routeDragInterval: options.lrm.routeDragInterval,
-  collapsible: options.lrm.collapsible
+  collapsible: options.lrm.collapsible,
+  itineraryBuilder: new ItineraryBuilder(),
 };
 var router = (new L.Routing.OSRMv1(controlOptions));
 router._convertRouteOriginal = router._convertRoute;

src/itinerary_builder.js

@@ -82,9 +82,7 @@ module.exports = function (language) {
     });
   }
 
-  L.Routing = L.Routing || {};
-
-  L.Routing.ItineraryBuilder = L.Class.extend({
+  return L.Class.extend({
     options: {
       containerClassName: ''
     },
@@ -145,6 +143,4 @@ module.exports = function (language) {
       return row;
     }
   });
-
-  return L.Routing;
-}
+};