[hibernate-commits] Hibernate SVN: r15250 - core/branches/Branch_3_2/src/org/hibernate/transaction.

[hibernate-commits at lists.jboss.org]

Author: steve.ebersole at jboss.com
Date: 2008-10-02 23:02:25 -0400 (Thu, 02 Oct 2008)
New Revision: 15250

Modified:
   core/branches/Branch_3_2/src/org/hibernate/transaction/JTATransactionFactory.java
Log:
HHH-3481 : minor bug in previous JTATransactionFactory patch

Modified: core/branches/Branch_3_2/src/org/hibernate/transaction/JTATransactionFactory.java
===================================================================
--- core/branches/Branch_3_2/src/org/hibernate/transaction/JTATransactionFactory.java	2008-10-02 21:14:24 UTC (rev 15249)
+++ core/branches/Branch_3_2/src/org/hibernate/transaction/JTATransactionFactory.java	2008-10-03 03:02:25 UTC (rev 15250)
@@ -155,12 +155,13 @@
 	 * @return The appropriate {@link UserTransaction} reference.
 	 */
 	protected UserTransaction getUserTransaction() {
-		log.trace( "Attempting to locate UserTransaction via JNDI [" + getUserTransactionName() + "]" );
+		final String utName = getUserTransactionName();
+		log.trace( "Attempting to locate UserTransaction via JNDI [" + utName + "]" );
 
 		try {
-			UserTransaction ut = ( UserTransaction ) getInitialContext().lookup( getUserTransactionName() );
+			UserTransaction ut = ( UserTransaction ) getInitialContext().lookup( utName );
 			if ( ut == null ) {
-				throw new TransactionException( "Naming service lookup for UserTransaction returned null [" + getUserTransactionName() +"]" );
+				throw new TransactionException( "Naming service lookup for UserTransaction returned null [" + utName +"]" );
 			}
 
 			log.trace( "Obtained UserTransaction" );
@@ -168,7 +169,7 @@
 			return ut;
 		}
 		catch ( NamingException ne ) {
-			throw new TransactionException( "Could not find UserTransaction in JNDI [" + getUserTransaction() + "]", ne );
+			throw new TransactionException( "Could not find UserTransaction in JNDI [" + utName + "]", ne );
 		}
 	}