Merge lp:~openerp-dev/openobject-client/6.0-opw-15282-jvo into lp:openobject-client/6.0
Proposed by
Jay Vora (Serpent Consulting Services)
Status: | Merged |
---|---|
Merged at revision: | 1868 |
Proposed branch: | lp:~openerp-dev/openobject-client/6.0-opw-15282-jvo |
Merge into: | lp:openobject-client/6.0 |
Diff against target: |
41 lines (+18/-8) 1 file modified
bin/modules/gui/window/form.py (+18/-8) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-client/6.0-opw-15282-jvo |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
OpenERP Core Team | Pending | ||
Review via email: mp+66888@code.launchpad.net |
Description of the change
GTK calls an extra time search() with domain=[]
Steps:
* create a lot of records (example: partners), for example by duplicating existing one. I tested with more or less 200 partners
* execute a search query: try to get only 2 or 3 records. I searched for "name = asus"
* switch to form view
* switch back to list view
-> sql query uses "where active = 't'" but no other criteria is used, not even name ilike '%asus%'
the domain sued by the sarch is an empty list: [] instead of [('name', 'ilike', 'asus')]
Note: v5 client used the correct domain
Problem is with get_resource() which forcefully called search([]).
Thanks.
To post a comment you must log in.
Thanks for approving it Olivier.
This merge will remove unnecessary search call and will let the system work faster while switching the views and while searching a desired record(resource) by Pressing Ctrl+ G (in GTK).