Merge lp://staging/~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM into lp://staging/ocb-addons

Proposed by Lionel Sausin - Initiatives/Numérigraphe
Status: Merged
Approved by: Alexandre Fayolle - camptocamp
Approved revision: no longer in the source branch.
Merged at revision: 10215
Proposed branch: lp://staging/~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM
Merge into: lp://staging/ocb-addons
Diff against target: 15 lines (+2/-3)
1 file modified
stock/wizard/stock_fill_inventory.py (+2/-3)
To merge this branch: bzr merge lp://staging/~numerigraphe-team/ocb-addons/7.0-fix-regression-fill-inventory-OOM
Reviewer Review Type Date Requested Status
Laetitia Gangloff (Acsone) (community) code Approve
Leonardo Pistone Approve
Loïc Bellier - Numérigraphe (community) code review Approve
Guewen Baconnier @ Camptocamp code review Approve
Weste (community) Approve
Review via email: mp+223928@code.staging.launchpad.net

Description of the change

Fixes a regression introduced while fixing #1312045 - we were confused and mixed 2 notations, so we skipped stock moves and the results were wrong.

To post a comment you must log in.
Revision history for this message
Weste (julien-weste) wrote :

That will fix the bug, thanks!

review: Approve
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

LGTM

review: Approve (code review)
Revision history for this message
Loïc Bellier - Numérigraphe (lb-b) wrote :

Great !

review: Approve (code review)
Revision history for this message
Leonardo Pistone (lepistone) wrote :

+1

review: Approve
Revision history for this message
Laetitia Gangloff (Acsone) (laetitia-gangloff) wrote :

Thank you !

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.