You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Patrick Antivackis (JIRA)" <ji...@apache.org> on 2009/10/28 11:53:59 UTC

[jira] Updated: (OFBIZ-3099) PriceServices.calculateProductPrice should not process or return obsolete PriceRules

     [ https://issues.apache.org/jira/browse/OFBIZ-3099?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Antivackis updated OFBIZ-3099:
--------------------------------------

    Attachment: Patch-OFBIZ-3099.txt

This patch add a filterByDate to the PriceRules obtained after the call to the makeProducePriceRuleList function 

> PriceServices.calculateProductPrice should not process or return obsolete PriceRules
> ------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-3099
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3099
>             Project: OFBiz
>          Issue Type: Bug
>          Components: product
>    Affects Versions: SVN trunk
>            Reporter: Patrick Antivackis
>             Fix For: SVN trunk
>
>         Attachments: Patch-OFBIZ-3099.txt
>
>
> There is no filterByDate done on the allProductPriceRules returned line 559.
> The consequences are that a findAllQuantityPrices=Y return a Map with some obsolete PriceRule
> The calcPriceResultFromRules process some PriceRules that could be filtered before.
> I don't see the point of returning all the PriceRules found when using findAllQuantityPrices=Y as this map is used to display active price quantity condition to the customer.
> If the service do need to return all PriceRules, as it is today, the service should be updated in order to allow to specify if the returned list should be only active or all.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.