Merge remote-tracking branch 'upstream/pull/3767'

[rails.git] / app / assets / javascripts / router.js

diff --git a/app/assets/javascripts/router.js b/app/assets/javascripts/router.js
index a8659d308762024606b7f1aa0df85398839326a6..2e4b375d79691b0bd7253fce4bcf8671530465b9 100644 (file)
--- a/app/assets/javascripts/router.js
+++ b/app/assets/javascripts/router.js
@@ -47,13 +47,13 @@
    move the map without the hash changing.
  */
 OSM.Router = function (map, rts) {
    move the map without the hash changing.
  */
 OSM.Router = function (map, rts) {

-  var escapeRegExp = /[-{}[\]+?.,\\^$|#\s]/g;
-  var optionalParam = /\((.*?)\)/g;
-  var namedParam = /(\(\?)?:\w+/g;
-  var splatParam = /\*\w+/g;
+  const escapeRegExp = /[-{}[\]+?.,\\^$|#\s]/g;
+  const optionalParam = /\((.*?)\)/g;
+  const namedParam = /(\(\?)?:\w+/g;
+  const splatParam = /\*\w+/g;

 
   function Route(path, controller) {
 
   function Route(path, controller) {

-    var regexp = new RegExp("^" +
+    const regexp = new RegExp("^" +

       path.replace(escapeRegExp, "\\$&")
         .replace(optionalParam, "(?:$1)?")
         .replace(namedParam, function (match, optional) {
       path.replace(escapeRegExp, "\\$&")
         .replace(optionalParam, "(?:$1)?")
         .replace(namedParam, function (match, optional) {


@@ -61,14 +61,14 @@ OSM.Router = function (map, rts) {
         })
         .replace(splatParam, "(.*?)") + "(?:\\?.*)?$");
 
         })
         .replace(splatParam, "(.*?)") + "(?:\\?.*)?$");
 

-    var route = {};
+    const route = {};

 
     route.match = function (path) {
       return regexp.test(path);
     };
 
     route.run = function (action, path) {
 
     route.match = function (path) {
       return regexp.test(path);
     };
 
     route.run = function (action, path) {

-      var params = [];
+      let params = [];

 
       if (path) {
         params = regexp.exec(path).map(function (param, i) {
 
       if (path) {
         params = regexp.exec(path).map(function (param, i) {


@@ -84,26 +84,24 @@ OSM.Router = function (map, rts) {
     return route;
   }
 
     return route;
   }
 

-  var routes = [];
-  for (var r in rts) {
-    routes.push(new Route(r, rts[r]));
-  }
+  const routes = Object.entries(rts)
+    .map(([r, t]) => new Route(r, t));

 
   routes.recognize = function (path) {
 
   routes.recognize = function (path) {

-    for (var i = 0; i < this.length; i++) {
-      if (this[i].match(path)) return this[i];
+    for (const route of this) {
+      if (route.match(path)) return route;

     }
   };
 
     }
   };
 

-  var currentPath = window.location.pathname.replace(/(.)\/$/, "$1") + window.location.search,
+  let currentPath = window.location.pathname.replace(/(.)\/$/, "$1") + window.location.search,

       currentRoute = routes.recognize(currentPath),
       currentHash = location.hash || OSM.formatHash(map);
 
       currentRoute = routes.recognize(currentPath),
       currentHash = location.hash || OSM.formatHash(map);
 

-  var router = {};
+  const router = {};

 
   function updateSecondaryNav() {
     $("header nav.secondary > ul > li > a").each(function () {
 
   function updateSecondaryNav() {
     $("header nav.secondary > ul > li > a").each(function () {

-      var active = $(this).attr("href") === window.location.pathname;
+      const active = $(this).attr("href") === window.location.pathname;

 
       $(this)
         .toggleClass("text-secondary", !active)
 
       $(this)
         .toggleClass("text-secondary", !active)


@@ -113,8 +111,8 @@ OSM.Router = function (map, rts) {
 
   $(window).on("popstate", function (e) {
     if (!e.originalEvent.state) return; // Is it a real popstate event or just a hash change?
 
   $(window).on("popstate", function (e) {
     if (!e.originalEvent.state) return; // Is it a real popstate event or just a hash change?

-    var path = window.location.pathname + window.location.search,
-        route = routes.recognize(path);
+    const path = window.location.pathname + window.location.search,
+          route = routes.recognize(path);

     if (path === currentPath) return;
     currentRoute.run("unload", null, route === currentRoute);
     currentPath = path;
     if (path === currentPath) return;
     currentRoute.run("unload", null, route === currentRoute);
     currentPath = path;


@@ -125,11 +123,11 @@ OSM.Router = function (map, rts) {
   });
 
   router.route = function (url) {
   });
 
   router.route = function (url) {

-    var path = url.replace(/#.*/, ""),
-        route = routes.recognize(path);
+    const path = url.replace(/#.*/, ""),
+          route = routes.recognize(path);

     if (!route) return false;
     currentRoute.run("unload", null, route === currentRoute);
     if (!route) return false;
     currentRoute.run("unload", null, route === currentRoute);

-    var state = OSM.parseHash(url);
+    const state = OSM.parseHash(url);

     map.setState(state);
     window.history.pushState(state, document.title, url);
     currentPath = path;
     map.setState(state);
     window.history.pushState(state, document.title, url);
     currentPath = path;


@@ -144,25 +142,22 @@ OSM.Router = function (map, rts) {
   };
 
   router.stateChange = function (state) {
   };
 
   router.stateChange = function (state) {

-    if (state.center) {
-      window.history.replaceState(state, document.title, OSM.formatHash(state));
-    } else {
-      window.history.replaceState(state, document.title, window.location);
-    }
+    const url = state.center ? OSM.formatHash(state) : window.location;
+    window.history.replaceState(state, document.title, url);

   };
 
   router.updateHash = function () {
   };
 
   router.updateHash = function () {

-    var hash = OSM.formatHash(map);
+    const hash = OSM.formatHash(map);

     if (hash === currentHash) return;
     currentHash = hash;
     router.stateChange(OSM.parseHash(hash));
   };
 
   router.hashUpdated = function () {
     if (hash === currentHash) return;
     currentHash = hash;
     router.stateChange(OSM.parseHash(hash));
   };
 
   router.hashUpdated = function () {

-    var hash = location.hash;
+    const hash = location.hash;

     if (hash === currentHash) return;
     currentHash = hash;
     if (hash === currentHash) return;
     currentHash = hash;

-    var state = OSM.parseHash(hash);
+    const state = OSM.parseHash(hash);

     map.setState(state);
     router.stateChange(state, hash);
   };
     map.setState(state);
     router.stateChange(state, hash);
   };


@@ -181,7 +176,7 @@ OSM.Router = function (map, rts) {
   };
 
   router.load = function () {
   };
 
   router.load = function () {

-    var loadState = currentRoute.run("load", currentPath);
+    const loadState = currentRoute.run("load", currentPath);

     router.stateChange(loadState || {});
   };
 
     router.stateChange(loadState || {});
   };