[jbosscache-commits] JBoss Cache SVN: r7582 - core/branches/flat/src/main/java/org/horizon/transaction.

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2009-01-23 06:45:16 -0500 (Fri, 23 Jan 2009)
New Revision: 7582

Modified:
   core/branches/flat/src/main/java/org/horizon/transaction/DummyTransaction.java
Log:
JBCACHE-1469:  DummyBaseTransactionManager behaves incorrectly with Status.STATUS_MARKED_ROLLBACK

Modified: core/branches/flat/src/main/java/org/horizon/transaction/DummyTransaction.java
===================================================================
--- core/branches/flat/src/main/java/org/horizon/transaction/DummyTransaction.java	2009-01-23 11:42:35 UTC (rev 7581)
+++ core/branches/flat/src/main/java/org/horizon/transaction/DummyTransaction.java	2009-01-23 11:45:16 UTC (rev 7582)
@@ -189,6 +189,7 @@
          throw new IllegalArgumentException("null synchronization " + this);
 
       switch (status) {
+         case Status.STATUS_MARKED_ROLLBACK:  // we should allow synchronizations to be registered even if the status is MARKED_ROLLBACK
          case Status.STATUS_ACTIVE:
          case Status.STATUS_PREPARING:
             break;
@@ -198,8 +199,6 @@
             throw new IllegalStateException("already started committing. " + this);
          case Status.STATUS_COMMITTED:
             throw new IllegalStateException("already committed. " + this);
-         case Status.STATUS_MARKED_ROLLBACK:
-            throw new RollbackException("already marked for rollback " + this);
          case Status.STATUS_ROLLING_BACK:
             throw new RollbackException("already started rolling back. " + this);
          case Status.STATUS_ROLLEDBACK: