[ofbiz-framework] branch release18.12 updated: Fixed: liniting issues introduced by commit d00208c (OFBIZ-10585)

[pawan]

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

pawan pushed a commit to branch release18.12
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/release18.12 by this push:
     new 0a181c7  Fixed: liniting issues introduced by commit d00208c 
(OFBIZ-10585)
0a181c7 is described below

commit 0a181c7c36a73e2f5981ddb831b313a24fc2d379
Author: Pawan Verma <pawan.ve...@hotwaxsystems.com>
AuthorDate: Mon Nov 25 12:52:44 2019 +0530

    Fixed: liniting issues introduced by commit d00208c
    (OFBIZ-10585)
    
    Thanks: Mathieu Lirzin for report
---
 .../manufacturing/jobshopmgt/ProductionRunServices.java  | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git 
a/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java
 
b/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java
index af8731f..9ab6242 100644
--- 
a/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java
+++ 
b/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java
@@ -2802,10 +2802,11 @@ public class ProductionRunServices {
                 BigDecimal qtyToProduce = 
qtyRequired.min(mktgPackagesAvailable);
                 /*
                     Creating production run job for remaining quantity in 
created status
-                    This will handle cases like if production run job creates 
for partial quantities or in case of fully backordered scenario.
+                    This will handle cases like if production run job creates 
for partial quantities
+                     or in case of fully backordered scenario.
                  */
                 BigDecimal remainingQty = 
orderItem.getBigDecimal("quantity").subtract(qtyToProduce);
-                if(remainingQty.compareTo(ZERO) > 0) {
+                if (remainingQty.compareTo(ZERO) > 0) {
                     serviceContext.clear();
                     serviceContext.put("facilityId", facilityId);
                     serviceContext.put("userLogin", userLogin);
@@ -2816,11 +2817,16 @@ public class ProductionRunServices {
                     if (ServiceUtil.isError(serviceResult)) {
                         return 
ServiceUtil.returnError(ServiceUtil.getErrorMessage(serviceResult));
                     }
-                    String productionRunIdForRemainingQty = 
(String)serviceResult.get("productionRunId");
+                    String productionRunIdForRemainingQty = (String) 
serviceResult.get("productionRunId");
                     try {
-                        delegator.create("WorkOrderItemFulfillment", 
UtilMisc.toMap("workEffortId", productionRunIdForRemainingQty, "orderId", 
orderId, "orderItemSeqId", orderItemSeqId));
+                        delegator.create("WorkOrderItemFulfillment",
+                                UtilMisc.toMap("workEffortId", 
productionRunIdForRemainingQty,
+                                        "orderId", orderId, "orderItemSeqId", 
orderItemSeqId));
                     } catch (GenericEntityException e) {
-                        return 
ServiceUtil.returnError(UtilProperties.getMessage(resource, 
"ManufacturingProductionRunForMarketingPackagesCreationError", 
UtilMisc.toMap("orderId", orderId, "orderItemSeqId", orderItemSeqId, 
"errorString", e.getMessage()), locale));
+                        return 
ServiceUtil.returnError(UtilProperties.getMessage(resource,
+                                
"ManufacturingProductionRunForMarketingPackagesCreationError",
+                                UtilMisc.toMap("orderId", orderId, 
"orderItemSeqId", orderItemSeqId,
+                                        "errorString", e.getMessage()), 
locale));
                     }
                 }