Merge pull request #86 from cnblogs/fix-empty-result

ikesnowy · web-flow · commit 9335fdb53ecd · 2023-04-13T14:56:19.000+08:00
fix: allow model validation failed result to be returned
diff --git a/src/Cnblogs.Architecture.Ddd.Cqrs.AspNetCore/CommandEndpointHandler.cs b/src/Cnblogs.Architecture.Ddd.Cqrs.AspNetCore/CommandEndpointHandler.cs
@@ -34,6 +34,12 @@ public CommandEndpointHandler(IMediator mediator, IOptions<CqrsHttpOptions> opti
                 "Expected ICommand<>, but got null, check if your delegate in MapCommand(route, delegate) returned non-null command");
         }
 
+        if (command is not IBaseRequest)
+        {
+            // not command, return as-is
+            return command;
+        }
+
         var response = await _mediator.Send(command);
         if (response is null)
         {

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,12 @@ public CommandEndpointHandler(IMediator mediator, IOptions<CqrsHttpOptions> opti`
`34`	`34`	`"Expected ICommand<>, but got null, check if your delegate in MapCommand(route, delegate) returned non-null command");`
`35`	`35`	`}`
`36`	`36`
	`37`	`+ if (command is not IBaseRequest)`
	`38`	`+ {`
	`39`	`+ // not command, return as-is`
	`40`	`+ return command;`
	`41`	`+ }`
	`42`	`+`
`37`	`43`	`var response = await _mediator.Send(command);`
`38`	`44`	`if (response is null)`
`39`	`45`	`{`