Code review comment for lp:~peter-langenberg/openobject-addons/addons12_001-1134365

Revision history for this message
Chris Biersbach (OpenERP) (cbi-openerp) wrote :

This fix seems ok. In fact, the order is random, because on one sale order, the order_id is the same for all the lines, and the sequence is initially 10 (it is only changed if you change the order manually afterwards), so the order could actually be anything.

Your fix works because we are in a case with editable=bottom, so a line that is added afterwards, which will get a higher id, will be at the bottom.

« Back to merge proposal