Property-ref on many-to-one with composite id fails

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

@@ -0,0 +1,59 @@
+//------------------------------------------------------------------------------
+// <auto-generated>
+//     This code was generated by AsyncGenerator.
+//
+//     Changes to this file may cause incorrect behavior and will be lost if
+//     the code is regenerated.
+// </auto-generated>
+//------------------------------------------------------------------------------
+
+
+using NUnit.Framework;
+
+namespace NHibernate.Test.NHSpecificTest.GH1918
+{
+	using System.Threading.Tasks;
+	[TestFixture]
+	public class FixtureAsync : BugTestCase
+	{
+		private const string _ingData = "data_ing";
+		private readonly EmbId _ingId = new EmbId { X = 5, Y = 6 };
+
+		protected override void OnSetUp()
+		{
+			var edId = new EmbId { X = 1, Y = 2 };
+			var ed = new BiEmbIdRefEdEntity { Id = edId, Data = "data_ed" };
+			var ing = new BiEmbIdRefIngEntity { Id = _ingId, Data = _ingData, Reference = ed };
+			using (var s = OpenSession())
+			using (var tx = s.BeginTransaction())
+			{
+				s.Save(ed);
+				s.Save(ing);
+				tx.Commit();
+			}
+		}
+
+		[Test]
+		public async Task CanReadBidirectionalEntitiesWithEmbeddedIdAsync()
+		{
+			using (var s = OpenSession())
+			using (var tx = s.BeginTransaction())
+			{
+				var ingEntity = await (s.GetAsync<BiEmbIdRefIngEntity>(_ingId));
+				Assert.That(ingEntity.Data, Is.EqualTo(_ingData));
+				await (tx.CommitAsync());
+			}
+		}
+
+		protected override void OnTearDown()
+		{
+			using (var session = OpenSession())
+			using (var tx = session.BeginTransaction())
+			{
+				session.CreateQuery("delete from BiEmbIdRefIngEntity").ExecuteUpdate();
+				session.CreateQuery("delete from BiEmbIdRefEdEntity").ExecuteUpdate();
+				tx.Commit();
+			}
+		}
+	}
+}

src/NHibernate.Test/NHSpecificTest/GH1918/BiEmbIdRefEdEntity.cs

@@ -0,0 +1,22 @@
+namespace NHibernate.Test.NHSpecificTest.GH1918
+{
+	public class BiEmbIdRefEdEntity
+	{
+		public virtual EmbId Id { get; set; }
+		public virtual string Data { get; set; }
+		public virtual BiEmbIdRefIngEntity Referencing { get; set; }
+
+		public override bool Equals(object obj)
+		{
+			var casted = obj as BiEmbIdRefEdEntity;
+			if (casted == null)
+				return false;
+			return Id.Equals(casted.Id) && Data.Equals(casted.Data);
+		}
+
+		public override int GetHashCode()
+		{
+			return Id.GetHashCode() ^ Data.GetHashCode();
+		}
+	}
+}

src/NHibernate.Test/NHSpecificTest/GH1918/BiEmbIdRefIngEntity.cs

@@ -0,0 +1,22 @@
+namespace NHibernate.Test.NHSpecificTest.GH1918
+{
+	public class BiEmbIdRefIngEntity
+	{
+		public virtual EmbId Id { get; set; }
+		public virtual string Data { get; set; }
+		public virtual BiEmbIdRefEdEntity Reference { get; set; }
+
+		public override bool Equals(object obj)
+		{
+			var casted = obj as BiEmbIdRefIngEntity;
+			if (casted == null)
+				return false;
+			return Id.Equals(casted.Id) && Data.Equals(casted.Data);
+		}
+
+		public override int GetHashCode()
+		{
+			return Id.GetHashCode() ^ Data.GetHashCode();
+		}
+	}
+}

src/NHibernate.Test/NHSpecificTest/GH1918/EmbId.cs

@@ -0,0 +1,21 @@
+namespace NHibernate.Test.NHSpecificTest.GH1918
+{
+	public class EmbId
+	{
+		public virtual int X { get; set; }
+		public virtual int Y { get; set; }
+
+		public override bool Equals(object obj)
+		{
+			var casted = obj as EmbId;
+			if (casted == null)
+				return false;
+			return X == casted.X && Y == casted.Y;
+		}
+
+		public override int GetHashCode()
+		{
+			return X ^ Y;
+		}
+	}
+}

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

