From 594e01f1064dafa63d2b77433aa0d782c1abef4c Mon Sep 17 00:00:00 2001
From: "Ujjvala Collins (OpenERP)" <uco@tinyerp.com>
Date: Wed, 22 Jun 2011 11:25:58 +0530
Subject: [PATCH] [FIX] sale: Fixed problem of unselect/make empty packaging
 field in the sale order line.

lp bug: https://launchpad.net/bugs/795487 fixed

bzr revid: uco@tinyerp.com-20110622055558-sk5lidkpklygqws5
---
 addons/sale/sale.py       | 9 ++++++---
 addons/sale/sale_view.xml | 2 +-
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/addons/sale/sale.py b/addons/sale/sale.py
index 0b436f0dc91..431e0a3d63a 100644
--- a/addons/sale/sale.py
+++ b/addons/sale/sale.py
@@ -1013,9 +1013,11 @@ class sale_order_line(osv.osv):
 
     def product_id_change(self, cr, uid, ids, pricelist, product, qty=0,
             uom=False, qty_uos=0, uos=False, name='', partner_id=False,
-            lang=False, update_tax=True, date_order=False, packaging=False, fiscal_position=False, flag=False):
+            lang=False, update_tax=True, date_order=False, packaging=False, fiscal_position=False, flag=False, context=None):
         if not  partner_id:
             raise osv.except_osv(_('No Customer Defined !'), _('You have to select a customer in the sales form !\nPlease set one customer before choosing a product.'))
+        if not context:
+            context = {}
         warning = {}
         warning_msgs = ''
         product_uom_obj = self.pool.get('product.uom')
@@ -1023,7 +1025,7 @@ class sale_order_line(osv.osv):
         product_obj = self.pool.get('product.product')
         if partner_id:
             lang = partner_obj.browse(cr, uid, partner_id).lang
-        context = {'lang': lang, 'partner_id': partner_id}
+        context.update({'lang': lang, 'partner_id': partner_id})
 
         if not product:
             return {'value': {'th_weight': 0, 'product_packaging': False,
@@ -1034,7 +1036,8 @@ class sale_order_line(osv.osv):
 
         result = {}
         product_obj = product_obj.browse(cr, uid, product, context=context)
-        if not packaging and product_obj.packaging:
+        from_packaging = context.get('from_packaging', False)
+        if not packaging and product_obj.packaging and not from_packaging:
             packaging = product_obj.packaging[0].id
             result['product_packaging'] = packaging
         
diff --git a/addons/sale/sale_view.xml b/addons/sale/sale_view.xml
index 6c444ad174d..efaca80c221 100644
--- a/addons/sale/sale_view.xml
+++ b/addons/sale/sale_view.xml
@@ -142,7 +142,7 @@
                                             <field
                                                 name="product_packaging"
                                                 context="{'partner_id':parent.partner_id, 'quantity':product_uom_qty, 'pricelist':parent.pricelist_id, 'shop':parent.shop_id, 'uom':product_uom}"
-                                                on_change="product_id_change(parent.pricelist_id,product_id,product_uom_qty,product_uom,product_uos_qty,product_uos,name,parent.partner_id, 'lang' in context and context['lang'], False, parent.date_order, product_packaging, parent.fiscal_position, False)"
+                                                on_change="product_id_change(parent.pricelist_id,product_id,product_uom_qty,product_uom,product_uos_qty,product_uos,name,parent.partner_id, 'lang' in context and context['lang'], False, parent.date_order, product_packaging, parent.fiscal_position, False, {'from_packaging': True})"
                                                 domain="[('product_id','=',product_id)]"
                                                 groups="base.group_extended"/>