Fix flyTo and flyToBounds crash on leaflet initialization (#552)

* use non compressed leaflet for easier debugging for everyone

* fit the world before flying if no bounds have been set

* undo leaflet-src change

* add news

* do not reset bounds if the map has already rendered

Author: schloerke

NEWS

@@ -11,6 +11,7 @@ BUG FIXES
 * Default marker icon locations will now use unpkg.com instead of the leaflet cdn when using https or file protocols. (#544)
 * `.leaflet-map-pane` `z-index` switched to 'auto'. Allows for map panes to appear above the map if they appear later in the dom. (#537)
 * Use correct Leaflet.js scale control remove method. (#547)
+* Start from world view if using flyTo or flyToBounds. (#552)
 
 
 leaflet 2.0.0

inst/htmlwidgets/leaflet.js

@@ -720,10 +720,18 @@ _htmlwidgets2.default.widget({
           methods.fitBounds.apply(map, data.fitBounds);
         }
         if (data.flyTo) {
+          if (!explicitView && !map.leafletr.hasRendered) {
+            // must be done to give a initial starting point
+            map.fitWorld();
+          }
           explicitView = true;
           map.flyTo.apply(map, data.flyTo);
         }
         if (data.flyToBounds) {
+          if (!explicitView && !map.leafletr.hasRendered) {
+            // must be done to give a initial starting point
+            map.fitWorld();
+          }
           explicitView = true;
           methods.flyToBounds.apply(map, data.flyToBounds);
         }

javascript/src/index.js

@@ -205,10 +205,18 @@ HTMLWidgets.widget({
           methods.fitBounds.apply(map, data.fitBounds);
         }
         if (data.flyTo) {
+          if (!explicitView && !map.leafletr.hasRendered) {
+            // must be done to give a initial starting point
+            map.fitWorld();
+          }
           explicitView = true;
           map.flyTo.apply(map, data.flyTo);
         }
         if (data.flyToBounds) {
+          if (!explicitView && !map.leafletr.hasRendered) {
+            // must be done to give a initial starting point
+            map.fitWorld();
+          }
           explicitView = true;
           methods.flyToBounds.apply(map, data.flyToBounds);
         }