[fix] enforce sideffects for event listeners registration

Author: smee

bower.json

@@ -10,8 +10,5 @@
   },
   "devDependencies": {
     "angular-mocks": "latest"
-  },
-  "resolutions": {
-    "crossfilter": "~1.3.0"
   }
 }

dist/angular-dc.js

@@ -1,8 +1,8 @@
 (function(root, factory) {
     if (typeof define === 'function' && define.amd) {
         // AMD. Register as an anonymous module unless amdModuleId is set
-        define(['angular', 'dc', 'lodash', 'd3'], function(angular, dc, lodash, d3) {
-            return (root['angularDc'] = factory(angular, dc, lodash, d3));
+        define(['angular', 'dc', 'lodash', 'd3'], function(a0, b1, c2, d3) {
+            return (root['angularDc'] = factory(a0, b1, c2, d3));
         });
     } else if (typeof exports === 'object') {
         // Node. Does not work with strict CommonJS, but
@@ -89,6 +89,8 @@
                 eventHandlers.each(function(handler, evt) {
                     chart.on(evt, handler);
                 });
+                // run the chain to enforce side effects (registration of handlers), ignore the result
+                eventHandlers.value();
                 // Run the postSetupChart callback, if provided
                 if (_.isFunction(options.postSetupChart)) {
                     options.postSetupChart(chart, options);

dist/angular-dc.min.js

@@ -87,7 +87,8 @@ angularDc.directive('dcChart', ['$timeout',
             eventHandlers.each(function(handler, evt) {
                 chart.on(evt, handler);
             });
-
+            // run the chain to enforce side effects (registration of handlers), ignore the result
+            eventHandlers.value();
             // Run the postSetupChart callback, if provided
             if (_.isFunction(options.postSetupChart)) {
                 options.postSetupChart(chart, options);