From 788eb62ac2964fa2827bafead1a6aafbab49eb92 Mon Sep 17 00:00:00 2001
From: Raphael Collet <rco@openerp.com>
Date: Tue, 6 Dec 2011 13:56:42 +0100
Subject: [PATCH] [IMP] res_users: improve access rights on user form

bzr revid: rco@openerp.com-20111206125642-hatpewnnxsg9nerx
---
 openerp/addons/base/res/res_users.py | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/openerp/addons/base/res/res_users.py b/openerp/addons/base/res/res_users.py
index 0a8bd2b0684..19658c26f0e 100644
--- a/openerp/addons/base/res/res_users.py
+++ b/openerp/addons/base/res/res_users.py
@@ -739,15 +739,17 @@ class groups_view(osv.osv):
             xml = []
             xml.append('<!-- GENERATED AUTOMATICALLY BY GROUPS -->')
             xml.append('<field name="groups_id" position="replace">')
+            xml.append('<group col="6">')
             for app, selects in self.get_groups_by_application(cr, uid, context):
-                category = app and app.name or _('Other')
-                xml.append('<separator string="%s" colspan="6"/>' % category)
+                app_name = app and app.name or _('Other')
+                xml.append('<separator string="%s" colspan="6"/>' % app_name)
                 for gs in selects:
                     if len(gs) == 1:
                         field_name = name_boolean_group(gs[0].id)
                     else:
                         field_name = name_selection_groups(map(int, gs))
                     xml.append('<field name="%s"/>' % field_name)
+            xml.append('</group>')
             xml.append('</field>')
             view.write({'arch': '\n'.join(xml)})
         return True
@@ -869,11 +871,6 @@ class users_view(osv.osv):
         res = super(users_view, self).fields_get(cr, uid, allfields, context, write_access)
         # add reified groups fields
         for app, selects in self.pool.get('res.groups').get_groups_by_application(cr, uid, context):
-            all_gs = reduce(lambda x, y: x + y, selects)
-            res[name_boolean_groups(map(int, all_gs))] = {
-                'type': 'boolean',
-                'string': app and app.name or _('Other'),
-            }
             for gs in selects:
                 if len(gs) == 1:
                     g = gs[0]