You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2006/10/27 14:31:08 UTC

svn commit: r468358 - /incubator/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java

Author: jleroux
Date: Fri Oct 27 05:31:07 2006
New Revision: 468358

URL: http://svn.apache.org/viewvc?view=rev&rev=468358
Log:
A patch from Joe Eckard from Jira issue " null Weight Shipping error" (http://issues.apache.org/jira/browse/OFBIZ-275)

Modified:
    incubator/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java

Modified: incubator/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java
URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java?view=diff&rev=468358&r1=468357&r2=468358
==============================================================================
--- incubator/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java (original)
+++ incubator/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/thirdparty/usps/UspsServices.java Fri Oct 27 05:31:07 2006
@@ -64,6 +64,7 @@
         Double shippableWeight = (Double) context.get("shippableWeight");
         if (shippableWeight.doubleValue() == 0) {
             // TODO: should we return an error, or $0.00 ?
+            return ServiceUtil.returnError("shippableWeight must be greater than 0");
         }
 
         // get the origination ZIP