[jboss-svn-commits] JBL Code SVN: r33854 - in labs/jbosstm/trunk: ArjunaJTS/jtax/classes/com/arjuna/ats/internal/jta/transaction/jts and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: jhalliday
Date: 2010-07-13 08:30:37 -0400 (Tue, 13 Jul 2010)
New Revision: 33854

Modified:
   labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/transaction/arjunacore/TransactionImple.java
   labs/jbosstm/trunk/ArjunaJTS/jtax/classes/com/arjuna/ats/internal/jta/transaction/jts/TransactionImple.java
Log:
Improve TransactionImple.isAlive JBTM-757


Modified: labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/transaction/arjunacore/TransactionImple.java
===================================================================
--- labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/transaction/arjunacore/TransactionImple.java	2010-07-13 12:29:58 UTC (rev 33853)
+++ labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/transaction/arjunacore/TransactionImple.java	2010-07-13 12:30:37 UTC (rev 33854)
@@ -1063,7 +1063,15 @@
      * @return
      */
     public boolean isAlive() {
-        return _transactions.contains(this);
+        try {
+            if(_theTransaction != null) {
+                return _transactions.containsKey(this.get_uid());
+            } else {
+                return false;
+            }
+        } catch(NullPointerException e) {
+            return false; // there is no tx/action, therefore it's not alive.
+        }
     }
 
     protected TransactionImple()

Modified: labs/jbosstm/trunk/ArjunaJTS/jtax/classes/com/arjuna/ats/internal/jta/transaction/jts/TransactionImple.java
===================================================================
--- labs/jbosstm/trunk/ArjunaJTS/jtax/classes/com/arjuna/ats/internal/jta/transaction/jts/TransactionImple.java	2010-07-13 12:29:58 UTC (rev 33853)
+++ labs/jbosstm/trunk/ArjunaJTS/jtax/classes/com/arjuna/ats/internal/jta/transaction/jts/TransactionImple.java	2010-07-13 12:30:37 UTC (rev 33854)
@@ -1193,7 +1193,15 @@
      * @return
      */
     public boolean isAlive() {
-        return _transactions.contains(this);
+        try {
+            if(_theTransaction != null) {
+                return _transactions.containsKey(this.get_uid());
+            } else {
+                return false;
+            }
+        } catch(NullPointerException e) {
+            return false; // there is no tx/action, therefore it's not alive.
+        }
     }