[13.0] [REGRESSION] Orderpoints broken with multi-step reception & multi-warehouse

[jbaudoux]

This recent commit 006ab22 in odoo regarding this issue #75236 is causing some regressions

Setup:

• 2 warehouses configured in 2 step reception: WH & WH2
• Product A with no stock, the default buy route and a supplier
• An orderpoint in each warehouse
  • orderpoint on WH (max 30)
  • orderpoint on WH2 (max 20)

Case 1
Run the scheduler.
Expected result: A PO of 30 for the WH and a PO of 20 for WH2.
Wrong result: A PO of 30 for the WH and no PO of 20 for WH2.

Case 2
Resupply WH2 from WH by configuring it on the warehouse and enabling the route on the product. Configure the product or rule as MTO (so the resupply will purchase and not take from WH stock)
Run the scheduler.
Expected result: A PO line of 30 for the WH, a move of 20 from WH to WH2, and a PO line of 20 for WH.
Wrong result: A PO of 30 for the WH, a move of 20 from WH to WH2 and no PO of 20 for WH2.

@agr-odoo @amoyaux

[agr-odoo]

Hello,
apologizes for the inconvenience, I'm working on a tentative solution here

[mathieu-mgm]

@jbaudoux Hello, for case 2, may I ask why you have set up two different routes to resupply the same warehouse for the same product ? In my opinion the expected result for case 2 should be : A PO line for 50 qty, and a move of 20 from WH to WH2

[jbaudoux]

@mathieu-mgm The second case is related to this test case that is currently failing since the change: https://github.com/OCA/purchase-workflow/blob/13.0/purchase_line_procurement_group/tests/test_po_line_proc_group.py

I think you are right that it should create one PO line of 50. Since the change, it is only creating a PO line of 30 and nothing for the MTO need of 20.