Revert previous behaviour

Author: hazzik

src/NHibernate/Async/Event/Default/AbstractSaveEventListener.cs

@@ -186,10 +186,8 @@ protected virtual async Task<object> PerformSaveOrReplicateAsync(object entity,
 
 			object id = key == null ? null : key.Identifier;
 
-			// NH Different behavior (shouldDelayIdentityInserts=false anyway)
-			//bool inTxn = source.ConnectionManager.IsInActiveTransaction;
-			//bool shouldDelayIdentityInserts = !inTxn && !requiresImmediateIdAccess;
-			bool shouldDelayIdentityInserts = false;
+			bool inTxn = source.ConnectionManager.IsInActiveTransaction;
+			bool shouldDelayIdentityInserts = !inTxn && !requiresImmediateIdAccess;
 
 			// Put a placeholder in entries, so we don't recurse back and try to save() the
 			// same object again. QUESTION: should this be done before onSave() is called?

src/NHibernate/Event/Default/AbstractSaveEventListener.cs

@@ -211,10 +211,8 @@ protected virtual object PerformSaveOrReplicate(object entity, EntityKey key, IE
 
 			object id = key == null ? null : key.Identifier;
 
-			// NH Different behavior (shouldDelayIdentityInserts=false anyway)
-			//bool inTxn = source.ConnectionManager.IsInActiveTransaction;
-			//bool shouldDelayIdentityInserts = !inTxn && !requiresImmediateIdAccess;
-			bool shouldDelayIdentityInserts = false;
+			bool inTxn = source.ConnectionManager.IsInActiveTransaction;
+			bool shouldDelayIdentityInserts = !inTxn && !requiresImmediateIdAccess;
 
 			// Put a placeholder in entries, so we don't recurse back and try to save() the
 			// same object again. QUESTION: should this be done before onSave() is called?