svn commit: r522996 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java

[jaz-3]

Author: jaz
Date: Tue Mar 27 10:00:23 2007
New Revision: 522996

URL: http://svn.apache.org/viewvc?view=rev&rev=522996
Log:
more non-product order related fixes

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

Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java?view=diff&rev=522996&r1=522995&r2=522996
==============================================================================
--- ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java (original)
+++ ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java Tue Mar 27 10:00:23 2007
@@ -32,6 +32,7 @@
 
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.UtilMisc;
+import org.ofbiz.base.util.UtilValidate;
 import org.ofbiz.entity.GenericDelegator;
 import org.ofbiz.entity.GenericEntity;
 import org.ofbiz.entity.GenericEntityException;
@@ -180,23 +181,26 @@
 
                         while (orderItemsIter != null && orderItemsIter.hasNext()) {
                             GenericValue orderItem = (GenericValue) orderItemsIter.next();
-                            // for each order item get the associated product
-                            GenericValue product = orderItem.getRelatedOneCache("Product");
+                            String productId = orderItem.getString("productId");
+                            if (UtilValidate.isNotEmpty(productId)) {
+                                // for each order item get the associated product
+                                GenericValue product = orderItem.getRelatedOneCache("Product");
 
-                            products.put(product.get("productId"), product);
+                                products.put(product.get("productId"), product);
 
-                            Integer curQuant = (Integer) productQuantities.get(product.get("productId"));
+                                Integer curQuant = (Integer) productQuantities.get(product.get("productId"));
 
-                            if (curQuant == null) curQuant = new Integer(0);
-                            Double orderQuant = orderItem.getDouble("quantity");
+                                if (curQuant == null) curQuant = new Integer(0);
+                                Double orderQuant = orderItem.getDouble("quantity");
 
-                            if (orderQuant == null) orderQuant = new Double(0.0);
-                            productQuantities.put(product.get("productId"), new Integer(curQuant.intValue() + orderQuant.intValue()));
+                                if (orderQuant == null) orderQuant = new Double(0.0);
+                                productQuantities.put(product.get("productId"), new Integer(curQuant.intValue() + orderQuant.intValue()));
 
-                            Integer curOcc = (Integer) productOccurances.get(product.get("productId"));
+                                Integer curOcc = (Integer) productOccurances.get(product.get("productId"));
 
-                            if (curOcc == null) curOcc = new Integer(0);
-                            productOccurances.put(product.get("productId"), new Integer(curOcc.intValue() + 1));
+                                if (curOcc == null) curOcc = new Integer(0);
+                                productOccurances.put(product.get("productId"), new Integer(curOcc.intValue() + 1));
+                            }
                         }
                     }
                 }