fixed issue where date copy creates an object instead of date

mhevery · mhevery · commit 805753dba47f · 2010-10-13T12:47:10.000-07:00
diff --git a/regression/resource_json_date.html b/regression/resource_json_date.html
@@ -0,0 +1,30 @@
+<!DOCTYPE HTML>
+<html xmlns:ng="http://angularjs.org">
+<head>
+  <script type="text/javascript" src="../src/angular-bootstrap.js" ng:autobind></script>
+  </script>
+  <script type="text/javascript">
+    angular.service('myService', function($resource){
+      this.myData = $resource('resource_json_date.json');
+    }, {$inject:['$resource'], $creation:'eager'});
+
+    /* The Controller object */
+    MyController = function() {
+      this.inlineData = angular.fromJson('{reportDate:"2010-10-13T17:37:00Z"}');
+      this.jsonData = this.myData.get();
+    };
+
+
+
+  </script>
+</head>
+<body ng:controller="MyController" ng:init="$window.$root = this">
+  <h3>This data is loaded with angular.fromJson:</h3>
+  {{ inlineData.reportDate | date }}
+  <hr/>
+  <h3>This data is loaded from a resource using a service:</h3>
+  <p>Name: {{ jsonData.name }}</p>
+  <p>Parsed date: {{ jsonData.reportDate }} (A date should be displayed here)</p>
+
+</body>
+</html>
diff --git a/regression/resource_json_date.json b/regression/resource_json_date.json
@@ -0,0 +1 @@
+{reportDate:"2010-10-13T17:37:00Z", name:"camilo"}
diff --git a/src/Angular.js b/src/Angular.js
@@ -226,14 +226,16 @@ function isLeafNode (node) {
  */
 function copy(source, destination){
   if (!destination) {
+    destination = source;
     if (source) {
       if (isArray(source)) {
-        return copy(source, []);
+        destination = copy(source, []);
+      } else if (source instanceof Date) {
+        destination = new Date(source.getTime());
       } else if (isObject(source)) {
-        return copy(source, {});
+        destination = copy(source, {});
       }
     }
-    return source;
   } else {
     if (isArray(source)) {
       while(destination.length) {
@@ -250,8 +252,8 @@ function copy(source, destination){
         destination[key] = copy(source[key]);
       }
     }
-    return destination;
   }
+  return destination;
 }
 
 function equals(o1, o2) {
diff --git a/test/AngularSpec.js b/test/AngularSpec.js
@@ -21,6 +21,13 @@ describe("copy", function(){
     assertSame(arr, copy([], arr));
   });
 
+  it("should copy Date", function(){
+    var date = new Date(123);
+    expect(copy(date) instanceof Date).toBeTruthy();
+    expect(copy(date).getTime()).toEqual(123);
+    expect(copy(date) === date).toBeFalsy();
+  });
+
   it("should copy array", function(){
     var src = [1, {name:"value"}];
     var dst = [{key:"v"}];

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+{reportDate:"2010-10-13T17:37:00Z", name:"camilo"}`
Original file line number	Diff line number	Diff line change
`@@ -226,14 +226,16 @@ function isLeafNode (node) {`
`226`	`226`	`*/`
`227`	`227`	`function copy(source, destination){`
`228`	`228`	`if (!destination) {`
	`229`	`+ destination = source;`
`229`	`230`	`if (source) {`
`230`	`231`	`if (isArray(source)) {`
`231`		`- return copy(source, []);`
	`232`	`+ destination = copy(source, []);`
	`233`	`+ } else if (source instanceof Date) {`
	`234`	`+ destination = new Date(source.getTime());`
`232`	`235`	`} else if (isObject(source)) {`
`233`		`- return copy(source, {});`
	`236`	`+ destination = copy(source, {});`
`234`	`237`	`}`
`235`	`238`	`}`
`236`		`- return source;`
`237`	`239`	`} else {`
`238`	`240`	`if (isArray(source)) {`
`239`	`241`	`while(destination.length) {`
`@@ -250,8 +252,8 @@ function copy(source, destination){`
`250`	`252`	`destination[key] = copy(source[key]);`
`251`	`253`	`}`
`252`	`254`	`}`
`253`		`- return destination;`
`254`	`255`	`}`
	`256`	`+ return destination;`
`255`	`257`	`}`
`256`	`258`
`257`	`259`	`function equals(o1, o2) {`