Merge lp:~openerp-dev/openobject-addons/6.1-opw-579011-rha into lp:openobject-addons/6.1

[Rifakat Husen (OpenERP)]

Hello,

Fix domain of search() on product.supplierinfo, it searches for supplier on
onchange_product_id() with product_id instead of product_tmpl_id.

product.supplierinfo has relation with product.template, not with product.product
so when product_id and product_tmpl_id differs for any product in that case it
will cause problem.
(Courtesy to Niels Huylebroeck)

Thanks for your review.

Regards,
Rifakat Haradwala

[lp:~openerp-dev/openobject-addons/6.1-opw-579011-rha]

6985. By Rifakat Husen (OpenERP) on 2012-09-11

[IMP] purchase: useability, added space after comma

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Not Relevant on Trunk (means that architecture on trunk has changed and so this bug has no meaning anymore). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6985. By Rifakat Husen (OpenERP) on 2012-09-11

[IMP] purchase: useability, added space after comma

6984. By Niels Huylebroeck <email address hidden> on 2012-09-11

[FIX] purchase: search() on supplierinfo obj using product_tmpl_id instead od product_id,
product.supplierinfo has m2o relation with product.template, not product.product

=== modified file 'purchase/purchase.py'
--- purchase/purchase.py	2012-07-25 17:07:19 +0000
+++ purchase/purchase.py	2012-09-11 05:57:22 +0000
@@ -777,7 +777,7 @@
 
         qty = qty or 1.0
         supplierinfo = False
-        supplierinfo_ids = product_supplierinfo.search(cr, uid, [('name','=',partner_id),('product_id','=',product.id)])
+        supplierinfo_ids = product_supplierinfo.search(cr, uid, [('name', '=', partner_id), ('product_id', '=', product.product_tmpl_id.id)])
         if supplierinfo_ids:
             supplierinfo = product_supplierinfo.browse(cr, uid, supplierinfo_ids[0], context=context)
             if supplierinfo.product_uom.id != uom_id: