bug fix

Author: glassonion1

copier.go

@@ -223,7 +223,7 @@ func copySlice(src, dst reflect.Value) error {
 }
 
 func setTimeField(src, dst reflect.Value) (bool, error) {
-	const format = "2006-01-02T15:04:05Z07:00"
+	const format = time.RFC3339Nano
 
 	switch t := src.Interface().(type) {
 	case time.Time:
@@ -302,10 +302,11 @@ func setTimeField(src, dst reflect.Value) (bool, error) {
 
 	case string:
 		// string -> time.Time or *time.Time
-		v, err := time.Parse(t, format)
+		v, err := time.Parse(format, t)
 		if err != nil {
 			return true, nil
 		}
+
 		switch dst.Interface().(type) {
 		case time.Time:
 			dst.Set(reflect.ValueOf(v))
@@ -322,7 +323,7 @@ func setTimeField(src, dst reflect.Value) (bool, error) {
 			return true, nil
 		}
 		// *string -> time.Time or *time.Time
-		v, err := time.Parse(*t, format)
+		v, err := time.Parse(format, *t)
 		if err != nil {
 			return true, nil
 		}

copier_test.go

@@ -231,14 +231,20 @@ func TestDeepCopy_time(t *testing.T) {
 		ReplacedAt *int64
 	}
 
+	type ModelE struct {
+		CreatedAt  string
+		UpdatedAt  *string
+		DeletedAt  string
+		ReplacedAt *string
+	}
+
 	type args struct {
 		src  interface{}
 		dest interface{}
 	}
 
-	//dt := time.Now()
-	//now := time.Unix(dt.Unix(), 0)
 	now := time.Now()
+	const format = time.RFC3339Nano
 
 	tests := []struct {
 		name string
@@ -299,6 +305,44 @@ func TestDeepCopy_time(t *testing.T) {
 			},
 			err: nil,
 		},
+		{
+			name: "time.Time to string",
+			in: args{
+				src: ModelC{
+					CreatedAt:  now,
+					UpdatedAt:  now,
+					DeletedAt:  &now,
+					ReplacedAt: &now,
+				},
+				dest: &ModelE{},
+			},
+			want: &ModelE{
+				CreatedAt:  now.Format(format),
+				UpdatedAt:  xgo.ToPtr(now.Format(format)),
+				DeletedAt:  now.Format(format),
+				ReplacedAt: xgo.ToPtr(now.Format(format)),
+			},
+			err: nil,
+		},
+		{
+			name: "string to time.Time",
+			in: args{
+				src: ModelE{
+					CreatedAt:  now.Format(format),
+					UpdatedAt:  xgo.ToPtr(now.Format(format)),
+					DeletedAt:  now.Format(format),
+					ReplacedAt: xgo.ToPtr(now.Format(format)),
+				},
+				dest: &ModelC{},
+			},
+			want: &ModelC{
+				CreatedAt:  now,
+				UpdatedAt:  now,
+				DeletedAt:  &now,
+				ReplacedAt: &now,
+			},
+			err: nil,
+		},
 	}
 
 	for _, tt := range tests {