Add check

unosquare · Sep 13, 2019 · e2e77af · e2e77af
1 parent 6874e1f
commit e2e77af
Showing 1 changed file with 20 additions and 3 deletions.
diff --git a/src/Swan.Lite/Extensions.cs b/src/Swan.Lite/Extensions.cs
@@ -67,7 +67,7 @@ public static int CopyOnlyPropertiesTo<T>(this T source, object target)
         /// <returns>
         /// Number of properties that were successfully copied.
         /// </returns>
-        public static int CopyOnlyPropertiesTo(this object source, object target, params string[] propertiesToCopy) 
+        public static int CopyOnlyPropertiesTo(this object source, object target, params string[] propertiesToCopy)
             => ObjectMapper.Copy(source, target, propertiesToCopy);
 
         /// <summary>
@@ -237,7 +237,7 @@ public static IEnumerable<string> GetCopyableProperties(this object @this)
 
             return PropertyTypeCache.DefaultCache.Value
                 .RetrieveAllProperties(@this.GetType(), true)
-                .Select(x => new { x.Name, HasAttribute = AttributeCache.DefaultCache.Value.RetrieveOne<CopyableAttribute>(x) != null})
+                .Select(x => new { x.Name, HasAttribute = AttributeCache.DefaultCache.Value.RetrieveOne<CopyableAttribute>(x) != null })
                 .Where(x => x.HasAttribute)
                 .Select(x => x.Name);
         }
@@ -259,7 +259,24 @@ public static IEnumerable<string> GetCopyableProperties(this object @this)
                         target = Array.CreateInstance(elementType, sourceObjectList.Count);
                     break;
                 default:
-                    target = Activator.CreateInstance(targetType, includeNonPublic);
+                    var ctors = targetType.GetConstructors()
+                        .Select(x => new { Ctor = x, Parameters = x.GetParameters() })
+                        .OrderBy(x => x.Parameters.Length);
+
+                    // Try to check if empty constructor is available
+                    if (ctors.Any(x => x.Parameters.Length == 0))
+                    {
+                        target = Activator.CreateInstance(targetType, includeNonPublic);
+                    }
+                    else
+                    {
+                        var firstCtor = ctors
+                            .OrderBy(x => x.Parameters.Length)
+                            .FirstOrDefault();
+
+                        target = Activator.CreateInstance(targetType, firstCtor?.Parameters.Select(arg => arg.GetType().GetDefault()).ToArray());
+                    }
+
                     break;
             }
         }