Merge lp:~camptocamp/account-consolidation/fix-domain-nbi into lp:~account-core-editors/account-consolidation/7.0

[Nicolas Bessi - Camptocamp]

[FIX] doamin on function fields that can badly impact performance

[Leonardo Pistone (lepistone)]

On a low-level, I see that elements of curr_ids that are not in ids should not change the result. So I approve the fix.

On the other hand, I don't get what the method does. Could you take the occasion to add a docstring or comment, and maybe rename tid, ids, curr_ids ...

thanks!

review: Approve

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

Good catch

review: Approve

=== 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