From 66922c7199304ef06b04d2b762bfa824ec957799 Mon Sep 17 00:00:00 2001
From: "Quentin (OpenERP)" <qdp-launchpad@openerp.com>
Date: Tue, 3 Jul 2012 17:58:43 +0200
Subject: [PATCH] [IMP] code review

bzr revid: qdp-launchpad@openerp.com-20120703155843-awb230fppn22wbs1
---
 .../report/hr_expense_report_view.xml         |  2 +-
 .../wizard/hr_timesheet_print_users_view.xml  | 29 +++++++++----------
 .../hr_timesheet_analytic_profit_view.xml     |  3 +-
 3 files changed, 16 insertions(+), 18 deletions(-)

diff --git a/addons/hr_expense/report/hr_expense_report_view.xml b/addons/hr_expense/report/hr_expense_report_view.xml
index 8b00e447ab9..7877edf4b3a 100644
--- a/addons/hr_expense/report/hr_expense_report_view.xml
+++ b/addons/hr_expense/report/hr_expense_report_view.xml
@@ -96,7 +96,7 @@
     </record>
 
     <record id="action_hr_expense_report_all" model="ir.actions.act_window">
-        <field name="name">Expense Analysis</field>
+        <field name="name">Expenses Analysis</field>
         <field name="res_model">hr.expense.report</field>
         <field name="view_type">form</field>
         <field name="view_mode">tree,graph</field>
diff --git a/addons/hr_timesheet/wizard/hr_timesheet_print_users_view.xml b/addons/hr_timesheet/wizard/hr_timesheet_print_users_view.xml
index c4ef649f12e..ebce5ca34c4 100644
--- a/addons/hr_timesheet/wizard/hr_timesheet_print_users_view.xml
+++ b/addons/hr_timesheet/wizard/hr_timesheet_print_users_view.xml
@@ -8,26 +8,24 @@
             <field name="model">hr.analytical.timesheet.users</field>
             <field name="type">form</field>
             <field name="arch" type="xml">
-				<form string="Monthly Employees Timesheet" version="7.0">
+                <form string="Monthly Employees Timesheet" version="7.0">
                     <header>
-                    	<button string="Print" name="print_report" type="object" class="oe_highlight"/> or
-                        <button string="Cancel" class="oe_link" special="cancel" />
-	                </header>
+                        <button string="Print" name="print_report" type="object" class="oe_highlight"/> or
+                        <button string="Cancel" class="oe_link" special="cancel"/>
+                    </header>
                     <sheet>
-                        <group><label for="month" string="Period" />
-             				<div>
-								<field name="month" nolabel="1" class="oe_inline"/>
-								<field name="year" nolabel="1" class="oe_inline"/>
-							</div> 
-                        </group>
                         <group>
-                            <div>
-                                <separator string="Employees" colspan="4"/>
-							    <field name="employee_ids" colspan="4" nolabel="1"/>
+                             <label for="month" string="Period"/>
+                             <div>
+                                <field name="month" class="oe_inline"/>
+                                <field name="year" class="oe_inline"/>
                             </div> 
-                        </group>						
+                        </group>
+                        <group string="Employees"/>
+                            <field name="employee_ids" nolabel="1"/>
+                        </group>
                     </sheet>
-				</form>
+                </form>
             </field>
         </record>
 
@@ -45,6 +43,5 @@
             parent="hr.menu_hr_reporting_timesheet"
             groups="base.group_hr_manager" sequence="2" icon="STOCK_PRINT"/>
 
-
      </data>
 </openerp>
diff --git a/addons/hr_timesheet_invoice/wizard/hr_timesheet_analytic_profit_view.xml b/addons/hr_timesheet_invoice/wizard/hr_timesheet_analytic_profit_view.xml
index d7b6cd1135b..4e4eb7b36d2 100644
--- a/addons/hr_timesheet_invoice/wizard/hr_timesheet_analytic_profit_view.xml
+++ b/addons/hr_timesheet_invoice/wizard/hr_timesheet_analytic_profit_view.xml
@@ -13,7 +13,8 @@
                         <button special="cancel" string="Cancel" class="oe_link"/>
                    </header>
                    <sheet>
-                       <group> <label for="date_from" string="Duration" />					
+                       <group> 
+                           <label for="date_from" string="Duration" />					
                            <div>
  						       <field name="date_from" nolabel="1" class="oe_inline" />
                                - <field name="date_to" nolabel="1" class="oe_inline"/>