You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by si...@apache.org on 2007/03/05 18:47:41 UTC
svn commit: r514747 -
/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/packing/PackingSession.java
Author: sichen
Date: Mon Mar 5 09:47:39 2007
New Revision: 514747
URL: http://svn.apache.org/viewvc?view=rev&rev=514747
Log:
NPE protection in PackingSession.setPackageWeight()
Modified:
ofbiz/trunk/applications/product/src/org/ofbiz/shipment/packing/PackingSession.java
Modified: ofbiz/trunk/applications/product/src/org/ofbiz/shipment/packing/PackingSession.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/shipment/packing/PackingSession.java?view=diff&rev=514747&r1=514746&r2=514747
==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/shipment/packing/PackingSession.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/shipment/packing/PackingSession.java Mon Mar 5 09:47:39 2007
@@ -893,7 +893,11 @@
}
public void setPackageWeight(int packageSeqId, Double packageWeight) {
- packageWeights.put(new Integer(packageSeqId), packageWeight);
+ if (UtilValidate.isEmpty(packageWeight)) {
+ packageWeights.remove(new Integer(packageSeqId));
+ } else {
+ packageWeights.put(new Integer(packageSeqId), packageWeight);
+ }
}
public Double getPackageWeight(int packageSeqId) {