This is an automated email from the ASF dual-hosted git repository.

mridulpathak pushed a commit to branch release24.09
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/release24.09 by this push:
     new 268f546519 Fixed: Regression in the minilang-to-Groovy conversion of 
ProductStoreServices.groovy (OFBIZ-13613)
268f546519 is described below

commit 268f546519f08c6d2d1cdd5572f6dcaa692b7571
Author: Mridul Pathak <[email protected]>
AuthorDate: Sat Sep 26 18:11:04 2026 +0530

    Fixed: Regression in the minilang-to-Groovy conversion of 
ProductStoreServices.groovy (OFBIZ-13613)
    
    reserveStoreInventory() no longer throws when the optional orderId is 
omitted or matches no OrderHeader, matching the original mini-lang which 
resolved to null with no exception in that case instead of aborting the whole 
reservation.
---
 .../org/apache/ofbiz/product/product/store/ProductStoreServices.groovy  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/applications/product/src/main/groovy/org/apache/ofbiz/product/product/store/ProductStoreServices.groovy
 
b/applications/product/src/main/groovy/org/apache/ofbiz/product/product/store/ProductStoreServices.groovy
index 023abf8c5c..c8dfa237d1 100644
--- 
a/applications/product/src/main/groovy/org/apache/ofbiz/product/product/store/ProductStoreServices.groovy
+++ 
b/applications/product/src/main/groovy/org/apache/ofbiz/product/product/store/ProductStoreServices.groovy
@@ -136,7 +136,7 @@ Map reserveStoreInventory() {
 
     GenericValue product = from('Product').where(parameters).cache().queryOne()
     GenericValue orderHeader = 
from('OrderHeader').where(parameters).cache().queryOne()
-    parameters.priority = orderHeader.priority
+    parameters.priority = orderHeader?.priority
 
     // if prodCatalog is set to not reserve inventory, break here
     if (productStore.reserveInventory == 'N') {

Reply via email to