You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by le...@apache.org on 2010/04/20 13:10:34 UTC

svn commit: r935869 - /ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java

Author: lektran
Date: Tue Apr 20 11:10:34 2010
New Revision: 935869

URL: http://svn.apache.org/viewvc?rev=935869&view=rev
Log:
Fix a problem reported by Bob Morley in OFBIZ-3663.  When a delegator is cloned during test execution then the test mode isn't inherited by the new delegator and subsequently any changes made by that delegator are not tracked for rollback.  Fixed by setting the testMode of the new delegator to match that of the one being cloned.

Modified:
    ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java

Modified: ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java?rev=935869&r1=935868&r2=935869&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java Tue Apr 20 11:10:34 2010
@@ -3236,6 +3236,7 @@ public class GenericDelegator implements
         newDelegator.crypto = this.crypto;
         // In case this delegator is in testMode give it a reference to
         // the rollback list
+        newDelegator.testMode = this.testMode;
         testOperationsUpdater.set(newDelegator, this.testOperations);
         // not setting the sequencer so that we have unique sequences.