From 68ce176f513149acc10956b22558e4c4b51b7745 Mon Sep 17 00:00:00 2001 From: Maxime Chambreuil Date: Thu, 11 Aug 2016 11:34:45 -0500 Subject: [PATCH] [IMP] Combining methods --- base_export_manager/models/res_users.py | 31 +++++++++---------------- 1 file changed, 11 insertions(+), 20 deletions(-) diff --git a/base_export_manager/models/res_users.py b/base_export_manager/models/res_users.py index 89731fd0e..759bf266a 100644 --- a/base_export_manager/models/res_users.py +++ b/base_export_manager/models/res_users.py @@ -9,23 +9,14 @@ from openerp import api, models class ResUsers(models.Model): _inherit = 'res.users' - @api.v7 - def get_export_models(self, cr, uid): - return self.fetch_export_models(cr, uid) - - @api.v8 - def get_export_models(self): - uid = self.id or self.env.uid - return self.fetch_export_models(self.env.cr, uid) - - def fetch_export_models(self, cr, uid): - groups_id = [group.id for group in self.browse(cr, uid, uid).groups_id] - accessobj = self.pool['ir.model.access'] - accessobj_ids = accessobj.search(cr, uid, [('perm_export', '=', True), - ('group_id', 'in', - groups_id)]) - model_names = [access_obj.model_id.model for access_obj in - accessobj.browse(cr, uid, accessobj_ids)] - # Make distinct value in list - model_names = list(set(model_names)) - return model_names + @api.model + def fetch_export_models(self): + accessobj = self.env['ir.model.access'] + accessobj_ids = accessobj.search([ + ('perm_export', '=', True), + ('group_id', 'in', self.env.user.groups_id.ids), + ]) + model_names = [ + access_obj.model_id.model for access_obj in accessobj_ids + ] + return list(set(model_names))