[6.0] sale - order of sales order lines
Bug #722699 reported by
Jacques-Etienne Baudoux
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Odoo Addons (MOVED TO GITHUB) |
Fix Released
|
Low
|
OpenERP R&D Addons Team 3 | ||
Bug Description
Hi,
I think the order of the sales order lines is wrong. In sale/sale.py : sale.order.line : it is defined as "sequence, id desc". This means that the line with the highest sequence number will be the first one. This is contradictory with the logic of the sequence in OpenERP. Moreover, if no sequence is used, last created order lines will be listed first (on the screen and in the reports) which is not logical. When I create a sale order, I want to see the lines in the order of creation.
The _order should be "sequence, id" instead of "sequence, id desc".
kr.
Related branches
lp:~openerp-dev/openobject-addons/trunk-bug-722699-ron
- Mustufa Rangwala (Open ERP) (community): Approve
-
Diff: 12 lines (+1/-1)1 file modifiedsale/sale.py (+1/-1)
Revision history for this message
| Jacques-Etienne Baudoux (jbaudoux) wrote | #1 |
| tags: | added: profserv |
| Changed in openobject-addons: | |
| assignee: | nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3) |
| importance: | Undecided → Low |
| status: | New → Confirmed |
| Changed in openobject-addons: | |
| status: | Confirmed → In Progress |
Revision history for this message
| Rohan Nayani(Open ERP) (ron-tinyerp) wrote | #2 |
| Changed in openobject-addons: | |
| status: | In Progress → Fix Committed |
| Changed in openobject-addons: | |
| milestone: | none → 6.1 |
| Changed in openobject-addons: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Small correction on my bug report. Of course, desc only apply to id and not sequence according to sql syntax. It means that sequence ordering is correct. However, I think the sort order on the id is wrong and it should be "sequence, id"