From 25a883b2aad12bbb957b25ba88e0a2c657f2b32b Mon Sep 17 00:00:00 2001
From: Vo Minh Thu <vmt@openerp.com>
Date: Wed, 5 Oct 2011 15:33:37 +0200
Subject: [PATCH] [FIX] expression: we dont add any restriction on the main ids
 as the possible active=True/False is already present in the main domain.

bzr revid: vmt@openerp.com-20111005133337-p8nvv26tlii7x253
---
 openerp/addons/base/test/test_osv_expression.yml | 2 +-
 openerp/osv/expression.py                        | 9 +--------
 2 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/openerp/addons/base/test/test_osv_expression.yml b/openerp/addons/base/test/test_osv_expression.yml
index d7e342926d0..f67a99cdf46 100644
--- a/openerp/addons/base/test/test_osv_expression.yml
+++ b/openerp/addons/base/test/test_osv_expression.yml
@@ -500,7 +500,7 @@
         except psycopg2.DataError:
             # Should give: 'DataError: invalid input syntax for type timestamp' or similar
             cr.execute('ROLLBACK TO SAVEPOINT expression_failure_test')
-
+-
     Testing for Many2Many field with category supplier and active=False
 -
     !python {model: res.partner }: |
diff --git a/openerp/osv/expression.py b/openerp/osv/expression.py
index 25cdffd2b83..cf7ef3b70cf 100644
--- a/openerp/osv/expression.py
+++ b/openerp/osv/expression.py
@@ -405,10 +405,6 @@ class expression(object):
                 return [value]
             return list(value)
 
-        active = False
-        for exp in self.__exp:
-            if exp[0] == 'active':
-                active = exp
         i = -1
         while i + 1<len(self.__exp):
             i += 1
@@ -466,10 +462,7 @@ class expression(object):
                 # Making search easier when there is a left operand as field.o2m or field.m2m
                 if field._type in ['many2many', 'one2many']:
                     right = field_obj.search(cr, uid, [(field_path[1], operator, right)], context=context)
-                    domain = [(fargs[0],'in', right)]
-                    if active:
-                        domain.append(active)
-                    right1 = table.search(cr, uid, domain, context=context)
+                    right1 = table.search(cr, uid, [(field_path[0],'in', right)], context=dict(context, active_test=False))
                     self.__exp[i] = ('id', 'in', right1)
 
                 if not isinstance(field, fields.property):