svn commit: r1804844 - /ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/

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

svn commit: r1804844 - /ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/

Author: jleroux
Date: Fri Aug 11 19:15:52 2017
New Revision: 1804844

Improved: Deprecated Methods in OrderChangeHelper

The methods abortOrderProcessing and releaseInitialOrderHold have been
deprecated since the workflow engine has been replaced by the ECA.
Both methods only return _true_ and other code code that uses these methods
is deprecated as well.
The revision in OrderChangeHelper shows that the methods used to have a
comment which stated that the methods are no longer in use.

jleroux: forgot to send the removing of some useless code

Thanks: Kyra Pritzel-Hentley for starting the analysis and creating the Jira


Modified: ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/
--- ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/ (original)
+++ ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/ Fri Aug 11 19:15:52 2017
@@ -2372,16 +2372,6 @@ public class OrderServices {
                     "OrderErrorCouldNotChangeOrderStatus",locale) + e.getMessage() + ").");
-        // release the inital hold if we are cancelled or approved
-        if ("ORDER_CANCELLED".equals(statusId) || "ORDER_APPROVED".equals(statusId)) {
-            OrderChangeHelper.releaseInitialOrderHold(ctx.getDispatcher(), orderId);
-            // cancel any order processing if we are cancelled
-            if ("ORDER_CANCELLED".equals(statusId)) {
-                OrderChangeHelper.abortOrderProcessing(ctx.getDispatcher(), orderId);
-            }
-        }
         if ("Y".equals(context.get("setItemStatus"))) {
             String newItemStatusId = null;
             if ("ORDER_APPROVED".equals(statusId)) {