diff options
| author | Simon Lejeune <sle@openerp.com> | 2020-06-29 10:11:28 +0000 |
|---|---|---|
| committer | Simon Lejeune <sle@openerp.com> | 2020-07-01 16:09:31 +0000 |
| commit | 058698e10491d73009824cbb83dd9bcc6aff3633 (patch) | |
| tree | 1361c4de4a2203cb1f9c254879c7ab7b37ba785f /addons/web/static/src/js/views/basic/basic_controller.js | |
| parent | 476028c14a0bbe05db12e184b4efb6cd33297363 (diff) | |
[FIX] stock: move line: write
Write is overridden on reserved move lines to make sure the quants are
correctly updated. It is divided in two parts: if you update the reserved
quantity and if you update a characteristic. If somehow both are updated
in the same call (which doesn't happen in the standard interface), then
the quants are unreserved two times, which is wrong.
This issue showcased by [0] and on databases where the reserved
quantities are made editable by customization.
We grouped the two update in one, such as the TODO indicated us since a
long, long time.
We removed `result_packaged_id` from the trigger because it should not
impact the reservation whatsoever.
We needed to remove a tricky logic in the stock move's backordering part
but it was fishy anyway.
[0] eac8c06e2233
closes odoo/odoo#53928
X-original-commit: 507101434f9221f2cfa6de66e1952676fff322fd
Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
Diffstat (limited to 'addons/web/static/src/js/views/basic/basic_controller.js')
0 files changed, 0 insertions, 0 deletions
