You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2010/02/22 18:57:11 UTC
svn commit: r914991 -
/ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/bom/BOMNode.java
Author: jacopoc
Date: Mon Feb 22 17:57:11 2010
New Revision: 914991
URL: http://svn.apache.org/viewvc?rev=914991&view=rev
Log:
Fixed bug as suggested in the user list: a wrong condition on scrap factor was introduced during conversion to BigDecimal.
Modified:
ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/bom/BOMNode.java
Modified: ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/bom/BOMNode.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/bom/BOMNode.java?rev=914991&r1=914990&r2=914991&view=diff
==============================================================================
--- ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/bom/BOMNode.java (original)
+++ ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/bom/BOMNode.java Mon Feb 22 17:57:11 2010
@@ -209,7 +209,7 @@
// A negative scrap factor is a salvage factor
BigDecimal bdHundred = new BigDecimal("100");
- if (percScrapFactor.compareTo(bdHundred.negate()) > 0 && percScrapFactor.compareTo(bdHundred.negate()) < 0) {
+ if (percScrapFactor.compareTo(bdHundred.negate()) > 0 && percScrapFactor.compareTo(bdHundred) < 0) {
percScrapFactor = BigDecimal.ONE.add(percScrapFactor.movePointLeft(2));
} else {
Debug.logWarning("A scrap factor of [" + percScrapFactor + "] was ignored", module);