From: Martin Trigaux <mat@openerp.com>
Date: Thu, 9 Oct 2014 11:37:06 +0000 (+0200)
Subject: [FIX] product.uom: safer handling of factor/factor_inv in UI
X-Git-Url: http://git.inspyration.org/?a=commitdiff_plain;h=3a0af6af7b5a67dfb1ed196bd3e5da41c1dfa57a;p=odoo%2Fodoo.git

[FIX] product.uom: safer handling of factor/factor_inv in UI

Add readonly attribute to avoid sending both factor and factor_inv value to the backend when saving.
This was possible if the user switched between uom_type to fill the two fields.
---

diff --git a/addons/product/product_view.xml b/addons/product/product_view.xml
index 7b4539e..a59a77c 100644
--- a/addons/product/product_view.xml
+++ b/addons/product/product_view.xml
@@ -421,8 +421,14 @@
                             <field name="uom_type" on_change="onchange_type(uom_type)"/>
                             <label for="factor"/>
                             <div>
-                                <field name="factor" digits="[42,5]" attrs="{'invisible':[('uom_type','!=','smaller')]}"/>
-                                <field name="factor_inv" digits="[42,5]" attrs="{'invisible':[('uom_type','!=','bigger')]}"/>
+                                <field name="factor"
+                                    digits="[42,5]"
+                                    attrs="{'invisible':[('uom_type','!=','smaller')],
+                                            'readonly':[('uom_type','!=','smaller')]}"/>
+                                <field name="factor_inv"
+                                    digits="[42,5]"
+                                    attrs="{'invisible':[('uom_type','!=','bigger')],
+                                            'readonly':[('uom_type','!=','bigger')]}"/>
                                 <p attrs="{'invisible':[('uom_type','!=','smaller')]}" class="oe_grey">
                                     e.g: 1 * (reference unit) = ratio * (this unit)
                                 </p>