You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by jo...@apache.org on 2006/06/23 01:03:47 UTC

svn commit: r416496 - in /jakarta/commons/proper/transaction/trunk: RELEASE-NOTES.txt src/java/org/apache/commons/transaction/file/FileResourceManager.java

Author: joerg
Date: Thu Jun 22 16:03:46 2006
New Revision: 416496

URL: http://svn.apache.org/viewvc?rev=416496&view=rev
Log:
Fixed issue with deleteResource(..) and createResource(..) of FileResourceManager seen as read-only operations.

Modified:
    jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt
    jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/file/FileResourceManager.java

Modified: jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt?rev=416496&r1=416495&r2=416496&view=diff
==============================================================================
--- jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt (original)
+++ jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt Thu Jun 22 16:03:46 2006
@@ -36,6 +36,7 @@
 - Fixed bugzilla issue 35377. Synchronizing on Set activeTransactions while iterating over its items in
   OptimisticMapWrapper.copyChangesToConcurrentTransactions() for preventing ConcurrentModificationException.
 - Fixed bugzilla issue 37379. The construction of the transaction specific path is now done via a TransactionIdToPathMapper.
+- Fixed issue with deleteResource(..) and createResource(..) of FileResourceManager seen as read-only operations.
 
 KNOWN ISSUES
 ------------

Modified: jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/file/FileResourceManager.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/file/FileResourceManager.java?rev=416496&r1=416495&r2=416496&view=diff
==============================================================================
--- jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/file/FileResourceManager.java (original)
+++ jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/file/FileResourceManager.java Thu Jun 22 16:03:46 2006
@@ -729,6 +729,7 @@
         String txDeletePath = getDeletePath(txId, resourceId);
         String mainPath = getMainPath(resourceId);
         try {
+            getContext(txId).readOnly = false;
 
             // first undo change / create when there was one
             undoScheduledChangeOrCreate(txId, resourceId);
@@ -769,7 +770,8 @@
 
         String txChangePath = getChangePath(txId, resourceId);
         try {
-
+            getContext(txId).readOnly = false;
+            
             // creation means either undoing a delete or actually scheduling a create
             if (!undoScheduledDelete(txId, resourceId)) {
                 FileHelper.createFile(txChangePath);



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org