[16.0][MIG] purchase_stock_price_unit_sync: Migration to 16.0 #2137
Conversation
…k moves already done [UPD] Update purchase_stock_price_unit_sync.pot [UPD] README.rst
[UPD] Update purchase_stock_price_unit_sync.pot
[UPD] Update purchase_stock_price_unit_sync.pot
If we receive a kit, we shouldn't write the kit purchase price on the components price units. TT32040 purchase_stock_price_unit_sync 13.0.1.0.1
…nded costs for the update if the stock_landed_costs addon is installed. TT35009
Steps to reproduce: - Create a purchase order with an storable product and confirm it. - Validate the reception picking. - Create a landed cost linked to the reception picking and validate it. - Now change the unit price in the purchase order. Result: Both SVLs (stock valuation layers), the one for the reception, and the landed cost one, result overwritten with the new purchase unit price, while only the reception one should. As the landed cost SVL is put as children (through the field `stock_valuation_layer_id`), we filter out these ones from the write.
In purchase_discount the price_unit needs to be written in order to be able to compute the discount as the hook method isn't being used in core (for the moment). These write should be ignored as they'll lead to an infinite loop TT36816
…rors when accessing stock.valuation.layer TT45036
|
@AungKoKoLin1997 Thank you for you explanation in here: OCA/stock-logistics-workflow#1465 I created a product and set it as AVG cost with automated valuation. Then I received one @ 10USD, then I receive another @ 10 USD. Then change the price on the first PO to be 20 USD. The layer unit price was updated but not the total value:
The cost in the product card was still 10 USD, shouldn't it be updated to 15 USD? |
|
@AungKoKoLin1997 Just did the same in v15 and the behavior is the same, I see that the module product_cost_price_avco_sync is not dependency here, so please disregard my previous comment. |
|
/ocabot migration purchase_stock_price_unit_sync |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at d08b546. Thanks a lot for contributing to OCA. ❤️ |
Standard Migration.
@qrtl