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.