]> git.openstreetmap.org Git - rails.git/commitdiff
Fix new eslint warnings
authorTom Hughes <tom@compton.nu>
Sun, 14 Apr 2019 18:39:40 +0000 (19:39 +0100)
committerTom Hughes <tom@compton.nu>
Wed, 12 Jun 2019 18:27:58 +0000 (19:27 +0100)
app/assets/javascripts/leaflet.zoom.js
app/assets/javascripts/router.js

index e7c048b2d933c568f8eacfa5c7fcd987c85f9f7b..342993347ccdfc206882c1f20b80a241929df8bd 100644 (file)
@@ -1,72 +1,72 @@
 L.OSM.Zoom = L.Control.extend({
-       options: {
-               position: "topright"
-       },
+        options: {
+                position: "topright"
+        },
 
-       onAdd: function (map) {
-               var zoomName = "zoom",
-                   container = L.DomUtil.create("div", zoomName);
+        onAdd: function (map) {
+                var zoomName = "zoom",
+                    container = L.DomUtil.create("div", zoomName);
 
-               this._map = map;
+                this._map = map;
 
-               this._zoomInButton = this._createButton(
-                       "", I18n.t("javascripts.map.zoom.in"), zoomName + "in", container, this._zoomIn, this);
-               this._zoomOutButton = this._createButton(
-                       "", I18n.t("javascripts.map.zoom.out"), zoomName + "out", container, this._zoomOut, this);
+                this._zoomInButton = this._createButton(
+                        "", I18n.t("javascripts.map.zoom.in"), zoomName + "in", container, this._zoomIn, this);
+                this._zoomOutButton = this._createButton(
+                        "", I18n.t("javascripts.map.zoom.out"), zoomName + "out", container, this._zoomOut, this);
 
-               map.on("zoomend zoomlevelschange", this._updateDisabled, this);
+                map.on("zoomend zoomlevelschange", this._updateDisabled, this);
 
-               return container;
-       },
+                return container;
+        },
 
-       onRemove: function (map) {
-               map.off("zoomend zoomlevelschange", this._updateDisabled, this);
-       },
+        onRemove: function (map) {
+                map.off("zoomend zoomlevelschange", this._updateDisabled, this);
+        },
 
-       _zoomIn: function (e) {
-               this._map.zoomIn(e.shiftKey ? 3 : 1);
-       },
+        _zoomIn: function (e) {
+                this._map.zoomIn(e.shiftKey ? 3 : 1);
+        },
 
-       _zoomOut: function (e) {
-               this._map.zoomOut(e.shiftKey ? 3 : 1);
-       },
+        _zoomOut: function (e) {
+                this._map.zoomOut(e.shiftKey ? 3 : 1);
+        },
 
-       _createButton: function (html, title, className, container, fn, context) {
-               var link = L.DomUtil.create("a", "control-button " + className, container);
-               link.innerHTML = html;
-               link.href = "#";
-               link.title = title;
+        _createButton: function (html, title, className, container, fn, context) {
+                var link = L.DomUtil.create("a", "control-button " + className, container);
+                link.innerHTML = html;
+                link.href = "#";
+                link.title = title;
 
-               L.DomUtil.create("span", "icon " + className, link);
+                L.DomUtil.create("span", "icon " + className, link);
 
-               var stop = L.DomEvent.stopPropagation;
+                var stop = L.DomEvent.stopPropagation;
 
-               L.DomEvent
-                   .on(link, "click", stop)
-                   .on(link, "mousedown", stop)
-                   .on(link, "dblclick", stop)
-                   .on(link, "click", L.DomEvent.preventDefault)
-                   .on(link, "click", fn, context);
+                L.DomEvent
+                    .on(link, "click", stop)
+                    .on(link, "mousedown", stop)
+                    .on(link, "dblclick", stop)
+                    .on(link, "click", L.DomEvent.preventDefault)
+                    .on(link, "click", fn, context);
 
-               return link;
-       },
+                return link;
+        },
 
-       _updateDisabled: function () {
-               var map = this._map,
-                       className = "disabled";
+        _updateDisabled: function () {
+                var map = this._map,
+                        className = "disabled";
 
-               L.DomUtil.removeClass(this._zoomInButton, className);
-               L.DomUtil.removeClass(this._zoomOutButton, className);
+                L.DomUtil.removeClass(this._zoomInButton, className);
+                L.DomUtil.removeClass(this._zoomOutButton, className);
 
-               if (map._zoom === map.getMinZoom()) {
-                       L.DomUtil.addClass(this._zoomOutButton, className);
-               }
-               if (map._zoom === map.getMaxZoom()) {
-                       L.DomUtil.addClass(this._zoomInButton, className);
-               }
-       }
+                if (map._zoom === map.getMinZoom()) {
+                        L.DomUtil.addClass(this._zoomOutButton, className);
+                }
+                if (map._zoom === map.getMaxZoom()) {
+                        L.DomUtil.addClass(this._zoomInButton, className);
+                }
+        }
 });
 
 L.OSM.zoom = function (options) {
-       return new L.OSM.Zoom(options);
+        return new L.OSM.Zoom(options);
 };
index 08a6c7268579f65ca341de933c02ca66c2336964..a2a20b8469031d4753e5d5dc8f299303b4a6dfc3 100644 (file)
@@ -47,7 +47,7 @@
    move the map without the hash changing.
  */
 OSM.Router = function (map, rts) {
-  var escapeRegExp  = /[\-{}\[\]+?.,\\\^$|#\s]/g;
+  var escapeRegExp  = /[-{}[\]+?.,\\^$|#\s]/g;
   var optionalParam = /\((.*?)\)/g;
   var namedParam    = /(\(\?)?:\w+/g;
   var splatParam    = /\*\w+/g;
@@ -57,7 +57,7 @@ OSM.Router = function (map, rts) {
       path.replace(escapeRegExp, "\\$&")
         .replace(optionalParam, "(?:$1)?")
         .replace(namedParam, function (match, optional) {
-          return optional ? match : "([^\/]+)";
+          return optional ? match : "([^/]+)";
         })
         .replace(splatParam, "(.*?)") + "(?:\\?.*)?$");