Filter validation problems

[KaJIbI4]

Hello!
There were some difficulties with validating the values coming in the filter.
I use EntityFramework, Automapper and extension from doc GridifyToAsync.
For example, entity:

public sealed class My
{
    public int MyInt {get; set;}
    public DateTime MyDate {get; set;}
    public MyEnum MyEnum {get; set;} // int Enum
}

For the enum and DateTime, I send queries with incorrect filters MyDate=xyz or MyEnum > 999999999999999 and errors are ignored, I get an empty array. But for int, int? (I haven't checked the others) with incorrect filters MyInt=xyz or MyInt < 999999999999999 i get the error Object of type 'System.String' cannot be converted to type 'System.Int32'. Шt turns out a different pattern of behavior for different types.

I found a function query.IsValid to validate the request, but it returns true in all these situations. Does it check only the field names?

Here are the error traces, maybe they will be useful:

• When compiling a query:

   at Gridify.QueryBuilders.LinqQueryBuilder`1.BuildQueryAccordingToValueType(Expression body, ParameterExpression parameter, Object value, SyntaxNode op, ValueExpressionSyntax valueExpression)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.BuildQuery(Expression body, ParameterExpression parameter, ValueExpressionSyntax valueExpression, SyntaxNode op, Func`2 convertor)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.ConvertBinaryExpressionSyntaxToQuery(BinaryExpressionSyntax binarySyntax)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.BuildQuery(ExpressionSyntax expression, Boolean isParenthesisOpen)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.Build(ExpressionSyntax expression)
   at Gridify.GridifyExtensions.CreateQuery[T](SyntaxTree syntaxTree, IGridifyMapper`1 mapper)
   at Gridify.QueryBuilder`1.ConvertConditionToExpression(String condition)
   at System.Linq.Enumerable.SelectListIterator`2.Fill(ReadOnlySpan`1 source, Span`1 destination, Func`2 func)
   at System.Linq.Enumerable.SelectListIterator`2.ToList()
   at Gridify.QueryBuilder`1.BuildFilteringExpression()
   at Gridify.QueryBuilder`1.BuildCompiled()

• When GridifyToAsync ( PgSql, Entity, EntityFrameworkCompatibilityLayer = true ):

   at System.RuntimeType.CheckValue(Object& value, Binder binder, CultureInfo culture, BindingFlags invokeAttr)
   at System.Reflection.RuntimePropertyInfo.SetValue(Object obj, Object value, Object[] index)
   at Gridify.Syntax.GridifyTypeBuilder.CreateNewObject(Type type, String fieldName, Object value)
   at Gridify.QueryBuilders.LinqQueryBuilder`1.GetValueExpression(Type type, Object value)
   at Gridify.QueryBuilders.LinqQueryBuilder`1.BuildQueryAccordingToValueType(Expression body, ParameterExpression parameter, Object value, SyntaxNode op, ValueExpressionSyntax valueExpression)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.BuildQuery(Expression body, ParameterExpression parameter, ValueExpressionSyntax valueExpression, SyntaxNode op, Func`2 convertor)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.ConvertBinaryExpressionSyntaxToQuery(BinaryExpressionSyntax binarySyntax)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.BuildQuery(ExpressionSyntax expression, Boolean isParenthesisOpen)
   at Gridify.QueryBuilders.BaseQueryBuilder`2.Build(ExpressionSyntax expression)
   at Gridify.GridifyExtensions.ApplyFiltering[T](IQueryable`1 query, String filter, IGridifyMapper`1 mapper)
   at Gridify.GridifyExtensions.ApplyFiltering[T](IQueryable`1 query, IGridifyFiltering gridifyFiltering, IGridifyMapper`1 mapper)
   at Gridify.EntityFramework.GridifyExtensions.GridifyQueryableAsync[T](IQueryable`1 query, IGridifyQuery gridifyQuery, IGridifyMapper`1 mapper)
   at ...GridifyExtension.GridifyToAsync[TSource,TDestination](IQueryable`1 query, IMapper autoMapper, IGridifyQuery gridifyQuery, IGridifyMapper`1 mapper)

As I understand it, I will have to do parsing of the filter syntax tree at the validation stage and check all my fields manually?
#187 Discussion

[alirezanet]

Hi @KaJIbI4,
Yes, the current implementation of the GridifyQuery isValid method checks the field names. As for exceptions, we depend on dotnet parsers. If the value you provide can be parsed by the built-in dotnet parser and converters, Gridify tries to avoid any exceptions since the value does not exist. However, if parsing fails, an exception is thrown (for example, when converting a string to an integer, which should not happen).
You can improve the functionality by accessing the SyntaxTree or using the mapper's value converter to parse the value before it gets to the GridifyQueryBuilder...
However, if you believe there is a scenario that Gridify should inherently support regarding these validations and parsing, please feel free to open a new feature request issue and share your ideas.