Merge pull request #1226 from progressonderwijs/robin/move-back-down

Changed SQL date type mapping to prioritize DateOnly over DateTime

Author: Bazeloth

src/ProgressOnderwijsUtils/ProgressOnderwijsUtils.csproj

@@ -1,8 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <Import Project="..\NugetPackagesCommon.props" />
   <PropertyGroup Label="Configuration">
-    <Version>112.0.1</Version>
-    <PackageReleaseNotes>Extended the lookup to determine the underlying sql type for DateOnly</PackageReleaseNotes>
+    <Version>113.0.0</Version>
+    <PackageReleaseNotes>Changed SQL date type mapping to prioritize DateOnly over DateTime</PackageReleaseNotes>
     <Title>ProgressOnderwijsUtils</Title>
     <Description>Collection of utilities developed by ProgressOnderwijs</Description>
     <PackageTags>ProgressOnderwijs</PackageTags>

src/ProgressOnderwijsUtils/SchemaReflection/SqlSystemTypeIdExtensions.cs

@@ -68,8 +68,8 @@ static readonly (Type clrType, SqlSystemTypeId typeId)[] typeLookup = {
         (typeof(uint), SqlSystemTypeId.Binary),
         (typeof(DateTime), SqlSystemTypeId.DateTime2),
         (typeof(DateTime), SqlSystemTypeId.DateTime),
-        (typeof(DateTime), SqlSystemTypeId.Date),
         (typeof(DateOnly), SqlSystemTypeId.Date),
+        (typeof(DateTime), SqlSystemTypeId.Date),
         (typeof(DateTime), SqlSystemTypeId.SmallDateTime),
         (typeof(DateTimeOffset), SqlSystemTypeId.DateTimeOffset),
         (typeof(decimal), SqlSystemTypeId.Decimal),