← Back to team overview

c2c-oerpscenario team mailing list archive

Re: [Bug 673122] Re: trunk v6 Error on calc price of a product on sale.order.line

 

Cristian,

The problem occurs when I add a new pricelist, and a new pricelist_version,
and try to use these on my sale order.

Best regards,

2010/11/9 Cristian Salamea (Gnuthink) <ovnicraft@xxxxxxxxx>

> @Diego i am at revno 3898 in addons, trying to reproduce it but i cant, can
> give us the steps to reproduce it.
> In pricelist.py line#167,  check if pricelist_ids was passed else we need
> reads a version active (afer in code search pricelist with in active date)
> and returned.
>
> Regards,
>
> ** Changed in: openobject-addons
>       Status: New => Invalid
>
> --
> trunk v6 Error on calc price of a product on sale.order.line
> https://bugs.launchpad.net/bugs/673122
> You received this bug notification because you are a direct subscriber
> of the bug.
>

-- 
trunk v6 Error on calc price of a product on sale.order.line
https://bugs.launchpad.net/bugs/673122
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.

Status in OpenObject Addons Modules: Invalid

Bug description:
on sale module when I try to add a sale line, I got a keyerror.

I think that error was that the function price_get_multi return a dict with
the pricelist_version id, and the sale.order.line try to use the pricelist id.

See this code on product_id_change on sale.py, and review price_get_multi,
           price = self.pool.get('product.pricelist').price_get(cr, uid, [pricelist],
                    product, qty or 1.0, partner_id, {
                        'uom': uom,
                        'date': date_order,
                        })[pricelist]

To solve the problem I think price_get_multi should return pricelist id instead of pricelist_version id





References