Skip to content

Fixed reservation logic for Auto Inventory allocation process. OFBIZ-13386#1254

Open
nameet-jain wants to merge 1 commit into
apache:trunkfrom
nameet-jain:OFBIZ-13386
Open

Fixed reservation logic for Auto Inventory allocation process. OFBIZ-13386#1254
nameet-jain wants to merge 1 commit into
apache:trunkfrom
nameet-jain:OFBIZ-13386

Conversation

@nameet-jain
Copy link
Copy Markdown
Contributor

Fixed reservation logic for Auto Inventory allocation process. OFBIZ-13386

@jacopoc
Copy link
Copy Markdown
Contributor

jacopoc commented May 26, 2026

Thank you, @nameet-jain.

Could you please provide the steps you followed to test the fix?

I’m having a hard time understanding the expected workflow from the descriptions provided in OFBIZ-10518 and OFBIZ-13386

@nameet-jain
Copy link
Copy Markdown
Contributor Author

Steps to Reproduce

  1. Go to Catalog → Stores and select any store (for example, 9000).
  2. Set the Allocate Inventory flag to Y. This enables the Auto Reserve flag on the order entry screen.
  3. Place an order from Order Manager with the Auto Reserve flag checked.

Actual Result

  • The order is placed successfully, but the inventory is not reserved. You can verify this on the Order View screen.
  • Also navigate to Order → Allocation Plan → Find All Allocation Plans. The Allocation Plan method is shown as Manual, which indicates that the reservation is planned and must be processed manually through the Allocation Plan screens.

Expected Result

  • The order should be placed successfully, and the inventory should be reserved automatically.
  • The system creates the Allocation Plan with the Auto method so that the planned allocation is processed automatically.

@jacopoc , here is another PR that fixes the display value of the Allocate Inventory flag. The default behavior for this flag should be N, but it is currently displayed as Y when the database value is empty (null), which is misleading.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants