From 3edcb0e8f3f8d5e71169104add4d76adaaa08fd4 Mon Sep 17 00:00:00 2001
From: Raphael Collet <rco@openerp.com>
Date: Fri, 29 Jul 2011 12:12:38 +0200
Subject: [PATCH] small code improvements

bzr revid: rco@openerp.com-20110729101238-udkcecldiy9ybfwq
---
 openerp/addons/base/res/res_user.py | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/openerp/addons/base/res/res_user.py b/openerp/addons/base/res/res_user.py
index 0d0023c9694..9f8cb7d889c 100644
--- a/openerp/addons/base/res/res_user.py
+++ b/openerp/addons/base/res/res_user.py
@@ -714,9 +714,13 @@ class users2(osv.osv):
                     sel_name = name_selection_groups(ids)
                     fields[app_name] = {'type': 'boolean', 'string': app}
                     fields[sel_name] = {'type': 'selection', 'string': '', 'selection': selection}
-                    elems.append('<field name="%s"/>' % app_name)
-                    elems.append("""<field name="%s" nolabel="1" attrs="{'invisible': [('%s', '=', False)]}"/>""" % (sel_name, app_name))
-                    elems.append('<newline/>')
+                    elems.append("""
+                        <field name="%(app)s"/>
+                        <field name="%(sel)s" nolabel="1"
+                            attrs="{'invisible': [('%(app)s', '=', False)]}"
+                            modifiers="{'invisible': [('%(app)s', '=', False)]}"/>
+                        <newline/>
+                        """ % {'app': app_name, 'sel': sel_name})
                 elems.append('</group>')
                 # create other sections
                 sections = sorted(others.items(), key=lambda pair: pair[0])