Odoo GTK Client (MOVED TO GITHUB)

[6.0/trunk] GTK : Group By on Boolean Field doesn't work!

Bug #876619 reported by Serpent Consulting Services
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo GTK Client (MOVED TO GITHUB)
Status tracked in Trunk
6.0
Fix Released
Medium
OpenERP Publisher's Warranty Team
Odoo GTK Client (MOVED TO GITHUB) 6.0.4
Trunk
Fix Released
Medium
OpenERP sa GTK client R&D
Odoo GTK Client (MOVED TO GITHUB) 6.1

Bug Description

When you have a boolean field and you wish to apply groupby on it, it doesn't work.

The logical impact should be : 2 sets of records, one with TRUE and another with FALSE.

With current code, the symbol on top level record seems always to be TRUE.

As a matter of the fact, its due to the code which manipulates the values to be bool(value) everytime whereas the result of group by may return False as (False(3)) where 3 is the no. of records.

But system understands bool(False(3)) as True because the evaluator is a string.

Please find the attached branches(for 6.0 and trunk) which fixes the issue.

As an example : One can test with 'Group by Customer' on Partner Model.

Thanks,
Serpent Consulting Services.

Tags: maintenance

Related branches

lp:~serpent-consulting-services/openobject-client/gtk-groupby-boolean-6.0
Naresh(OpenERP) (community): Approve
Diff: 15 lines (+4/-0)
1 file modified
bin/widget/view/tree_gtk/parser.py (+4/-0)
lp:~serpent-consulting-services/openobject-client/gtk-groupby-boolean-6.1
Naresh(OpenERP): Needs Fixing
Diff: 15 lines (+5/-0)
1 file modified
bin/widget/view/tree_gtk/parser.py (+5/-0)
lp:openobject-client/6.0
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

Thanks for reporting !

The fix has been merged @ trunk client with some improvements @ revision-info 1983 <email address hidden>

Thanks,

Changed in openobject-client:
status: New → Fix Released
Revision history for this message
Niels Huylebroeck (red15) wrote :

When will this be merged in 6.0 ? It's marked as fix released but I don't see it in 6.0 client code at r1918

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Thanks for coming back on this Niels.

It has been fixed for trunk and has been set as released for it.

The bug still happens on 6.0 and not fixed yet.

For the fix, you can see the merge proposal attached.

Hope this helps.

Thanks,
Serpent Consulting Services.

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

I wonder why anyone else is not affected by this bug.

This fix should go to in stable as well.

Thanks.

Naresh(OpenERP) (nch-openerp)
tags: added: maintenance
Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Thanks Olivier, Naresh and Niels,

Finally, this bug has been fixed in OpenERP Client version 6.0.

Thanks,
Serpent Consulting Services.
http://www.serpentcs.com

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.