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

[jacopoc]

Author: jacopoc
Date: Tue Jul 21 13:13:14 2009
New Revision: 796286

URL: http://svn.apache.org/viewvc?rev=796286&view=rev
Log:
Mass change orders to approved was only changing the header, not the items.

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=796286&r1=796285&r2=796286&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 Tue Jul 21 13:13:14 2009
@@ -4242,7 +4242,7 @@
     }
 
     public static Map massChangeApproved(DispatchContext dctx, Map context) {
-        return massChangeItemStatus(dctx, context, "ITEM_APPROVED");
+        return massChangeOrderStatus(dctx, context, "ORDER_APPROVED");
     }
 
     public static Map massCancelOrders(DispatchContext dctx, Map context) {
@@ -4286,6 +4286,7 @@
             Map ctx = FastMap.newInstance();
             ctx.put("statusId", statusId);
             ctx.put("orderId", orderId);
+            ctx.put("setItemStatus", "Y");
             ctx.put("userLogin", userLogin);
             Map resp = null;
             try {
@@ -4295,7 +4296,7 @@
                 return ServiceUtil.returnError(e.getMessage());
             }
             if (ServiceUtil.isError(resp)) {
-                return ServiceUtil.returnError("Error changing order item status: ", null, null, resp);
+                return ServiceUtil.returnError("Error changing order status: ", null, null, resp);
             }
         }
         return ServiceUtil.returnSuccess();