Fix NH-1587

SVN: trunk@3927

Author: fabiomaulo

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

@@ -8,7 +8,7 @@ namespace NHibernate.Test.NHSpecificTest.NH1587
 	[TestFixture]
 	public class Fixture
 	{
-		[Test, Ignore("Not fixed yet")]
+		[Test]
 		public void Bug()
 		{
 			XmlConfigurator.Configure();

src/NHibernate/Tuple/Entity/AbstractEntityTuplizer.cs

@@ -23,7 +23,7 @@ public abstract class AbstractEntityTuplizer : IEntityTuplizer
 		protected internal IGetter[] getters;
 		protected internal ISetter[] setters;
 		protected internal bool hasCustomAccessors;
-		private readonly IInstantiator instantiator;
+		protected IInstantiator instantiator;
 		private readonly IProxyFactory proxyFactory;
 		private readonly IAbstractComponentType identifierMapperType;
 
@@ -67,7 +67,8 @@ public AbstractEntityTuplizer(EntityMetamodel entityMetamodel, PersistentClass m
 			}
 			hasCustomAccessors = foundCustomAccessor;
 
-			instantiator = BuildInstantiator(mappingInfo);
+			//NH-1587
+			//instantiator = BuildInstantiator(mappingInfo);
 
 			if (entityMetamodel.IsLazy)
 			{

src/NHibernate/Tuple/Entity/DynamicMapEntityTuplizer.cs

@@ -14,7 +14,11 @@ public class DynamicMapEntityTuplizer : AbstractEntityTuplizer
 		private static readonly ILog log = LogManager.GetLogger(typeof(PocoEntityTuplizer));
 
 		internal DynamicMapEntityTuplizer(EntityMetamodel entityMetamodel, PersistentClass mappingInfo)
-			: base(entityMetamodel, mappingInfo) {}
+			: base(entityMetamodel, mappingInfo)
+		{
+			// NH different behavior fo NH-1587
+			instantiator = BuildInstantiator(mappingInfo);
+		}
 
 		public override System.Type ConcreteProxyClass
 		{

src/NHibernate/Tuple/Entity/PocoEntityTuplizer.cs

@@ -40,13 +40,16 @@ public PocoEntityTuplizer(EntityMetamodel entityMetamodel, PersistentClass mappe
 					lazyPropertyNames.Add(property.Name);
 			}
 
-			if (hasCustomAccessors || !Cfg.Environment.UseReflectionOptimizer)
+			if (Cfg.Environment.UseReflectionOptimizer)
 			{
-				optimizer = null;
+				// NH different behavior fo NH-1587
+				optimizer = Cfg.Environment.BytecodeProvider.GetReflectionOptimizer(mappedClass, getters, setters);
 			}
-			else
+			instantiator = BuildInstantiator(mappedEntity);
+
+			if (hasCustomAccessors)
 			{
-				optimizer = Cfg.Environment.BytecodeProvider.GetReflectionOptimizer(mappedClass, getters, setters);
+				optimizer = null;
 			}
 
 			proxyValidator = Cfg.Environment.BytecodeProvider.ProxyFactoryFactory.ProxyValidator;
@@ -81,10 +84,12 @@ protected internal override IInstantiator BuildInstantiator(PersistentClass pers
 		{
 			if (optimizer == null)
 			{
+				log.Debug("Create Instantiator without optimizer for:" + persistentClass.MappedClass.FullName);
 				return new PocoInstantiator(persistentClass, null);
 			}
 			else
 			{
+				log.Debug("Create Instantiator using optimizer for:" + persistentClass.MappedClass.FullName);
 				return new PocoInstantiator(persistentClass, optimizer.InstantiationOptimizer);
 			}
 		}