Merge pull request #220 from linq2db/version2

Release 2.5.0

Author: sdanyliv

Build/linq2db.Default.props

@@ -1,6 +1,6 @@
 <Project>
 	<PropertyGroup>
-		<Version>2.4.0</Version>
+		<Version>2.5.0</Version>
 
 		<Authors>Svyatoslav Danyliv, Igor Tkachev, Dmitry Lukashenko, Ilya Chudin</Authors>
 		<Product>Linq to DB</Product>

Directory.Packages.props

@@ -5,11 +5,11 @@
 		<PackageVersion Include="NUnit" Version="3.13.2" />
 		<PackageVersion Include="FluentAssertions" Version="5.10.3" />
 
-		<PackageVersion Include="linq2db" Version="3.5.2" />
-		<PackageVersion Include="linq2db.Tools" Version="3.5.2" />
+		<PackageVersion Include="linq2db" Version="3.7.0" />
+		<PackageVersion Include="linq2db.Tools" Version="3.7.0" />
 
 		<PackageVersion Include="Microsoft.SourceLink.GitHub" Version="1.1.1" />
-		<PackageVersion Include="Microsoft.Bcl.AsyncInterfaces" Version="5.0.0" />
+		<PackageVersion Include="Microsoft.Bcl.AsyncInterfaces" Version="6.0.0" />
 		<PackageVersion Include="System.Interactive.Async" Version="3.2.0"/>
 
 		<PackageVersion Include="Microsoft.EntityFrameworkCore.Relational" Version="2.2.6" />

NuGet/linq2db.EntityFrameworkCore.nuspec

@@ -16,7 +16,7 @@
 		<dependencies>
 			<group targetFramework=".NETStandard2.0">
 				<dependency id="Microsoft.EntityFrameworkCore.Relational" version="2.2.6" />
-				<dependency id="linq2db" version="3.5.2" />
+				<dependency id="linq2db" version="3.7.0" />
 			</group>
 		</dependencies>
 	</metadata>

Source/LinqToDB.EntityFrameworkCore/LinqToDBForEFToolsImplDefault.cs

@@ -780,12 +780,24 @@ public virtual Expression TransformExpression(Expression expression, IDataContex
 			var tracking           = true;
 			var ignoreTracking     = false;
 
+			var nonEvaluatableParameters = new HashSet<ParameterExpression>();
+
 			TransformInfo LocalTransform(Expression e)
 			{
 				e = CompactExpression(e);
 
 				switch (e.NodeType)
 				{
+					case ExpressionType.Lambda:
+					{
+						foreach (var parameter in ((LambdaExpression)e).Parameters)
+						{
+							nonEvaluatableParameters.Add(parameter);
+						}
+
+						break;
+					}
+
 					case ExpressionType.Constant:
 					{
 						if (dc != null && typeof(EntityQueryable<>).IsSameOrParentOf(e.Type) || typeof(DbSet<>).IsSameOrParentOf(e.Type))
@@ -939,14 +951,18 @@ TransformInfo LocalTransform(Expression e)
 
 						if (typeof(IQueryable<>).IsSameOrParentOf(methodCall.Type))
 						{
-							// Invoking function to evaluate EF's Subquery located in function
-
-							var obj = EvaluateExpression(methodCall.Object);
-							var arguments = methodCall.Arguments.Select(EvaluateExpression).ToArray();
-							if (methodCall.Method.Invoke(obj, arguments) is IQueryable result)
+							if (null == methodCall.Find(nonEvaluatableParameters,
+								    (c, t) => t.NodeType == ExpressionType.Parameter && c.Contains(t)))
 							{
-								if (!ExpressionEqualityComparer.Instance.Equals(methodCall, result.Expression))
-									return new TransformInfo(result.Expression, false, true);
+								// Invoking function to evaluate EF's Subquery located in function
+
+								var obj = EvaluateExpression(methodCall.Object);
+								var arguments = methodCall.Arguments.Select(EvaluateExpression).ToArray();
+								if (methodCall.Method.Invoke(obj, arguments) is IQueryable result)
+								{
+									if (!ExpressionEqualityComparer.Instance.Equals(methodCall, result.Expression))
+										return new TransformInfo(result.Expression, false, true);
+								}
 							}
 						}
 

Tests/LinqToDB.EntityFrameworkCore.PostgreSQL.Tests/Models/NpgSqlEntities/NpgSqlEnititesContext.cs

@@ -22,9 +22,13 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
 					entity.ToView("EventsView", "views");
 				});
 			*/
+
+			modelBuilder.Entity<EntityWithArrays>(entity =>
+			{
+			});
 		}
 
-		// public virtual DbSet<Event> Events { get; set; } = null!;
+		public virtual DbSet<Event> Events { get; set; } = null!;
 		public virtual DbSet<EntityWithArrays> EntityWithArrays { get; set; } = null!;
 
 	}

Tests/LinqToDB.EntityFrameworkCore.PostgreSQL.Tests/NpgSqlTests.cs

@@ -89,5 +89,23 @@ where Sql.Ext.PostgreSQL().Overlaps(m.Guids, guids)
 		}
 		*/
 
+		[Test]
+		public void TestUnnest()
+		{
+			using var db = CreateNpgSqlEntitiesContext();
+			using var dc = db.CreateLinqToDbConnection();
+
+			var guids = new Guid[] { Guid.Parse("271425b1-ebe8-400d-b71d-a6e47a460ae3"),
+				Guid.Parse("b75de94e-6d7b-4c70-bfa1-f8639a6a5b35") };
+
+			var query = 
+				from m in db.EntityWithArrays.ToLinqToDBTable()
+				from g in dc.Unnest(m.Guids) 
+				where Sql.Ext.PostgreSQL().Overlaps(m.Guids, guids)
+				select m;
+
+			query.Invoking(q => q.ToArray()).Should().NotThrow();
+		}
+
 	}
 }

azure-pipelines.yml

@@ -1,8 +1,8 @@
 variables:
   solution: 'linq2db.EFCore.sln'
   build_configuration: 'Release'
-  assemblyVersion: 2.4.0
-  nugetVersion: 2.4.0
+  assemblyVersion: 2.5.0
+  nugetVersion: 2.5.0
   artifact_nugets: 'nugets'
 
 # build on commits to important branches (master + release branches):
@@ -26,7 +26,7 @@ stages:
   jobs:
   - job: build_job
     pool:
-      vmImage: 'windows-2019'
+      vmImage: 'windows-2022'
     variables:
       packageVersion: $(nugetVersion)
     displayName: 'Build'