Author: eckardjf
Date: Tue Apr 13 15:38:45 2010
New Revision: 933680
URL: http://svn.apache.org/viewvc?rev=933680&view=rev
Log:
replace references to this.delegator with this.getDelegator(), this.delegator
may be null if the cart comes from restored session
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=933680&r1=933679&r2=933680&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
Tue Apr 13 15:38:45 2010
@@ -2470,7 +2470,7 @@ public class ShoppingCart implements Ite
// set as the default shipping location the first from the list of
available shipping locations
if (this.getPartyId() != null &&
!this.getPartyId().equals("_NA_")) {
try {
- GenericValue orderParty =
delegator.findByPrimaryKey("Party", UtilMisc.toMap("partyId",
this.getPartyId()));
+ GenericValue orderParty =
this.getDelegator().findByPrimaryKey("Party", UtilMisc.toMap("partyId",
this.getPartyId()));
Collection shippingContactMechList =
ContactHelper.getContactMech(orderParty, "SHIPPING_LOCATION", "POSTAL_ADDRESS",
false);
if (UtilValidate.isNotEmpty(shippingContactMechList)) {
GenericValue shippingContactMech =
(GenericValue)(shippingContactMechList.iterator()).next();
@@ -2498,7 +2498,7 @@ public class ShoppingCart implements Ite
// the facilityId should be set prior to triggering default
options, otherwise we do not set up facility information
String defaultFacilityId = getFacilityId();
if (defaultFacilityId != null) {
- GenericValue facilityContactMech =
ContactMechWorker.getFacilityContactMechByPurpose(delegator, facilityId,
UtilMisc.toList("SHIPPING_LOCATION", "PRIMARY_LOCATION"));
+ GenericValue facilityContactMech =
ContactMechWorker.getFacilityContactMechByPurpose(this.getDelegator(),
facilityId, UtilMisc.toList("SHIPPING_LOCATION", "PRIMARY_LOCATION"));
if (facilityContactMech != null) {
this.setShippingContactMechId(0,
facilityContactMech.getString("contactMechId"));
}
@@ -2714,7 +2714,7 @@ public class ShoppingCart implements Ite
/** Add an orderTerm to the order */
public int addOrderTerm(String termTypeId, BigDecimal termValue, Long
termDays, String textValue) {
- GenericValue orderTerm =
GenericValue.create(delegator.getModelEntity("OrderTerm"));
+ GenericValue orderTerm = this.getDelegator().makeValue("OrderTerm");
orderTerm.put("termTypeId", termTypeId);
orderTerm.put("termValue", termValue);
orderTerm.put("termDays", termDays);
@@ -3311,6 +3311,8 @@ public class ShoppingCart implements Ite
return newProductId;
}
+ Delegator delegator = this.getDelegator();
+
//create new product and associate it
GenericValue product = item.getProduct();
String productName = product.getString("productName");
@@ -3350,7 +3352,7 @@ public class ShoppingCart implements Ite
//create a new WorkEffortGoodStandard based on existing one of
AGGREGATED product .
//Another approach could be to get WorkEffortGoodStandard of the
AGGREGATED product while creating production run.
- List productionRunTemplates =
getDelegator().findByAnd("WorkEffortGoodStandard", UtilMisc.toMap("productId",
item.getProductId(), "workEffortGoodStdTypeId", "ROU_PROD_TEMPLATE",
"statusId", "WEGS_CREATED"));
+ List productionRunTemplates =
delegator.findByAnd("WorkEffortGoodStandard", UtilMisc.toMap("productId",
item.getProductId(), "workEffortGoodStdTypeId", "ROU_PROD_TEMPLATE",
"statusId", "WEGS_CREATED"));
GenericValue productionRunTemplate =
EntityUtil.getFirst(EntityUtil.filterByDate(productionRunTemplates));
if (productionRunTemplate != null) {
serviceContext.clear();
@@ -3660,6 +3662,7 @@ public class ShoppingCart implements Ite
/** make a list of all OrderPaymentPreferences and Billing info including
all payment methods and types */
public List makeAllOrderPaymentInfos(LocalDispatcher dispatcher) {
+ Delegator delegator = this.getDelegator();
List allOpPrefs = new LinkedList();
BigDecimal remainingAmount =
this.getGrandTotal().subtract(this.getPaymentTotal());
remainingAmount = remainingAmount.setScale(2,
BigDecimal.ROUND_HALF_UP);
@@ -3694,7 +3697,7 @@ public class ShoppingCart implements Ite
inf.amount = remainingAmount;
remainingAmount = BigDecimal.ZERO;
}
- allOpPrefs.addAll(inf.makeOrderPaymentInfos(this.getDelegator(),
this));
+ allOpPrefs.addAll(inf.makeOrderPaymentInfos(delegator, this));
}
return allOpPrefs;
}
@@ -4058,7 +4061,7 @@ public class ShoppingCart implements Ite
String facilityId = null;
if (UtilValidate.isNotEmpty(this.getProductStoreId())) {
try {
- GenericValue productStore =
delegator.findByPrimaryKeyCache("ProductStore",
UtilMisc.toMap("productStoreId", this.getProductStoreId()));
+ GenericValue productStore =
this.getDelegator().findByPrimaryKeyCache("ProductStore",
UtilMisc.toMap("productStoreId", this.getProductStoreId()));
facilityId = productStore.getString("inventoryFacilityId");
} catch (Exception e) {
Debug.logError(UtilProperties.getMessage(resource_error,"OrderProblemGettingProductStoreRecords",
locale) + e.getMessage(), module);