You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by bi...@apache.org on 2009/04/06 10:12:03 UTC

svn commit: r762248 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCart.java

Author: bibryam
Date: Mon Apr  6 08:12:03 2009
New Revision: 762248

URL: http://svn.apache.org/viewvc?rev=762248&view=rev
Log:
Fixed a bug reported by CJay Harton in user ML, that caused new product creation for the same aggregated product configuration.

Modified:
    ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCart.java

Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCart.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCart.java?rev=762248&r1=762247&r2=762248&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCart.java (original)
+++ ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCart.java Mon Apr  6 08:12:03 2009
@@ -3265,7 +3265,7 @@
         try {
             //first search for existing productId
             newProductId = ProductWorker.getAggregatedInstanceId(getDelegator(), item.getProductId(), configId);
-            if (configId.equals(newProductId)) {
+            if (newProductId != null) {
                 return newProductId;
             }