From e421eb864ec41bc3f46237d807933e165e88b2d9 Mon Sep 17 00:00:00 2001 From: Holger Brunn Date: Wed, 13 Aug 2014 14:35:42 +0200 Subject: [PATCH] [RFR] use refactoring in partner_relations --- partner_relations_in_tab/model/res_partner.py | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/partner_relations_in_tab/model/res_partner.py b/partner_relations_in_tab/model/res_partner.py index 636a5e259..ab7b3e57a 100644 --- a/partner_relations_in_tab/model/res_partner.py +++ b/partner_relations_in_tab/model/res_partner.py @@ -27,12 +27,8 @@ from openerp.tools.translate import _ class ResPartner(Model): _inherit = 'res.partner' - def _get_relation_ids( - self, cr, uid, ids, dummy_name, dummy_arg, context=None): - if context is None: - context = {} - - #TODO: do a permission test on returned ids + def _get_relation_ids_select(self, cr, uid, ids, field_name, arg, + context=None): cr.execute( '''select r.id, left_partner_id, right_partner_id from res_partner_relation r @@ -43,13 +39,7 @@ class ResPartner(Model): ' order by ' + self.pool['res.partner.relation']._order, (tuple(ids), tuple(ids)) ) - result = dict([(i, []) for i in ids]) - for row in cr.fetchall(): - if row[1] in result: - result[row[1]].append(row[0]) - if row[2] in result: - result[row[2]].append(row[0]) - return result + return cr.fetchall() def _create_relation_type_tab( self, cr, uid, rel_type, inverse, field_names, context=None):