You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Jacopo Cappellato <ti...@sastau.it> on 2007/12/12 06:34:35 UTC
Re: svn commit: r603423 - in /ofbiz/trunk/applications/product/src/org/ofbiz:
product/price/PriceServices.java shipment/thirdparty/usps/UspsServices.java
Bilgin,
this is a good catch, thanks.
Jacopo
bibryam@apache.org wrote:
> Author: bibryam
> Date: Tue Dec 11 16:21:27 2007
> New Revision: 603423
>
> URL: http://svn.apache.org/viewvc?rev=603423&view=rev
> Log:
> Fixed a bug in calculateProductPrice service when called for products with agreement.
> Removed non UTF character causing warnings.
>
> Modified:
> ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java
> ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java
>
> 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=603423&r1=603422&r2=603423&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 Tue Dec 11 16:21:27 2007
> @@ -485,7 +485,7 @@
> List orderItemPriceInfos = FastList.newInstance();
> if (defaultPriceValue != null) {
> // If a price calc formula (service) is specified, then use it to get the unit price
> - if (UtilValidate.isNotEmpty(defaultPriceValue.getString("customPriceCalcService"))) {
> + if ("ProductPrice".equals(defaultPriceValue.getEntityName()) && UtilValidate.isNotEmpty(defaultPriceValue.getString("customPriceCalcService"))) {
> GenericValue customMethod = null;
> try {
> customMethod = defaultPriceValue.getRelatedOne("CustomMethod");