From bc8250b428d41aacd84d8d829e1a4e578a96cd69 Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Fri, 21 Sep 2012 09:58:41 +0200
Subject: [PATCH] [IMP] usability product form

bzr revid: fp@tinyerp.com-20120921075841-qzmqr64ucm2yhdsc
---
 addons/product/product_view.xml | 18 ++++++++++--------
 addons/stock/product_view.xml   |  2 +-
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/addons/product/product_view.xml b/addons/product/product_view.xml
index d04e6f17fe8..c08bbc412e3 100644
--- a/addons/product/product_view.xml
+++ b/addons/product/product_view.xml
@@ -110,23 +110,25 @@
                                         <field name="cost_method" groups="product.group_costing_method"/>
                                         <field name="standard_price" attrs="{'readonly':[('cost_method','=','average')]}"/>
                                     </group>
-                                    <group name="procurement_help" class="oe_grey" col="1">
+                                    <group name="procurement_help" class="oe_grey" col="1" groups="base.group_user">
                                         <p attrs="{'invisible': [('type','&lt;&gt;','service'),('procure_method','&lt;&gt;','make_to_stock')]}">
                                             When you sell this service, nothing special will be trigered
-                                            to deliver the customer.
+                                            to deliver the customer, as you set the procurement method as
+                                            'Make to Stock'.
                                         </p>
                                         <p attrs="{'invisible': [('type','&lt;&gt;','product'),('procure_method','&lt;&gt;','make_to_stock')]}">
-                                            When you sell this product, OpenERP will <b>use the available inventory</b>
-                                            for the delivery order.
-                                            <br/>
+                                            When you sell this product, OpenERP will <b>use the available
+                                            inventory</b> for the delivery order.
+                                            <br/><br/>
                                             If there are not enough quantities available, the delivery order
-                                            will wait for new products. You should create others rules
-                                            (orderpoints, manual purchase orders) to fulfill the inventory.
+                                            will wait for new products. To fulfill the inventory, you should
+                                            create others rules like orderpoints.
                                         </p>
                                         <p attrs="{'invisible': [('type','&lt;&gt;','consu'),('procure_method','&lt;&gt;','make_to_stock')]}">
                                             When you sell this product, a delivery order will be created.
                                             OpenERP will consider that the <b>required quantities are always
-                                            available</b> as it's a consumable (the stock on hand may become negative).
+                                            available</b> as it's a consumable (as a result of this, the quantity
+                                            on hand may become negative).
                                         </p>
                                     </group>
                                     <group groups="product.group_uom">
diff --git a/addons/stock/product_view.xml b/addons/stock/product_view.xml
index 2e0eef44da9..a7dff7fcc0d 100644
--- a/addons/stock/product_view.xml
+++ b/addons/stock/product_view.xml
@@ -56,7 +56,7 @@
             <field name="model">product.product</field>
             <field name="inherit_id" ref="product.product_normal_form_view"/>
             <field name="arch" type="xml">
-                <group name="procurement" position="inside">
+                <group name="procurement_help" position="after">
                     <group name="delay" string="Delays">
                         <label for="produce_delay" attrs="{'invisible':[('type','=','service')]}"/>
                         <div attrs="{'invisible':[('type','=','service')]}">