summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/views/basic/basic_model.js
diff options
context:
space:
mode:
authorSimon Lejeune <sle@openerp.com>2018-02-14 17:08:05 +0100
committerSimon Lejeune <sle@openerp.com>2018-02-20 14:42:44 +0100
commit7ed86b53cdfbab9a7867de328604850138692caf (patch)
tree46345d20036687b9c3c8a825faf11aec81841b2b /addons/web/static/src/js/views/basic/basic_model.js
parent980d786b09db3186681f0416496542ddec2fa985 (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