From 7191bbfb24ccbefddf0137a3556becedc6f1112c Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@odoo.com>
Date: Tue, 5 Aug 2014 15:16:53 +0200
Subject: [PATCH] [IMP] is_product_variant not needed in most views + no delete
 in stock valuation history

---
 addons/account/product_view.xml                          | 9 +++------
 addons/product_email_template/views/product_view.xml     | 1 -
 .../wizard/stock_valuation_history_view.xml              | 2 +-
 3 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/addons/account/product_view.xml b/addons/account/product_view.xml
index 44b9099a00a..e88a45a77a9 100644
--- a/addons/account/product_view.xml
+++ b/addons/account/product_view.xml
@@ -15,16 +15,13 @@
                         </group>
                         <group name="properties">
                             <group>
-                                <field name="property_account_income" domain="[('type','=','other')]" groups="account.group_account_user"
-                                    attrs="{'readonly': [('is_product_variant', '=', True)]}"/>
+                                <field name="property_account_income" domain="[('type','=','other')]" groups="account.group_account_user"/>
                                 <field name="taxes_id" colspan="2" widget="many2many_tags"
                                     attrs="{'readonly':[('sale_ok','=',0)]}"/>
                             </group>
                             <group>
-                                <field name="property_account_expense" domain="[('type','=','other')]" groups="account.group_account_user"
-                                    attrs="{'readonly': [('is_product_variant', '=', True)]}"/>
-                                <field name="supplier_taxes_id" colspan="2" widget="many2many_tags"
-                                    attrs="{'readonly': [('is_product_variant', '=', True)]}"/>
+                                <field name="property_account_expense" domain="[('type','=','other')]" groups="account.group_account_user"/>
+                                <field name="supplier_taxes_id" colspan="2" widget="many2many_tags"/>
                             </group>
                         </group>
                     </page>
diff --git a/addons/product_email_template/views/product_view.xml b/addons/product_email_template/views/product_view.xml
index 3c98d8f9109..31cf5f5754e 100644
--- a/addons/product_email_template/views/product_view.xml
+++ b/addons/product_email_template/views/product_view.xml
@@ -15,7 +15,6 @@
                             'default_subject': name,
                             'default_name': name,
                         }"
-                        attrs="{'invisible': [('is_product_variant', '=', False)]}"
                     />
                 </field>
             </field>
diff --git a/addons/stock_account/wizard/stock_valuation_history_view.xml b/addons/stock_account/wizard/stock_valuation_history_view.xml
index 6343aee26c4..ea2450e3fae 100644
--- a/addons/stock_account/wizard/stock_valuation_history_view.xml
+++ b/addons/stock_account/wizard/stock_valuation_history_view.xml
@@ -42,7 +42,7 @@
         <field name="name">Stock Value At Date</field>
         <field name="model">stock.history</field>
         <field name="arch" type="xml">
-            <tree string="Stock Value At Date" create="0">
+            <tree string="Stock Value At Date" create="0" delete="0">
                 <field name="location_id" invisible="1"/>
                 <field name="product_id" invisible="1"/>
                 <field name="move_id"/>