From d5a9dfd30b68bb21057b581c5866822fe749477c Mon Sep 17 00:00:00 2001
From: "Quentin (OpenERP)" <qdp-launchpad@openerp.com>
Date: Fri, 4 Oct 2013 11:01:52 +0200
Subject: [PATCH] [IMP] sale_stock, stock: change the 'view delivery orders' in
 order to see all the pcikings related and not only the deliveries (include
 pick and pack operations)

bzr revid: qdp-launchpad@openerp.com-20131004090152-wmlgx9npiuus1z43
---
 addons/sale_stock/sale_stock.py | 17 +++++------------
 addons/stock/stock_view.xml     | 14 ++++++++++++++
 2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/addons/sale_stock/sale_stock.py b/addons/sale_stock/sale_stock.py
index a9940be3b08..9f45bf0be59 100644
--- a/addons/sale_stock/sale_stock.py
+++ b/addons/sale_stock/sale_stock.py
@@ -77,12 +77,12 @@ class sale_order(osv.osv):
             if not sale.procurement_group_id:
                 res[sale.id] = []
                 continue
-            picking_ids = {}
+            picking_ids = set()
             for procurement in sale.procurement_group_id.procurement_ids:
                 for move in procurement.move_ids:
                     if move.picking_id:
-                        picking_ids[move.picking_id.id] = True
-            res[sale.id] = picking_ids.keys()
+                        picking_ids.add(move.picking_id.id)
+            res[sale.id] = list(picking_ids)
         return res
 
     def _prepare_order_line_procurement(self, cr, uid, order, line, group_id = False, context=None):
@@ -134,28 +134,21 @@ class sale_order(osv.osv):
         mod_obj = self.pool.get('ir.model.data')
         act_obj = self.pool.get('ir.actions.act_window')
 
-        result = mod_obj.get_object_reference(cr, uid, 'stock', 'action_picking_tree')
+        result = mod_obj.get_object_reference(cr, uid, 'stock', 'action_picking_tree_all')
         id = result and result[1] or False
         result = act_obj.read(cr, uid, [id], context=context)[0]
 
         #compute the number of delivery orders to display
         pick_ids = []
-        ctx = {}
         for so in self.browse(cr, uid, ids, context=context):
             pick_ids += [picking.id for picking in so.picking_ids]
-        if len(pick_ids)>0:
-            pick_type = self.pool.get('stock.picking').browse(cr, uid, pick_ids[0], context=context)['picking_type_id'].id
-            ctx = eval(result['context'],{'active_id': pick_type}, nocopy=True)
         #choose the view_mode accordingly
         if len(pick_ids) > 1:
-            result['domain'] = "[('id','in',["+','.join(map(str, pick_ids))+"])]"
+            result['domain'] = "[('id','in',[" + ','.join(map(str, pick_ids)) + "])]"
         else:
             res = mod_obj.get_object_reference(cr, uid, 'stock', 'view_picking_form')
             result['views'] = [(res and res[1] or False, 'form')]
             result['res_id'] = pick_ids and pick_ids[0] or False
-        result.update({
-            'context': ctx,
-        })
         return result
 
 
diff --git a/addons/stock/stock_view.xml b/addons/stock/stock_view.xml
index d2abec65ee4..e3eaad45abc 100644
--- a/addons/stock/stock_view.xml
+++ b/addons/stock/stock_view.xml
@@ -682,6 +682,20 @@
             </field>
         </record>
 
+        <record id="action_picking_tree_all" model="ir.actions.act_window">
+            <field name="name">Stock Operations</field>
+            <field name="res_model">stock.picking</field>
+            <field name="type">ir.actions.act_window</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">tree,form,calendar</field>
+            <field name="domain"></field>
+            <field name="context">{
+                    'contact_display': 'partner_address',
+            }
+            </field>
+            <field name="search_view_id" ref="view_picking_internal_search"/>
+        </record>
+
         <record id="action_picking_tree_draft" model="ir.actions.act_window">
             <field name="name">Draft Transfers</field>
             <field name="res_model">stock.picking</field>