small refactorings

.eslintrc

@@ -2,6 +2,7 @@
   "extends": ["airbnb-base", "prettier"],
   "rules": {
     "no-underscore-dangle": 0,
+    "import/prefer-default-export": 0,
     "keyword-spacing": 0,
     "no-param-reassign": 0,
     "no-restricted-syntax": 0,

package.json

@@ -29,7 +29,7 @@
     "@babel/preset-env": "^7.3.1",
     "babel-loader": "^8.0.5",
     "css-loader": "^0.28.11",
-    "cypress": "^3.1.4",
+    "cypress": "^3.1.5",
     "eslint": "^4.19.1",
     "eslint-config-airbnb-base": "^12.1.0",
     "eslint-config-prettier": "^3.6.0",

src/js/Edit/L.PM.Edit.Line.js

@@ -2,6 +2,7 @@ import kinks from '@turf/kinks';
 import get from 'lodash/get';
 import Edit from './L.PM.Edit';
 import Utils from '../L.PM.Utils';
+import { isEmptyDeep } from '../helpers';
 
 // Shit's getting complicated in here with Multipolygon Support. So here's a quick note about it:
 // Multipolygons with holes means lots of nested, multidimensional arrays.
@@ -23,6 +24,8 @@ Edit.Line = Edit.extend({
     } else {
       this.disable();
     }
+
+    return this.enabled();
   },
 
   enable(options) {
@@ -395,7 +398,7 @@ Edit.Line = Edit.extend({
     // TODO: we may should remove all empty coord-rings here as well.
 
     // if no coords are left, remove the layer
-    if (this.isEmptyDeep(coords)) {
+    if (isEmptyDeep(coords)) {
       this._layer.remove();
     }
 
@@ -445,15 +448,6 @@ Edit.Line = Edit.extend({
       // TODO: maybe add latlng as well?
     });
   },
-  isEmptyDeep(l) {
-    // thanks for the function, Felix Heck
-    const flatten = list =>
-      list
-        .filter(x => ![null, '', undefined].includes(x))
-        .reduce((a, b) => a.concat(Array.isArray(b) ? flatten(b) : b), []);
-
-    return !flatten(l).length;
-  },
   findDeepMarkerIndex(arr, marker) {
     // thanks for the function, Felix Heck
     let result;

src/js/helpers/index.js

@@ -0,0 +1,9 @@
+export function isEmptyDeep(l) {
+  // thanks for the function, Felix Heck
+  const flatten = list =>
+    list
+      .filter(x => ![null, '', undefined].includes(x))
+      .reduce((a, b) => a.concat(Array.isArray(b) ? flatten(b) : b), []);
+
+  return !flatten(l).length;
+}