Updated $firebaseUtils.deepCopy() to properly support Date object

Author: mcleanra

src/utils/utils.js

@@ -183,7 +183,7 @@
           },
 
           deepCopy: function(obj) {
-            if( !angular.isObject(obj) ) { return obj; }
+            if( !angular.isObject(obj) || angular.isDate(obj) ) { return obj; }
             var newCopy = angular.isArray(obj) ? obj.slice() : angular.extend({}, obj);
             for (var key in newCopy) {
               if (newCopy.hasOwnProperty(key)) {
@@ -395,7 +395,7 @@
     ]);
 
     function stripDollarPrefixedKeys(data) {
-      if( !angular.isObject(data) ) { return data; }
+      if( !angular.isObject(data) || angular.isDate(data)) { return data; }
       var out = angular.isArray(data)? [] : {};
       angular.forEach(data, function(v,k) {
         if(typeof k !== 'string' || k.charAt(0) !== '$') {

tests/unit/utils.spec.js

@@ -136,6 +136,33 @@ describe('$firebaseUtils', function () {
       expect(spy).toHaveBeenCalledWith('baz', 'biz');
     });
   });
+  
+  describe('#deepCopy', function() {
+    it('should work for empty objects', function() {
+      var obj = {};
+      expect($utils.deepCopy(obj)).toEqual(obj);
+    });
+    it('should work for primitives', function() {
+      var obj = 'foo';
+      expect($utils.deepCopy(obj)).toEqual(obj);
+    });
+    it('should work for dates', function() {
+      var obj = new Date();
+      expect($utils.deepCopy(obj)).toEqual(obj);
+    });
+    it('should work for nested objects', function() {
+      var d = new Date();
+      var obj = { date: {date: [{date: d}, {int: 1}, {str: "foo"}, {}]}};
+      expect($utils.deepCopy(obj)).toEqual(obj);
+    });
+    it('should work for functions', function() {
+      var f = function(){ 
+        var s = 'foo';
+      };
+      var obj = f;
+      expect($utils.deepCopy(obj)).toEqual(obj);
+    });
+  });
 
   describe('#updateRec', function() {
     it('should return true if changes applied', function() {
@@ -229,6 +256,13 @@ describe('$firebaseUtils', function () {
       var json = {foo: 'bar', $foo: '$bar'};
       expect($utils.toJSON(json)).toEqual({foo: json.foo});
     });
+    
+    it('should be able to handle date objects', function(){
+      var d = new Date();
+      var json = {date: d};
+
+      expect($utils.toJSON(json)).toEqual({date: d});
+    });
 
     it('should remove any deeply nested variables prefixed with $', function() {
       var json = {