openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde into lp:ocb-addons

To: mp+214486@xxxxxxxxxxxxxxxxxx
From: "Guewen Baconnier @ Camptocamp" <guewen.baconnier@xxxxxxxxxxxxxx>
Date: Fri, 11 Apr 2014 08:17:15 -0000
In-reply-to: <20140411075149.17829.39564.codereview@chaenomeles.canonical.com>
Reply-to: mp+214486@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

> - wouldn't it be the same to write "result = list(ids)" at line 21?

`list(ids)` wouldn't keep the order of ids.

-- 
https://code.launchpad.net/~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde/+merge/214486
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde into lp:ocb-addons.

References

Re: [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1302630_document_search_order_by-rde into lp:ocb-addons
From: Lionel Sausin - Numérigraphe, 2014-04-11