Replaced suppression with fix

Bart Koelman · Bart Koelman · commit 8534c41d9c6c · 2021-09-20T11:13:58.000+02:00
diff --git a/src/JsonApiDotNetCore/Serialization/JsonApiReader.cs b/src/JsonApiDotNetCore/Serialization/JsonApiReader.cs
@@ -83,9 +83,7 @@ public async Task<InputFormatterResult> ReadAsync(InputFormatterContext context)
                 ValidateRequestBody(model, body, context.HttpContext.Request);
             }
 
-            // ReSharper disable once AssignNullToNotNullAttribute
-            // Justification: According to JSON:API we must return 200 OK without a body in some cases.
-            return await InputFormatterResult.SuccessAsync(model);
+            return model == null ? await InputFormatterResult.NoValueAsync() : await InputFormatterResult.SuccessAsync(model);
         }
 
         private async Task<string> GetRequestBodyAsync(Stream bodyStream)

Original file line number	Diff line number	Diff line change
`@@ -83,9 +83,7 @@ public async Task<InputFormatterResult> ReadAsync(InputFormatterContext context)`
`83`	`83`	`ValidateRequestBody(model, body, context.HttpContext.Request);`
`84`	`84`	`}`
`85`	`85`
`86`		`- // ReSharper disable once AssignNullToNotNullAttribute`
`87`		`- // Justification: According to JSON:API we must return 200 OK without a body in some cases.`
`88`		`- return await InputFormatterResult.SuccessAsync(model);`
	`86`	`+ return model == null ? await InputFormatterResult.NoValueAsync() : await InputFormatterResult.SuccessAsync(model);`
`89`	`87`	`}`
`90`	`88`
`91`	`89`	`private async Task<string> GetRequestBodyAsync(Stream bodyStream)`