NH-3757 - Allow dynamic entity mapped with entity-name to have a component of a fixed class

Author: ogvolkov

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

@@ -0,0 +1,36 @@
+using System.Collections.Generic;
+using NUnit.Framework;
+
+namespace NHibernate.Test.NHSpecificTest.NH3757
+{
+    [TestFixture]
+    public class Fixture : BugTestCase
+    {
+        protected override void OnTearDown()
+        {
+            using (ISession s = OpenSession())
+            {
+                using (ITransaction tx = s.BeginTransaction())
+                {
+                    s.CreateSQLQuery("delete from EntityName").ExecuteUpdate();
+                    tx.Commit();
+                }
+            }
+        }
+
+        [Test]
+        public void ShouldBePossibleToHaveComponentInEntityNameMappedEntity()
+        {
+            using (ISession session = OpenSession())
+            using (ITransaction transaction = session.BeginTransaction())
+            {
+                var e1 = new Dictionary<string, object>();
+                e1["Money"] = new Money { Amount = 100m, Currency = "USD" };
+                session.Save("EntityName", e1);
+
+                session.Flush();
+                transaction.Commit();
+            }
+        }
+    }
+}

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

@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<hibernate-mapping xmlns="urn:nhibernate-mapping-2.2" assembly="NHibernate.Test" namespace="NHibernate.Test.NHSpecificTest.NH3757">
+
+  <class entity-name="EntityName">
+    <id name="Id" type="Int32" generator="native" />
+    <component name="Money" class="Money">
+      <property name="Amount" type="decimal"/>
+      <property name="Currency" type="string"/>      
+    </component>
+  </class>
+
+</hibernate-mapping>

src/NHibernate.Test/NHSpecificTest/NH3757/Money.cs

@@ -0,0 +1,9 @@
+namespace NHibernate.Test.NHSpecificTest.NH3757
+{
+    public class Money
+    {
+        public virtual decimal Amount { get; set; }
+
+        public virtual string Currency { get; set; }
+    }
+}

src/NHibernate.Test/NHibernate.Test.csproj

@@ -1321,6 +1321,8 @@
     <Compile Include="NHSpecificTest\NH3641\TestFixture.cs" />
     <Compile Include="NHSpecificTest\NH3800\Domain.cs" />
     <Compile Include="NHSpecificTest\NH3800\Fixture.cs" />
+    <Compile Include="NHSpecificTest\NH3757\Fixture.cs" />
+    <Compile Include="NHSpecificTest\NH3757\Money.cs" />
     <Compile Include="NHSpecificTest\Properties\CompositePropertyRefTest.cs" />
     <Compile Include="NHSpecificTest\Properties\DynamicEntityTest.cs" />
     <Compile Include="NHSpecificTest\Properties\Model.cs" />
@@ -3221,6 +3223,7 @@
     <EmbeddedResource Include="NHSpecificTest\NH3800\Mappings.hbm.xml">
       <SubType>Designer</SubType>
     </EmbeddedResource>
+    <EmbeddedResource Include="NHSpecificTest\NH3757\Mappings.hbm.xml" />
     <EmbeddedResource Include="LazyComponentTest\Person.hbm.xml" />
     <EmbeddedResource Include="NHSpecificTest\NH3372\Mappings.hbm.xml" />
     <EmbeddedResource Include="NHSpecificTest\NH3567\Mappings.hbm.xml" />

src/NHibernate/Tuple/Component/ComponentEntityModeToTuplizerMapping.cs

@@ -46,7 +46,7 @@ public ComponentEntityModeToTuplizerMapping(Mapping.Component component)
 			userSuppliedTuplizerImpls.TryGetValue(EntityMode.Poco, out tempObject2);
 			userSuppliedTuplizerImpls.Remove(EntityMode.Poco);
 			tuplizerImpl = tempObject2;
-			if (owner.HasPocoRepresentation && component.HasPocoRepresentation)
+			if (component.HasPocoRepresentation)
 			{
 				if (tuplizerImpl == null)
 				{