Merge lp:~openerp-dev/openobject-addons/6.1-opw-576606-han into lp:openobject-addons/6.1

[Hardik Ansodariya (OpenERP)]

Hello,

Fix the issue;
When pass a value as None It was throwing error below
unsupported operand type(s) for /: 'NoneType' and 'float'

Steps to reproduce:

 * menu "Accounting / Periodical Processing / End of Period / Generate Opening Entries"
 * fields:
   - Fiscal Year to close: 2012
   - New Fiscal Year: 2013
   - Opening Entries Journal: Opening Entries Journal (EUR)
   - Opening Entries Period: Opening Period 2013
 * button: create

[Vinay Rana (OpenERP) (vra-openerp)]

The Propose patch will fix the issue.

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug is not reproducible on Trunk so if this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6882. By Hardik Ansodariya (OpenERP) on 2012-07-17

[FIX] account: set coalesce(sum(amount_currency), 0.0) as it was throwing a error when none value passed (Maintanece Case: 576606)

=== modified file 'account/account.py'
--- account/account.py	2012-07-04 16:33:07 +0000
+++ account/account.py	2012-07-17 11:03:21 +0000
@@ -1517,7 +1517,7 @@
         cr.execute('update account_move_line set '+mode2+'=%s where id=%s', (result, line_id))
 
         #adjust also the amount in currency if needed
-        cr.execute("select currency_id, sum(amount_currency) as amount_currency from account_move_line where move_id = %s and currency_id is not null group by currency_id", (move.id,))
+        cr.execute("select currency_id, coalesce(sum(amount_currency), 0.0) as amount_currency from account_move_line where move_id = %s and currency_id is not null group by currency_id", (move.id,))
         for row in cr.dictfetchall():
             currency_id = currency_obj.browse(cr, uid, row['currency_id'], context=context)
             if not currency_obj.is_zero(cr, uid, currency_id, row['amount_currency']):