Merge pull request #135 from STRML/removeIsMounted

Remove isMounted()
BinaryMuse · Sep 17, 2015 · 6c4579e · 6c4579e
2 parents 0537787 + bc70bb9
commit 6c4579e
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/store_watch_mixin.js b/lib/store_watch_mixin.js
@@ -5,20 +5,24 @@ var StoreWatchMixin = function() {
   return {
     componentDidMount: function() {
       var flux = this.props.flux || this.context.flux;
+      this.mounted = true;
+      // No autobinding in ES6 classes
+      this._setStateFromFlux = this._setStateFromFlux.bind(this);
       _each(storeNames, function(store) {
         flux.store(store).on("change", this._setStateFromFlux);
       }, this);
     },
 
     componentWillUnmount: function() {
       var flux = this.props.flux || this.context.flux;
+      this.mounted = false;
       _each(storeNames, function(store) {
         flux.store(store).removeListener("change", this._setStateFromFlux);
       }, this);
     },
 
     _setStateFromFlux: function() {
-      if(this.isMounted()) {
+      if(this.mounted) {
         this.setState(this.getStateFromFlux());
       }
     },