Merge lp:~camptocamp/openobject-addons/6.1-opening-entry-fix-mdh into lp:~camptocamp/openobject-addons/6.1-c2c-official

[Matthieu Dietrich @ camptocamp]

[FIX] lp:1257288 - fixed amount in foreign currency for opening entries

Found this issue: when computing an opening entry, the move lines' domains differ between the amount in system currency (all move lines in the latest FY before the current one, for example FY2012) and the amount in foreign currency (all move lines in previous FY, for example FY2012, 2011, 2010...).

I removed the old method to retrieve the sum of foreign amounts, which used a domain of multiple FYs, and retrieved the correct sum by using the function field "foreign_balance" in the account (since a single FY is specified in the browse() context, only the balance for this FY is retrieved; this is how the correct "account.balance" works).

New 6.1 branch, after another commit was "stored" in this one (I'll make a separate MP to keep XA and c2c-official consistent).

[Romain Deheele - Camptocamp (romaindeheele)]

LGTM,

Romain

review: Approve (code review)

[Leonardo Pistone (lepistone)]

I have a doubt about """if account.currency_id.id else 0.0"""

If the currency_id field is empty, then if I remember correctly, account.currency_id is a browse_null
object, something that should be False when cast to boolean. On the other hand, I am not sure that account.currency_id.id will work.

I suggest changing that line to """if account.currency_id else 0.0"""

Thanks

review: Needs Information

[lp:~camptocamp/openobject-addons/6.1-opening-entry-fix-mdh]

6641. By Matthieu Dietrich @ camptocamp on 2014-04-17

[FIX] remove .id from boolean test

[Matthieu Dietrich @ camptocamp (mdietrich-c2c)]

@Leonardo : good point, I missed this and it is now corrected.

[Leonardo Pistone (lepistone)]

thanks!

review: Approve

[Romain Deheele - Camptocamp (romaindeheele)]

Remark : if currency_id is null, currency_id.id returns False (in 7.0, I don't know in 6.1).
but I'm ok with this change.

Romain

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

In 6.1 and 7.0 browse_null should act the same. Thus returning None when __getattr__ is called:

http://bazaar.launchpad.net/~openerp/openobject-server/6.1/view/head:/openerp/osv/orm.py#L257
http://bazaar.launchpad.net/~openerp/openobject-server/7.0/view/head:/openerp/osv/orm.py#L266

However for 6.1 there are very few use of this mechanism so it is fine that way.

review: Approve

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

review: Approve

=== modified file 'account/wizard/account_fiscalyear_close.py'
--- account/wizard/account_fiscalyear_close.py	2012-02-09 17:03:33 +0000
+++ account/wizard/account_fiscalyear_close.py	2014-04-17 07:54:37 +0000
@@ -224,14 +224,6 @@
         query_2nd_part = ""
         query_2nd_part_args = []
         for account in obj_acc_account.browse(cr, uid, account_ids, context={'fiscalyear': fy_id}):
-            balance_in_currency = 0.0
-            if account.currency_id:
-                cr.execute('SELECT sum(amount_currency) as balance_in_currency FROM account_move_line ' \
-                        'WHERE account_id = %s ' \
-                            'AND ' + query_line + ' ' \
-                            'AND currency_id = %s', (account.id, account.currency_id.id))
-                balance_in_currency = cr.dictfetchone()['balance_in_currency']
-
             company_currency_id = self.pool.get('res.users').browse(cr, uid, uid).company_id.currency_id
             if not currency_obj.is_zero(cr, uid, company_currency_id, abs(account.balance)):
                 if query_2nd_part:
@@ -246,7 +238,7 @@
                        period.id,
                        account.id,
                        account.currency_id and account.currency_id.id or None,
-                       balance_in_currency,
+                       account.foreign_balance if account.currency_id else 0.0,
                        account.company_id.id,
                        'draft')
         if query_2nd_part: