X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/5f2a5cadcb693b34b81a9232b8879c3b4f12843d..4027d0c30abb5c50ac61785a57a8699de000ddd7:/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 e72f6d75f..191475873 100644
--- a/app/assets/javascripts/index/directions/graphhopper.js
+++ b/app/assets/javascripts/index/directions/graphhopper.js
@@ -1,90 +1,88 @@
-function GraphHopperEngine(id, vehicleType) {
-  var GH_INSTR_MAP = {
-    "-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
-    "-7": 19, // keep left
-    "7": 18, // keep right
-    "-98": 4, // unknown direction u-turn
-    "-8": 4, // left u-turn
-    "8": 4 // right u-turn
-  };
+(function () {
+  function GraphHopperEngine(id, vehicleType) {
+    const GH_INSTR_MAP = {
+      "-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
+      "-7": 19, // keep left
+      "7": 18, // keep right
+      "-98": 4, // unknown direction u-turn
+      "-8": 4, // left u-turn
+      "8": 4 // right u-turn
+    };
 
-  return {
-    id: id,
-    creditline: "<a href=\"https://www.graphhopper.com/\" target=\"_blank\">GraphHopper</a>",
-    draggable: false,
+    function _processDirections(path) {
+      const line = L.PolylineUtil.decode(path.points);
 
-    getRoute: function (points, callback) {
-      // GraphHopper Directions API documentation
-      // https://graphhopper.com/api/1/docs/routing/
-      return $.ajax({
-        url: OSM.GRAPHHOPPER_URL,
-        data: {
+      const steps = path.instructions.map(function (instr, i) {
+        const num = `<b>${i + 1}.</b> `;
+        const lineseg = line
+          .slice(instr.interval[0], instr.interval[1] + 1)
+          .map(([lat, lng]) => ({ lat, lng }));
+        return [
+          lineseg[0],
+          GH_INSTR_MAP[instr.sign],
+          num + instr.text,
+          instr.distance,
+          lineseg
+        ]; // TODO does graphhopper map instructions onto line indices?
+      });
+      steps.at(-1)[1] = 14;
+
+      return {
+        line: line,
+        steps: steps,
+        distance: path.distance,
+        time: path.time / 1000,
+        ascend: path.ascend,
+        descend: path.descend
+      };
+    }
+
+    return {
+      id: id,
+      creditline: "<a href=\"https://www.graphhopper.com/\" target=\"_blank\">GraphHopper</a>",
+      draggable: false,
+
+      getRoute: function (points, callback) {
+        // GraphHopper Directions API documentation
+        // https://graphhopper.com/api/1/docs/routing/
+        const data = {
           vehicle: vehicleType,
           locale: I18n.currentLocale(),
           key: "LijBPDQGfu7Iiq80w3HzwB4RUDJbMbhs6BU0dEnn",
           elevation: false,
           instructions: true,
           turn_costs: vehicleType === "car",
-          point: points.map(function (p) { return p.lat + "," + p.lng; })
-        },
-        traditional: true,
-        dataType: "json",
-        success: function (data) {
-          if (!data.paths || data.paths.length === 0) {
-            return callback(true);
-          }
-
-          var path = data.paths[0];
-          var line = L.PolylineUtil.decode(path.points);
-
-          var steps = [];
-          var len = path.instructions.length;
-          for (var i = 0; i < len; i++) {
-            var instr = path.instructions[i];
-            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 lineseg = [];
-            for (var j = instr.interval[0]; j <= instr.interval[1]; j++) {
-              lineseg.push({ lat: line[j][0], lng: line[j][1] });
+          point: points.map(p => p.lat + "," + p.lng)
+        };
+        return $.ajax({
+          url: OSM.GRAPHHOPPER_URL,
+          data,
+          traditional: true,
+          dataType: "json",
+          success: function ({ paths }) {
+            if (!paths || paths.length === 0) {
+              return callback(true);
             }
-            steps.push([
-              { lat: latLng[0], lng: latLng[1] },
-              instrCode,
-              instrText,
-              distInMeter,
-              lineseg
-            ]); // TODO does graphhopper map instructions onto line indices?
+            callback(false, _processDirections(paths[0]));
+          },
+          error: function () {
+            callback(true);
           }
+        });
+      }
+    };
+  }
 
-          callback(false, {
-            line: line,
-            steps: steps,
-            distance: path.distance,
-            time: path.time / 1000,
-            ascend: path.ascend,
-            descend: path.descend
-          });
-        },
-        error: function () {
-          callback(true);
-        }
-      });
-    }
-  };
-}
-
-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_car", "car"), true);
+  OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_bicycle", "bike"), true);
+  OSM.Directions.addEngine(new GraphHopperEngine("graphhopper_foot", "foot"), true);
+}());