[14.0][IMP] stock_reserve_rule: add full bin strategy

[TDu]

The full empty bin rule is like the empty bin rule, but it should take everything. That means there can not be any pre existing reservations.

ref.: rau-206

[jbaudoux]

Thanks, LGTM

[elvise]

@jbaudoux can i have more details about this PR ?

[jbaudoux]

@jbaudoux can i have more details about this PR ?

This is to prefer a location where the move takes everything from it. Useful for locations with everything on single pallet. Then you know you can take the full pallet.

[elvise]

@jbaudoux can i have more details about this PR ?

This is to prefer a location where the move takes everything from it. Useful for locations with everything on single pallet. Then you know you can take the full pallet.

Hi @jbaudoux i’m testing this PR in the runboat.
i created:

• Product: my test
• Location: My location for test (wh/stock/mylocationfortest)
• Reservation rule: Test 1
  

• Created an inventory for “my test” product

• Location: Shelf 1 > 10 units for “my test” and 10 units for “Bolt-White”
  

• Location: My location for test > 10 units for “my test”
  

Then i have installed sale app and i followed these steps:

• Create new SO with product “my test”
• Confirm SO
• Opened related picking

What i’m aspecting that user get the location “My location for test” but actually he gets only product unreserved.

[francesco-ooops]

@jbaudoux any feedback on @elvise 's functional test?

[jbaudoux]

@elvise You put 2 products on the location and you try to reserve only one of them, so you are not reserving the full location. So works as expected. Thanks for the test.

[jbaudoux]

/ocabot merge minor

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-1831-by-jbaudoux-bump-minor, awaiting test results.

[elvise]

@jbaudoux My pleasure 🤟🏻

[OCA-git-bot]

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