Update of /var/cvs/src/org/mmbase/module/core
In directory james.mmbase.org:/tmp/cvs-serv15451

Modified Files:
        TransactionManager.java 
Log Message:
this solves the other failing testcase in TransactionTest


See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/module/core


Index: TransactionManager.java
===================================================================
RCS file: /var/cvs/src/org/mmbase/module/core/TransactionManager.java,v
retrieving revision 1.44
retrieving revision 1.45
diff -u -b -r1.44 -r1.45
--- TransactionManager.java     28 Jul 2008 17:42:17 -0000      1.44
+++ TransactionManager.java     28 Jul 2008 17:49:01 -0000      1.45
@@ -21,7 +21,7 @@
  * @javadoc
  *
  * @author Rico Jansen
- * @version $Id: TransactionManager.java,v 1.44 2008/07/28 17:42:17 michiel 
Exp $
+ * @version $Id: TransactionManager.java,v 1.45 2008/07/28 17:49:01 michiel 
Exp $
  */
 public class TransactionManager {
 
@@ -237,7 +237,13 @@
     private void commitNode(Object user, MMObjectNode node, int i, int[] 
nodestate, int[] nodeexist) {
 
         if (nodeexist[i] == I_EXISTS_YES ) {
-            if (! node.isChanged()) return;
+            //if (! node.isChanged()) return;
+            // Commit also if not changed, because the node may have been 
deleted or changed by
+            // someone else. It is like this in the transaction it should be 
saved like this.
+            // See also MMB-1680
+            // TODO: what is the performance penalty here?
+
+
             // use safe commit, which locks the node cache
             boolean commitOK;
             if (user instanceof UserContext) {
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to