Fix support for converting to and from DateOnly

Author: rhuijben

RepoDb.Core/RepoDb/Extensions/DbCommandExtension.cs

@@ -885,6 +885,12 @@ private static object AutomaticConvert(object value,
             {
                 return AutomaticConvertGuidToString(value);
             }
+#if NET6_0_OR_GREATER
+            else if (fromType == StaticType.DateOnly && targetType == StaticType.DateTime)
+            {
+                return AutomaticConvertDateOnlyToDateTime(value);
+            }
+#endif
             else
             {
                 return (value != DBNull.Value) ? Convert.ChangeType(value, targetType) : Activator.CreateInstance(targetType);
@@ -915,6 +921,10 @@ private static object AutomaticConvertStringToGuid(object value)
         private static object AutomaticConvertGuidToString(object value) =>
             value?.ToString();
 
+#if NET6_0_OR_GREATER
+        private static object AutomaticConvertDateOnlyToDateTime(object value) =>
+            (value is DateOnly dateOnly ? dateOnly.ToDateTime(default(TimeOnly)) : null);
+#endif
         #endregion
     }
 }

RepoDb.Core/RepoDb/Extensions/ExpressionExtension.cs

@@ -382,7 +382,7 @@ public static object GetValue(this NewExpression expression)
             if (expression.Arguments.Count > 0)
             {
                 return Activator.CreateInstance(expression.Type,
-                    expression.Arguments.Select(arg => arg.GetValue()));
+                    expression.Arguments.Select(arg => arg.GetValue()).ToArray());
             }
             else
             {

RepoDb.Core/RepoDb/Reflection/Compiler/Compiler.cs

@@ -1,15 +1,15 @@
+using RepoDb.Enumerations;
+using RepoDb.Exceptions;
+using RepoDb.Extensions;
+using RepoDb.Interfaces;
+using RepoDb.Resolvers;
 using System;
 using System.Collections.Generic;
 using System.Data;
 using System.Data.Common;
 using System.Linq;
 using System.Linq.Expressions;
 using System.Reflection;
-using RepoDb.Enumerations;
-using RepoDb.Exceptions;
-using RepoDb.Extensions;
-using RepoDb.Interfaces;
-using RepoDb.Resolvers;
 
 namespace RepoDb.Reflection
 {
@@ -510,7 +510,15 @@ internal static MethodInfo GetDateTimeTimeOfDayPropertyGetMethod() =>
         /// <returns></returns>
         internal static MethodInfo GetDateOnlyFromDateTimeStaticMethod() =>
             StaticType.DateOnly.GetMethod("FromDateTime");
+
+        /// <summary>
+        ///
+        /// </summary>
+        /// <returns></returns>
+        internal static MethodInfo GetDateTimeFromDateOnlyMethod() =>
+            StaticType.DateOnly.GetMethod("ToDateTime", new Type[] { StaticType.TimeOnly });
 #endif
+
         /// <summary>
         ///
         /// </summary>
@@ -548,6 +556,24 @@ internal static Expression ConvertExpressionToNullableGetValueOrDefaultExpressio
             return expression;
         }
 
+#if NET6_0_OR_GREATER
+        internal static Expression ConvertExpressionToNullableGetValueOrDefaultExpression(Func<Expression, Expression> converter, Expression expression)
+        {
+            if (Nullable.GetUnderlyingType(expression.Type) != null)
+            {
+                var converted = converter(ConvertExpressionToNullableGetValueOrDefaultExpression(expression));
+                var nullableType = typeof(Nullable<>).MakeGenericType(converted.Type);
+                return Expression.Condition(
+                    Expression.Property(expression, nameof(Nullable<int>.HasValue)),
+                    Expression.Convert(converted, nullableType),
+                    Expression.Constant(null, nullableType)
+                );
+            }
+
+            return converter(expression);
+        }
+#endif
+
         /// <summary>
         ///
         /// </summary>
@@ -601,7 +627,15 @@ internal static Expression ConvertExpressionToDateTimeToTimeSpanExpression(Expre
         /// <param name="expression"></param>
         /// <returns></returns>
         internal static Expression ConvertExpressionToDateTimeToDateOnlyExpression(Expression expression) =>
-            ConvertExpressionToNullableGetValueOrDefaultExpression(ConvertExpressionToDateOnlyFromDateTimeExpression(expression));
+            ConvertExpressionToNullableGetValueOrDefaultExpression(ConvertExpressionToDateTimeFromDateOnlyExpression, expression);
+
+        /// <summary>
+        ///
+        /// </summary>
+        /// <param name="expression"></param>
+        /// <returns></returns>
+        internal static Expression ConvertExpressionToDateOnlyToDateTimeExpression(Expression expression) =>
+            ConvertExpressionToNullableGetValueOrDefaultExpression(ConvertExpressionToDateOnlyFromDateTimeExpression, expression);
 #endif
         /// <summary>
         ///
@@ -625,6 +659,14 @@ internal static Expression ConvertExpressionToDateTimeTimeOfDayExpression(Expres
         /// <param name="expression"></param>
         /// <returns></returns>
         internal static Expression ConvertExpressionToDateOnlyFromDateTimeExpression(Expression expression) =>
+            Expression.Call(expression, GetDateTimeFromDateOnlyMethod(), Expression.Constant(default(TimeOnly)));
+
+        /// <summary>
+        ///
+        /// </summary>
+        /// <param name="expression"></param>
+        /// <returns></returns>
+        internal static Expression ConvertExpressionToDateTimeFromDateOnlyExpression(Expression expression) =>
             Expression.Call(null, GetDateOnlyFromDateTimeStaticMethod(), expression);
 #endif
         /// <summary>
@@ -907,6 +949,12 @@ internal static Expression ConvertExpressionWithAutomaticConversion(Expression e
             {
                 expression = ConvertExpressionToDateTimeToDateOnlyExpression(expression);
             }
+
+            // DateOnly to DateTime
+            else if (fromType == StaticType.DateOnly && toType == StaticType.DateTime)
+            {
+                expression = ConvertExpressionToDateOnlyToDateTimeExpression(expression);
+            }
 #endif
             // Others
             else
@@ -1084,7 +1132,7 @@ internal static Expression ConvertExpressionToClassHandlerSetExpression(Expressi
             return entityOrEntitiesExpression;
         }
 
-#endregion
+        #endregion
 
         #region Common
 

RepoDb.Core/RepoDb/StaticType.cs

@@ -95,8 +95,8 @@ internal static class StaticType
         /// Gets a type of the <see cref="System.DateTimeOffset"/> .NET CLR type.
         /// </summary>
         public static Type DateTimeOffset => typeof(DateTimeOffset);
-        
-        #if NET6_0_OR_GREATER
+
+#if NET6_0_OR_GREATER
         /// <summary>
         /// Gets a type of the <see cref="System.DateOnly"/> .NET CLR type.
         /// </summary>
@@ -105,8 +105,8 @@ internal static class StaticType
         /// Gets a type of the <see cref="System.TimeOnly"/> .NET CLR type.
         /// </summary>
         public static Type TimeOnly => typeof(TimeOnly);
-        #endif
-        
+#endif
+
         /// <summary>
         /// Gets a type of the <see cref="System.Data.Common.DbCommand"/> .NET CLR type.
         /// </summary>