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

Reply via email to