fix(select): defensive foundation check before unmounting - also incl…

…udes radio, checkbox, text-field (#512)
material-components · Dec 12, 2018 · 2ee4c80 · 2ee4c80
1 parent 179b865
commit 2ee4c80
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 6 deletions.
diff --git a/packages/checkbox/index.js b/packages/checkbox/index.js
@@ -69,7 +69,9 @@ export class Checkbox extends Component {
   }
 
   componentWillUnmount() {
-    this.foundation_.destroy();
+    if (this.foundation_) {
+      this.foundation_.destroy();
+    }
   }
 
   init = (el) => {

diff --git a/packages/radio/index.js b/packages/radio/index.js
@@ -59,7 +59,9 @@ class Radio extends React.Component {
   }
 
   componentWillUnmount() {
-    this.foundation_.destroy();
+    if (this.foundation_) {
+      this.foundation_.destroy();
+    }
   }
 
   componentDidUpdate(prevProps, prevState) {

diff --git a/packages/select/index.js b/packages/select/index.js
@@ -72,7 +72,9 @@ export default class Select extends React.Component {
   }
 
   componentWillUnmount() {
-    this.foundation_.destroy();
+    if (this.foundation_) {
+      this.foundation_.destroy();
+    }
   }
 
   onChange = (evt) => {

diff --git a/packages/text-field/helper-text/index.js b/packages/text-field/helper-text/index.js
@@ -66,7 +66,9 @@ export default class HelperText extends React.Component {
   }
 
   componentWillUnmount() {
-    this.foundation_.destroy();
+    if (this.foundation_) {
+      this.foundation_.destroy();
+    }
   }
 
   get classes() {

diff --git a/packages/text-field/icon/index.js b/packages/text-field/icon/index.js
@@ -57,7 +57,9 @@ export default class Icon extends React.Component {
   }
 
   componentWillUnmount() {
-    this.foundation_.destroy();
+    if (this.foundation_) {
+      this.foundation_.destroy();
+    }
   }
 
   get adapter() {

diff --git a/packages/text-field/index.js b/packages/text-field/index.js
@@ -79,7 +79,9 @@ class TextField extends React.Component {
   }
 
   componentWillUnmount() {
-    this.state.foundation.destroy();
+    if (this.state.foundation) {
+      this.state.foundation.destroy();
+    }
   }
 
   /**