Ravi Gohil (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-server/6.1-opw-575909-rgo into 
lp:openobject-server/6.1.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/6.1-opw-575909-rgo/+merge/110751

Hello,

Sorting `id` in descending order doesn't work. It always stays in ascending 
order.

You can reproduce the issue by defining a class with attribute _order like 
_order = "id desc" or by calling search like obj.search(cr, uid, ids, 
context=context, order="id desc").

This fix fixes this issue. Kindly review it.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.1-opw-575909-rgo/+merge/110751
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.1-opw-575909-rgo.
=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2012-06-15 13:14:27 +0000
+++ openerp/osv/orm.py	2012-06-18 09:08:18 +0000
@@ -4536,7 +4536,7 @@
                 order_direction = order_split[1].strip() if len(order_split) == 2 else ''
                 inner_clause = None
                 if order_field == 'id':
-                    order_by_clause = '"%s"."%s"' % (self._table, order_field)
+                    inner_clause = '"%s"."%s"' % (self._table, order_field)
                 elif order_field in self._columns:
                     order_column = self._columns[order_field]
                     if order_column._classic_read:

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to