From 99b27fa6b4b5497d6d5457839a7bfdf941da97a7 Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Mon, 4 Nov 2013 11:26:46 +0100 Subject: [PATCH] [FIX] open_product_line() returns an action, the action should not propagate the current context but use its own one (so it will have the correct active_model, active_ids and so on, otherwise, it propagates a wrong active_ids and active_model). bzr revid: guewen.baconnier@camptocamp.com-20131104102646-pjdorjqfaic3sw7b --- addons/purchase_requisition/purchase_requisition.py | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/addons/purchase_requisition/purchase_requisition.py b/addons/purchase_requisition/purchase_requisition.py index 0e2d2cc5e85..119bbd1b997 100644 --- a/addons/purchase_requisition/purchase_requisition.py +++ b/addons/purchase_requisition/purchase_requisition.py @@ -119,15 +119,12 @@ class purchase_requisition(osv.osv): context = {} res = self.pool.get('ir.actions.act_window').for_xml_id(cr, uid, 'purchase_requisition', 'purchase_line_tree', context=context) res['context'] = context - po_ids_browse = self.browse(cr, uid, ids, context=context)[0].po_line_ids - po_ids = [] - for po in po_ids_browse: - po_ids.append(po.id) - res['context'].update({ + po_lines = self.browse(cr, uid, ids, context=context)[0].po_line_ids + res['context'] = { 'search_default_groupby_product': True, 'search_default_hide_cancelled': True, - }) - res['domain'] = [('id', 'in', po_ids)] + } + res['domain'] = [('id', 'in', [line.id for line in po_lines])] return res def open_rfq(self, cr, uid, ids, context=None):