Merge branch 'v2.1'

2 files changed, 14 insertions, 2 deletions

diff --git a/chimere/static/chimere/js/jquery.chimere-ol.js b/chimere/static/chimere/js/jquery.chimere-ol.js
index a04e0b3..251c09c 100644
--- a/chimere/static/chimere/js/jquery.chimere-ol.js
+++ b/chimere/static/chimere/js/jquery.chimere-ol.js
@@ -1249,8 +1249,7 @@ if (typeof(OpenLayers) != 'undefined'){
                 uri += settings.routing_transport + "/"
             }
             if(settings.routing_speed){
-                //uri += speed.split('_')[1] + "/"
-                uri += routing_settings.speed + "/"
+                uri += settings.routing_speed + "/"
             }
             for (var i = 0; i < steps.length; i++) {
                 var step = steps[i].transform(
diff --git a/chimere/views.py b/chimere/views.py
index 0c512d4..01179e4 100644
--- a/chimere/views.py
+++ b/chimere/views.py
@@ -819,6 +819,19 @@ def route(request, map_name, lon1, lat1, lonlat_steps, lon2, lat2,
         steps = [(steps[i*2], steps[i*2+1]) for i in range(len(steps)/2)]
     except ValueError:
         return HttpResponse('no results')
+
+    # prevent incoherent transport and speed
+    if transport not in dict(settings.CHIMERE_ROUTING_TRANSPORT):
+        transport = settings.CHIMERE_ROUTING_TRANSPORT[0][0]
+    if speed:
+        speed = unicode(speed)
+        available_speed = [unicode(sp)
+                      for sp, lbl in settings.CHIMERE_ROUTING_SPEEDS[transport]]
+        if speed not in available_speed:
+            speed = None
+    if not speed:
+        speed = settings.CHIMERE_ROUTING_SPEEDS[transport][0][0]
+
     jsons, desc, total = router.route(lon1, lat1, lon2, lat2, steps=steps,
                                       transport=transport, speed=speed)
     if not jsons: