Author: ashish
Date: Fri Dec 26 09:55:20 2014
New Revision: 1647947

URL: http://svn.apache.org/r1647947
Log:
Applied hand merged bug fix changes from trunk r1647946.
==========================================================
Applied patch from jira issue - OFBIZ-5385 - Purchase order 
immutable/unchangeable errors when adding variant products.
In this changes Arun has taken care of cloning related concern raised by 
Adrian. Thanks Arun for the contribution.
==========================================================

Modified:
    
ofbiz/branches/release13.07/applications/product/src/org/ofbiz/product/product/ProductWorker.java

Modified: 
ofbiz/branches/release13.07/applications/product/src/org/ofbiz/product/product/ProductWorker.java
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/product/src/org/ofbiz/product/product/ProductWorker.java?rev=1647947&r1=1647946&r2=1647947&view=diff
==============================================================================
--- 
ofbiz/branches/release13.07/applications/product/src/org/ofbiz/product/product/ProductWorker.java
 (original)
+++ 
ofbiz/branches/release13.07/applications/product/src/org/ofbiz/product/product/ProductWorker.java
 Fri Dec 26 09:55:20 2014
@@ -1104,15 +1104,17 @@ nextProd:
                 Debug.logInfo("set the productId to: " + 
product.getString("productId"), module);
 
                 // copy the supplier
-                List<GenericValue> supplierProducts = 
delegator.findByAnd("SupplierProduct", UtilMisc.toMap("productId", productId), 
null, false);
+                List<GenericValue> supplierProducts = 
delegator.findByAnd("SupplierProduct", UtilMisc.toMap("productId", productId), 
null, true);
                 for (GenericValue supplierProduct: supplierProducts) {
+                    supplierProduct = (GenericValue) supplierProduct.clone();
                     supplierProduct.set("productId",  
product.getString("productId"));
                     supplierProduct.create();
                 }
 
                 // copy the content
-                List<GenericValue> productContents = 
delegator.findByAnd("ProductContent", UtilMisc.toMap("productId", productId), 
null, false);
+                List<GenericValue> productContents = 
delegator.findByAnd("ProductContent", UtilMisc.toMap("productId", productId), 
null, true);
                 for (GenericValue productContent: productContents) {
+                    productContent = (GenericValue) productContent.clone();
                     productContent.set("productId",  
product.getString("productId"));
                     productContent.create();
                 }


Reply via email to