Fixes after merging v5 into openapi branch

Author: bkoelman

src/JsonApiDotNetCore.OpenApi.Client/JsonApiDotNetCore.OpenApi.Client.csproj

@@ -28,7 +28,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0">
+    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

src/JsonApiDotNetCore.OpenApi/JsonApiActionDescriptorCollectionProvider.cs

@@ -72,7 +72,7 @@ private static IEnumerable<ActionDescriptor> AddJsonApiMetadataToAction(ActionDe
             }
             case PrimaryRequestMetadata primaryMetadata:
             {
-                UpdateBodyParameterDescriptor(endpoint, primaryMetadata.DocumentType);
+                UpdateBodyParameterDescriptor(endpoint, primaryMetadata.DocumentType, null);
                 return Array.Empty<ActionDescriptor>();
             }
             case NonPrimaryEndpointMetadata nonPrimaryEndpointMetadata and (RelationshipResponseMetadata or SecondaryResponseMetadata):
@@ -140,7 +140,7 @@ private static IEnumerable<ActionDescriptor> Expand(ActionDescriptor genericEndp
         return expansion;
     }
 
-    private static void UpdateBodyParameterDescriptor(ActionDescriptor endpoint, Type documentType, string? parameterName = null)
+    private static void UpdateBodyParameterDescriptor(ActionDescriptor endpoint, Type documentType, string? parameterName)
     {
         ControllerParameterDescriptor? requestBodyDescriptor = endpoint.GetBodyParameterDescriptor();
 

src/JsonApiDotNetCore.OpenApi/JsonApiDotNetCore.OpenApi.csproj

@@ -32,7 +32,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0">
+    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

src/JsonApiDotNetCore.OpenApi/SwaggerComponents/CachingSwaggerGenerator.cs

@@ -3,6 +3,8 @@
 using Swashbuckle.AspNetCore.Swagger;
 using Swashbuckle.AspNetCore.SwaggerGen;
 
+#pragma warning disable AV1553 // Do not use optional parameters with default value null for strings, collections or tasks
+
 namespace JsonApiDotNetCore.OpenApi.SwaggerComponents;
 
 /// <summary>

test/OpenApiTests/OpenApiStartup.cs

@@ -1,10 +1,8 @@
 using JsonApiDotNetCore.Configuration;
 using JsonApiDotNetCore.OpenApi;
 using Microsoft.AspNetCore.Builder;
-using Microsoft.AspNetCore.Hosting;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Logging;
 using TestBuildingBlocks;
 
 namespace OpenApiTests;
@@ -21,7 +19,7 @@ public override void ConfigureServices(IServiceCollection services)
         services.AddOpenApi(mvcBuilder);
     }
 
-    public override void Configure(IApplicationBuilder app, IWebHostEnvironment environment, ILoggerFactory loggerFactory)
+    public override void Configure(IApplicationBuilder app)
     {
         app.UseRouting();
         app.UseJsonApi();