DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG� RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=32250>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND� INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=32250 ------- Additional Comments From [EMAIL PROTECTED] 2004-11-15 22:55 ------- The fix you propose is a little simplistic as it removes the lock check and cleanup code for all external transactions and this is not correct. This code was added to the AbstractWebdavMethod so that all methods will run it when they are run and all "internally" controlled transactions currently do this. The same should be true for external transactions. The only external transaction for which this code should not be run is the UnlockMethod and only when it is performing its role as the transaction committer or aborter -- regardless of whether the tx is internal or external. With the fix I have supplied, which checks for this specific case, both internal and external transactions work as they should. I have tested this with our own client and tested that both tx commits and aborts work correctly. Aborted tx's cleanup any operations in progress including unlocking resources that were locked during the operation which would not have been possible if the LockMethod and UnlockMethod on the client did not enlist themselves to the transaction too. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
