diff options
| author | Xavier Morel <xmo@tinyerp.com> | 2010-01-26 20:30:48 +0100 |
|---|---|---|
| committer | Xavier Morel <xmo@tinyerp.com> | 2010-01-26 20:30:48 +0100 |
| commit | bedf69d3f01304c66ca07bda475fe77f5dfbb9bb (patch) | |
| tree | 4f8bc499608a052799def1889bbc0a128d724cf5 /addons/base_setup | |
| parent | 2e49aaa7ce33b601b983ec6bb368535426cdc148 (diff) | |
[fix] try to fix intermittent error where the user's action_id gets set to the wrong item
bzr revid: xmo@tinyerp.com-20100126193048-lxhqvyljnjjjzfx2
Diffstat (limited to 'addons/base_setup')
| -rw-r--r-- | addons/base_setup/__init__.py | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/addons/base_setup/__init__.py b/addons/base_setup/__init__.py index 5c89388aa93..22e81a488b0 100644 --- a/addons/base_setup/__init__.py +++ b/addons/base_setup/__init__.py @@ -33,19 +33,11 @@ class base_setup_config_choice(osv.osv_memory): logger = netsvc.Logger() def set_default_menu(self, cr, uid, menu, context=None): - users = self.pool.get('res.users') - users.write(cr, uid, - users.search(cr, uid, - [('action_id','=','Setup')], - context=context), - {'action_id': menu.id}, - context=context) - users.write(cr, uid, - users.search(cr, uid, - [('menu_id','=','Setup')], - context=context), - {'menu_id': menu.id}, - context=context) + user = self.pool.get('res.users')\ + .browse(cr, uid, uid, context=context) + + user.write({'action_id': menu.id, + 'menu_id': menu.id}) def get_default_menu(self, cr, uid, context=None): actions = self.pool.get('ir.actions.act_window') @@ -53,7 +45,7 @@ class base_setup_config_choice(osv.osv_memory): current_menu_id = actions.search(cr, uid, [('name','=','Menu')], context=context) assert len(current_menu_id) == 1,\ - 'A given user should only have menu item' + 'A given user should only have one menu item' return actions.browse(cr, uid, current_menu_id[0], context=context) def menu(self, cr, uid, ids, context=None): |