This is an automated email from the ASF dual-hosted git repository.
gsperi pushed a commit to branch release22.01
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git
The following commit(s) were added to refs/heads/release22.01 by this push:
new ae37f44 Fixed: multiple facility inventory reservation issue
(OFBIZ-12264)
ae37f44 is described below
commit ae37f446d7054c9b2e513a3af36752b802a03af5
Author: Giulio Speri <giulio.sp...@mpstyle.it>
AuthorDate: Sun Apr 3 12:44:44 2022 +0200
Fixed: multiple facility inventory reservation issue (OFBIZ-12264)
The ProductStore is set up to reserve inventory from more than one
facility, so the flag oneInventoryFacility is set to N.
Flags requireInventory and reserveInventory are both to Y.
Flag allocateInventory is N.
A product can be anyway reserved in a ProductStoreFacility with its
thru date set and past, so is disabled, and if is not present enough
inventory quantity on other facilities.
Added filtering by date on multiple facility code in
reserveStoreInventoryMethod() and on isStoreInventoryAvailable().
This is the first part of the problem resolution; the second part is
accomplished by path to OFBIZ-12455 Jira.
Thanks: Jacques Le Roux and Pierre SMith for Jira feedbacks, Nicola
Mazzoni for helping in bug analaysis and Jacopo Cappellato for path
review.
---
.../groovyScripts/product/store/ProductStoreServices.groovy | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git
a/applications/product/groovyScripts/product/store/ProductStoreServices.groovy
b/applications/product/groovyScripts/product/store/ProductStoreServices.groovy
index 9f05ebb..e4c22d0 100644
---
a/applications/product/groovyScripts/product/store/ProductStoreServices.groovy
+++
b/applications/product/groovyScripts/product/store/ProductStoreServices.groovy
@@ -176,6 +176,7 @@ def reserveStoreInventory() {
List productStoreFacilities = from("ProductStoreFacility")
.where(productStoreId: productStore.productStoreId)
.orderBy("sequenceNum")
+ .filterByDate()
.cache()
.queryList()
for (GenericValue productStoreFacility : productStoreFacilities) {
@@ -184,29 +185,28 @@ def reserveStoreInventory() {
// TODO: must entire quantity be available in one location?
// Right now the answer is yes, it only succeeds if one
facility has sufficient inventory for the order.
Map callServiceMapIABF = [productId: parameters.productId,
facilityId: productStoreFacility.facilityId]
- logInfo("ProductStoreService:In productStoreFacilities
loop: [" + parameters.facilityId + "]")
Map serviceResultIABF = run service:
"getInventoryAvailableByFacility", with: callServiceMapIABF
BigDecimal availableToPromiseTotal =
serviceResultIABF.availableToPromiseTotal
-
if (availableToPromiseTotal >= parameters.quantity) {
storeFound = productStoreFacility
}
}
}
// didn't find anything? Take the first facility from list
+ GenericValue defaultStoreFound
if (!storeFound) {
- storeFound = productStoreFacilities.get(0)
+ defaultStoreFound = productStore.getRelatedOne("Facility",
true)
}
- facilityId = storeFound.facilityId ?: ""
+ facilityId = storeFound ? storeFound.facilityId :
defaultStoreFound.facilityId
Map serviceResult = run service:
"reserveProductInventoryByFacility", with: [*: parameters,
facilityId: facilityId,
requireInventory: requireInventory,
reserveOrderEnumId: productStore.reserveOrderEnumId]
quantityNotReserved = serviceResult.quantityNotReserved
- logInfo("Inventory IS reserved in facility with id
[${storeFound.facilityId}] for product id [${parameters.productId}]; desired
quantity was ${parameters.quantity}")
+ logInfo("Inventory IS reserved in facility with id [${facilityId}]
for product id [${parameters.productId}]; desired quantity was
${parameters.quantity}")
}
} else {
- List productStoreFacilities =
from("ProductStoreFacility").where(productStoreId: productStore.productStoreId,
facilityId: facilityId).cache().orderBy("sequenceNum").queryList()
+ List productStoreFacilities =
from("ProductStoreFacility").where(productStoreId: productStore.productStoreId,
facilityId:
facilityId).cache().orderBy("sequenceNum").filterByDate().queryList()
GenericValue facilityFound
for (GenericValue productStoreFacility : productStoreFacilities) {
// Search Product Store Facilities to insure the facility passed
in is associated to the Product Store passed in
@@ -314,6 +314,7 @@ def isStoreInventoryAvailable() {
List productStoreFacilities = from("ProductStoreFacility")
.where(productStoreId: productStore.productStoreId)
.orderBy("sequenceNum")
+ .filterByDate()
.cache()
.queryList()
available = "N"
