Merge pull request #4112 from AntonKhorev/no-float-inside-sidebar

[rails.git] / app / assets / javascripts / index / directions / graphhopper.js

diff --git a/app/assets/javascripts/index/directions/graphhopper.js b/app/assets/javascripts/index/directions/graphhopper.js
index c35085b35482fcbf336a7dd76c96522675affbb2..11d600316db8e5ddb978dc6d8011223f947d0c9a 100644 (file)
--- a/app/assets/javascripts/index/directions/graphhopper.js
+++ b/app/assets/javascripts/index/directions/graphhopper.js
@@ -1,41 +1,42 @@
 function GraphHopperEngine(id, vehicleType) {
   var GH_INSTR_MAP = {
 function GraphHopperEngine(id, vehicleType) {
   var GH_INSTR_MAP = {

-    "-3": 6, // sharp left
-    "-2": 7, // left
-    "-1": 8, // slight left
-    0: 0, // straight
-    1: 1, // slight right
-    2: 2, // right
-    3: 3, // sharp right
-    4: -1, // finish reached
-    5: -1, // via reached
-    6: 11 // roundabout
+    "-3": 7, // sharp left
+    "-2": 6, // left
+    "-1": 5, // slight left
+    "0": 0, // straight
+    "1": 1, // slight right
+    "2": 2, // right
+    "3": 3, // sharp right
+    "4": 14, // finish reached
+    "5": 14, // via reached
+    "6": 10 // roundabout

   };
 
   return {
     id: id,
   };
 
   return {
     id: id,

-    creditline: '<a href="https://graphhopper.com/" target="_blank">Graphhopper</a>',
+    creditline: "<a href=\"https://www.graphhopper.com/\" target=\"_blank\">GraphHopper</a>",

     draggable: false,
 
     getRoute: function (points, callback) {
       // GraphHopper Directions API documentation
     draggable: false,
 
     getRoute: function (points, callback) {
       // GraphHopper Directions API documentation

-      // https://github.com/graphhopper/directions-api/blob/master/docs-routing.md
+      // https://graphhopper.com/api/1/docs/routing/

       return $.ajax({
       return $.ajax({

-        url: document.location.protocol + OSM.GRAPHHOPPER_URL,
+        url: OSM.GRAPHHOPPER_URL,

         data: {
           vehicle: vehicleType,
           locale: I18n.currentLocale(),
           key: "LijBPDQGfu7Iiq80w3HzwB4RUDJbMbhs6BU0dEnn",
         data: {
           vehicle: vehicleType,
           locale: I18n.currentLocale(),
           key: "LijBPDQGfu7Iiq80w3HzwB4RUDJbMbhs6BU0dEnn",

-          type: "jsonp",

           elevation: false,
           instructions: true,
           elevation: false,
           instructions: true,

+          turn_costs: vehicleType === "car",

           point: points.map(function (p) { return p.lat + "," + p.lng; })
         },
         traditional: true,
           point: points.map(function (p) { return p.lat + "," + p.lng; })
         },
         traditional: true,

-        dataType: "jsonp",
+        dataType: "json",

         success: function (data) {
         success: function (data) {

-          if (!data.paths || data.paths.length === 0)
+          if (!data.paths || data.paths.length === 0) {

             return callback(true);
             return callback(true);

+          }

 
           var path = data.paths[0];
           var line = L.PolylineUtil.decode(path.points);
 
           var path = data.paths[0];
           var line = L.PolylineUtil.decode(path.points);


@@ -44,17 +45,21 @@ function GraphHopperEngine(id, vehicleType) {
           var len = path.instructions.length;
           for (var i = 0; i < len; i++) {
             var instr = path.instructions[i];
           var len = path.instructions.length;
           for (var i = 0; i < len; i++) {
             var instr = path.instructions[i];

-            var instrCode = (i === len - 1) ? 15 : GH_INSTR_MAP[instr.sign];
+            var instrCode = (i === len - 1) ? 14 : GH_INSTR_MAP[instr.sign];

             var instrText = "<b>" + (i + 1) + ".</b> ";
             instrText += instr.text;
             var latLng = line[instr.interval[0]];
             var distInMeter = instr.distance;
             var instrText = "<b>" + (i + 1) + ".</b> ";
             instrText += instr.text;
             var latLng = line[instr.interval[0]];
             var distInMeter = instr.distance;

+            var lineseg = [];
+            for (var j = instr.interval[0]; j <= instr.interval[1]; j++) {
+              lineseg.push({ lat: line[j][0], lng: line[j][1] });
+            }

             steps.push([
             steps.push([

-              {lat: latLng.lat, lng: latLng.lng},
+              { lat: latLng[0], lng: latLng[1] },

               instrCode,
               instrText,
               distInMeter,
               instrCode,
               instrText,
               distInMeter,

-              []
+              lineseg

             ]); // TODO does graphhopper map instructions onto line indices?
           }
 
             ]); // TODO does graphhopper map instructions onto line indices?
           }
 


@@ -62,7 +67,9 @@ function GraphHopperEngine(id, vehicleType) {
             line: line,
             steps: steps,
             distance: path.distance,
             line: line,
             steps: steps,
             distance: path.distance,

-            time: path.time / 1000
+            time: path.time / 1000,
+            ascend: path.ascend,
+            descend: path.descend

           });
         },
         error: function () {
           });
         },
         error: function () {


@@ -73,5 +80,6 @@ function GraphHopperEngine(id, vehicleType) {
   };
 }
 
   };
 }
 

+OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_car", "car"), true);

 OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_bicycle", "bike"), true);
 OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_foot", "foot"), true);
 OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_bicycle", "bike"), true);
 OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_foot", "foot"), true);