Merge lp:~arthru/ocb-addons/fix-1334657 into lp:ocb-addons

[arthru]

Fix the bug lp:1334657 : makes it possible to exit the loop

[Pedro Manuel Baeza (pedro.baeza)]

This project is now hosted on https://github.com/OCA/OCB. Please move your proposal there. This guide may help you https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub

review: Needs Resubmitting

[arthru (arthru)]

Resubmited as pull request #29.

The method you recommended does not work for OCB as OCB in github is a combination of server, addons and web, and the branch in LP is just addons.

I just used the patch in git...

[Holger Brunn (Therp) (hbrunn)]

Development for 7.0 has moved to github on https://github.com/OCA/ocb - please move your merge proposal there if it is still valid.

(I close and reject this in order to have a cleaner overview for 6.1 MPs which indeed have to be done on launchpad)

review: Disapprove

Unmerged revisions

10220. By Arthur Vuillard <email address hidden> on 2014-06-26

Fix bug lp:1334657

=== modified file 'procurement/schedulers.py'
--- procurement/schedulers.py	2014-03-10 08:54:20 +0000
+++ procurement/schedulers.py	2014-06-26 13:31:59 +0000
@@ -82,7 +82,6 @@
                     if maxdate >= proc.date_planned:
                         wf_service.trg_validate(uid, 'procurement.order', proc.id, 'button_check', cr)
                     else:
-                        offset += 1
                         report_later += 1
 
                     if proc.state == 'exception':
@@ -91,6 +90,7 @@
                                     proc.product_id.name))
                         report_except += 1
                     report_total += 1
+                offset += len(ids)
                 if use_new_cursor:
                     cr.commit()
                 if not ids: