Ravi Gohil (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/6.1-opw-581791-rgo into lp:openobject-addons/6.1.
Requested reviews: Naresh(OpenERP) (nch-openerp) For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-581791-rgo/+merge/135360 Steps to reproduce(From Web-Client): 1) Create a db with demo data and install module `warning`, 2) Create a product and set it's default UoM to `PCE` and Purchase UoM to `Dozen`, 3) Edit an RFQ which has a order line for `PCE` and select the product created in step 2, On changing the product, UoM expected to be set to `Dozen`, instead, it stays `PCE`. This is due to context not passed when called `product_id_change()` with `super` because purchase UoM being set on the basis of context. This branch also contains fix for the traceback faced saying, TypeError: onchange_product_id() got an unexpected keyword argument 'fiscal_position_id'" when changing the product in purchase order line. Kindly review the fix. Thanks. -- https://code.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-581791-rgo/+merge/135360 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/6.1-opw-581791-rgo.
=== modified file 'warning/warning.py' --- warning/warning.py 2012-09-06 10:32:09 +0000 +++ warning/warning.py 2012-11-21 10:27:34 +0000 @@ -231,7 +231,7 @@ class purchase_order_line(osv.osv): _inherit = 'purchase.order.line' def onchange_product_id(self,cr, uid, ids, pricelist, product, qty, uom, - partner_id, date_order=False, fiscal_position=False, date_planned=False, + partner_id, date_order=False, fiscal_position_id=False, date_planned=False, name=False, price_unit=False, notes=False, context=None): warning = {} if not product: @@ -250,7 +250,7 @@ warning['message'] = message result = super(purchase_order_line, self).product_id_change(cr, uid, ids, pricelist, product, qty, uom, - partner_id, date_order, fiscal_position) + partner_id, date_order, fiscal_position_id, context=context) if result.get('warning',False): warning['title'] = title and title +' & '+result['warning']['title'] or result['warning']['title']
_______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : openerp-dev-gtk@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp