pricelist: no match when first item is based on supplier info and partner is not in it

Bug #860625 reported by Numérigraphe
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Medium
Albert Cervera i Areny - http://www.NaN-tic.com

Bug Description

In v5.0, in the module "product" and in nan_pricelist_extended, it seems that there is a case where no price can be found even though an item would match :
If the first item is based on the partner section, and the current partner is not in the supplier info, then this item should be skipped and the next item should be considered.
Instead, this item is not skipped.
Lionel Sausin

Revision history for this message
Numérigraphe (numerigraphe) wrote :

I think this is due to price_get_extended() beeing allowed to select pricelist items with base=-2 regardless of whether the partner is among the suppliers of the product or not.

description: updated
summary: - nan_pricelist_extended: no pricelist can match when first item is based
- partnerinfo
+ nan_pricelist_extended: no match when first item is based on supplier
+ info and partner is not in it
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote : Re: nan_pricelist_extended: no match when first item is based on supplier info and partner is not in it

As I don't know of any specific project for NaN's nan_pricelist_extended module, I suppose openobject-addons in the right place.
Let's assign it to Albert and let him manage the validation of the bug and merge proposal.

Changed in openobject-addons:
assignee: nobody → Albert Cervera i Areny - http://www.NaN-tic.com (albert-nan)
Revision history for this message
Numérigraphe (numerigraphe) wrote :

It turns out that the module "product" in the standard addons in v5.0 is affected too.
I'll push a merge proposal in a moment.
Lionel Sausin.

summary: - nan_pricelist_extended: no match when first item is based on supplier
- info and partner is not in it
+ pricelist: no match when first item is based on supplier info and
+ partner is not in it
description: updated
Revision history for this message
Numérigraphe (numerigraphe) wrote :

The modules "product" in 6.0 and trunk are affected too, so Olivier could you please assign this to the core team ?
I've pushed merge requests for a tentative fix on all the branches.
Lionel.

Revision history for this message
qdp (OpenERP) (qdp) wrote :

Hi Lionel,

your patch has landed on trunk in revision 5705. Thanks a lot for your contribution!
Quentin

Changed in openobject-addons:
status: New → Fix Released
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.