Revert "[FIX] product,float_utils: perform ceiling via float_round with new rounding_...
authorDenis Ledoux <dle@odoo.com>
Fri, 26 Sep 2014 19:21:06 +0000 (21:21 +0200)
committerDenis Ledoux <dle@odoo.com>
Fri, 26 Sep 2014 19:21:06 +0000 (21:21 +0200)
commit333852e19dd3d5174e07e29f9d8f6c087b2c09f6
treefd180c81bae8f0483d329bfb44474c171fb10be5
parentceff8ef8992b598d44e95ed349dc63b2d6c6b5c1
Revert "[FIX] product,float_utils: perform ceiling via float_round with new rounding_method UP"

This reverts commit d4972ffdb6b9356a524eef1dbc11f455ff4473f2.

Seems to break some cases, at least in _product_reserve from stock/stock.py

Actual use case:

SELECT product_uom, sum(product_qty) AS product_qty FROM stock_move WHERE location_dest_id=%s AND location_id<>%s AND product_id=3645 AND state='done' GROUP BY product_uom;
returning 1 | 6

SELECT product_uom,-sum(product_qty) AS product_qty FROM stock_move WHERE location_id=%s AND location_dest_id<>%s AND product_id=%s AND state in ('done', 'assigned') GROUP BY product_uom;
returning 1 | -6

results += cr.dictfetchall()
    total = 0.0
    results2 = 0.0
    for r in results:
        amount = uom_obj._compute_qty(cr, uid, r['product_uom'], r['product_qty'], context.get('uom', False))
        results2 += amount
        total += amount
Total = 1, amount = -5

It should actually be
Total = 0, amount = -6
addons/product/_common.py
openerp/addons/base/test/base_test.yml
openerp/tools/float_utils.py