Odoo Server (MOVED TO GITHUB)

sorting in report view

Bug #1060086 reported by Pollet Alexandre on 2012-10-02

This bug affects 7 people

Affects		Status	Importance	Assigned to	Milestone
	Odoo Server (MOVED TO GITHUB)	Fix Released	Low	OpenERP Publisher's Warranty Team

Bug Description

Hello,

In the sale report view, when i try to sort by "total price" (it's does'nt work with other fields either), the sorting is incorrect.

So I can't see my best partner (for example)

I don't have the problem with classic tree view.

See the screeshot attached.

Tags:

Related branches

lp:~olivier-laurent/openobject-server/7.0-bug1060086

Rejected for merging into lp:openobject-server/7.0

Anaël Closson (openerp): Pending requested 2014-01-21

lp:~openerp-dev/openobject-server/7.0-opw-598942b-acl

Merged into lp:openobject-server/7.0 at revision 5237

Olivier Laurent (Acsone) (community): Needs Fixing on 2014-02-19

Martin Trigaux (OpenERP) (community): Approve on 2014-02-18

Revision history for this message

Pollet Alexandre (a-pollet) wrote on 2012-10-02:

screenshot sorting bug Edit (82.6 KiB, image/png)

Revision history for this message

Amit Parik (amit-parik) wrote on 2012-11-27:

Hello Pollet,

Would you please try with the same thing with trunk 7.0 cause I have checked this issue with trunk and it's working fine.

Please try with trunk 7.0 and let us know about it.

Thanks and waiting for your reply!

Changed in openerp-web:
status:	New → Incomplete

Revision history for this message

Pollet Alexandre (a-pollet) wrote on 2012-11-27:

sort_stats_v7.png Edit (15.1 KiB, image/png)

Still doesn't work...

serv rev : 4601
addons rev : 8141
web rev : 3502

Jignesh Rathod(OpenERP) (jir-openerp) on 2012-11-29

affects:

openerp-web → openobject-addons

Revision history for this message

Jignesh Rathod(OpenERP) (jir-openerp) wrote on 2012-12-04:

Sales Analysis.ogv Edit (1.7 MiB, video/ogg)

Changed in openobject-addons:
assignee:	nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2)
importance:	Undecided → Low
status:	Incomplete → Confirmed

Amit Parik (amit-parik) on 2012-12-14

tags:	added: usability
summary:	- [6.1] sorting in report view + sorting in report view

Revision history for this message

Javier Fuentes (javier-codeback) wrote on 2013-05-22:

Problem persists in v7. Seems related with the grouping of data.

Regards.

Naresh(OpenERP) (nch-openerp) on 2013-10-11

tags:	added: maintenance
Changed in openobject-addons:
assignee:	OpenERP R&D Addons Team 2 (openerp-dev-addons2) → OpenERP Publisher's Warranty Team (openerp-opw)

Revision history for this message

Olivier Laurent (Acsone) (olivier-laurent) wrote on 2013-10-28:

Hello OpenERP Team,

Any news about the fixing of this issue ?

The bug is located in the read_group method of the BaseModel in orm.py arround this line of code:
data_ids = self.search(cr, uid, [('id', 'in', alldata.keys())], order=order, context=context)
that tries to introduce the sort _AFTER_ the fetch of data rows with the sql query that preceeds.

Unfortunately, the domain passed to the search method is based on a collection of aggregate ids (i.e. min(id)) for which the sort is made on their own fields values _NOT_ on the aggregate values computed by the sql query.

It would be better to introduce the sort directly in the sql query with a classic "ORDER BY" clause.

Many thanks in advance for the fix.

Revision history for this message

Olivier Dony (Odoo) (odo-openerp) wrote on 2014-03-05:

#10

This issue has been initially fixed in server 7.0 at revision 5237, and further fixed/refined in the next revisions (please grab the latest).

Thanks for reporting!

PS: moving this bug to server, as it was a generic framework concern

Changed in openobject-addons:
milestone:	none → 7.0
status:	Confirmed → Fix Released
affects:	openobject-addons → openobject-server
Changed in openobject-server:
milestone:	7.0 → none