From 4e5ab1870b81011f3490d6625f2f085231c408a4 Mon Sep 17 00:00:00 2001
From: Olivier Dony <odo@openerp.com>
Date: Wed, 18 Aug 2010 18:44:55 +0200
Subject: [PATCH] [REVERT] orm/expression: reverted revision 2563 (it breaks
 fields.related) until we do a better fix

bzr revid: odo@openerp.com-20100818164455-nq6altxdrywwfz3i
---
 bin/osv/expression.py | 28 +++-------------------------
 1 file changed, 3 insertions(+), 25 deletions(-)

diff --git a/bin/osv/expression.py b/bin/osv/expression.py
index 95ab02bdeee..322ee3cd18d 100644
--- a/bin/osv/expression.py
+++ b/bin/osv/expression.py
@@ -115,7 +115,6 @@ class expression(object):
             if self._is_operator(e) or e == self.__DUMMY_LEAF:
                 continue
             left, operator, right = e
-            operator = operator.lower()
             working_table = table
             main_table = table
             fargs = left.split('.', 1)
@@ -279,36 +278,15 @@ class expression(object):
                         dom = _rec_get(ids2, working_table, parent=left)
                     self.__exp = self.__exp[:i] + dom + self.__exp[i+1:]
                 else:
-                    def _get_expression(field_obj,cr, uid, left, right, operator, context=None):
-                        if context is None:
-                            context = {}
+                    if isinstance(right, basestring): # and not isinstance(field, fields.related):
                         c = context.copy()
                         c['active_test'] = False
-                        dict_op = {'not in':'!=','in':'='}
-                        if isinstance(right,tuple):
-                            right = list(right)
-                        if (not isinstance(right,list)) and operator in ['not in','in']:
-                            operator = dict_op[operator]
-                            
                         res_ids = field_obj.name_search(cr, uid, right, [], operator, limit=None, context=c)
                         if not res_ids:
-                             return ('id','=',0)
+                            self.__exp[i] = ('id','=',0)
                         else:
                             right = map(lambda x: x[0], res_ids)
-                            return (left, 'in', right)
-
-                    m2o_str = False
-                    if isinstance(right, basestring): # and not isinstance(field, fields.related):
-                        m2o_str = True
-                    elif isinstance(right, list) or isinstance(right, tuple):
-                        m2o_str = True
-                        for ele in right:
-                            if not isinstance(ele, basestring): 
-                                m2o_str = False
-                                break
-
-                    if m2o_str:
-                        self.__exp[i] = _get_expression(field_obj,cr, uid, left, right, operator, context=context)
+                            self.__exp[i] = (left, 'in', right)
             else:
                 # other field type
                 # add the time part to datetime field when it's not there: