From acde6d86c16d7541f3529ea798a58fe45756b362 Mon Sep 17 00:00:00 2001
From: "Darshan Kalola (OpenERP)" <dka@tinyerp.com>
Date: Thu, 16 Jan 2014 13:15:35 +0530
Subject: [PATCH] [IMP]rename has_variants field to standard_variants in
 product form and improved fields_view_get method.

bzr revid: dka@tinyerp.com-20140116074535-m40q8q7dy3kzfguf
---
 addons/product/product.py       | 19 ++++++++++++-------
 addons/product/product_view.xml |  2 +-
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/addons/product/product.py b/addons/product/product.py
index aa9f93e176a..f8c676f87d6 100644
--- a/addons/product/product.py
+++ b/addons/product/product.py
@@ -749,14 +749,19 @@ class product_product(osv.osv):
             context = {}
         res = super(product_product, self).fields_view_get(cr, uid, view_id=view_id, view_type=view_type, context=context, toolbar=toolbar, submenu=submenu)
         #check the current user in group_product_variant
-        if self.pool['res.users'].has_group(cr, uid, 'product.group_product_variant') and view_type == 'form':
+        if view_type == 'form':
             doc = etree.XML(res['arch'])
-            for node in doc.xpath("//field[@name='name']"):
-                node.set('invisible', '1')
-                node.set('required', '0')
-                setup_modifiers(node, res['fields']['name'])
-            for node in doc.xpath("//label[@name='label_name']"):
-                node.set('string','Product Template')
+            if self.pool['res.users'].has_group(cr, uid, 'product.group_product_variant'):
+                for node in doc.xpath("//field[@name='name']"):
+                    node.set('invisible', '1')
+                    node.set('required', '0')
+                    setup_modifiers(node, res['fields']['name'])
+                for node in doc.xpath("//label[@name='label_name']"):
+                    node.set('string','Product Template')
+            else:
+                for node in doc.xpath("//field[@name='product_tmpl_id']"):
+                    node.set('required', '0')
+                    setup_modifiers(node, res['fields']['name'])
             res['arch'] = etree.tostring(doc)
         return res
 
diff --git a/addons/product/product_view.xml b/addons/product/product_view.xml
index c98125eecf8..2301df63279 100644
--- a/addons/product/product_view.xml
+++ b/addons/product/product_view.xml
@@ -61,7 +61,7 @@
                 <form string="Product" version="7.0">
                     <sheet>
                         <field name="image_medium" widget="image" class="oe_avatar oe_left"/>
-                        <field name="has_variants" invisible="1"/>
+                        <field name="standard_variants" invisible="1"/>
                         <field name="lst_price" invisible="1"/>
                         <field name="price_extra" invisible="1"/>
                         <field name="price_margin" invisible="1"/>