Issue#128: Added fix to have reference equality check for datarows.

src/directives/hotTable.js

@@ -117,14 +117,26 @@
                 } else {
                   scope.hotInstance.loadData(newValue);
                   scope.htSettings.data = newValue;
-
                 }
               } else if (newValue !== oldValue) {
                 scope.htSettings[key] = newValue;
                 settingFactory.updateHandsontableSettings(scope.hotInstance, scope.htSettings);
               }
             }, ['datarows', 'columns', 'rowHeights', 'colWidths', 'rowHeaders', 'colHeaders'].indexOf(key) >= 0);
           });
+
+          /**
+           * Check for reference equality changes for datarows
+           * TODO: must the remaining bindingsKeys need to be added also if their reference changes
+           */
+          scope.$watch('datarows', function(newValue) {
+            if (newValue === void 0) {
+              return;
+            }
+            if (scope.hotInstance.getSettings().data !== newValue) {
+              scope.hotInstance.loadData(newValue);
+            }
+          });
 
           /**
            * Check if data length has been changed

test/directives/hotTable/watchingOptions.spec.js

@@ -20,9 +20,29 @@ describe('hotTable - Watching options', function() {
 
     scope.$digest();
 
-    expect(scope.hotInstance.getData()).toBe(rootScope.value);
+    expect(scope.hotInstance.getSettings().data).toBe(rootScope.value);
   });
+
+  it('should ensure a new copy of `datarows` is made when the reference is changed but it has the same value', function() {
+    rootScope.value = [[1,2,3,4]];
+    var scope = angular.element(compile('<hot-table datarows="value"></hot-table>')(rootScope)).isolateScope();
+
+    scope.$digest();
+
+    expect(scope.hotInstance.getSettings().data).toBe(rootScope.value);
+
+    rootScope.value = [[1, 2, 3, 4]];
 
+    rootScope.$digest();
+
+    expect(scope.hotInstance.getSettings().data).toBe(rootScope.value);
+
+    rootScope.value.push([[5, 6, 7, 8]]);
+    scope.$digest();
+
+    expect(scope.hotInstance.getSettings().data).toBe(rootScope.value);
+  });
+
   it('should create table with `dataSchema` attribute', function() {
     rootScope.value = {id: null};
     var scope = angular.element(compile('<hot-table dataschema="value"></hot-table>')(rootScope)).isolateScope();
@@ -831,4 +851,4 @@ describe('hotTable - Watching options', function() {
     expect(scope.hotInstance.getSettings().className).toBe('foo');
     expect(scope.hotInstance.getSettings().data).toEqual([[1]]);
   });
-});
+});