Explicitly hande DateOnly in HttpParameterFormatter #341 (#342)

DarkWanderer · Jul 25, 2023 · a1b0daa · a1b0daa
1 parent fb5bb78
commit a1b0daa
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/ClickHouse.Client/Formats/HttpParameterFormatter.cs b/ClickHouse.Client/Formats/HttpParameterFormatter.cs
@@ -38,14 +38,14 @@ internal static string Format(ClickHouseType type, object value, bool quote)
             case DecimalType dt:
                 return Convert.ToDecimal(value, CultureInfo.InvariantCulture).ToString(CultureInfo.InvariantCulture);
 
-            case DateType dt when value is DateTimeOffset @do:
-                return @do.Date.ToString("yyyy-MM-dd", CultureInfo.InvariantCulture);
+            case DateType dt when value is DateTimeOffset @dto:
+                return @dto.Date.ToString("yyyy-MM-dd", CultureInfo.InvariantCulture);
+#if NET6_0_OR_GREATER
+            case DateType dt when value is DateOnly @do:
+                return @do.ToString("yyyy-MM-dd", CultureInfo.InvariantCulture);
+#endif
             case DateType dt:
                 return Convert.ToDateTime(value, CultureInfo.InvariantCulture).ToString("yyyy-MM-dd", CultureInfo.InvariantCulture);
-            case Date32Type dt when value is DateTimeOffset @do:
-                return @do.Date.ToString("yyyy-MM-dd", CultureInfo.InvariantCulture);
-            case Date32Type dt:
-                return Convert.ToDateTime(value, CultureInfo.InvariantCulture).ToString("yyyy-MM-dd", CultureInfo.InvariantCulture);
 
             case StringType st:
             case FixedStringType tt:

diff --git a/ClickHouse.Client/Types/Date32Type.cs b/ClickHouse.Client/Types/Date32Type.cs
@@ -4,7 +4,7 @@
 
 namespace ClickHouse.Client.Types;
 
-internal class Date32Type : AbstractDateTimeType
+internal class Date32Type : DateType
 {
     public override string Name { get; }