toGeoJSON() should still work if no values in coords array (#8737)

Leaflet · Dec 22, 2022 · f42cab9 · f42cab9
1 parent 11049bc
commit f42cab9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/spec/suites/layer/GeoJSONSpec.js b/spec/suites/layer/GeoJSONSpec.js
@@ -889,6 +889,11 @@ describe("L.GeoJSON functions", function () {
 			expect(coords).to.eql([[1, 2, 3], [4, 5, 6], [1, 2, 3]]);
 			expect(coords[0] === coords[2]).to.not.ok();
 		});
+		it('still works if no values in coords array', () => {
+			expect(function () {
+				L.GeoJSON.latLngsToCoords([[]], 1, true);
+			}).to.not.throwException();
+		});
 	});
 
 	describe("#asFeature", function () {

diff --git a/src/layer/GeoJSON.js b/src/layer/GeoJSON.js
@@ -288,7 +288,7 @@ export function latLngsToCoords(latlngs, levelsDeep, closed, precision) {
 			latLngToCoords(latlngs[i], precision));
 	}
 
-	if (!levelsDeep && closed) {
+	if (!levelsDeep && closed && coords.length > 0) {
 		coords.push(coords[0].slice());
 	}