openerp-community-reviewer team mailing list archive

[Weste <julien.weste@xxxxxxxxx>]

Public bug reported:

rev 10204 introduce this bad loop:

MAX_IMPORT_LINES = 10000
for i in range(0, len(all_move_ids), self.MAX_IMPORT_LINES):
                move_ids = all_move_ids[i * self.MAX_IMPORT_LINES:
                                        (i + 1) * self.MAX_IMPORT_LINES]

i will have the values: 0, 10000, 20000, etc.
so move_ids will be all_move_ids[0:10000], all_move_ids[100000000:100010000], all_move_ids[200000000:200010000] etc.

suggestion: move_ids = all_move_ids[i:i + self.MAX_RECORDS]

** Affects: ocb-addons
     Importance: Undecided
         Status: New


** Tags: inventory

** Branch linked: lp:ocb-addons

-- 
You received this bug notification because you are a member of OpenERP
Community Backports, which is subscribed to OpenERP Community Backports
(Addons).
https://bugs.launchpad.net/bugs/1329342

Title:
  stock.fill.inventory doesn't consider all the records

Status in OpenERP Community Backports (Addons):
  New

Bug description:
  rev 10204 introduce this bad loop:

  MAX_IMPORT_LINES = 10000
  for i in range(0, len(all_move_ids), self.MAX_IMPORT_LINES):
                  move_ids = all_move_ids[i * self.MAX_IMPORT_LINES:
                                          (i + 1) * self.MAX_IMPORT_LINES]

  i will have the values: 0, 10000, 20000, etc.
  so move_ids will be all_move_ids[0:10000], all_move_ids[100000000:100010000], all_move_ids[200000000:200010000] etc.

  suggestion: move_ids = all_move_ids[i:i + self.MAX_RECORDS]

To manage notifications about this bug go to:
https://bugs.launchpad.net/ocb-addons/+bug/1329342/+subscriptions