From 4f69be7eadbcde6d9f65427877e29b17e5b19bfe Mon Sep 17 00:00:00 2001
From: Toufik Benjaa <tbe@odoo.com>
Date: Tue, 14 Aug 2018 11:05:45 +0200
Subject: [PATCH] [FIX] sale_timesheet: avoid adding buttonbox on invoices

- Instead of adding the "timesheet" statbutton on the invoice's
buttonbox, this inherit view is adding a new buttonbox which causes
conflict with the original buttonbox.
---
 addons/sale_timesheet/views/account_invoice_views.xml | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/addons/sale_timesheet/views/account_invoice_views.xml b/addons/sale_timesheet/views/account_invoice_views.xml
index 20e25720343..cb8b2ef54db 100644
--- a/addons/sale_timesheet/views/account_invoice_views.xml
+++ b/addons/sale_timesheet/views/account_invoice_views.xml
@@ -15,12 +15,10 @@
         <field name="model">account.invoice</field>
         <field name="inherit_id" ref="account.invoice_form"/>
         <field name="arch" type="xml">
-            <xpath expr="//label" position="before">
-                <div class="oe_button_box" name="button_box" groups="base.group_user">
-                    <button name="%(sale_timesheet.action_timesheet_from_invoice)d" type="action" class="oe_stat_button" icon="fa-calendar" attrs="{'invisible':[('timesheet_count','=', 0)]}">
-                        <field name="timesheet_count" widget="statinfo" string="Timesheets"/>
-                    </button>
-                </div>
+            <xpath expr="//div[@name='button_box']" position="inside">
+                <button name="%(sale_timesheet.action_timesheet_from_invoice)d" type="action" class="oe_stat_button" icon="fa-calendar" attrs="{'invisible':[('timesheet_count','=', 0)]}">
+                    <field name="timesheet_count" widget="statinfo" string="Timesheets"/>
+                </button>
             </xpath>
         </field>
     </record>