[14.0][FIX] purchase_sale_inter_company: sync confirmed unlocked purchase updates

[chienandalu]

When the purchase user updates the PO after the SO is generated and always if both are confirmed and unlocked, we sync the new lines to update the demand.

cc @Tecnativa TT46773

please take a look @pedrobaeza

[OCA-git-bot]

Hi @renda-dev, @aleuffre, some modules you are maintaining are being modified, check this out!

[chienandalu]

I miss a write override for also synchronizing changed quantities/prices.

I added that logic as well. For the moment only to sync quantities

[francesco-ooops]

When the purchase user updates the PO after the SO is generated and always if both are open, we sync the new lines to update the demand.

thanks for the PR @chienandalu , just to understand better functionally: what states are you referring to with "open"?

[chienandalu]

thanks for the PR @chienandalu , just to understand better functionally: what states are you referring to with "open"?

Basically, both the purchase and its sale order are in states purchase/sale

[francesco-ooops]

@chienandalu ok, so they're both "confirmed" :)

Will test and review

[chienandalu]

@chienandalu ok, so they're both "confirmed" :)

Yes, but not done (locked)

[chienandalu]

Chicago purchases from San Francisco, product Acoustic Block Screen qty 10 > confirm Chicago updates POL qty to 5 San Francisco still has qty 10 on SOL and picking OUT

I just tested it on runboat and it works as expected:

• the sale order gets the updated qty to the decreased one and the picking schedules an warning activity for the decreased qty (this is the standard in this version when you decrease the demand from the sale order)

[pedrobaeza]

Please rebase, as there's now a conflict.

[chienandalu]

• Rebased
• Added a test with @francesco-ooops case

[pedrobaeza]

/ocabot merge minor

[pedrobaeza]

Please fw-port it.

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-559-by-pedrobaeza-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 6552d3bfec2226c0e66e8bdd299774304387de12. Thanks a lot for contributing to OCA. ❤️