Fixes nextProps and nextState to observe when component updates. Otherwise, when componentWillUpdate is triggered, the observe function gets called and the nextProps and nextState gets dropped even if changed

boopathi · boopathi · commit 7d2dc7b42875 · 2015-04-28T02:32:29.000+05:30
diff --git a/lib/Mixin.js b/lib/Mixin.js
@@ -52,7 +52,7 @@ var Mixin = {
   componentWillUpdate: function componentWillUpdate(nextProps, nextState) {
     // only subscribe if props or state changed
     if (nextProps !== this.props || nextState !== this.state) {
-      this._subscribe();
+      this._subscribe(nextProps, nextState);
     }
   },
 

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ var Mixin = {`
`52`	`52`	`componentWillUpdate: function componentWillUpdate(nextProps, nextState) {`
`53`	`53`	`// only subscribe if props or state changed`
`54`	`54`	`if (nextProps !== this.props \|\| nextState !== this.state) {`
`55`		`- this._subscribe();`
	`55`	`+ this._subscribe(nextProps, nextState);`
`56`	`56`	`}`
`57`	`57`	`},`
`58`	`58`