From 4e9a0c1ef3c2f71331f557a358d3a8f958b5a92c Mon Sep 17 00:00:00 2001
From: ced <>
Date: Tue, 7 Aug 2007 14:30:10 +0000
Subject: [PATCH] client,base_setup: improve login/setup

bzr revid: ced-08c2f9b2f4cd31c8f82b7ba518d6af7f23a1afc6
---
 addons/base_setup/base_setup_data.xml         |  1 +
 addons/base_setup/wizard/wizard_base_setup.py | 38 ++++++++-----------
 2 files changed, 16 insertions(+), 23 deletions(-)

diff --git a/addons/base_setup/base_setup_data.xml b/addons/base_setup/base_setup_data.xml
index 1616ce8f0ed..0b79f3d9a09 100644
--- a/addons/base_setup/base_setup_data.xml
+++ b/addons/base_setup/base_setup_data.xml
@@ -10,6 +10,7 @@
 
 		<record model="res.users" id="base.user_admin">
 			<field name="action_id" ref="action_wizard_setup"/>
+			<field name="menu_id" ref="action_wizard_setup"/>
 		</record>
 	</data>
 </terp>
diff --git a/addons/base_setup/wizard/wizard_base_setup.py b/addons/base_setup/wizard/wizard_base_setup.py
index 4b645e2b448..97a65fa4b05 100644
--- a/addons/base_setup/wizard/wizard_base_setup.py
+++ b/addons/base_setup/wizard/wizard_base_setup.py
@@ -227,38 +227,30 @@ class wizard_base_setup(wizard.interface):
 				filename=os.path.join(tools.config["root_path"], "i18n", lang.code + ".csv")
 				tools.trans_load(cr.dbname, filename, lang.code)
 		return {}
+
 	def _menu(self, cr, uid, data, context):
 		users_obj=pooler.get_pool(cr.dbname).get('res.users')
 		action_obj=pooler.get_pool(cr.dbname).get('ir.actions.act_window')
 
 		ids=action_obj.search(cr, uid, [('name', '=', 'Menu')])
 		menu=action_obj.browse(cr, uid, ids)[0]
+
 		ids=users_obj.search(cr, uid, [('action_id', '=', 'Setup')])
 		users_obj.write(cr, uid, ids, {'action_id': menu.id})
+		ids=users_obj.search(cr, uid, [('menu_id', '=', 'Setup')])
+		users_obj.write(cr, uid, ids, {'menu_id': menu.id})
 
-		user = users_obj.browse(cr, uid, uid)
-		action = user.action_id
-		if action.type == 'ir.actions.act_window':
-			action = action_obj.browse(cr, uid, action.id)
-			return {
-				'name': action.name,
-				'type': action.type,
-				'view_id': (action.view_id and\
-						(action.view_id.id, action.view_id.name)) or False,
-				'domain': action.domain,
-				'res_model': action.res_model,
-				'src_model': action.src_model,
-				'view_type': action.view_type,
-				'view_mode': action.view_mode,
-				'views': action.views,
-			}
 		return {
-			'name': 'Menu',
-			'type': 'ir.actions.act_window',
-			'res_model': 'ir.ui.menu',
-			'view_type': 'tree',
-			'domain': str([('parent_id', '=', False)]),
-			'view_id': (menu.view_id.id, menu.view_id.name),
+			'name': menu.name,
+			'type': menu.type,
+			'view_id': (menu.view_id and\
+					(menu.view_id.id, menu.view_id.name)) or False,
+			'domain': menu.domain,
+			'res_model': menu.res_model,
+			'src_model': menu.src_model,
+			'view_type': menu.view_type,
+			'view_mode': menu.view_mode,
+			'views': menu.views,
 		}
 
 	def _next(self, cr, uid, data, context):
@@ -407,6 +399,6 @@ class wizard_base_setup(wizard.interface):
 		'menu': {
 			'actions': [],
 			'result': {'type': 'action', 'action': _menu, 'state': 'end'}
-		}
+		},
 	}
 wizard_base_setup('base_setup.base_setup')