You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@ofbiz.apache.org by ha...@apache.org on 2008/08/29 05:02:17 UTC

svn commit: r690081 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java

Author: hansbak
Date: Thu Aug 28 20:02:17 2008
New Revision: 690081

URL: http://svn.apache.org/viewvc?rev=690081&view=rev
Log:
removed another condition which prevented the calculation of sales tax on purchase orders

Modified:
    ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java

Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=690081&r1=690080&r2=690081&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java (original)
+++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java Thu Aug 28 20:02:17 2008
@@ -1376,10 +1376,10 @@
             return ServiceUtil.returnError(UtilProperties.getMessage(resource_error,"OrderErrorNoValidOrderHeaderFoundForOrderId", UtilMisc.toMap("orderId",orderId), locale));
         }
 
-        // don't charge tax on purchase orders
-        if ("PURCHASE_ORDER".equals(orderHeader.getString("orderTypeId"))) {
-            return ServiceUtil.returnSuccess();
-        }
+        // don't charge tax on purchase orders, better we still do.....
+//        if ("PURCHASE_ORDER".equals(orderHeader.getString("orderTypeId"))) {
+//            return ServiceUtil.returnSuccess();
+//        }
 
         // Retrieve the order tax adjustments
         List orderTaxAdjustments = null;