Merge lp:~openerp-groupes/openobject-client-web/fix-unlimited-sorted-tfr into lp:~openerp-groupes/openobject-client-web/6.0

Proposed by tfr (Openerp)
Status: Needs review
Proposed branch: lp:~openerp-groupes/openobject-client-web/fix-unlimited-sorted-tfr
Merge into: lp:~openerp-groupes/openobject-client-web/6.0
Diff against target: 30 lines (+9/-11)
1 file modified
addons/openerp/widgets/listgrid.py (+9/-11)
To merge this branch: bzr merge lp:~openerp-groupes/openobject-client-web/fix-unlimited-sorted-tfr
Reviewer Review Type Date Requested Status
OpenERP Groupe S Pending
Review via email: mp+83386@code.launchpad.net
To post a comment you must log in.

Unmerged revisions

4492. By tfr (Openerp)

[FIX] unlimited not sorted + refactor a little bit the code

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/openerp/widgets/listgrid.py'
2--- addons/openerp/widgets/listgrid.py 2011-03-29 15:51:28 +0000
3+++ addons/openerp/widgets/listgrid.py 2011-11-25 11:01:36 +0000
4@@ -148,17 +148,15 @@
5 if not self.source:
6 self.source = terp_params.get('_terp_source', None)
7 if not default_data and not self.o2m and not self.m2m:
8- if self.limit > 0:
9- if self.sort_key:
10- ids = proxy.search(search_param, self.offset, self.limit, self.sort_key + ' ' +self.sort_order, context)
11- else:
12- if search_text:
13- if self.source == '_terp_list':
14- ids = proxy.search(search_param, self.offset, self.limit, False, context)
15- else:
16- ids = proxy.search(search_param, self.offset, self.limit, False, context)
17- else:
18- ids = proxy.search(search_param, 0, 0, 0, context)
19+ limit = 0
20+ sort_info = False
21+ if self.limit >= 0:
22+ limit = self.limit
23+ if self.sort_key:
24+ sort_info = self.sort_key + ' ' +self.sort_order
25+
26+ ids = proxy.search(search_param, self.offset, limit, sort_info, context)
27+
28 if len(ids) < self.limit:
29 if self.offset > 0:
30 self.count = len(ids) + self.offset

Subscribers

People subscribed via source and target branches