Merge pull request #1 from erictse/master

Potential bug fix with prop updates not rerendering with new data
reactjs · Jan 12, 2015 · 554885a · 554885a
2 parents 42eed23 + ad79670
commit 554885a
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/vars.js b/vars.js
@@ -33,16 +33,16 @@ module.exports = {
         chart.destroy();
       };
 
-      classData.componentWillReceiveProps = function(props) {
+      classData.componentWillReceiveProps = function(nextProps) {
         var chart = this.state.chart;
         chart.destroy();
-        this.initializeChart(props);
+        this.initializeChart(nextProps);
       };
 
-      classData.initializeChart = function(props) {
+      classData.initializeChart = function(nextProps) {
         var el = this.getDOMNode();
         var ctx = el.getContext("2d");
-        var chart = new Chart(ctx)[chartType](this.props.data, this.props.options || {});
+        var chart = new Chart(ctx)[chartType](nextProps.data, nextProps.options || {});
         this.state.chart = chart;
       };