Merge lp:~openerp-dev/openobject-addons/7.0-opw-604693-ado into lp:openobject-addons/7.0

Proposed by Amit Dodiya (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/7.0-opw-604693-ado
Merge into: lp:openobject-addons/7.0
Diff against target: 12 lines (+1/-1)
1 file modified
analytic/analytic.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-604693-ado
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+209868@code.launchpad.net

Description of the change

Hello,

[FIX] hr_timesheet_sheet with analytic: while searching on many2one field based on analytic account with filter does not contains and contain gives same result on timsheet activity tree view

Steps to reproduce:
1. Install hr_timesheet_sheet module
2. Go to timesheet activities list view.
3. Create an advance filter with Analytic Account, operator - "doesn't contain" and value from one of the analytic accounts that is seen in the list view and search on it.
4. Again add a filter with Analytic Account, operator - "contain" and value, you will find that both the filter gives you same result.

Regards,
Amit

To post a comment you must log in.

Unmerged revisions

9857. By Amit Dodiya (OpenERP)

[FIX] hr_timesheet_sheet with analytic: while searching on many2one field based on analytic account with filter does not contains and contain gives same result on timsheet activity tree view

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'analytic/analytic.py'
2--- analytic/analytic.py 2014-01-06 15:38:18 +0000
3+++ analytic/analytic.py 2014-03-07 09:25:57 +0000
4@@ -301,7 +301,7 @@
5 dom = []
6 for name2 in name.split('/'):
7 name = name2.strip()
8- account_ids = self.search(cr, uid, dom + [('name', 'ilike', name)] + args, limit=limit, context=context)
9+ account_ids = self.search(cr, uid, dom + [('name', operator, name)] + args, limit=limit, context=context)
10 if not account_ids: break
11 dom = [('parent_id','in',account_ids)]
12 else: