[jbosscache-commits] JBoss Cache SVN: r7581 - core/trunk/src/main/java/org/jboss/cache/transaction.

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2009-01-23 06:42:35 -0500 (Fri, 23 Jan 2009)
New Revision: 7581

Modified:
   core/trunk/src/main/java/org/jboss/cache/transaction/DummyTransaction.java
Log:
JBCACHE-1469:  DummyBaseTransactionManager behaves incorrectly with Status.STATUS_MARKED_ROLLBACK

Modified: core/trunk/src/main/java/org/jboss/cache/transaction/DummyTransaction.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/transaction/DummyTransaction.java	2009-01-23 11:22:21 UTC (rev 7580)
+++ core/trunk/src/main/java/org/jboss/cache/transaction/DummyTransaction.java	2009-01-23 11:42:35 UTC (rev 7581)
@@ -229,10 +229,13 @@
          throws RollbackException, IllegalStateException, SystemException
    {
       if (sync == null)
+      {
          throw new IllegalArgumentException("null synchronization " + this);
+      }
 
       switch (status)
       {
+         case Status.STATUS_MARKED_ROLLBACK:
          case Status.STATUS_ACTIVE:
          case Status.STATUS_PREPARING:
             break;
@@ -242,8 +245,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: