svn commit: r471127 - /incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r471127 - /incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java

jonesde
Author: jonesde
Date: Fri Nov  3 21:48:51 2006
New Revision: 471127

URL: http://svn.apache.org/viewvc?view=rev&rev=471127
Log:
Fixed bug when ignoring quantity so that it always returns true instead of having that depend on a compare to 0

Modified:
    incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java

Modified: incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java
URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java?view=diff&rev=471127&r1=471126&r2=471127
==============================================================================
--- incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java (original)
+++ incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/price/PriceServices.java Fri Nov  3 21:48:51 2006
@@ -1129,7 +1129,8 @@
         } else if ("PRIP_QUANTITY".equals(productPriceCond.getString("inputParamEnumId"))) {
             if (quantity == null) {
                 // if no quantity is passed in, assume all quantity conditions pass
-                compare = 0;
+                // NOTE: setting compare = 0 won't do the trick here because the condition won't always be or include and equal
+                return true;
             } else {
                 compare = quantity.compareTo(Double.valueOf(productPriceCond.getString("condValue")));
             }