Update (#21)

- Fixed `null` property value default validation (needs to be skipped, because the default behavior of .NET validation attributes is to return an error for a `null` value)
nd1012 · Oct 21, 2023 · 063665c · 063665c
1 parent 9195c1d
commit 063665c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/ObjectValidation/ObjectValidation.csproj b/src/ObjectValidation/ObjectValidation.csproj
@@ -22,7 +22,7 @@
     <PackageLicenseFile>LICENSE</PackageLicenseFile>
     <PackageRequireLicenseAcceptance>True</PackageRequireLicenseAcceptance>
     <PackageReadmeFile>README.md</PackageReadmeFile>
-    <Version>1.12.1</Version>
+    <Version>1.12.2</Version>
     <DebugType>embedded</DebugType>
     <EmbedAllSources>true</EmbedAllSources>
   </PropertyGroup>

diff --git a/src/ObjectValidation/ValidationExtensions.Internal.cs b/src/ObjectValidation/ValidationExtensions.Internal.cs
@@ -226,11 +226,14 @@ orderby pi.Name
                             noValidation = noValidationAttr is not null;
                             if (isObjectValidatable)
                             {
-                                validationContext = new(obj, serviceProvider, items: null)
+                                if (value is not null)
                                 {
-                                    MemberName = pi.Name
-                                };
-                                res &= Validator.TryValidateProperty(value, validationContext, validationResults);
+                                    validationContext = new(obj, serviceProvider, items: null)
+                                    {
+                                        MemberName = pi.Name
+                                    };
+                                    res &= Validator.TryValidateProperty(value, validationContext, validationResults);
+                                }
                                 if (noValidation && noValidationAttr!.SkipNullValueCheck)
                                 {
 #if DEBUG