Skip to content

Commit 60b9640

Browse files
committed
fix: merge I
1 parent a0d4cfc commit 60b9640

File tree

7 files changed

+19
-13
lines changed

7 files changed

+19
-13
lines changed

src/JsonApiDotNetCore/Configuration/ApplicationBuilderExtensions.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
using JsonApiDotNetCore.Builders;
21
using JsonApiDotNetCore.Formatters;
32
using JsonApiDotNetCore.Middleware;
43
using Microsoft.AspNetCore.Builder;
@@ -34,6 +33,8 @@ public static void UseJsonApi(this IApplicationBuilder builder)
3433
options.OutputFormatters.Insert(0, builder.ApplicationServices.GetRequiredService<IJsonApiOutputFormatter>());
3534
options.Conventions.Insert(0, builder.ApplicationServices.GetRequiredService<IJsonApiRoutingConvention>());
3635
};
36+
37+
builder.UseMiddleware<JsonApiMiddleware>();
3738
}
3839
}
3940
}

src/JsonApiDotNetCore/Configuration/IJsonApiApplicationBuilder.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
using System;
22
using Microsoft.AspNetCore.Mvc;
33

4-
namespace JsonApiDotNetCore.Builders
4+
namespace JsonApiDotNetCore.Configuration
55
{
66
internal interface IJsonApiApplicationBuilder
77
{

src/JsonApiDotNetCore/Configuration/JsonApiApplicationBuilder.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
21
using System;
32
using JsonApiDotNetCore.Configuration;
43
using JsonApiDotNetCore.Formatters;
@@ -23,7 +22,7 @@
2322
using Microsoft.Extensions.DependencyInjection.Extensions;
2423
using Microsoft.Extensions.Logging;
2524

26-
namespace JsonApiDotNetCore.Builders
25+
namespace JsonApiDotNetCore.Configuration
2726
{
2827
/// <summary>
2928
/// A utility class that builds a JsonApi application. It registers all required services

src/JsonApiDotNetCore/Configuration/ResourceGraph.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,15 +24,21 @@ public ResourceGraph(IReadOnlyCollection<ResourceContext> resources)
2424
/// <inheritdoc />
2525
public ResourceContext GetResourceContext(string resourceName)
2626
{
27-
if (resourceName == null) throw new ArgumentNullException(nameof(resourceName));
27+
if (resourceName == null)
28+
{
29+
throw new ArgumentNullException(nameof(resourceName));
30+
}
2831

2932
return _resources.SingleOrDefault(e => e.ResourceName == resourceName);
3033
}
3134

3235
/// <inheritdoc />
3336
public ResourceContext GetResourceContext(Type resourceType)
3437
{
35-
if (resourceType == null) throw new ArgumentNullException(nameof(resourceType));
38+
if (resourceType == null)
39+
{
40+
throw new ArgumentNullException(nameof(resourceType));
41+
}
3642

3743
return IsLazyLoadingProxyForResourceType(resourceType)
3844
? _resources.SingleOrDefault(e => e.ResourceType == resourceType.BaseType)

src/JsonApiDotNetCore/Configuration/ServiceCollectionExtensions.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
1-
21
using System;
32
using System.Collections.Generic;
43
using System.Linq;
54
using System.Reflection;
6-
using JsonApiDotNetCore.Builders;
75
using JsonApiDotNetCore.Errors;
86
using JsonApiDotNetCore.Serialization.Building;
97
using JsonApiDotNetCore.Serialization.Client.Internal;

src/JsonApiDotNetCore/Configuration/ServiceDiscoveryFacade.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
21
using System;
32
using System.Collections.Generic;
43
using System.Linq;
@@ -8,7 +7,6 @@
87
using JsonApiDotNetCore.Resources;
98
using JsonApiDotNetCore.Services;
109
using Microsoft.EntityFrameworkCore;
11-
using Microsoft.EntityFrameworkCore.Internal;
1210
using Microsoft.Extensions.DependencyInjection;
1311
using Microsoft.Extensions.Logging;
1412

src/JsonApiDotNetCore/Middleware/JsonApiRoutingConvention.cs

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,15 @@ public void Apply(ApplicationModel application)
6262
foreach (var controller in application.Controllers)
6363
{
6464
var resourceType = ExtractResourceTypeFromController(controller.ControllerType);
65-
var resourceContext = _resourceGraph.GetResourceContext(resourceType);
6665

67-
if (resourceContext != null)
66+
if (resourceType != null)
6867
{
69-
_registeredResources.Add(controller.ControllerName, resourceContext);
68+
var resourceContext = _resourceGraph.GetResourceContext(resourceType);
69+
70+
if (resourceContext != null)
71+
{
72+
_registeredResources.Add(controller.ControllerName, resourceContext);
73+
}
7074
}
7175

7276
if (RoutingConventionDisabled(controller) == false)

0 commit comments

Comments
 (0)