openerp-wms-expert team mailing list archive

[Numérigraphe <informatique@xxxxxxxxxxxxxxxx>]

You have been requested to review the proposed merge of lp:~numerigraphe/openobject-addons/trunk-stock-reset-negative-qty-onchange into lp:openobject-addons.

For more details, see:
https://code.launchpad.net/~numerigraphe/openobject-addons/trunk-stock-reset-negative-qty-onchange/+merge/94979

The onchange methods on stock.move set the UoS qty to 0.0 when a negative qty is keyed in.
I propose to also set the UoM qty to 0.0, effectivly forbidding users to enter negative values.
Lionel Sausin.

-- 
https://code.launchpad.net/~numerigraphe/openobject-addons/trunk-stock-reset-negative-qty-onchange/+merge/94979
Your team OpenERP WMS Expert is requested to review the proposed merge of lp:~numerigraphe/openobject-addons/trunk-stock-reset-negative-qty-onchange into lp:openobject-addons.

=== modified file 'stock/stock.py'
--- stock/stock.py	2012-02-15 18:00:28 +0000
+++ stock/stock.py	2012-02-28 15:15:23 +0000
@@ -1764,6 +1764,7 @@
           }
 
         if (not product_id) or (product_qty <=0.0):
+            result['product_qty'] = 0.0
             return {'value': result}
 
         product_obj = self.pool.get('product.product')
@@ -1790,6 +1791,7 @@
           }
 
         if (not product_id) or (product_uos_qty <=0.0):
+            result['product_uos_qty'] = 0.0
             return {'value': result}
 
         product_obj = self.pool.get('product.product')