Merge pull request #376 from clauderic/fix-remove-listener-bug

Fix issues with removeEventListener when container is undefined
clauderic · May 18, 2018 · de05b98 · de05b98
2 parents 0716d75 + 9be29a9
commit de05b98
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/SortableContainer/index.js b/src/SortableContainer/index.js
@@ -134,11 +134,13 @@ export default function sortableContainer(WrappedComponent, config = {withRef: f
     }
 
     componentWillUnmount() {
-      for (const key in this.events) {
-        if (this.events.hasOwnProperty(key)) {
-          events[key].forEach(eventName =>
-            this.container.removeEventListener(eventName, this.events[key])
-          );
+      if (this.container) {
+        for (const key in this.events) {
+          if (this.events.hasOwnProperty(key)) {
+            events[key].forEach(eventName =>
+              this.container.removeEventListener(eventName, this.events[key])
+            );
+          }
         }
       }
     }