diff options
| author | Simon Lejeune <sle@openerp.com> | 2018-02-14 17:08:05 +0100 |
|---|---|---|
| committer | Simon Lejeune <sle@openerp.com> | 2018-02-20 14:42:44 +0100 |
| commit | 7ed86b53cdfbab9a7867de328604850138692caf (patch) | |
| tree | 46345d20036687b9c3c8a825faf11aec81841b2b /addons/web/static/src/js/views/basic/basic_model.js | |
| parent | 980d786b09db3186681f0416496542ddec2fa985 (diff) | |
[FIX] stock: immediate transfer and unreserved move
Before the patch, validating a picking where one of its move do not
have any quantity done nor quantity reserved resulted in the complete
validation of the said move. This doesn't make sense and does not do
what the help in the wizard indicate (process only *reserved* quantities).
We also use this patch to clarify what the tests on this workflow is
doing (test_immediate_validate_*). We completely replace the 3 as it was
testing the `force_assign` mechanism, which methods are still present in
v11 but unused. Instead this test is now testing the failing flow this
commit is fixing.
We thus adapt the `process` method of the immediate transfer to only
process the move lines (move line = reservation). We also fix a case
where nothing is reserved and nothing is done. In this case, the
backorder wizard should not opens but instead an UserError telling the
user to either cancel or process something. The testcase for this flow
is test_immediate_validate_7.
With these changes, the test `test_04_create_picking_update_saleorderline`
was failing because one of the items in the so lines was a stockable
product without stock. So before this patch, everything was processed,
now there's a backorder wizard. We complete the patch and duplicate it
with enough products in stock, so that the backorder wizard is not
needed.
opw-815056
opw-808126
Diffstat (limited to 'addons/web/static/src/js/views/basic/basic_model.js')
0 files changed, 0 insertions, 0 deletions
