Merge lp:~openerp-dev/openobject-client/trunk-bug-816207-nch into lp:openobject-client

Proposed by Naresh(OpenERP)
Status: Merged
Merged at revision: 1937
Proposed branch: lp:~openerp-dev/openobject-client/trunk-bug-816207-nch
Merge into: lp:openobject-client
Diff against target: 42 lines (+4/-17)
2 files modified
bin/widget_search/custom_filter.py (+4/-3)
bin/widget_search/form.py (+0/-14)
To merge this branch: bzr merge lp:~openerp-dev/openobject-client/trunk-bug-816207-nch
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Approve
Review via email: mp+70520@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/widget_search/custom_filter.py'
2--- bin/widget_search/custom_filter.py 2011-04-07 05:32:21 +0000
3+++ bin/widget_search/custom_filter.py 2011-08-05 06:02:15 +0000
4@@ -112,9 +112,10 @@
5 self.widget_obj.field_left = self.field_selection[field_string][0]
6 self.widget_obj.selected_oper_text = self.operators_lst[self.combo_op.get_active()]
7 self.widget_obj.selected_oper = self.op_selection[self.widget_obj.selected_oper_text]
8- wid_domain = self.widget_obj._value_get()
9- condition = self.condition_next.get_active() == 0 and '&' or '|'
10- domain = [condition] + wid_domain
11+ domain = self.widget_obj._value_get()
12+ condition = self.condition_next.get_active() == 1 and '|' or False
13+ if condition:
14+ domain = [condition] + domain
15 return {'domain':domain}
16
17 def sig_exec(self, widget):
18
19=== modified file 'bin/widget_search/form.py'
20--- bin/widget_search/form.py 2011-07-27 12:55:13 +0000
21+++ bin/widget_search/form.py 2011-08-05 06:02:15 +0000
22@@ -460,20 +460,6 @@
23 ordered_gp += self.groupby[val]
24 if ordered_gp:
25 context.update({'group_by':ordered_gp})
26- if domain:
27- pos = False
28- if '&' in domain or '|' in domain:
29- if domain[-2] in ['&','|']:
30- pos = 2
31- elif len(domain) >= 4 and domain[-4] in ['&','|']:
32- pos = 4
33- if len(domain) == 4 and domain[0] in ['&','|'] and domain[1] in ['&','|']:
34- domain = domain[1:]
35- else:
36- if pos:
37- res1 = domain[:-pos]
38- res2 = domain[-(pos-1):]
39- domain = res1 + res2
40 return {'domain':domain, 'context':context}
41
42 def destroy(self):