HHH-13790 Temporary session not being closed

Author: flexo0

hibernate-core/src/main/java/org/hibernate/collection/spi/AbstractPersistentCollection.java

@@ -238,31 +238,30 @@ else if ( !session.isConnected() ) {
 
 		SharedSessionContractImplementor originalSession = null;
 		boolean isJTA = false;
+		try {
+			if ( tempSession != null ) {
+				isTempSession = true;
+				originalSession = session;
+				session = tempSession;
+
+				isJTA = session.getTransactionCoordinator().getTransactionCoordinatorBuilder().isJta();
+
+				if ( !isJTA ) {
+					// Explicitly handle the transactions only if we're not in
+					// a JTA environment.  A lazy loading temporary session can
+					// be created even if a current session and transaction are
+					// open (ex: session.clear() was used).  We must prevent
+					// multiple transactions.
+					session.beginTransaction();
+				}
 
-		if ( tempSession != null ) {
-			isTempSession = true;
-			originalSession = session;
-			session = tempSession;
-
-			isJTA = session.getTransactionCoordinator().getTransactionCoordinatorBuilder().isJta();
-
-			if ( !isJTA ) {
-				// Explicitly handle the transactions only if we're not in
-				// a JTA environment.  A lazy loading temporary session can
-				// be created even if a current session and transaction are
-				// open (ex: session.clear() was used).  We must prevent
-				// multiple transactions.
-				session.beginTransaction();
+				final CollectionPersister collectionDescriptor =
+						session.getSessionFactory()
+								.getMappingMetamodel()
+								.getCollectionDescriptor( getRole() );
+				session.getPersistenceContextInternal()
+						.addUninitializedDetachedCollection( collectionDescriptor, this );
 			}
-
-			final CollectionPersister collectionDescriptor =
-					session.getSessionFactory()
-							.getMappingMetamodel()
-							.getCollectionDescriptor( getRole() );
-			session.getPersistenceContextInternal().addUninitializedDetachedCollection( collectionDescriptor, this );
-		}
-
-		try {
 			return lazyInitializationWork.doWork();
 		}
 		finally {