Commit 80a6875

committed

Throw better exception in corner cases

1 parent d94b2e5 commit 80a6875Copy full SHA for 80a6875

File tree

-3

lines changed

-3

lines changed

Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,6 @@ public AttrAttribute GetAttributeByPublicName(string publicName)`
`173`	`173`	`public AttrAttribute GetAttributeByPropertyName(string propertyName)`
`174`	`174`	`{`
`175`	`175`	`AttrAttribute? attribute = FindAttributeByPropertyName(propertyName);`
`176`		`-`
`177`	`176`	`return attribute ?? throw new InvalidOperationException($"Attribute for property '{propertyName}' does not exist on resource type '{ClrType.Name}'.");`
`178`	`177`	`}`
`179`	`178`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+using JsonApiDotNetCore.Errors;`
`1`	`2`	`using JsonApiDotNetCore.Middleware;`
`2`	`3`	`using Microsoft.AspNetCore.Builder;`
`3`	`4`	`using Microsoft.Extensions.DependencyInjection;`
`@@ -60,7 +61,7 @@ private static void AssertAspNetCoreOpenApiIsNotRegistered(IServiceProvider serv`
`60`	`61`
`61`	`62`	`if (configureInstance != null)`
`62`	`63`	`{`
`63`		`- throw new InvalidOperationException("JsonApiDotNetCore is incompatible with ASP.NET OpenAPI. " +`
	`64`	`+ throw new InvalidConfigurationException("JsonApiDotNetCore is incompatible with ASP.NET OpenAPI. " +`
`64`	`65`	`"Replace 'services.AddOpenApi()' with 'services.AddOpenApiForJsonApi()' from the JsonApiDotNetCore.OpenApi.Swashbuckle NuGet package.");`
`65`	`66`	`}`
`66`	`67`	`}`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -455,7 +455,7 @@ private static void AssertNoInfiniteRecursion(int recursionDepth)`
`455`	`455`	`{`
`456`	`456`	`if (recursionDepth >= 500)`
`457`	`457`	`{`
`458`		`- throw new InvalidOperationException("Infinite recursion detected in eager-load chain.");`
	`458`	`+ throw new InvalidConfigurationException("Infinite recursion detected in eager-load chain.");`
`459`	`459`	`}`
`460`	`460`	`}`
`461`	`461`

Comments

(0)