openerp-community-reviewer team mailing list archive

["Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>]

Review: Needs Fixing code review

I think you'd be better off using search(..., limit=your_limit, offset=your_offset).

If you don't want to make that change, at least the 10000 should be in a variable and you should use range(start, end, step)
-- 
https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM/+merge/217049
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons.