Odoo Web Client

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

  1. 6.0-opw-594176-han
  2. Merge into 6.0
Proposed by Hardik Ansodariya (OpenERP)
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 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)

[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
=== modified file 'addons/openerp/controllers/listgrid.py'
--- addons/openerp/controllers/listgrid.py 2012-08-24 10:57:38 +0000
+++ addons/openerp/controllers/listgrid.py 2013-06-26 11:22:43 +0000
@@ -283,7 +283,8 @@
283 if not params.view_type == 'graph':283 if not params.view_type == 'graph':
284 params.view_type = 'tree'284 params.view_type = 'tree'
285 if params.search_domain:285 if params.search_domain:
286 params.domain += params.search_domain286 search_domain = [x for x in params.search_domain if x not in params.domain]
287 params.domain += search_domain
287288
288 params.domain = params.domain or []289 params.domain = params.domain or []
289 if params.filter_domain:290 if params.filter_domain: