[14.0] Dont force Done state on PO picking

[thomaspaulb]

Currently the code forces PO picking state to Done when SO picking is done, but this can be harmful:

• If there was a mismatch in SO and PO move lines, activity is created for user to fix, but picking is already set to Done, then user can't correct anything

---

I do think this is a good idea because the PO picking state should IMO never be force-set to Done; better behaviour is that the move lines are processed like they should and then the picking will automatically be set to Done when everything is in order; if not, it should leave the picking open and create an activity for a user.

MR is still set to Draft because:

• I don't know if this is actually a case that happens in real life of if it was caused by https://github.com/OCA/multi-company/pull/646 or https://github.com/OCA/stock-logistics-warehouse/pull/2025
• Because of the above I also can't make tests, because I fail to reproduce a case where this would be needed

[OCA-git-bot]

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