You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by de...@apache.org on 2024/05/18 12:56:49 UTC

(ofbiz-framework) branch trunk updated: Improved: Corrected the condition to fetch currency UOM id from shipment instead of shipment route segment (#801)

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

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


The following commit(s) were added to refs/heads/trunk by this push:
     new 776eacc2a4 Improved: Corrected the condition to fetch currency UOM id from shipment instead of shipment route segment (#801)
776eacc2a4 is described below

commit 776eacc2a46de3779cea1f9b9748077d701e7929
Author: Sandeep Rajput <90...@users.noreply.github.com>
AuthorDate: Sat May 18 18:26:43 2024 +0530

    Improved: Corrected the condition to fetch currency UOM id from shipment instead of shipment route segment (#801)
    
    There is a typo in the condition while fetching a currency UOM id from the shipment route segment entity instead of shipment at the time of creating a shipment on the FedEx carrier.
---
 .../java/org/apache/ofbiz/shipment/thirdparty/fedex/FedexServices.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/applications/product/src/main/java/org/apache/ofbiz/shipment/thirdparty/fedex/FedexServices.java b/applications/product/src/main/java/org/apache/ofbiz/shipment/thirdparty/fedex/FedexServices.java
index 3dd00a0439..80d208bebb 100644
--- a/applications/product/src/main/java/org/apache/ofbiz/shipment/thirdparty/fedex/FedexServices.java
+++ b/applications/product/src/main/java/org/apache/ofbiz/shipment/thirdparty/fedex/FedexServices.java
@@ -538,7 +538,7 @@ public class FedexServices {
             String currencyCode = null;
             if (UtilValidate.isNotEmpty(shipmentRouteSegment.getString("currencyUomId"))) {
                 currencyCode = shipmentRouteSegment.getString("currencyUomId");
-            } else if (UtilValidate.isNotEmpty(shipmentRouteSegment.getString("currencyUomId"))) {
+            } else if (UtilValidate.isNotEmpty(shipment.getString("currencyUomId"))) {
                 currencyCode = shipment.getString("currencyUomId");
             } else {
                 currencyCode = EntityUtilProperties.getPropertyValue("general", "currency.uom.id.default", "USD", delegator);