[CLEAN] cleaning: set(list), use directly a set
authorThibault Delavallée <tde@openerp.com>
Thu, 25 Apr 2013 14:28:01 +0000 (16:28 +0200)
committerThibault Delavallée <tde@openerp.com>
Thu, 25 Apr 2013 14:28:01 +0000 (16:28 +0200)
bzr revid: tde@openerp.com-20130425142801-gwk9xsnhb3r20jji

openerp/addons/base/ir/ir_ui_menu.py

index b878681..647f714 100644 (file)
@@ -299,9 +299,9 @@ class ir_ui_menu(osv.osv):
               the action domain
         """
         res = {}
-        menu_ids = []
+        menu_ids = set()
         for menu in self.browse(cr, uid, ids, context=context):
-            menu_ids.append(menu.id)
+            menu_ids.add(menu.id)
             ctx = None
             if menu.action and menu.action.type in ('ir.actions.act_window', 'ir.actions.client') and menu.action.context:
                 ctx = eval(menu.action.context, context) or None
@@ -313,8 +313,8 @@ class ir_ui_menu(osv.osv):
                 for menu_data in menu_ref:
                     model, id = model_data_obj.get_object_reference(cr, uid, menu_data.split('.')[0], menu_data.split('.')[1])
                     if (model == 'ir.ui.menu'):
-                        menu_ids.append(id)
-        menu_ids = list(set(menu_ids))
+                        menu_ids.add(id)
+        menu_ids = list(menu_ids)
 
         for menu in self.browse(cr, uid, menu_ids, context=context):
             res[menu.id] = {