Fix for #103. Corrected expression tree transformation for IgnoreQueryFilters. (#105)

Author: sdanyliv

Source/LinqToDB.EntityFrameworkCore/LinqToDBForEFToolsImplDefault.cs

@@ -758,7 +758,6 @@ static List<Expression> CompactTree(List<Expression> items, ExpressionType nodeT
 		/// <returns>Transformed expression.</returns>
 		public virtual Expression TransformExpression(Expression expression, IDataContext dc, DbContext? ctx, IModel? model)
 		{
-			var ignoreQueryFilters = false;
 			var tracking           = true;
 			var ignoreTracking     = false;
 
@@ -807,8 +806,10 @@ TransformInfo LocalTransform(Expression e)
 
 								if (generic == IgnoreQueryFiltersMethodInfo)
 								{
-									ignoreQueryFilters = true;
-									isTunnel = true;
+									var newMethod = Expression.Call(
+										Methods.LinqToDB.IgnoreFilters.MakeGenericMethod(methodCall.Method.GetGenericArguments()),
+										methodCall.Arguments[0], Expression.NewArrayInit(typeof(Type)));
+									return new TransformInfo(newMethod, false, true);
 								}
 								else if (generic == AsNoTrackingMethodInfo)
 								{
@@ -953,13 +954,6 @@ TransformInfo LocalTransform(Expression e)
 
 			var newExpression = expression.Transform(e => LocalTransform(e));
 
-			if (ignoreQueryFilters)
-			{
-				var elementType = newExpression.Type.GetGenericArguments()[0];
-				newExpression = Expression.Call(Methods.LinqToDB.IgnoreFilters.MakeGenericMethod(elementType),
-					newExpression, Expression.NewArrayInit(typeof(Type)));
-			}
-
 			if (!ignoreTracking && dc is LinqToDBForEFToolsDataConnection dataConnection)
 			{
 				// ReSharper disable once ConditionIsAlwaysTrueOrFalse

Tests/LinqToDB.EntityFrameworkCore.SqlServer.Tests/ToolsTests.cs

@@ -769,6 +769,35 @@ public void TestNullability([Values(true, false)] bool enableFilter)
 		}
 
 
+		[Test]
+		public void TestUpdate([Values(true, false)] bool enableFilter)
+		{
+			using (var ctx = CreateContext(enableFilter))
+			{
+				int? test = 1;
+				ctx.Employees.IgnoreQueryFilters().Where(e => e.EmployeeId == test).Update(x => new Employee
+				{
+					Address = x.Address
+
+				});
+			}
+		}
+
+		[Test]
+		public async Task TestUpdateAsync([Values(true, false)] bool enableFilter)
+		{
+			using (var ctx = CreateContext(enableFilter))
+			{
+				int? test = 1;
+				await ctx.Employees.IgnoreQueryFilters().Where(e => e.EmployeeId == test).UpdateAsync(x => new Employee
+				{
+					Address = x.Address
+
+				});
+			}
+		}
+
+
 		[Test]
 		public void TestCommandTimeout()
 		{