fix broken mouse events within shiny apps caused by leaflet-search in leaflet.extras (#563)

* trim latLng that value is actually retrieved. Do not allow non standard values to propagate

* added news

Author: schloerke

NEWS

@@ -3,6 +3,9 @@ leaflet 2.0.1.9000
 
 BUG FIXES and IMPROVEMENTS
 * Require viridis >= 0.5.1 to avoid namespace issues with viridisLite (#557)
+* Fixed broken mouse events after using leaflet-search from leaflet.extras within shiny applications (#563)
+
+
 
 leaflet 2.0.1
 --------------------------------------------------------------------------------

inst/htmlwidgets/leaflet.js

@@ -1287,10 +1287,18 @@ function mouseHandler(mapId, layerId, group, eventName, extraInfo) {
   return function (e) {
     if (!_htmlwidgets2.default.shinyMode) return;
 
+    var latLng = e.target.getLatLng ? e.target.getLatLng() : e.latlng;
+    if (latLng) {
+      // retrieve only lat, lon values to remove prototype
+      //   and extra parameters added by 3rd party modules
+      // these objects are for json serialization, not javascript
+      var latLngVal = _leaflet2.default.latLng(latLng); // make sure it has consistent shape
+      latLng = { lat: latLngVal.lat, lon: latLngVal.lon };
+    }
     var eventInfo = _jquery2.default.extend({
       id: layerId,
       ".nonce": Math.random() // force reactivity
-    }, group !== null ? { group: group } : null, e.target.getLatLng ? e.target.getLatLng() : e.latlng, extraInfo);
+    }, group !== null ? { group: group } : null, latLng, extraInfo);
 
     _shiny2.default.onInputChange(mapId + "_" + eventName, eventInfo);
   };

javascript/src/methods.js

@@ -18,13 +18,21 @@ function mouseHandler(mapId, layerId, group, eventName, extraInfo) {
   return function(e) {
     if (!HTMLWidgets.shinyMode) return;
 
+    let latLng = e.target.getLatLng ? e.target.getLatLng() : e.latlng;
+    if (latLng) {
+      // retrieve only lat, lon values to remove prototype
+      //   and extra parameters added by 3rd party modules
+      // these objects are for json serialization, not javascript
+      let latLngVal = L.latLng(latLng); // make sure it has consistent shape
+      latLng = {lat: latLngVal.lat, lon: latLngVal.lon};
+    }
     let eventInfo = $.extend(
       {
         id: layerId,
         ".nonce": Math.random()  // force reactivity
       },
       group !== null ? {group: group} : null,
-      e.target.getLatLng ? e.target.getLatLng() : e.latlng,
+      latLng,
       extraInfo
     );