From a5aea1df135d92292d02c2bca42e768ea823ad71 Mon Sep 17 00:00:00 2001
From: "Amit Patel (OpenERP)" <apa@tinyerp.com>
Date: Fri, 20 Jul 2012 12:42:50 +0530
Subject: [PATCH] [IMP]:improved code

bzr revid: apa@tinyerp.com-20120720071250-1h5cmcaxdwkqmsju
---
 addons/hr/res_config_view.xml         | 90 +++++++++++++--------------
 addons/hr_payroll/res_config_view.xml |  1 -
 2 files changed, 45 insertions(+), 46 deletions(-)

diff --git a/addons/hr/res_config_view.xml b/addons/hr/res_config_view.xml
index 76f052f873a..520dd4dd94f 100644
--- a/addons/hr/res_config_view.xml
+++ b/addons/hr/res_config_view.xml
@@ -20,57 +20,57 @@
                         or
                         <button string="Cancel" type="object" name="cancel" class="oe_link"/>
                     </header>
-                        <separator string="Additional Features"/>
-                        <group>
-                            <label for="id" string="Additional Features"/>
+                    <separator string="Additional Features"/>
+                    <group>
+                        <label for="id" string="Additional Features"/>
+                        <div>
                             <div>
-                                <div>
-                                    <field name="module_hr_expense" class="oe_inline"/>
-                                    <label for="module_hr_expense"/>
-                                </div>
-                                <div>
-                                    <field name="module_hr_evaluation" class="oe_inline"/>
-                                    <label for="module_hr_evaluation"/>
-                                </div>
-                                <div>
-                                    <field name="module_hr_holidays" class="oe_inline"/>
-                                    <label for="module_hr_holidays"/>
-                                </div>
-                                <div>
-                                    <field name="module_hr_recruitment" class="oe_inline"/>
-                                    <label for="module_hr_recruitment"/>
-                                </div>
-                                <div>
-                                    <field name="module_hr_contract" class="oe_inline"/>
-                                    <label for="module_hr_contract"/>
-                                </div>
-                                <div name="hr_payroll">
-                                    <field name="module_hr_payroll" class="oe_inline"/>
-                                    <label for="module_hr_payroll"/>
-                                    <button name="%(open_payroll_modules)d" type="action"
+                                <field name="module_hr_expense" class="oe_inline"/>
+                                <label for="module_hr_expense"/>
+                            </div>
+                            <div>
+                                <field name="module_hr_evaluation" class="oe_inline"/>
+                                <label for="module_hr_evaluation"/>
+                            </div>
+                            <div>
+                                <field name="module_hr_holidays" class="oe_inline"/>
+                                <label for="module_hr_holidays"/>
+                            </div>
+                            <div>
+                                <field name="module_hr_recruitment" class="oe_inline"/>
+                                <label for="module_hr_recruitment"/>
+                            </div>
+                            <div>
+                                <field name="module_hr_contract" class="oe_inline"/>
+                                <label for="module_hr_contract"/>
+                            </div>
+                            <div name="hr_payroll">
+                                <field name="module_hr_payroll" class="oe_inline"/>
+                                <label for="module_hr_payroll"/>
+                                <button name="%(open_payroll_modules)d" type="action"
                                     string="Install Country-Specific Payroll" icon="gtk-go-forward"
                                     attrs="{'invisible': [('module_hr_payroll','=',False)]}" class="oe_link"/>
-                                </div>
                             </div>
-                        </group>
-                        <separator string="Timesheets"/>
-                        <group name="timesheet_grp">
-                            <label for="id" string="Timesheet Features"/>
+                        </div>
+                    </group>
+                    <separator string="Timesheets"/>
+                    <group name="timesheet_grp">
+                        <label for="id" string="Timesheet Features"/>
+                        <div>
                             <div>
-                                <div>
-                                    <field name="module_hr_attendance" on_change="onchange_hr_attendance(module_hr_attendance)" class="oe_inline"/>
-                                    <label for="module_hr_attendance"/>
-                                </div>
-                                <div>
-                                    <field name="module_hr_timesheet" on_change="onchange_hr_timesheet(module_hr_timesheet)" class="oe_inline"/>
-                                    <label for="module_hr_timesheet"/>
-                                </div>
-                                <div name="hr_timesheet">
-                                    <field name="module_hr_timesheet_sheet" class="oe_inline"/>
-                                    <label for="module_hr_timesheet_sheet"/>
-                                </div>
+                                <field name="module_hr_attendance" on_change="onchange_hr_attendance(module_hr_attendance)" class="oe_inline"/>
+                                <label for="module_hr_attendance"/>
                             </div>
-                        </group>
+                            <div>
+                                <field name="module_hr_timesheet" on_change="onchange_hr_timesheet(module_hr_timesheet)" class="oe_inline"/>
+                                <label for="module_hr_timesheet"/>
+                            </div>
+                            <div name="hr_timesheet">
+                                <field name="module_hr_timesheet_sheet" class="oe_inline"/>
+                                <label for="module_hr_timesheet_sheet"/>
+                            </div>
+                        </div>
+                    </group>
                 </form>
             </field>
         </record>
diff --git a/addons/hr_payroll/res_config_view.xml b/addons/hr_payroll/res_config_view.xml
index eb486a7a8d5..23b4adde536 100644
--- a/addons/hr_payroll/res_config_view.xml
+++ b/addons/hr_payroll/res_config_view.xml
@@ -7,7 +7,6 @@
             <field name="type">form</field>
             <field name="inherit_id" ref="hr.view_human_resources_configuration"/>
             <field name="arch" type="xml">
-                <!-- put after the element <div><field name="module_hr_payroll" class="oe_inline"/> ...</div> -->
                 <xpath expr="//div[@name='hr_payroll']" position="after">
                     <div>
                         <field name="module_hr_payroll_account" class="oe_inline"/>