@@ -0,0 +1,48 @@
+using NUnit.Framework;
+
+namespace NHibernate.Test.NHSpecificTest.GH1918
+{
+	[TestFixture]
+	public class Fixture : BugTestCase
+	{
+		private const string _ingData = "data_ing";
+		private readonly EmbId _ingId = new EmbId { X = 5, Y = 6 };
+
+		protected override void OnSetUp()
+		{
+			var edId = new EmbId { X = 1, Y = 2 };
+			var ed = new BiEmbIdRefEdEntity { Id = edId, Data = "data_ed" };
+			var ing = new BiEmbIdRefIngEntity { Id = _ingId, Data = _ingData, Reference = ed };
+			using (var s = OpenSession())
+			using (var tx = s.BeginTransaction())
+			{
+				s.Save(ed);
+				s.Save(ing);
+				tx.Commit();
+			}
+		}
+
+		[Test]
+		public void CanReadBidirectionalEntitiesWithEmbeddedId()
+		{
+			using (var s = OpenSession())
+			using (var tx = s.BeginTransaction())
+			{
+				var ingEntity = s.Get<BiEmbIdRefIngEntity>(_ingId);
+				Assert.That(ingEntity.Data, Is.EqualTo(_ingData));
+				tx.Commit();
+			}
+		}
+
+		protected override void OnTearDown()
+		{
+			using (var session = OpenSession())
+			using (var tx = session.BeginTransaction())
+			{
+				session.CreateQuery("delete from BiEmbIdRefIngEntity").ExecuteUpdate();
+				session.CreateQuery("delete from BiEmbIdRefEdEntity").ExecuteUpdate();
+				tx.Commit();
+			}
+		}
+	}
+}

src/NHibernate.Test/NHSpecificTest/GH1918/Mappings.hbm.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8"?>
+<hibernate-mapping xmlns="urn:nhibernate-mapping-2.2"
+                   assembly="NHibernate.Test"
+                   namespace="NHibernate.Test.NHSpecificTest.GH1918">
+
+  <class name="BiEmbIdRefEdEntity">
+    <composite-id name="Id">
+      <key-property name="X"/>
+      <key-property name="Y"/>
+    </composite-id>
+    <property name="Data" />
+    <one-to-one name="Referencing"
+                class="BiEmbIdRefIngEntity"
+                property-ref="Reference"/>
+  </class>
+
+  <class name="BiEmbIdRefIngEntity">
+    <composite-id name="Id">
+      <key-property name="X"/>
+      <key-property name="Y"/>
+    </composite-id>
+    <property name="Data" />
+    <many-to-one name="Reference">
+      <column name="RefX" />
+      <column name="RefY" />
+    </many-to-one>
+  </class>
+</hibernate-mapping>

src/NHibernate/Async/Type/ComponentType.cs

@@ -324,9 +324,18 @@ public override Task<object> SemiResolveAsync(object value, ISessionImplementor
 			{
 				return Task.FromCanceled<object>(cancellationToken);
 			}
-			//note that this implementation is kinda broken
-			//for components with many-to-one associations
-			return ResolveIdentifierAsync(value, session, owner, cancellationToken);
+			try
+			{
+				//note that this implementation is kinda broken
+				//for components with many-to-one associations
+				if (ReturnedClass.IsInstanceOfType(value))
+					return Task.FromResult<object>(value);
+				return ResolveIdentifierAsync(value, session, owner, cancellationToken);
+			}
+			catch (Exception ex)
+			{
+				return Task.FromException<object>(ex);
+			}
 		}
 
 		public override async Task<bool> IsModifiedAsync(object old, object current, bool[] checkable, ISessionImplementor session, CancellationToken cancellationToken)

src/NHibernate/Type/ComponentType.cs

@@ -520,6 +520,8 @@ public override object SemiResolve(object value, ISessionImplementor session, ob
 		{
 			//note that this implementation is kinda broken
 			//for components with many-to-one associations
+			if (ReturnedClass.IsInstanceOfType(value))
+				return value;
 			return ResolveIdentifier(value, session, owner);
 		}