enisn / AutoFilterer

AutoFilterer is a mini filtering framework library for dotnet. The main purpose of the library is to generate LINQ expressions for Entities over DTOs automatically. The first aim is to be compatible with Open API 3.0 Specifications
MIT License
458 stars 37 forks source link

FromQuery Error: InvalidOperationException: No public static bool GenericFilter.TryParse(string, out GenericFilter) method found for filter. #57

Open gpproton opened 1 year ago

gpproton commented 1 year ago

I'm Currently trying out AutoFilterer and it works well with using FromBody but not with FromQuery

Below are the full error message generated, i'm sticking to using FromBody for not just wanted to report the issue.

Microsoft.AspNetCore.Http.RequestDelegateFactory.BindParameterFromValue(ParameterInfo parameter, Expression valueExpression, RequestDelegateFactoryContext factoryContext, string source) Microsoft.AspNetCore.Http.RequestDelegateFactory.CreateArgument(ParameterInfo parameter, RequestDelegateFactoryContext factoryContext) Microsoft.AspNetCore.Http.RequestDelegateFactory.CreateArguments(ParameterInfo[] parameters, RequestDelegateFactoryContext factoryContext) Microsoft.AspNetCore.Http.RequestDelegateFactory.CreateArgumentsAndInferMetadata(MethodInfo methodInfo, RequestDelegateFactoryContext factoryContext) Microsoft.AspNetCore.Http.RequestDelegateFactory.InferMetadata(MethodInfo methodInfo, RequestDelegateFactoryOptions options) Microsoft.AspNetCore.Routing.RouteEndpointDataSource.CreateRouteEndpointBuilder(RouteEntry entry, RoutePattern groupPrefix, IReadOnlyList<Action> groupConventions, IReadOnlyList<Action> groupFinallyConventions) Microsoft.AspNetCore.Routing.RouteEndpointDataSource.GetGroupedEndpoints(RouteGroupContext context) Microsoft.AspNetCore.Routing.RouteGroupBuilder+GroupEndpointDataSource.GetGroupedEndpointsWithNullablePrefix(RoutePattern prefix, IReadOnlyList<Action> conventions, IReadOnlyList<Action> finallyConventions, IServiceProvider applicationServices) Microsoft.AspNetCore.Routing.RouteGroupBuilder+GroupEndpointDataSource.get_Endpoints() Microsoft.AspNetCore.Routing.CompositeEndpointDataSource.CreateEndpointsUnsynchronized() Microsoft.AspNetCore.Routing.CompositeEndpointDataSource.EnsureEndpointsInitialized() Microsoft.AspNetCore.Routing.DataSourceDependentCache.Initialize() System.Threading.LazyInitializer.EnsureInitializedCore(ref T target, ref bool initialized, ref object syncLock, Func valueFactory) Microsoft.AspNetCore.Routing.Matching.DataSourceDependentMatcher..ctor(EndpointDataSource dataSource, Lifetime lifetime, Func matcherBuilderFactory) Microsoft.AspNetCore.Routing.Matching.DfaMatcherFactory.CreateMatcher(EndpointDataSource dataSource) Microsoft.AspNetCore.Routing.EndpointRoutingMiddleware.InitializeCoreAsync() Microsoft.AspNetCore.Routing.EndpointRoutingMiddleware.g__AwaitMatcher|8_0(EndpointRoutingMiddleware middleware, HttpContext httpContext, Task matcherTask) Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext) Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider) Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddlewareImpl.Invoke(HttpContext context)

enisn commented 1 year ago

It seems it's an AspNetCore binding error. Your value might have some special characters or something that can break the binding

gpproton commented 1 year ago

I thought so initially, below is the sample filter used.

// Sample filter
public class GenericFilter : PaginationFilterBase{ }

// Sample DTO
public class SampleDto : FilterBase {
public Guid Id { get; set; }
public string Name { get; set; }
}
enisn commented 1 year ago

I understand the problem but nothing to do in autofilterer library. Autofilterer generates only LINQ expressions and it doesn't know about your aspnetcore application or how to bind object properties.

There is no aspnetcore dependency of autofilterer. https://github.com/enisn/AutoFilterer/blob/5e55bbf819bc1640ba2286d8fc7c8fc10a8dfdf1/src/AutoFilterer/AutoFilterer.csproj#L5-L22

You may ask your question at dotnet/aspnetcore repositroy

gpproton commented 1 year ago

I was able to use fine on dotnet 7 with the new AsParameters which is better for minimal API

Here

group.MapGet("/", async ([FromServices] CategoryService sv, [AsParameters] CategoryFilter filter) => await sv.GetAllAsync(filter))
            .WithName($"GetAll{name}")
            .WithOpenApi();
enisn commented 1 year ago

Might be similar with https://github.com/dotnet/aspnetcore/issues/40752


Most probably this is the reason https://stackoverflow.com/questions/70260354/net-6-minimal-api-no-public-static-bool-request-tryparsestring-out-request