L.multipolyline doesn't exist (#515)

* L.polyline accepts either array of polylines or a single polyline * added news line
rstudio · Apr 6, 2018 · 8d54fd4 · 8d54fd4
1 parent 7e3b374
commit 8d54fd4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/NEWS b/NEWS
@@ -30,6 +30,8 @@ BREAKING CHANGES
 
 BUG FIXES AND FEATURES
 
+* `L.multiPolyline` was absorbed into `L.polyline`, which accepts multiple an array of polyline information. http://leafletjs.com/reference-1.3.0.html#polyline. (#515)
+
 * Fix bug where icons where anchored to the top-center, not center-center (2a60751)
 
 * Fix bug where markers would not appear in self contained knitr files (cc79bc3)

diff --git a/inst/htmlwidgets/leaflet.js b/inst/htmlwidgets/leaflet.js
@@ -1624,7 +1624,7 @@ methods.addPolylines = function (polygons, layerId, group, options, popup, popup
         return _htmlwidgets2.default.dataframeToD3(shape[0]);
       });
       if (shapes.length > 1) {
-        return _leaflet2.default.multiPolyline(shapes, df.get(i));
+        return _leaflet2.default.polyline(shapes, df.get(i));
       } else {
         return _leaflet2.default.polyline(shapes[0], df.get(i));
       }

diff --git a/javascript/src/methods.js b/javascript/src/methods.js
@@ -460,8 +460,8 @@ methods.addPolylines = function(polygons, layerId, group, options, popup, popupO
     addLayers(this, "shape", df, function(df, i) {
       let shapes = df.get(i, "shapes");
       shapes = shapes.map(shape => HTMLWidgets.dataframeToD3(shape[0]));
-      if(shapes.length>1) {
-        return L.multiPolyline(shapes, df.get(i));
+      if(shapes.length > 1) {
+        return L.polyline(shapes, df.get(i));
       } else {
         return L.polyline(shapes[0], df.get(i));
       }