Merge lp:~serpent-consulting-services/openobject-client/gtk-groupby-boolean-6.0 into lp:openobject-client/6.0

Proposed by Serpent Consulting Services
Status: Merged
Approved by: Naresh(OpenERP)
Approved revision: 1904
Merge reported by: Olivier Laurent (Open ERP)
Merged at revision: not available
Proposed branch: lp:~serpent-consulting-services/openobject-client/gtk-groupby-boolean-6.0
Merge into: lp:openobject-client/6.0
Diff against target: 15 lines (+4/-0)
1 file modified
bin/widget/view/tree_gtk/parser.py (+4/-0)
To merge this branch: bzr merge lp:~serpent-consulting-services/openobject-client/gtk-groupby-boolean-6.0
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Review via email: mp+79573@code.launchpad.net
To post a comment you must log in.
1904. By Naresh(OpenERP)

[REF] Correction over the previous commit

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/view/tree_gtk/parser.py'
2--- bin/widget/view/tree_gtk/parser.py 2011-09-09 11:22:35 +0000
3+++ bin/widget/view/tree_gtk/parser.py 2011-10-20 07:20:28 +0000
4@@ -370,7 +370,11 @@
5 def setter(self, column, cell, store, iter):
6 model = store.get_value(iter, 0)
7 value = self.get_textual_value(model)
8+ # A Fix to manage the groupby on boolean fields
9+ if not isinstance(value,(bool,int)):
10+ value = eval(value.split(' ')[0])
11 cell.set_active(bool(value))
12+
13 if model.value.get('state',False):
14 self.state_set(model, model.value.get('state','draft'))
15 self.attrs_set(model, cell)