openerp-community-reviewer team mailing list archive

["Guewen Baconnier @ Camptocamp" <guewen.baconnier@xxxxxxxxxxxxxx>]

On 02/26/2014 04:41 PM, Pedro Manuel Baeza wrote:
> Review: Approve code review
>
> LGTM
>
> BTW, I think you could improve code making only one search with:
>
> move_ids = move_obj.search(cr, uid,
>                             [('dispatch_id', 'in', ids)],
>                             context=context)
>
> Regards.
>

Of course you are right :-)
I changed that. Thanks

-- 
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-picking_dispatch-check-availability/+merge/208402
Your team Stock and Logistic Core Editors is subscribed to branch lp:stock-logistic-flows.