Author: jleroux
Date: Wed May 7 11:44:44 2014
New Revision: 1592977
URL: http://svn.apache.org/r1592977
Log:
Fixes a regression introduced by r1125215 "Add an automatic product price
currency conversion which can be switched off in a properties file."
r1125215 broke the possibility to set a currency to a product store.
The automatic product price currency conversion should not be activated by
default
Correctly formats changes done by r1125215 in PriceServices.java
Modified:
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCartItem.java
ofbiz/trunk/applications/product/config/catalog.properties
ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java
Modified:
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCartItem.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCartItem.java?rev=1592977&r1=1592976&r2=1592977&view=diff
==============================================================================
---
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCartItem.java
(original)
+++
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/ShoppingCartItem.java
Wed May 7 11:44:44 2014
@@ -1198,7 +1198,11 @@ public class ShoppingCartItem implements
}
}
}
- priceContext.put("currencyUomIdTo", cart.getCurrency());
+ if
("true".equals(UtilProperties.getPropertyValue("catalog.properties",
"convertProductPriceCurrency"))){
+ priceContext.put("currencyUomIdTo",
cart.getCurrency());
+ } else {
+ priceContext.put("currencyUomId", cart.getCurrency());
+ }
priceContext.put("prodCatalogId", this.getProdCatalogId());
priceContext.put("webSiteId", cart.getWebSiteId());
priceContext.put("productStoreId",
cart.getProductStoreId());
Modified: ofbiz/trunk/applications/product/config/catalog.properties
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/config/catalog.properties?rev=1592977&r1=1592976&r2=1592977&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/config/catalog.properties (original)
+++ ofbiz/trunk/applications/product/config/catalog.properties Wed May 7
11:44:44 2014
@@ -43,4 +43,4 @@ image.management.autoApproveImage=Y
image.management.multipleApproval=N
# Automatic product price currency conversion
-convertProductPriceCurrency=true
+convertProductPriceCurrency=false
Modified:
ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java?rev=1592977&r1=1592976&r2=1592977&view=diff
==============================================================================
---
ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java
(original)
+++
ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java
Wed May 7 11:44:44 2014
@@ -522,9 +522,9 @@ public class PriceServices {
}
// Convert the value to the price currency, if required
- if("true".equals(UtilProperties.getPropertyValue("catalog.properties",
"convertProductPriceCurrency"))){
+ if
("true".equals(UtilProperties.getPropertyValue("catalog.properties",
"convertProductPriceCurrency"))) {
if (UtilValidate.isNotEmpty(currencyDefaultUomId) &&
UtilValidate.isNotEmpty(currencyUomIdTo) &&
!currencyDefaultUomId.equals(currencyUomIdTo)) {
- if(UtilValidate.isNotEmpty(result)){
+ if (UtilValidate.isNotEmpty(result)) {
Map<String, Object> convertPriceMap =
FastMap.newInstance();
for (Map.Entry<String, Object> entry : result.entrySet()) {
BigDecimal tempPrice = BigDecimal.ZERO;
@@ -545,22 +545,23 @@ public class PriceServices {
else if (entry.getKey() == "listPrice")
tempPrice = (BigDecimal) entry.getValue();
- if(tempPrice != null && tempPrice != BigDecimal.ZERO){
+ if (tempPrice != null && tempPrice != BigDecimal.ZERO)
{
Map<String, Object> priceResults =
FastMap.newInstance();
try {
- priceResults =
dispatcher.runSync("convertUom", UtilMisc.<String, Object>toMap("uomId",
currencyDefaultUomId, "uomIdTo", currencyUomIdTo, "originalValue", tempPrice ,
"defaultDecimalScale" , Long.valueOf(2) , "defaultRoundingMode" , "HalfUp"));
+ priceResults =
dispatcher.runSync("convertUom", UtilMisc.<String, Object> toMap("uomId",
currencyDefaultUomId, "uomIdTo", currencyUomIdTo,
+ "originalValue", tempPrice,
"defaultDecimalScale", Long.valueOf(2), "defaultRoundingMode", "HalfUp"));
if (ServiceUtil.isError(priceResults) ||
(priceResults.get("convertedValue") == null)) {
- Debug.logWarning("Unable to convert " +
entry.getKey() + " for product " + productId , module);
- }
+ Debug.logWarning("Unable to convert " +
entry.getKey() + " for product " + productId, module);
+ }
} catch (GenericServiceException e) {
Debug.logError(e, module);
}
convertPriceMap.put(entry.getKey(),
priceResults.get("convertedValue"));
- }else{
+ } else {
convertPriceMap.put(entry.getKey(),
entry.getValue());
}
}
- if(UtilValidate.isNotEmpty(convertPriceMap)){
+ if (UtilValidate.isNotEmpty(convertPriceMap)) {
convertPriceMap.put("currencyUsed", currencyUomIdTo);
result = convertPriceMap;
}
