feat: Control whether middleware sets context.Result to null on valid…

…ation error (#48)


Co-authored-by: Ben McCallum <[email protected]>

src/Directory.Build.props

@@ -17,7 +17,7 @@
     <PackageIconUrl>https://github.com/benmccallum/fairybread/raw/master/logo-400x400.png</PackageIconUrl>
     <PackageLicenseUrl>https://github.com/benmccallum/fairybread/blob/master/LICENSE</PackageLicenseUrl>
 
-    <Version>6.0.1</Version>
+    <Version>6.1.0</Version>
 
     <HotChocolateVersion>11.0.9</HotChocolateVersion>
 

src/FairyBread/DefaultFairyBreadOptions.cs

@@ -10,6 +10,9 @@ public class DefaultFairyBreadOptions : IFairyBreadOptions
     {
         public virtual bool ThrowIfNoValidatorsFound { get; set; } = true;
 
+        /// <inheritdoc/>
+        public virtual bool SetNullResultOnValidationError { get; set; } = true;
+
         /// <inheritdoc/>
         public virtual Func<IMiddlewareContext, IInputField, bool> ShouldValidate { get; set; }
             = DefaultImplementations.ShouldValidate;

src/FairyBread/IFairyBreadOptions.cs

@@ -13,6 +13,12 @@ public interface IFairyBreadOptions
         /// </summary>
         bool ThrowIfNoValidatorsFound { get; set; }
 
+        /// <summary>
+        /// If true, FairyBread will set the current field's <c>IResolverContext.Result</c> 
+        /// to <c>null</c> when a validation error occurs.
+        /// </summary>
+        bool SetNullResultOnValidationError { get; set; }
+
         /// <summary>
         /// Function used to determine if an argument should be validated by
         /// FairyBread's <see cref="InputValidationMiddleware"/>.

src/FairyBread/InputValidationMiddleware.cs

@@ -76,7 +76,11 @@ public async Task InvokeAsync(IMiddlewareContext context)
             if (invalidResults.Any())
             {
                 _validationErrorsHandler.Handle(context, invalidResults);
-                context.Result = null;
+
+                if (_options.SetNullResultOnValidationError)
+                {
+                    context.Result = null;
+                }
             }
             else
             {