Merge branch '5.3.x'

# Conflicts:
#	build-common/NHibernate.props
#	src/NHibernate/Engine/JoinSequence.cs
#	src/NHibernate/Loader/JoinWalker.cs
#	src/NHibernate/Loader/OuterJoinableAssociation.cs

Author: fredericDelaporte

releasenotes.txt

@@ -1,4 +1,19 @@
-Build 5.3.4
+Build 5.3.5
+=============================
+
+Release notes - NHibernate - Version 5.3.5
+
+2 issues were resolved in this release.
+
+** Bug
+
+   * #2599 WrongClassException in Linq query
+
+** Task
+
+   * #2606 Release 5.3.5
+
+Build 5.3.4
 =============================
 
 Release notes - NHibernate - Version 5.3.4

src/NHibernate.Test/Async/NHSpecificTest/GH2463/Fixture.cs

@@ -9,6 +9,7 @@
 
 
 using System.Linq;
+using NHibernate.Criterion;
 using NHibernate.DomainModel;
 using NUnit.Framework;
 using NHibernate.Linq;
@@ -19,23 +20,68 @@ namespace NHibernate.Test.NHSpecificTest.GH2463
 	[TestFixture]
 	public class FixtureAsync : TestCase
 	{
+		protected override bool AppliesTo(Dialect.Dialect dialect)
+		{
+			return Dialect.SupportsScalarSubSelects;
+		}
+
 		protected override string[] Mappings
 		{
 			get { return new[] {"ABC.hbm.xml"}; }
 		}
 
+		protected override void OnSetUp()
+		{
+			using (var session = OpenSession())
+			using (var transaction = session.BeginTransaction())
+			{
+				var a = new A {Name = "A", AnotherName = "X"};
+				session.Save(a);
+
+				var b = new B {Name = "B", AnotherName = "X"};
+				session.Save(b);
+
+				transaction.Commit();
+			}
+		}
+
+		protected override void OnTearDown()
+		{
+			using (var session = OpenSession())
+			using (var transaction = session.BeginTransaction())
+			{
+				session.CreateQuery("delete from System.Object").ExecuteUpdate();
+
+				transaction.Commit();
+			}
+		}
+
+		//Also see GH-2599
 		[Test]
-		public async Task CanJoinOnEntityWithDiscriminatorAsync()
+		public async Task CanJoinOnEntityWithDiscriminatorLinqAsync()
 		{
 			using (var s = OpenSession())
 			{
-				await (s.Query<A>().Join(
-					s.Query<A>(),
-					a => a.Id,
-					b => b.Id,
+				var list = await (s.Query<A>().Join(
+					s.Query<B>(),
+					a => a.AnotherName,
+					b => b.AnotherName,
 					(a, b) =>
 						new {a, b}).ToListAsync());
 			}
 		}
+
+		[Test]
+		public async Task CanJoinOnEntityWithDiscriminatorQueryOverAsync()
+		{
+			using (var s = OpenSession())
+			{
+				A a = null;
+				B b = null;
+				var list = await (s.QueryOver<A>(() => a)
+							.JoinEntityAlias(() => b, () => a.AnotherName == b.AnotherName)
+							.Select((x) => a.AsEntity(), (x) => b.AsEntity()).ListAsync<object[]>());
+			}
+		}
 	}
 }

src/NHibernate.Test/NHSpecificTest/GH2463/Fixture.cs

@@ -1,4 +1,5 @@
 using System.Linq;
+using NHibernate.Criterion;
 using NHibernate.DomainModel;
 using NUnit.Framework;
 
@@ -7,23 +8,68 @@ namespace NHibernate.Test.NHSpecificTest.GH2463
 	[TestFixture]
 	public class Fixture : TestCase
 	{
+		protected override bool AppliesTo(Dialect.Dialect dialect)
+		{
+			return Dialect.SupportsScalarSubSelects;
+		}
+
 		protected override string[] Mappings
 		{
 			get { return new[] {"ABC.hbm.xml"}; }
 		}
 
+		protected override void OnSetUp()
+		{
+			using (var session = OpenSession())
+			using (var transaction = session.BeginTransaction())
+			{
+				var a = new A {Name = "A", AnotherName = "X"};
+				session.Save(a);
+
+				var b = new B {Name = "B", AnotherName = "X"};
+				session.Save(b);
+
+				transaction.Commit();
+			}
+		}
+
+		protected override void OnTearDown()
+		{
+			using (var session = OpenSession())
+			using (var transaction = session.BeginTransaction())
+			{
+				session.CreateQuery("delete from System.Object").ExecuteUpdate();
+
+				transaction.Commit();
+			}
+		}
+
+		//Also see GH-2599
 		[Test]
-		public void CanJoinOnEntityWithDiscriminator()
+		public void CanJoinOnEntityWithDiscriminatorLinq()
 		{
 			using (var s = OpenSession())
 			{
-				s.Query<A>().Join(
-					s.Query<A>(),
-					a => a.Id,
-					b => b.Id,
+				var list = s.Query<A>().Join(
+					s.Query<B>(),
+					a => a.AnotherName,
+					b => b.AnotherName,
 					(a, b) =>
 						new {a, b}).ToList();
 			}
 		}
+
+		[Test]
+		public void CanJoinOnEntityWithDiscriminatorQueryOver()
+		{
+			using (var s = OpenSession())
+			{
+				A a = null;
+				B b = null;
+				var list = s.QueryOver<A>(() => a)
+							.JoinEntityAlias(() => b, () => a.AnotherName == b.AnotherName)
+							.Select((x) => a.AsEntity(), (x) => b.AsEntity()).List<object[]>();
+			}
+		}
 	}
 }

src/NHibernate/Engine/JoinSequence.cs

@@ -237,9 +237,9 @@ private SqlString GetWithClause(IDictionary<string, IFilter> enabledFilters, ref
 			else if (string.IsNullOrEmpty(on))
 			{
 				// NH Different behavior : NH1179 and NH1293
-				// Apply filters in Many-To-One association
+				// Apply filters for entity joins and Many-To-One association
 				var enabledForManyToOne = FilterHelper.GetEnabledForManyToOne(enabledFilters);
-				if (enabledForManyToOne.Count > 0)
+				if (ForceFilter || enabledForManyToOne.Count > 0)
 					withConditions.Add(join.Joinable.FilterFragment(join.Alias, enabledForManyToOne));
 			}
 
@@ -346,6 +346,8 @@ public interface ISelector
 
 		public ISessionFactoryImplementor Factory => factory;
 
+		internal bool ForceFilter { get; set; }
+
 		public JoinSequence AddJoin(FromElement fromElement)
 		{
 			joins.AddRange(fromElement.JoinSequence.joins);

src/NHibernate/Hql/Ast/ANTLR/Tree/EntityJoinFromElement.cs

@@ -18,7 +18,7 @@ public EntityJoinFromElement(FromClause fromClause, IQueryable entityPersister,
 			InitializeEntity(fromClause, entityPersister.EntityName, entityPersister, entityType, alias, tableAlias);
 
 			//NH Specific: hibernate uses special class EntityJoinJoinSequenceImpl
-			JoinSequence = new JoinSequence(SessionFactoryHelper.Factory)
+			JoinSequence = new JoinSequence(SessionFactoryHelper.Factory) {ForceFilter = true}
 				.AddJoin(entityType, tableAlias, joinType, Array.Empty<string>());
 
 			fromClause.Walker.AddQuerySpaces(entityPersister.QuerySpaces);

src/NHibernate/Loader/JoinWalker.cs

@@ -388,7 +388,8 @@ internal void AddExplicitEntityJoinAssociation(
 					GetWithClause(path, pathAlias),
 					Factory,
 					enabledFilters,
-					GetSelectMode(path)), path);
+					GetSelectMode(path)) {ForceFilter = true},
+				path);
 			AddAssociation(assoc);
 		}
 
@@ -849,9 +850,9 @@ protected JoinFragment MergeOuterJoins(IList<OuterJoinableAssociation> associati
 				else
 				{
 					// NH Different behavior : NH1179 and NH1293
-					// Apply filters in Many-To-One association
+					// Apply filters for entity joins and Many-To-One associations
 					SqlString filter = null;
-					if (enabledFiltersForManyToOne.Count > 0)
+					if (oj.ForceFilter || enabledFiltersForManyToOne.Count > 0)
 					{
 						string manyToOneFilterFragment = oj.Joinable.FilterFragment(oj.RHSAlias, enabledFiltersForManyToOne);
 						bool joinClauseDoesNotContainsFilterAlready =

src/NHibernate/Loader/OuterJoinableAssociation.cs

@@ -100,6 +100,8 @@ public SelectMode SelectMode
 
 		public ISet<string> EntityFetchLazyProperties { get; set; }
 
+		internal bool ForceFilter { get; set; }
+
 		string[] IJoin.LHSColumns => lhsColumns;
 		string IJoin.Alias => RHSAlias;
 		IAssociationType IJoin.AssociationType => JoinableType;