Merge lp:~openerp-dev/openobject-client-web/6.0-opw-594176-han into lp:openobject-client-web

Proposed by Hardik Ansodariya (OpenERP) on 2013-06-26
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-client-web/6.0-opw-594176-han
Merge into: lp:openobject-client-web
Diff against target: 13 lines (+2/-1)
1 file modified
addons/openerp/controllers/listgrid.py (+2/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-client-web/6.0-opw-594176-han
Reviewer Review Type Date Requested Status
OpenERP Core Team 2013-06-26 Pending
Review via email: mp+171507@code.launchpad.net

Description of the change

Hi,

Fixed the issue of duplicating domain when move to next page in list view.

To reproduce:

Add a domain 'project_id' = 1 in action of project.issue.

When try to move on next page args in search is like [('project_id', '=', 1), ('project_id', '=', 1)], it duplicates the domain.

Thanks,

To post a comment you must log in.

Unmerged revisions

4916. By Hardik Ansodariya (OpenERP) on 2013-06-26

[FIXED] Fixed the issue of duplicating domain when move to next page on list view (Maintenance case: 594176)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/openerp/controllers/listgrid.py'
2--- addons/openerp/controllers/listgrid.py 2012-08-24 10:57:38 +0000
3+++ addons/openerp/controllers/listgrid.py 2013-06-26 11:22:43 +0000
4@@ -283,7 +283,8 @@
5 if not params.view_type == 'graph':
6 params.view_type = 'tree'
7 if params.search_domain:
8- params.domain += params.search_domain
9+ search_domain = [x for x in params.search_domain if x not in params.domain]
10+ params.domain += search_domain
11
12 params.domain = params.domain or []
13 if params.filter_domain: