filter by date in move_line

[Eric Vernichon]

In 5.0.14 the filter by date doesn't work for move_line

If I try by script it's the same thing

ids=obj.execute(base,uid,pwd, 'account.move.line','search',[('partner_id', '=', 151), ('account_id.type', 'in', ['receivable', 'payable']),('date', '>=', '2010-01-01'), ('date', '<=', '2010-01-29')])
for id in ids:
     print obj.execute(base,uid,pwd, 'account.move.line','read',id)['date']

The result is

 2010-02-18
 2010-02-18
 2009-03-03
 2009-03-03
 2008-11-27
 2008-11-07
 2008-10-16
 2008-10-16
 2008-10-07
 2008-07-01
 2008-07-01

If I replace the expression.py by the file of the version 5.0.6, it works

[Christophe CHAUVET (christophe-chauvet)]

Hi

I confirmed the problem

Regards,

[jftempo (jfb-tempo-consulting)]

The faulty commit: 1950

http://bazaar.launchpad.net/~openerp/openobject-server/5.0/revision/1950

[Eric Vernichon (eric-vernichon)]

yes in 5.0.6 version is OK

but the problem is how to correct that.

For me this bug is really constraining

[Jay Vora (Serpent Consulting Services) (jayvora)]

Hello,

Would you please apply the attached patch on https://bugs.launchpad.net/openobject-server/+bug/626806?

This is an issue of search on fields.related.

Thanks.

[Eric Vernichon (eric-vernichon)]

sorry but not working

[Eric Vernichon (eric-vernichon)]

but if I supress the faulty commit 1950 in expression.py version 5.0.14 (thank you jftempo) , it's work

I wait your solution
But in waiting , I use mine

@ -149,22 +149,21 @@
                     self.__exp[i] = ('id', 'in', right1)

                 continue
-
- if field._properties and ((not field.store) or field._fnct_search):
-
+ if field._properties:
+ if not field.store:
                 # this is a function field
- if not field._fnct_search:
- # the function field doesn't provide a search function and doesn't store
- # values in the database, so we must ignore it : we generate a dummy leaf
- self.__exp[i] = self.__DUMMY_LEAF
- else:
- subexp = field.search(cr, uid, table, left, [self.__exp[i]], context=context)
- # we assume that the expression is valid
- # we create a dummy leaf for forcing the parsing of the resulting expression
- self.__exp[i] = '&'
- self.__exp.insert(i + 1, self.__DUMMY_LEAF)
- for j, se in enumerate(subexp):
- self.__exp.insert(i + 2 + j, se)
+ if not field._fnct_search :
+ # the function field doesn't provide a search function and doesn't store
+ # values in the database, so we must ignore it : we generate a dummy leaf
+ self.__exp[i] = self.__DUMMY_LEAF
+ else:
+ subexp = field.search(cr, uid, table, left, [self.__exp[i]], context=context)
+ # we assume that the expression is valid
+ # we create a dummy leaf for forcing the parsing of the resulting expression
+ self.__exp[i] = '&'
+ self.__exp.insert(i + 1, self.__DUMMY_LEAF)
+ for j, se in enumerate(subexp):
+ self.__exp.insert(i + 2 + j, se)

                 # else, the value of the field is store in the database, so we search on it

[Eric Vernichon (eric-vernichon)]

but if I supress the faulty commit 1950 in expression.py version 5.0.14 (thank you jftempo) , it's work

I wait your solution
But in waiting , I use this patch

[Jay Vora (Serpent Consulting Services) (jayvora)]

Hello,

Fixed in : by

Stable : 2138 <email address hidden>
Trunk : 2831 <email address hidden>

Thanks.