svn commit: r1855307 - /ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/OrderServices.java

[rishi]

Author: rishi
Date: Tue Mar 12 11:17:49 2019
New Revision: 1855307

URL: http://svn.apache.org/viewvc?rev=1855307&view=rev
Log:
Improved: Store Order Service Imrovements. Use UtilValidate method instead of checking null.
(OFBIZ-10885)
Revisited the commit for review and found these improvements. Same has been done for the other context params.

Modified:
    ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/OrderServices.java

Modified: ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/OrderServices.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/OrderServices.java?rev=1855307&r1=1855306&r2=1855307&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/OrderServices.java (original)
+++ ofbiz/ofbiz-framework/trunk/applications/order/src/main/java/org/apache/ofbiz/order/order/OrderServices.java Tue Mar 12 11:17:49 2019
@@ -548,11 +548,11 @@ public class OrderServices {
             orderHeader.set("createdBy", userLogin.getString("userLoginId"));
         }
         
-        if (context.get("isRushOrder") != null) {
+        if (UtilValidate.isNotEmpty(context.get("isRushOrder"))) {
             orderHeader.set("isRushOrder", context.get("isRushOrder"));
         }
         
-        if (context.get("priority") != null) {
+        if (UtilValidate.isNotEmpty(context.get("priority"))) {
             orderHeader.set("priority", context.get("priority"));
         }