openerp-community-reviewer team mailing list archive

[Nicolas Bessi - Camptocamp <nicolas.bessi@xxxxxxxxxxxxxx>]

Nicolas Bessi - Camptocamp has proposed merging lp:~camptocamp/account-consolidation/fix-domain-nbi into lp:account-consolidation/7.0.

Requested reviews:
  Account Core Editors (account-core-editors)

For more details, see:
https://code.launchpad.net/~camptocamp/account-consolidation/fix-domain-nbi/+merge/216331

[FIX] doamin on function fields that can badly impact performance
-- 
https://code.launchpad.net/~camptocamp/account-consolidation/fix-domain-nbi/+merge/216331
Your team Account Core Editors is requested to review the proposed merge of lp:~camptocamp/account-consolidation/fix-domain-nbi into lp:account-consolidation/7.0.

=== modified file 'account_consolidation/account_move_line.py'
--- account_consolidation/account_move_line.py	2013-03-12 10:21:10 +0000
+++ account_consolidation/account_move_line.py	2014-04-17 13:59:33 +0000
@@ -27,7 +27,8 @@
 
     def _current_company(self, cursor, uid, ids, name, args, context=None):
         company_id = self.pool['res.company']._company_default_get(cursor, uid)
-        curr_ids = self.search(cursor, uid, [('company_id', '=', company_id)])
+        curr_ids = self.search(cursor, uid, [('company_id', '=', company_id),
+                                             ('id', 'in', ids)])
         res = dict([(tid, tid in curr_ids) for tid in ids])
         return res