svn commit: r1647947 - /ofbiz/branches/release13.07/applications/product/src/org/ofbiz/product/product/ProductWorker.java

[ashish-18]

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();
                 }