Merge pull request #6 from pghalliday/peer-dependency

add chart.js to peer dependencies and require it in core

Author: jhudson8

lib/core.js

@@ -23,36 +23,35 @@ module.exports = {
       };
     }
 
-    if (global.Chart) {
-      classData.componentDidMount = function() {
-        this.initializeChart(this.props);
-      };
+    classData.componentDidMount = function() {
+      this.initializeChart(this.props);
+    };
 
-      classData.componentWillUnmount = function() {
-        var chart = this.state.chart;
-        chart.destroy();
-      };
+    classData.componentWillUnmount = function() {
+      var chart = this.state.chart;
+      chart.destroy();
+    };
 
-      classData.componentWillReceiveProps = function(nextProps) {
-        var chart = this.state.chart;
-        chart.destroy();
-        this.initializeChart(nextProps);
-      };
+    classData.componentWillReceiveProps = function(nextProps) {
+      var chart = this.state.chart;
+      chart.destroy();
+      this.initializeChart(nextProps);
+    };
 
-      classData.initializeChart = function(nextProps) {
-        var el = this.getDOMNode();
-        var ctx = el.getContext("2d");
-        var chart = new Chart(ctx)[chartType](nextProps.data, nextProps.options || {});
-        this.state.chart = chart;
-      };
+    classData.initializeChart = function(nextProps) {
+      var Chart = require('chart.js');
+      var el = this.getDOMNode();
+      var ctx = el.getContext("2d");
+      var chart = new Chart(ctx)[chartType](nextProps.data, nextProps.options || {});
+      this.state.chart = chart;
+    };
 
-      var i;
-      for (i=0; i<extras.length; i++) {
-        extra(extras[i]);
-      }
-      for (i=0; i<methodNames.length; i++) {
-        extra(methodNames[i]);
-      }
+    var i;
+    for (i=0; i<extras.length; i++) {
+      extra(extras[i]);
+    }
+    for (i=0; i<methodNames.length; i++) {
+      extra(methodNames[i]);
     }
 
     var React = require('react');

package.json

@@ -25,7 +25,8 @@
   },
   "homepage": "https://github.com/jhudson8/react-chartjs",
   "peerDependencies": {
-    "react": "*"
+    "react": "*",
+    "chart.js": "*"
   },
   "devDependencies": {
     "uglify-js": "^2.4.16",